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

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

ただいまの
回答率

87.92%

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

解決済

回答 2

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 1,900

score 16

前提・実現したいこと

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

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

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

  • m.ts10806

    2019/11/23 11:39

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

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

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

    キャンセル

  • m.ts10806

    2019/11/23 11:49

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

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

    キャンセル

  • m.ts10806

    2019/11/23 12:25 編集

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

    キャンセル

回答 2

+2

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).'
';


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

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2019/11/23 12:39 編集

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

    キャンセル

  • 2019/11/23 12:42

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

    キャンセル

  • 2019/11/23 12:43

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

    キャンセル

checkベストアンサー

+1

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

<?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);
}

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2019/11/23 13:10

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

    キャンセル

  • 2019/11/23 15:04 編集

    一応。

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

    キャンセル

  • 2019/11/23 16:23

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

    キャンセル

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

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

関連した質問

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