functions.phpに広告用のコードを貼りショートコードで呼び出しているが上手く動かない

Question

子テーマのfunctions.phpに広告用のコードを貼り、ショートコードで呼び出して呼び出しているのですが、上手く動いてくれません。
wp_is_mobileが動いていないのか、スマホでも忍者Admaxのコードが表示されてしまいます。

PCでは忍者Admaxの広告を二個横並びに、スマホではnendの広告のみを表示したいです。
どうすればよいでしょうか。

<?php if (wp_is_mobile()) :?>
<p style=”font-size:5pt; color:#999999″><center>スポンサードリンク</center></p>
nendのコード
<?php else: ?>
<p style=”font-size:5pt; color:#999999″><center>スポンサードリンク</center></p><table cellspacing="15">
<table>
 <tr>
 <td>
PC左側の忍者Admaxコード
</td>
 <td>
PC左側の忍者Admaxコード
</td>
 </tr>
 </table>
<?php endif; ?>

恥ずかしながら、私はphpやhtmlを始めプログラミングは全くわかりません。
頓珍漢な質問だとは思いますが、回答を頂けると幸いです。

Answer 1

キャッシュ系プラグインが入っていると、モバイル判別が効かないことがあります。

【モバイル端末かを判別するis_mobile関数の使い方と機能しないときの対処法】
http://locatimefree.com/how-smartphone-friendly-function-is_mobile-works/#is_mobile-2
【WP Super Cacheとwp_is_mobile関数の組合せに注意、PCでスマホ用のCSSが反映】
http://webcommu.net/pc-mobile-css/
【WP Super Cacheを使う時はwp_is_mobile関数は使わないほうがいい | work.log】
https://worklog.be/archives/3128

Comments

[nakana79]

回答ありがとうございます。
WP Fastest Cacheを入れているので、キャッシュを削除したあと無効化して試してみましたが駄目でした。
また、別の所(ダブルレクタングルではなくシングル?レクタングル)の広告では問題なくwp_is_mobileが働いているようです。

[kei344]

質問に書いてあるコードを、問題のない箇所（「別の所」とおっしゃっているコード部分）の次か前部分に挿入し、表示を確認してみてください。
もし表示されるなら、functions.phpから表示部分のテンプレートへの呼び出しに問題がある可能性があります。

[nakana79]

問題のない箇所にに挿入してみたところ、意図通りに表示されました
以下が呼び出しコードなのですが、問題ありますでしょうか。

```
function yobu1() {
return'

<?php if (wp_is_mobile()) :?>
<p style=”font-size:5pt; color:#999999″><center>スポンサードリンク</center></p>
nendのコード
<?php else: ?>
<p style=”font-size:5pt; color:#999999″><center>スポンサードリンク</center></p><table cellspacing="15">
<table>
<tr>
<td>
PC左側の忍者Admaxコード
</td>
<td>
PC左側の忍者Admaxコード
</td>
</tr>
</table>
<?php endif; ?>

';
}
add_shortcode('ad1','yobu1');
```

[kei344]

return で返す文字列の中にPHPコードって入れられましたっけ？（変数は展開できたと思いますが）
また、ヒアドキュメントをあまり使わないため、その書き方で問題ないかがわかりません。

function yobu1() {
if ( wp_is_mobile() ) :
return <<<'EOD'
<p style=”font-size:5pt; color:#999999″><center>スポンサードリンク</center></p>
nendのコード
EOD
;
else:
return <<<'EOF'
<p style=”font-size:5pt; color:#999999″><center>スポンサードリンク</center></p><table cellspacing="15">
<table>
<tr>
<td>
PC左側の忍者Admaxコード
</td>
<td>
PC左側の忍者Admaxコード
</td>
</tr>
</table>
EOF
;
endif;
}

【PHPの基本構文「ヒアドキュメント」 - PHP入門 - Webkaru】
http://webkaru.net/php/here-document/

[nakana79]

恥ずかしながら、私はphpやhtml等の知識がゼロなんです。
調べながらいろいろなサイトを参考に継ぎ接ぎで作ったコードなので、詳しいことはわかりません。
身の程をわきまえて、諦めようかと思います。
相談に乗っていただき誠にありがとうございました。

[kei344]

それならショートコードで書かずに、WordPressテンプレートのpage.phpかsingle.phpの挿入したい場所に直接書かれたら良いと思います。

[nakana79]

Jetpackを使って、SNSのリンクや関連記事を表示しているので、直接書くとほんとうの意味で本文の最後に広告を出す事が出来なくなってしまう(SNSリンクや関連記事のあとに広告が表示されてしまう)んです。

[kei344]

広告やSNS~関連記事までの高さ（ピクセル）を固定できるのであれば、本文直下にCSSで広告文の空間を作って、などできますけどね。
yobu1のコードはうまく動きませんでしたか？ヒアドキュメントを使わずに書くなら下記を参考にしてみてください。
function yobu1() {
if ( wp_is_mobile() ) {
return 'is mobile!';
} else {
return 'is not mobile!';
};
}
ちょっとしつこく返信しすぎました、すいません。

[nakana79]

私の使い方が悪かったのか、両方共エラーが出てしまいました。
あと、返信がいただけて嬉しいです。謝らないでください。