PHP 1～50までの中から、3の倍数と3がつく文字を配列にいれて表示したい

Question

閲覧ありがとうございます。

やりたいこと：1～50までの数字から3の倍数と3がつく文字を
配列にいれて表示させたい

現状：3の倍数を配列にいれて表示まで出来たのですが、$iを文字列
に変換し3の文字を検索判定して３があった場合配列に入れるという意味で下記コードを書いたら
「未定義の関数str_contains（）への呼び出し」と表示されてしまいます。

str_contains（）の使い方はhttps://mebee.info/2020/11/29/post-24224/
を参考に書いてみたのですが、どこが悪いのかわかる方いらっしゃいましたら教えて頂けませんでしょうか。

※バージョンphp7ではstr_contains（）はないようですが、自分のPHPのバージョンを確認したらPHP 8.0.1でした。

 　　<?php        
        　$stack =array();

        　for($i = 1; $i<=50;$i++){
          $str = strval($i);
          $strThree = '3';

         if($i%3 === 0 or str_contains($str,$strThree)){
            array_push($stack,$i);
          };
        };

        foreach($stack as $value){
          echo $value;
        };
        
  ?>

Answer 1

8 系で動かしたところ、str_contains() は正しく動作しました。
提示のコードには全角スペースが入っており、そのままだと syntax error が発生します。

Comments

[nakayamaakg]

ご指摘ありがとうございます。
確かに全角がないと思っていましたが、貼り付けしなおしてみたら全角がございました。

全角消したうえで、再度画面表示したらエラー53 の未定義関数 str_contains() の呼び出し
と表示されたのですが、te2ji様がstr_contains() が動作したと連絡頂いたコードは
私が上記に書いたコードでしょうか。

Answer 2

代替え案を探していたら「strpos」でうまくいきました・・・

$stack =array();

    for($i = 1; $i<=50;$i++){
      $str = strval($i);
      $strThree = '3';

     if($i%3 === 0 or strpos($str,$strThree)){
        array_push($stack,$i);
      };
    };

    foreach($stack as $value){
      echo $value;
    };

Comments

[otn]

うまく行かないと思いますが、ちゃんと結果見てますか？
仕様の違う関数を単純に差し替えても駄目です。関数に合わせた使い方をしないと。

[nakayamaakg]

コメントありがとうございます。
$str = strval($i);　ここが指摘頂いた箇所でしょうか。
一応、エラーがでず、36912131518212324273033363942434548
と3の倍数以外にも3がつく文字を表示してしまいました。。

[otn]

？？？？
何を言ってるのですか？
自分が何をしたかったのか、理解してますか？

> と3の倍数以外にも3がつく文字を表示してしまいました。。
がしたかったことでは？
それが出来てないという指摘ですが。

[nakayamaakg]

現在エラーがでず、36912131518212324273033363942434548　と画面表示されています。
上記にて、13，23，43と3で割り切れない3の文字が表示できているので
問題ないのではないでしょうか。

[退会済みユーザー]

https://www.youtube.com/watch?v=wjXoqcrLBbA

[otn]

> 上記にて、13，23，43と3で割り切れない3の文字が表示できているので
問題ないのではないでしょうか。

1～50のうち、3の倍数と3がつく文字を「すべて」求めるのかとばかり思っていましたが、
すべてじゃなくていいのでしょうか？
この手の問題は普通はすべて求めるので、てっきり「すべて」かと思っていました。

[nakayamaakg]

数字を文字として3がないか検索した目的は13，23，43を表示させるためでした。
※33は3で割り切れるので　or strpos($str,$strThree)がないコードにて
136912151821242730333639424548　と表示が出来ておりました。

[otn]

31 や 32 などは 3 がつくけど、表示されないのが正解という意味？？
3 を含むうち、何が対象で何が対象で無いのか質問文からは読み取れない。

[nakayamaakg]

31～39すべて該当します。
すいません。出題課題で解いていたのですが、31～39について構想外でした。

[otn]

そうですか。であれば、2021/06/09 23:16 のコメントの通りです。

[nakayamaakg]

コメント頂きありがとうございました。
https://www.flatflag.nir87.com/strpos-210　このサイトにて
「検索する文字列が先頭にある場合エラーになる」ことへの解消方法が記載されており
下記コードにて31～39すべての表示に成功しました。

$stack =array();

for($i = 1; $i<=50;$i++){
$str = strval($i);
$strThree = '3';

if($i%3 === 0 or strpos($str,$strThree) !== false){
array_push($stack,$i);
};
};

foreach($stack as $value){
echo $value;
};