前回書いた記事(最大値を探そう python (修正ver))についてアドバイスをいただいたのでさらにリファクタリングをしてみました。今回は可読性に重点を置きコードを改変してみました。
変更点
・for文のループ処理の時の変数の設定の仕方
コードは以下の通りです。
num = int(input("何個数値を代入しますか?"))
value_list = [] #数値を格納する配列を用意する
for i in range(num):
value = int(input("数値を代入して下さい"))
value_list.append(value) #用意したリストに値を追加していく
max_value = value_list[0] #リストの要素で初期化
for value in value_list:
if value > max_value:
max_value = value
print(max_value) #最大値を出力
変更したコード
変更前
for i in range(num):
if value_list[i] > max_value:
max_value = value_list[i]
変更後
for value in value_list:
if value > max_value:
max_value = value
こっちのコードの方がシンプルでよりわかりやすいですね。
記事を書けば書くほどいろいろな発見があり、とても勉強になります。