fetus Diary

2007/12/04(火) - 随所でわかる

    function setLocale($locale) {
        $urls = array(
            'us' => 'http://webservices.amazon.com/onca/xml2',
            'uk' => 'http://webservices.amazon.co.uk/onca/xml2',
            'de' => 'http://webservices.amazon.de/onca/xml2',
            'jp' => 'http://webservices.amazon.co.jp/onca/xml2',
            'fr' => 'http://webservices.amazon.fr/onca/xml2',
            'ca' => 'http://webservices.amazon.ca/onca/xml2',
        );
        $this->_locale = strtolower($locale);
        if (empty($urls[$locale])) {
            return;
        }
        $this->setBaseUrl($urls[$locale]);
    }

この作者、センスなさげ

# せっかく strtolower 使ってるのに、その直後で「正しいけど大文字」かも知れない $locale 使って参照してるし。

私だったら、setBaseUrl に null を渡すと現在のロケールを参照して自動設定するという風に作るなぁ。locale が URL を知っているんじゃなくて、URL リストが対応する locale を知っているんだから

コメント

コメントはありません。

名前
メール
コメント

※HTML タグは使えません。HTTP URL には自動リンクが張られます。

Captcha
画像から読み取れる文字を入力: