1

API呼び出しをRapidshareに送信し、その呼び出しの結果を返すクラスを構築しています。呼び出しを行う方法は次のとおりです。

$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 に答える