空のリスト[]のときに0を出力させたい

Question

前提・実現したいこと

pythonで与えられたリストの最大の数値と最少の数値の差に対し、リストの最後尾の数値を掛けた値を返す計算をしています。
number = [] の時だけ0を出力させたいです。

発生している問題・エラーメッセージ

Traceback (most recent call last):
  File "print_string.py", line 20, in <module>
    result = check_sample(number)
  File "print_string.py", line 5, in check_sample
    a = max(number)
ValueError: max() arg is an empty sequence

該当のソースコード

def check_sample(number):
    a = max(number)
    b = min(number)
    c = a- b
    d = number[-1] 
    e = d * c
    
    return e

    
    
number = []

if number == []:
    print(0)

result = check_sample(number)
print(result)

Answer 1

check_sample関数が引数にどこまで責任を持つかに依ります。
場合に依っては単に次のように書き換えるだけで良いのでは。

if number == []:
    print(0)

result = check_sample(number)
print(result)

↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓↓

if number == []:   # if not number: の方が Pythonic
    print(0)
else:
    result = check_sample(number)
    print(result)

number = [] の時だけ0を出力させたいです。

number = [1, 1, 1, 1] のときも、
number = [3, 2, 1, 0] のときも計算の結果0が出力されますが、それは問題無いのでしょうか。

Comments

[hacker_]

回答ありがとうございます。

その書き方はとてもわかりやすいですね。参考になりました。

number=[]　の時に0を表示させたいので、計算結果として0になっても大丈夫です。

丁寧な回答ありがとうございます。また機会があればよろしくお願いします。

[LouiS0616]

回答にも書きましたが、check_sampleがどこまで処理の責任を担うかに依ります。
私の方法がふさわしい場合もあれば、同様にotnさんの方法がふさわしい場合もあります。

Answer 2

len()という便利なものを使うという感じですかね？

Comments

[LouiS0616]

== [] でも比較としては問題無く機能しますし、そもそも空リストはfalsyなのでlenを使うのは冗長です。
PEP8にも「For sequences, (strings, lists, tuples), use the fact that empty sequences are false.」と書かれています。

[hacker_]

なるほど、lenを使っても解決しそうですね。回答ありがとうございます。

[hacker_]

>>LouiS0601

回答ありがとうございます。解決しました。

Answer 3

ifで[]かどうかを判断して別処理するしかないと思います。

Comments

[hacker_]

回答ありがとうございます。
それは関数内にということでよろしいでしょうか？

if number != []:
a = max(number)
b = min(number)
c = a- b
d = number[-1]
e = d * c

else:
return 0
return e

このような形でしょうか？

[otn]

はい。関数内です。
[]の時に早期にリターンするのが普通な気がします。

[hacker_]

回答ありがとうございます。無事解決しました！