1

我正在构建一个类来向 Rapidshare 发送 API 调用并返回所述调用的结果。这是我希望完成通话的方式：

$rs = new rs();

$params = array(
    'sub'   =>  'listfiles_v1',
    'type'  =>  'prem',
    'login' =>  '10347455',
    'password'  =>  'not_real_pass',
    'realfolder'    => '0',
    'fields'    => 'filename,downloads,size',
    );

print_r($rs->apiCall($params));

这是到目前为止的课程：

class RS
{
    var $baseUrl = 'http://api.rapidshare.com/cgi-bin/rsapi.cgi?sub=';

    function apiCall($params)
    {
        $newUrl = $baseUrl;
        $keys = array_keys($params);
        $count = count($params);
        for($i = 0; $i < $count; $i++)
        {
            $newUrl .= $keys[$i];
            $newUrl .= '&';
            $newUrl .= $params[$keys[$i]];
        }
        return $newUrl;
    }
}

显然我正在返回$newUrl并使用print_r()来测试查询字符串，这就是上面显示的代码的结果：

sub&listfiles_v1type&premlogin&10347455password&_not_real_passrealfolder&0fields&filename,downloads,size

什么时候应该：

http://api.rapidshare.com/cgi-bin/rsapi.cgi?sub=listfiles_v1&type=prem&login=10347455&password=not_real_pass&realfolder=0&fields=filename,downloads,size

希望您能在这里看到我正在尝试做的事情：P 这可能是我未能找到的愚蠢错误或逻辑错误。

提前致谢。

4

1 回答 1

1

你应该有：

$newUrl = $this->baseUrl;

您需要用于$this在该类中引用该类的成员。也不要var用于声明成员。它是 PHP4 并且 (afaik) 已弃用。而是使用private(等。

最后，您创建参数的循环可以大大简化，并且逻辑对于您想要实现的目标不正确。尝试：

class RS {
  private $baseUrl = 'http://api.rapidshare.com/cgi-bin/rsapi.cgi?';

  function apiCall($params) {
    $newUrl = $this->baseUrl;
    foreach ($params as $k => $v) {
      $newUrl .= urlencode($k) . '=' . urlencode($v) . '&';
    }
    return $newUrl;
  }
}

或者，更好的是，使用http_build_query()：

class RS {
  private $baseUrl = 'http://api.rapidshare.com/cgi-bin/rsapi.cgi?';

  function apiCall($params) {
    return $this->baseUrl . http_build_query($params);
  }
}
于 2010-04-03T01:24:25.457 回答