質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

90.98%

  • PHP

    17813questions

    PHPは、Webサイト構築に特化して開発されたプログラミング言語です。大きな特徴のひとつは、HTMLに直接プログラムを埋め込むことができるという点です。PHPを用いることで、HTMLを動的コンテンツとして出力できます。HTMLがそのままブラウザに表示されるのに対し、PHPプログラムはサーバ側で実行された結果がブラウザに表示されるため、PHPスクリプトは「サーバサイドスクリプト」と呼ばれています。

もっと簡潔にできると思うのですが…。

解決済

回答 6

投稿

  • 評価
  • クリップ 0
  • VIEW 246

chabiribon

score 6

function PCorPhone($ua){
    if(mb_strpos($ua,'Android')===false){
        $css = 'designPC.css';
    }
    if(mb_strpos($ua,'iPhone')===false){
        $css = 'designPC.css';
    }
    if(mb_strpos($ua,'iPad')===false){
        $css = 'designPC.css';
    }
    if(mb_strpos($ua,'Window')===false){
        $css = 'designPhone.css';
    }
    return $css;
}


PCとスマホでアクセスしたのを区別するプログラムを書いたのですが、もっと簡潔な書き方がありませんか?
アドバイスいただけると助かります。よろしくお願いします。

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • asahina1979

    2017/09/21 12:30

    Windows Phone, BlackBerry を忘れないで上げて

    キャンセル

回答 6

checkベストアンサー

+3

PCとスマホの切り分け条件は、ユーザーエージェント(以下 UA)に、
・Android か iPhone か iPad の文字列があれば、スマホ用
・Windows があれば PC 用
でしょうか?

提示されたソースの場合、Android も iPhone も iPad も Windows もない(例えば Mac からだったり Linux 機だったり)場合、$css が未定義のままですから、意図しない表示になるはずです。

上の切り分け条件をちょっと変えて
・Android か iPhone か iPad の文字列があれば、スマホ用
・そうでなければ PC 用
であれば、こういう感じになります。

function PCorPhone($ua) {
  $css = 'designPC.css';
  if (preg_match(/(Android|iPhone|iPad)/, $ua) === 1) {
    $css = 'designPhone.css';
  }
  return $css;
}

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/09/21 12:01

    なるほど!preg_matchを使えば簡潔ですね!ありがとうございます。早速やってみます。

    キャンセル

  • 2017/09/21 13:32

    こちらの回答を参考に作成したところ無事にできました!また、いろいろな意見を聞けてとても参考になりました。皆さんありがとうございました!

    キャンセル

+2

$cssのデフォ値が設定されていないですね
またifが排他的になっていないので常に後者で上書きされています。
基本をdesignPC.cssとして、特定の時designPhone.cssにするとよいのでは?

function PCorPhone($ua){
  $css = preg_match('/Android|iPhone|iPad/',$ua)?'designPhone.css':'designPC.css';
  return $css;
}

;

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/09/21 12:16

    なるほど、参考になります。ありがとうございます。

    キャンセル

+2

function PCorPhone($ua){
    if(mb_strpos($ua,'Window')===false){
        $css = 'designPhone.css';
    } else {
        $css = 'designPC.css';
    }
    return $css;
}

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/09/21 11:52

    あ!そうか!なるほど。確かにこれなら簡潔ですね!

    キャンセル

  • 2017/09/21 11:55

    Window以外がPhoneはさすがに乱暴すぎでは?

    キャンセル

  • 2017/09/21 11:57

    そうかもしれませんが、提示されたコードを単純に簡潔にしただけですので。

    キャンセル

+2

まずもって、このコードでは狙ったとおりに動きません。たとえば、macOSのSafariは以下のようなユーザーエージェントを返します。

Mozilla/5.0 (Macintosh; Intel Mac OS X 10_11_2) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/47.0.2526.106 Safari/537.36

最後のWindow条件に引っかかって、パソコンなのに'designPhone.css'を返してしまいます。

こういったものは、自分で書くより既存のライブラリを使ったほうが楽で確実だと思います。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/09/21 11:55

    このままではmacOSに対応できていないのですね?既存のライブラリとはどのようなものでしょうか?

    キャンセル

+1

tacsheavenさんやyambejpさんのようなロジック変更を行った方が良いかと思いますが、
質問文のソースコードを極限までシンプルにするとこうなります。

// この書き方は規約でアウト扱いの可能性もあり
function PCorPhone($ua){
    if (mb_strpos($ua, 'Window') === false) return 'designPhone.css';
    return 'designPC.css';
}
// その場合はこっち
function PCorPhone($ua){
    if (mb_strpos($ua, 'Window') === false) {
        return 'designPhone.css';
    }
    return 'designPC.css';
}

所謂ガード説という対応です。
さっさとreturn返して逃げれば一時変数も使わずに済んで万々歳ですね。


おまけ

PCorPhoneという関数を見て、他の人はどういう値を返す関数と想像するでしょうか?
まぁ大多数は文字列の"PC"か"Phone"なんだろうと想像するでしょうね。
ところが、実行すると返ってくるのが"designPC.css"なわけですよ。

なので下のようなアプローチが良いでしょうね。

function PCorPhone($ua){
    if (mb_strpos($ua, 'Window') === false) {
        return 'Phone';
    }
    return 'PC';
}

$device = PCorPhone($ua);
$css = "design{$device}.css";
function isPC($ua){
    if (mb_strpos($ua, 'Window') === false) {
        return false;
    }
    return true;
}

$device = isPC($ua) ? 'PC' : 'Phone';
$css = "design{$device}.css";

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

0

解決しました。これで今のところ上手くいっています。ありがとうございました。

//パソコンかスマホかを判断する関数
function PCorPhone($ua){

    $pattern = "/(Android|iPhone|iPad)/";
    $css = 'designPC.css';

    if(preg_match($pattern, $ua) === 1){
        $css = 'designPhone.css';
    }
    return $css;

}

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/09/21 18:23

    一応念のため今回のパターンにカッコは要らないです

    キャンセル

  • 2017/09/21 18:25

    そうなんですね。まだまだ理解不足で…。ありがとうございます。

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 90.98%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る

  • PHP

    17813questions

    PHPは、Webサイト構築に特化して開発されたプログラミング言語です。大きな特徴のひとつは、HTMLに直接プログラムを埋め込むことができるという点です。PHPを用いることで、HTMLを動的コンテンツとして出力できます。HTMLがそのままブラウザに表示されるのに対し、PHPプログラムはサーバ側で実行された結果がブラウザに表示されるため、PHPスクリプトは「サーバサイドスクリプト」と呼ばれています。