Noneが出るのはなぜですか？

Question

前提・実現したいこと

オンライン学習サービスのSPOJで簡単な課題に取り組んでいます。
インプットが
1
2
88
42
99
のものをアウトプットで
1
2
88
にしろと言うものです。

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

1
2
88
None

該当のソースコード

def select_number(numbers):
  for number in numbers:
    if number==42:
      break
    
      
    
      
    print(number)
  

  
    
  
    
print(select_number([1,2,88,42,99]))

試したこと

最後にNoneが出てしまうのはなぜでしょうか？

補足情報（FW/ツールのバージョンなど）

ここにより詳細な情報を記載してください。

Answer 1

関数select_numberにはreturn文が存在しません。pythonの関数は返るまでにreturn文が実行されなかった場合、勝手にNoneを返す仕様になっています。

print(select_number([1,2,88,42,99]))でそのNoneを表示してしまっています。

---

関数select_numberの中で表示処理を行っているのですから、返り値を更にprintする必要はないのです。以下のように書き換えてください。

# 略
select_number([1,2,88,42,99])

Comments

[khayato0512]

なるほど！分かりやすい回答ありがとうございます。納得しました！

Answer 2

質問文のコードで None が表示されるのは、hayataka2049 さんお回答のとうりです。

問題に対するコードとしては、
[1,2,88,42,99] の入力パラメータに対して、[1,2,88] を返すメソッド
をつくるべきでは？
x.pt

def select_number(numbers):
    ret = []
    for number in numbers:
        if number == 42:
            return ret
        ret.append(number)

numbers = []
while True:
    n = int(input())
    if n == 99:
        break;
    numbers.append(n)

for n in select_number(numbers):
    print(n)

でも問題の意図は 大小が逆転するまでの数字を出力することなのでは？
y.py

numbers = []
while True:
    n = int(input())
    if n == 99:
        break
    numbers.append(n)

print()
for i in range(len(numbers) - 1):
    print(numbers[i])
    if numbers[i] >= numbers[i + 1]:
        break