ループ処理で数字を文字列に置き換える方法とフィボナッチ数列で300を"超えるまでの"数列を表示させたいです。

Question

1から順番に数を表示する(100まで)
その数が3で割り切れるなら"Fizz",5で割り切れるなら"Buzz",両方で割り切れるなら"FizzBuzz"と表示する。
つまり、1 2 Fizz 4 Buzz …という風に100まで出力したいです。

フィボナッチ数列についてはタイトルの通り、300を"超えるまで"表示したいです。

自分の書いたコードです。

<?php
for($i=1;$i<=100;$i++){
    echo $i;
  if($i%3==0){
    echo "Fizz";
  }elseif($i%5==0){
    echo "Buzz";
  }elseif($i%5==0 && $i%3==0){
    echo "FizzBuzz";
  }
  echo "&nbsp;";
}
?>

<?php
$a=1;
$i=0;

while(($a+$i)<=300){
  $s=($a+$i);

echo $s."&nbsp;";

$a=$i;
$i=$s;
}
 ?>

###それぞれの出力結果です。

1 2 3Fizz 4 5Buzz 6Fizz 7 8 9Fizz 10Buzz 11 12Fizz 13 14 15Fizz 16 17 18Fizz 19 20Buzz 21Fizz 22 23 24Fizz 25Buzz 26 27Fizz 28 29 30Fizz 31 32 33Fizz 34 35Buzz 36Fizz 37 38 39Fizz 40Buzz 41 42Fizz 43 44 45Fizz 46 47 48Fizz 49 50Buzz 51Fizz 52 53 54Fizz 55Buzz 56 57Fizz 58 59 60Fizz 61 62 63Fizz 64 65Buzz 66Fizz 67 68 69Fizz 70Buzz 71 72Fizz 73 74 75Fizz 76 77 78Fizz 79 80Buzz 81Fizz 82 83 84Fizz 85Buzz 86 87Fizz 88 89 90Fizz 91 92 93Fizz 94 95Buzz 96Fizz 97 98 99Fizz 100Buzz  

1 1 2 3 5 8 13 21 34 55 89 144 233 

よろしければご回答をお願いします。

Comments

[m.ts10806]

現在のコードで起きている問題、分からないことは何でしょうか。　https://teratail.com/help/question-tips#questionTips3-4

Answer 1

15のところはFizzBuzzと表示されるべきなのに、15Fizzとなっています。この部分に問題があると考えられます。

そこで、条件判定の部分(以下のコード）みてみると

  echo $i;
  if($i%3==0){
    echo "Fizz";
  }elseif($i%5==0){
    echo "Buzz";
  }elseif($i%5==0 && $i%3==0){
    echo "FizzBuzz";
  }
  echo " ";

１）数を無条件に表示している（3,5,15の倍数の場合には、数は表示せずに、Fizz,Buzz,FizzBuzzを表示するべき）
２）先に「３で割り切れる」「５で割り切れる」でFizzやBuzzを表示しているので、FizzBuzzが表示される可能性が無い(15の倍数は、3の倍数で5の倍数でもある）
という問題があるのが判ります。

こうした問題の解決方法は、
１）の問題は、Fizz,Buzz,FizzBuzzを表示しない時だけ数を表示するようにする。
２）の問題は、最初に１５で割り切れるかどうかの判断をするようにする。

これらを上記のコードに反映すると

  if($i%5==0 && $i%3==0){　//最初に１５の倍数かどうか判断
    echo "FizzBuzz";
  }elseif($i%3==0){
    echo "Fizz";
  }elseif($i%5==0){
    echo "Buzz";
  }else{ // ３，５，１５の倍数ではなかった
    echo $i;  //数字を表示
  }
  echo " ";

Comments

[退会済みユーザー]

ご回答ありがとうございました。ベストアンサーに選ばせていただきました。
また機会があればよろしくお願いします。

Answer 2

直接の回答を出してしまうのは面白くないのでヒントのみ。

• FizzBuzz

if文の順序を工夫しましょう

• フィボナッチ

ループが300を超えないまで、なら、もう一回手動で回してやれば良いでしょう

Answer 3

プログラム的にはこんな感じでも

$r=range(0,100);
foreach([3=>"Fizz",5=>"Buzz",15=>"FizzBuzz"] as $key=>$val1){
  foreach(range($key,100,$key) as $val2){
    $r[$val2]=$val1;
  }
}
unset($r[0]);
print implode(",",$r);

Answer 4

変数で指定されていない問題は固定出力すべき

Answer 5

まず最初の問題は、FizzBuzzの条件が最後にあるので、そこに到達することができません。
最初に持ってくれば解決です。

<?php
for($i=1;$i<=100;$i++){
    echo $i;
  if($i%5==0 && $i%3==0){
    echo "FizzBuzz";
  }elseif($i%3==0){
    echo "Fizz";
  }elseif($i%5==0){
    echo "Buzz";
  }
  echo "&nbsp;";
}
?>

もしくはelseifをやめてFizzとBuzzをそれぞれ判定するようにするか。

<?php
for($i=1;$i<=100;$i++){
    echo $i;
    if($i%3==0){
        echo "Fizz";
    }
    if($i%5==0){
        echo "Buzz";
    }
    echo "&nbsp;";
}
?>

次の問題は、ループ条件で300を閾値にしてしまうとよろしくないので、計算結果が300を超えたらループを抜けるようにすればよいでしょう。

<?php
$a=1;
$i=0;

while(1){
    $s=($a+$i);

    echo $s."&nbsp;";
    if($s>300) break;

    $a=$i;
    $i=$s;
}