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

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

Python
1def fizzbuzz(count=100,fizzmod=3,buzzmod=5):
2    for cnt in range(1,count+1):
3        if cnt%fizzmod==0 and cnt%buzzmod==0:
4            print("FizzBuzz")
5        elif cnt%fizzmod==0:
6            print("Fizz")
7        elif cnt%buzzmod==0:
8            print("Buzz")
9        else:
10            print(cnt)

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

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

LouiS0616

2018/08/16 06:39

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

tachikoma

2018/08/16 06:40

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

行動規範の内容に同意します

回答3件

ベストアンサー

インデント

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

Python
1def fizzbuzz(count=100,fizzmod=3,buzzmod=5):
2    for cnt in range(1,count+1):
3        if cnt%fizzmod==0 and cnt%buzzmod==0:
4            print("FizzBuzz")
5        elif cnt%fizzmod==0:
6            print("Fizz")
7        elif cnt%buzzmod==0:
8            print("Buzz")
9        else:
10            print(cnt)

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

これでもだめなら

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

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

Python
1fizzbuzz()

Wandbox

投稿2018/08/16 06:47

LouiS0616

総合スコア35660

KazuhoYoshii

2018/08/16 08:10

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

LouiS0616

2018/08/16 08:12

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

KazuhoYoshii

2018/08/16 08:19

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

行動規範の内容に同意します

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

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

fizzbuzz2.py

python3
1def fizzbuzz(count=100, fizzmod=3, buzzmod=5):
2    for cnt in range(1, count + 1):
3        result = ""
4        if cnt % fizzmod == 0:
5            result += "Fizz"
6        if cnt % buzzmod == 0:
7            result += "Buzz"
8        if result == "":
9            result += str(cnt)
10
11        print(result)
12
13
14fizzbuzz(20)

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

投稿2018/08/16 09:53

katoy

総合スコア22324

KazuhoYoshii

2018/08/19 15:25

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

行動規範の内容に同意します

動きますね

あと，こういう書き方もあります（fizzmodとbuzzmodが互いに素である場合）

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