if文のorのエラー

Question

入力したキル数の平均を求める関数型プログラムです。
23行目のrecordsとnumの間に空白を空けているのはなぜでしょうか？

def get_kill_count():
    while True:
        try:
            return int(input("キル数は: "))
        except ValueError:
            print("整数値を入力してください")


def ask_quit():
    while True:
        str = input("終了しますか？ Yes/No: ")
        if str == "Yes" or "yes":
            return True
        if str == "No" or "no":
            return False
        print("Yes/Noを選択して入力してください")


def calc_average(records,num):
    return (sum(records) + num) / (len(records) + 1)

def show_average(records,num):
    print("平均キル数:{}".format(calc_average(records, num))) #なぜ空白？
    print("注意、記録は保持されません")


kill_records=[1,2,5,6]
while True:
    show_average(kill_records, get_kill_count())
    if ask_quit():
        break

この前ある方からいただいたコードですので、もし不快だったら言ってください。

追記１

すみません、該当の個所が違いました。
12行目と14行目の条件式でorを使うと、意図しない挙動を起こします。(YesでもNoでも、終了してしまう)
なぜか、わかりますか？

Answer 1

12行目と14行目の条件式でorを使うと、意図しない挙動を起こします。(YesでもNoでも、終了してしまう)

なぜか、わかりますか？

str == "Yes" or "yes"

という比較は、演算子の優先順位*1に従って

(str == "Yes") or ("yes")

という順番で処理されます。

「str == "Yes"はわかるけど、"yes"に対して論理演算はできないじゃん」と思われるかもしれませんが、pythonのオブジェクトは雑に言えばすべてbool値に変換できます。

>>> bool("yes")
True

細かい定義とかは参考リンク*2を見てくれれば良いのですが、とにかくこの真理値判定にもとづいてその式が評価されます。

bool("yes")が常にTrueであるために、その式は常にTrueになります。

参考：
*1 演算子の優先順位 - 数値 - Python入門
*2 4. 組み込み型 — Python 3.6.5 ドキュメント　4.1. 真理値判定

Comments

[sasuke_killer]

なるほど、比較演算子の順番がポイントだったのですね。
回答ありがとうございました！

Answer 2

12行目と14行目の条件式でorを使うと、意図しない挙動を起こします。

orの使い方を完全に誤っています。
ご提示のコードは次のように書いているのと同じです。

if (str == "Yes") or ("yes"):

Pythonでは空でない文字列は真値として取り扱われるので、上記の条件式は恒真です。

次のように書くと良いでしょう。

if str in ('Yes', 'yes',):

どうしてもorを使いたいなら、次のようにします。

if (str == "Yes") or (str == "yes"):

ただし

変数名にlist、str、intなどの命名をすることは絶対に避けてください。
不可解なエラーを引き起こします。

>>> str(42)
'42'
>>> str = 'spam'
>>> str(42)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
TypeError: 'str' object is not callable

元の質問に対して

PythonにはPEP8という規定があり、それに従うのが慣習となっています。
PEP 8 -- Style Guide for Python Code
PEP 8 -- Style Guide for Python Code (和訳)

ただ、『カンマの後には空白を開けましょう』って規定が見当たらないのですよね...
pycodestyleはしっかりとそれを報告してくれるのですが。

>python --version
Python 3.6.2 :: Continuum Analytics, Inc.

>python -m pycodestyle --version
2.4.0

>type tera3.py
print(1,2)

>python -m pycodestyle tera3.py --show-pep8
tera3.py:1:8: E231 missing whitespace after ','
    Each comma, semicolon or colon should be followed by whitespace.

    Okay: [a, b]
    Okay: (3,)
    Okay: a[1:4]
    Okay: a[:4]
    Okay: a[1:]
    Okay: a[1:4:2]
    E231: ['a','b']
    E231: foo(bar,baz)
    E231: [{'a':'b'}]

私の見ている情報が古いのだろうか？

Comments

[sasuke_killer]

なるほど。ありがとうございます！

[sasuke_killer]

式や文中の空白文字
イライラの元

という続いている二つの見出しにある

末尾のカンマと、その後に続く閉じカッコの間:

という項目は裏を返せば、(続く場合)『カンマの後には空白を開けましょう』ということなのではないでしょうか？

[LouiS0616]

『イライラする空白の置き方』が網羅されているという仮定の下で、『カンマの後には空白を開けてもイライラしない』ってことしか導かれないのですよね。
イライラしない空白なら置かねばならないと強制する記述は無いのです。

[sasuke_killer]

確かにそうですね...

Answer 3

特に理由はありません。「この方が見やすい」と思ったから、そういうルールにしているのでしょう。
なんか、よく分からないルールですね。「仮引数を区切るカンマの後は空けない。実引数を区切るカンマの後は空ける。配列の要素を区切るカンマの後も空けない」ですかね。

何故ここだけ気になるのでしょうか？
str = input("終了しますか？ Yes/No: ")の=の前後とか、他にもたくさん文法的には不要な空白があります。

Comments

[sasuke_killer]

あれ、すみません、この部分で意図しない挙動がおきているのかと思いましたが違いました。
とりあえず、回答ありがとうございます。

[otn]

カンマの後の空白のことですよね？

[sasuke_killer]

はい、そのことです。

[otn]

ありゃ、質問が全然別の物になってる。

[sasuke_killer]

すみません、勘違いしていたようです。

[sasuke_killer]

空白ではなく、if文が意図しない挙動の原因でした

Answer 4

質問文のコードを a.py とし、 style 修正しコードを b.py としました。

ｂ．ｐｙ

def get_kill_count():
    while True:
        try:
            return int(input("キル数は: "))
        except ValueError:
            print("整数値を入力してください")


def ask_quit():
    while True:
        str = input("終了しますか？ Yes/No: ")
        if str == "Yes" or "yes":
            return True
        if str == "No" or "no":
            return False
        print("Yes/Noを選択して入力してください")


def calc_average(records,num):
    return (sum(records) + num) / (len(records) + 1)

def show_average(records,num):
    print("平均キル数:{}".format(calc_average(records, num))) #なぜ空白？
    print("注意、記録は保持されません")


kill_records=[1,2,5,6]
while True:
    show_average(kill_records, get_kill_count())
    if ask_quit():
        break

a,py, b,py の style chekp 結果と 2つのフィアルの差分を示します。

a.py は警告が出てますが、b.py は警告は消えています。
この差分にでている SPACE 有無は、プログラムの動作には影響はないはずです。

追記：

if str == "Yes" or "yes":

は, 他の回答にあるように、変な記述ですね。

[https://teratail.com/questions/141409] の回答にあった類義のコードでは、

        str = input("終了しますか？ Y/N:")
        if str == 'Y':

となってます。
これを Yes/No の入力に対応sしようとした際にミスしたと思われます。

Comments

[sasuke_killer]

空白は問題ないのですね。
調べていただきありがとうございました！

[sasuke_killer]

はい、その通りでございます...!