FizzBuzz問題はこれで完璧ですよね

Question

コード
```i = ["Fizz Buzz" if i % 15 == 0 else "Fizz" if i % 3 == 0 else "Buzz" if i % 5 == 0 else i for i in range(1,101)]
print(i)

Comments

[TakaiY]

どういうプログラムを「完璧」と呼ぶのかはいろいろな考えかたがあると思いますが、どんな基準ですか？

[退会済みユーザー]

説明不足で申し訳ありません。リストの内包表記を使ったFizzBuzz問題を実装する時のプログラムです。

[kazuma-s]

「コード」と「```」という余計な文字があるためエラーになります。
「Fizz と Buzz と数字」以外に「[」と「］」と「,」と「’」いう余計な文字が表示されるので完璧とは言えません。
変数 i を使わずにリストを print の引数にしたほうが短くなります。

Answer 1

どこがどう完璧なのか意味不明。１点

Comments

[退会済みユーザー]

批判するなら自分の意見（コード）を示しましょう。０点。

[Zuishin]

コードがほしかったんですか？
採点してほしかったんじゃないんですか？

[y_waiwai]

ざんねんながら、おやの遺言でコード書いてはいけないというのがありまして

[退会済みユーザー]

おもしろい。100点

[退会済みユーザー]

面白いので高評価しました。気に入らなかったら言ってください。

[ikadzuchi]

ゴミに餌をやる輩が不快なのでカウンター低評価しました。

[deathed]

カウンター高評価しました。

[koegoorkekg]

カウンター高評価しました。

Answer 2

"Fizz Buzz"

Fizz と Buzz が別々に出力された場合と区別がつかないので、空白は削除した方ががいいですよ。

pythonならではのショートゴルフコードを示しておきます。

print([i%3//2*"Fizz"+i%5//4*"Buzz"or-~i for i in range(100)])

Comments

[退会済みユーザー]

なるほど！的確なアドバイスありがとうございます。
私のコードでは、変数iのリストの中にもiを含んでいるのですが、本来ならリストの中と違う文字で変数を作成した方がいいのでしょうか？
例えば、
num = ["Fizz Buzz" if i % 15 == 0 else "Fizz" if i % 3 == 0 else "Buzz" if i % 5 == 0 else i for i in range(1,101)]
print(num)
です。

[shiracamus]

変数名 i には index や integer という意味が込められているかと思います。
変数名 n や num には number や the number of ～ で数を表す意味が込められているでしょうから、もっとふさわしい名前にした方がいい気がします。
例えば fizzbuzz にするのはいかがですか？
「リーダブルコード」という書籍を読むと変数名の付け方などが書いてあります。インターネット検索すると記事やブログも見つかります。

Answer 3

lint チェックをしたり、自動テストコードを書いてそれが PASS することをせずに、"完璧” というのは無理があると思います。