如何在不重复代码的情况下与许多不同的 Twitter 用户一起执行多个功能？

Question

我正在使用 Twitter API 1.1 制作一个 wep 应用程序，并且已经成功创建了一些函数，这些函数可以提取指定的用户个人资料图像、他们的实际姓名和他们的最新推文。全部使用 OAuth 对请求进行身份验证。

我的问题是 - 创建的所有 3 个函数都需要定义一个 twitter 用户名。目前，我必须为我输入的每个 Twitter 用户名一遍又一遍地复制我的代码。肯定有更好的方法吗？目前我的代码看起来一点也不好看。如果有人可以提供帮助，我将不胜感激。我对 PHP 还很陌生，所以一直坚持要做什么。

请在下面查看我的代码： 功能：

function getTweets($user, $tmhOAuth){
$usertweets = array();

$code = $tmhOAuth->request('GET', $tmhOAuth->url('1.1/statuses/user_timeline'), array(
  'include_entities'    => '1',
  'include_rts'         => '1',
  'screen_name'         => $user,
  'count'               => 1,
  'exclude_replies'     => 'true',
  'contributor_details' => 'true'
));


if ($code == 200) {
  $timeline = json_decode($tmhOAuth->response['response'], true); 
  foreach ($timeline as $tweet) :
    $entified_tweet = tmhUtilities::entify_with_options($tweet);
    $is_retweet = isset($tweet['retweeted_status']);

    $diff = time() - strtotime($tweet['created_at']);
    if ($diff < 60*60)
      $created_at = floor($diff/60) . ' minutes ago';
    elseif ($diff < 60*60*24)
      $created_at = floor($diff/(60*60)) . ' hours ago';
    else
      $created_at = date('d M', strtotime($tweet['created_at']));

    $permalink  = str_replace(
      array(
        '%screen_name%',
        '%id%',
        '%created_at%'
      ),
      array(
        $tweet['user']['screen_name'],
        $tweet['id_str'],
        $created_at,
      ),
      '<a href="https://twitter.com/%screen_name%/%id%">%created_at%</a>'
    );
     $tweet['created_at'] = $created_at;
     $usertweets[] = $tweet;
   endforeach;
   } else {
       tmhUtilities::pr($tmhOAuth->response);
}
    return $usertweets;
}

function getImage($screen_name,$tmhOAuth,$size = ''){
$url = $tmhOAuth->request('GET', $tmhOAuth->url('1.1/users/show'), array(
  'screen_name' => $screen_name,
));

$results = json_decode($tmhOAuth->response['response'], true);

//Get the user's profile image
$profileImg = ($results['profile_image_url']);
return str_replace('_normal', $size, $profileImg);
}

function getName($screen_name,$tmhOAuth){
$url = $tmhOAuth->request('GET', $tmhOAuth->url('1.1/users/show'), array(
  'screen_name' => $screen_name,
));

$results = json_decode($tmhOAuth->response['response'], true);

//Get the user's name
$name = ($results['name']);
return $name;
}

功能是如何实现的：

$name2 = getName("JoeyEssex_",$tmhOAuth);
$user2 = getTweets("JoeyEssex_",$tmhOAuth);
$image2 = getImage("JoeyEssex_",$tmhOAuth);

$name3 = getName("piersmorgan",$tmhOAuth);
$user3 = getTweets("piersmorgan",$tmhOAuth);
$image3 = getImage("piersmorgan",$tmhOAuth);

$name4 = getName("BinkyFelstead",$tmhOAuth);
$user4 = getTweets("BinkyFelstead",$tmhOAuth);
$image4 = getImage("BinkyFelstead",$tmhOAuth);

//Display the profile images of specified users

echo '<img src="' . $image2 . '" width="240px" height="240px" />';
echo '<img src="' . $image3 . '" width="240px" height="240px" />';
echo '<img src="' . $image4 . '" width="240px" height="240px" />';

//Display user's name
echo '<p>Latest tweet from <b>' . $name1 . '</b>:<br />';
//Display their tweets
foreach($user1 as $tweet){ 
echo $tweet['text'] . '<br />';
echo "Sent: <b>" . $tweet['created_at'] . "</b></p>";
}

echo '<b>' . $name2 . '</b>&nbsp;';
foreach($user2 as $tweet){
echo $tweet['text'];
echo "&nbsp;" . $tweet['created_at'] . "<br />";
}

echo '<b>' . $name3 . '</b>&nbsp;';
foreach($user3 as $tweet){
echo $tweet['text'];
echo "&nbsp;" . $tweet['created_at'] . "<br />";
}

echo '<b>' . $name4 . '</b>&nbsp;';
foreach($user4 as $tweet){
echo $tweet['text'];
echo "&nbsp;" . $tweet['created_at'] . "<br />";
}

你看到可怕的重复了吗？一定有更好的方法！

Answer 1

重复代码中唯一不同的部分实际上是发推文的人的姓名。因此，创建一个所有高音扬声器名称的列表并对其进行迭代。更改foreach 循环内的代码以反映新的输入和处理：

$tweeters = array('JoeyEssex_', 'piersmorgan', 'BinkyFelstead');

foreach ($tweeters as $tweeter)
{
    $tweeter_name   = getName($tweeter, $tmhOAuth);
    $tweeter_tweets = getTweets($tweeter, $tmhOAuth);
    $tweeter_image  = getImage($tweeter, $tmhOAuth);    

    // Display the profile images of user
    echo '<img src="' . $tweeter_image . '" width="240px" height="240px" />';

    // Display user's name
    echo '<p>Latest tweet from <b>' . tweeter_name . '</b>:<br />';

    // Display user's tweets
    foreach ($tweeter_tweets as $tweet)
    {
        echo $tweet['text'] . '<br />';
        echo "Sent: <b>" . $tweet['created_at'] . "</b></p>";
    }
}

将重复放入循环是表达代码重复的常用方式，意思是一遍又一遍地做同样的事情。

此外，您甚至可以开始将功能包装到 Tweeter 类中：

class Tweeter {

    private $name;
    private $tmhOAuth;

    public function __construct($name, $tmhOAuth) {
        $this->name     = $name;
        $this->tmhOAuth = $tmhOAuth;
    }

    public function getName() {
        return getName($this->name, $this->tmhOAuth);
    }

    public function getTweets() {
        return getTweets($this->name, $this->tmhOAuth);
    }

    public function getImage() {
        return getImage($this->name, $this->tmhOAuth);
    }
}

这写得很快，当然你甚至可以将代码从现有的get... 函数移到类方法中，但我保持它一步一步快速地进行第一次更改而不删除现有代码。

那么用法就相当简单了：

$tweeters = array('JoeyEssex_', 'piersmorgan', 'BinkyFelstead');

// Create all Tweeter objects
foreach ($tweeters as $i => $tweeter)
{
    $tweeters[$i] = new Tweeter($tweeter, $tmhOAuth);
}

// Display all Tweeters

foreach ($tweeters as $tweeter)
{
    echo '<img src="' . $tweeter->getImage() . '" width="240px" height="240px" />';

    echo '<p>Latest tweet from <b>' . $tweeter->getName() . '</b>:<br />';

    foreach ($tweeter->getTweets() as $tweet)
    {
        echo $tweet['text'] . '<br />';
        echo "Sent: <b>" . $tweet['created_at'] . "</b></p>";
    }
}

这将使您开始从面向对象编程中获利。