FizzBuzz問題の仕様の誤りを正したい

Question

前提・実現したいこと

1から $max までの整数値についてFizzBuzzの関数を実行したいです。
特定の正の整数値を引数として受け取りのその数値によって戻り値が変わり、
・数値が3の倍数であれば、戻り値は”Fizz”
・数値が5の倍数であれば、戻り値は”Buzz”
・数値が3の倍数であり5の倍数でもある場合は、戻り値は”FizzBuzz”
・上記のどれも満たさない場合は、その数値自体を戻り値にする。という条件のものです。
1から $max までの整数値 を使用してfizzbuzz関数を実行するので、forはfizzbuzz関数の外にしたいです。
Fizzbuzz関数内でprintせず、一度returnしたうえで戻り値をprintするようにしたいです。

発生している問題・エラーメッセージ

エラーメッセージ
特になし
上記の仕様を満たせていない点

該当のソースコード

コード
<?php
    function fizz_buzz() {
        $max = 100;
        for($n=1; $n<=$max; $n++){
            if($n%15==0){
                print "FizzBuzz".PHP_EOL;
            }
            elseif($n%3==0){
                print "Fizz".PHP_EOL;
            }
            elseif($n%5==0){
                print "Buzz".PHP_EOL;
            }
            else{
                print $n.PHP_EOL;
            }
        }
        
        }
    fizz_buzz()
?>

ソースコード

試したこと

特になし

補足情報（FW/ツールのバージジョン)

Cloud9

Comments

[y_waiwai]

で、しつもんはなんでしょうか

[Zuishin]

for が入ってますが。fizz_buzz(3) のように使うんじゃないんですか？

[m.ts10806]

試したことが「テキストの理解」だけだと誰もそれがどんなものでどう理解できるのか確認できません。
また、現在のコードにおける問題点を記載してください。
具体的に何を解決したいのでしょうか。

[m.ts10806]

今の質問内容から言えるのはこれくらいですが、それでも良いですか？

> forはfizzbuzz関数の外にしたいです。
まずそのように書いてみましょう

> izzbuzz関数内でprintせず、一度returnしたうえで戻り値をprintするようにしたいです。
ではそのように書いてみましょう。

[m.ts10806]

そもそも構文エラーがでるコードなので「仕様を満たせてない」以前の問題であることは認識されているのでしょうか

それに、「どの仕様を満たせてないか」認識できているかこの質問内容では分かりません。
その「仕様を満たせてない」ことに対して何も試してないということですか？
修正前「テキストの理解」と書かれていましたが、実際に理解しなければいけないのはテキストの内容ではなくコード、ロジックでは？

[m.ts10806]

ちなみに初心者では説明不可能な記述をあえて入れています。
理解せず使おうとすると痛い目みますよ。
仕様満たしてるかどうかまで確認してないようですし。

Answer 1

ini の変更なしでイケるように書いてみた。

<?php
function fizz_buzz($i) {
return@[@Fizz][$i%3].@[@Buzz][$i%5]?:$i;
}
$max=100;
for($i=1;$i<=$max;$i++)print fizz_buzz($i).'
';

ちなみに初心者では説明不可能な記述をあえて入れています。

Comments

[m.ts10806]

これ前見た気がする。
初心者どころか上級者でもパッと見理解が。
（たぶん業務でも中々使うことがない表現な気がする）

[退会済みユーザー]

改良版ｗ
質問者は仕様満たしたかどうかぐらいチェックしてほしいよね。

[m.ts10806]

課題だした側が理解できるかも不明ですね（これ前も同じやり取りした気が）

Answer 2

課題っぽいので丸投げ回答。解説しません。
このまま提出して通っても通らなくても何の責任も持ちません。
「理解できないコードを書かない」が原則と言うのは忘れないでください。

<?php
function fizz_buzz(int $n):string
{
    if ($n % 15 == 0) {
        return 'FizzBuzz'.PHP_EOL;
    } elseif ($n % 3 == 0) {
        return 'Fizz'.PHP_EOL;
    } elseif ($n % 5 == 0) {
        return 'Buzz'.PHP_EOL;
    } else {
        return $n.PHP_EOL;
    }
}

$max = 100;
for ($n = 1; $n <= $max; ++$n) {
    echo fizz_buzz($n);
}

Comments

[m.ts10806]

ちなみに初心者では説明不可能な記述をあえて入れています。
提出先に突っ込まれて説明できなくても丸投げコードをそのまま使おうとしたあなたの責任だということです。
目的がコードを写すことだけの人にできるフォローはありません。
「もらったコードがなんか知らんけど動いたからそれでOK」というスタンスの人は私は嫌いですし、エンジニアは嫌う傾向にあります。
（これで提出先にこっぴどく怒られて自身のスタンスを見なおされるのを期待しています）

[m.ts10806]

ちなみに今高評価をした方は理解してないのに高評価をしているため、
私は迷惑しています。そもそもPHPやってない人だしなぜ高評価できるのか謎。

[m.ts10806]

高評価理由書いてもらったらそれで納得するんですけど、
何も言わないまま取り下げたということは多少はこちらの意図が伝わったということでしょうかね。翌日に忘れてなければいいんですが。

[m.ts10806]

>tonkoさん
遊ぶのはやめてください。
PHPでの活動が全くない人なのでなぜ評価できるのかと疑問を持っているのです。
「なぜ高評価したか」理由を書いてください。

[kyoya0819]

一応。

個人的にはte2jiさんのコードより端的？でわかりやすい かつ 初心者には説明不可能な記述があり好きです。

[m.ts10806]

質問者のコードを要件に合わせて改修しただけですしね。
もうひとつ追加してみました。
本当は返り値で判断して色々遊びたかったですけどね。大喜利にするにはあまり遊べない話題なのでこのあたりで。