我正在构建一个类,用于向 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 这可能是一个我没有找到的愚蠢错误或逻辑错误。
你应该:
$newUrl = $this->baseUrl;
您需要使用
$this
从该类中引用该类的成员。也不要使用 var
来声明成员。它是 PHP4 并且(据我所知)已被弃用。而是使用 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);
}
}