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

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

ただいまの
回答率

88.58%

みんなのPython第4版 p.145の内容について

解決済

回答 3

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 599

KazuhoYoshii

score 8

プログラミング初心者です。
みんなのPython第4版を読み進めておりますが、以下のコードで数字が表示されないため困っております。エラーメッセージは出ていないので、原因等をご教示頂けれると幸いです。
コードは以下の通りです。

def fizzbuzz(count=100,fizzmod=3,buzzmod=5):
    for cnt in range(1,count+1):
        if cnt%fizzmod==0 and cnt%buzzmod==0:
            print("FizzBuzz")
        elif cnt%fizzmod==0:
            print("Fizz")
        elif cnt%buzzmod==0:
            print("Buzz")
        else:
            print(cnt)

よろしくお願いいたします。

※ご指摘のコード部分を修正いたしました。

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

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

  • LouiS0616

    2018/08/16 15:39

    インデントが潰れて読めないので、次の画像に従ってコードを貼りなおしてください。https://teratail.storage.googleapis.com/uploads/contributed_images/56957fe805d9d7befa7dba6a98676d2b.gif

    キャンセル

  • tachikoma

    2018/08/16 15:40

    編集からコードの部分を<code>で囲んでインデントが正しく表示されるように修正お願いします。Pythonはインデントが崩れると意味が変わってしまうので。

    キャンセル

回答 3

checkベストアンサー

+3

インデント

Pythonはインデント(字下げ)が重要な意味を持つ言語です。
まず、次のようにインデントが整っているか確認してください。

def fizzbuzz(count=100,fizzmod=3,buzzmod=5):
    for cnt in range(1,count+1):
        if cnt%fizzmod==0 and cnt%buzzmod==0:
            print("FizzBuzz")
        elif cnt%fizzmod==0:
            print("Fizz")
        elif cnt%buzzmod==0:
            print("Buzz")
        else:
            print(cnt)

これを確かめる意味でも、質問の際はインデントが潰れないようにご留意ください。
Python

これでもだめなら

初心者の躓きポイントとしてよく有るのは、『関数を定義しただけ』で終わってしまうことです。

関数は定義しただけでは何も働きません。定義した後に呼び出す必要があります。
今回の場合、次のように書くだけで充分です。

fizzbuzz()

Wandbox

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/08/16 17:10

    大変失礼いたしました。
    質問を修正しましたのでご確認お願い申し上げます。

    キャンセル

  • 2018/08/16 17:12

    『これでもだめなら』以降は試しましたか?

    キャンセル

  • 2018/08/16 17:19

    ご指摘の通り、定義をしただけで記述が終わっており、呼び出しが完了しておりませんでした。
    また、インデントをfizzbuzzを定義した箇所と揃えることにより無事動きました。
    ご回答頂きまして、誠にありがとうございました。

    キャンセル

+2

動きますね
イメージ説明

あと,こういう書き方もあります(fizzmodとbuzzmodが互いに素である場合)

def fizzbuzz(count=100,fizzmod=3,buzzmod=5):
    for cnt in range(1,count+1):
        if cnt%(fizzmod*buzzmod) == 0: #MODIFIED
            print("FizzBuzz")
        elif cnt%fizzmod==0:
            print("Fizz")
        elif cnt%buzzmod==0:
            print("Buzz")
        else:
            print(cnt)

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/08/16 16:43

    fizzmodとbuzzmodが互いに素な関係にある時しかその書き換えは成立しないと思います。

    キャンセル

  • 2018/08/16 16:46

    確かに!

    失礼いたしました
    m(_ _)m

    キャンセル

  • 2018/08/16 17:21

    この度はご回答くださり、誠にありがとうございました!
    単に定義したfizzbuzzを呼び込んでいなかっただけでした。。。
    今後は定義しただけにならず、呼び出しまでを記述するよう気を付けていきます。
    誠にありがとうございました。

    キャンセル

+2

エディタの設定をおこなうと、コードの書き方のチェック(スタイルチェック)を行うことができます。
質問文のコードは、次のようにスタイルとして不適な箇所が多いです。
(赤マルがある行、赤波線のある行...)
イメージ説明

そこで、スタイルを整え、かつ % 演算の回数を減らす (4 回 -> 2回) ことをしてみました。
(1行目の赤表示は、メソッドの説明コメントがないよ というもの)
イメージ説明
fizzbuzz2.py

def fizzbuzz(count=100, fizzmod=3, buzzmod=5):
    for cnt in range(1, count + 1):
        result = ""
        if cnt % fizzmod == 0:
            result += "Fizz"
        if cnt % buzzmod == 0:
            result += "Buzz"
        if result == "":
            result += str(cnt)

        print(result)


fizzbuzz(20)

書籍のコードそのままの動作を確認するだけでなく、スタイル修正をしたり、すこし異なる書き方にしてみたりしながら勉強をすすめていくと良いと思います。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2018/08/20 00:25

    この度はご回答くださりありがとうございました。
    質問とは別の、より簡潔なコードを記述頂きまして大変勉強になります。
    今後はテキストに記載されたコードを追ってばかりでなく、書き方を変えてみることも試していきたいと思います。

    キャンセル

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

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

関連した質問

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