python 一定回数ループした後、処理を変化させる

Question

前提・実現したいこと

pythonで一定回数ループした後の処理を変化させたいです。
fileの名前を変えていく処理です。

該当のソースコード

flist = sorted(glob.glob(path + '*.m4a'))

for i, file in enumerate(flist, 1):
    if i <= 845:
        os.rename(file, path + str(i).zfill(4) + '.m4a')

    else:#845以降は1001から名前をつけたい。
        os.rename(file, path + str(i + 55) + '.m4a')

試したこと

上のコードでも出来るのですが他にいい方法があるような気がするので質問させて頂きました。

Answer 1

私なら、これぐらいのものなら単純に書きます。

番号が 845 以降は 1001 から始めたいようなので、処理を変えるっていうより、変数に入っているデータを変更すればいいかなと。

Pythonは不慣れなので、あくまでイメージでとらえてください。
(書き方自体が間違っていたらすみません…)

# 上記にはさらにコードがあるとして

delimNum = 845 # 基準値は 845

num = 0 # 初期値は 0で、0から開始する

for file in enumerate(flist, 1):
    if num == (delimNum + 1):
       num = 1000 # 1000 で初期化 (厳密には代入だが…)
    num += 1 # 1上昇
    # ここで num を使う

みたいにやりますね。

単純に考えてみてください。

x≦845 (xは仮の変数だとする) の間は そのままの数字で、
xが845 をオーバーしたら ( x＞845 ) 1001 から開始するのだから、
「846になったら1000からにする」みたいに考えることができますね。
846 になったときだけ作動するif文にするかなぁ。私なら。

ただし、処理自体を変えるなら今回の方法はきかないと思います。

あと、可読性のためにも、マジックナンバー ( 意味のあるデータを直接指定する方法. つまり変数なんかに入れずに指定するやつ ) はやめたほうがいいです。
もし基準値が 5 に切り替わったなら 845 をすべて 5 に書き換えないといけないので。

Comments

[hiranomo]

確かにそうですね。変に考え過ぎていました。
ありがとうございました。

Answer 2

enumerateでカウント変数を充分にコンパクトにかけていると思います。
可読性を無視すればbool値を掛け算に組み込み下のようにも書けますが普通が一番です。

flist = sorted(glob.glob(path + '*.m4a'))
# NOTE: 846以降は1001からカウントする
trigger_num = 846
add_num = 1001 - trigger_num

for i, file in enumerate(flist, 1):
    num = i + (add_num * (i >= trigger_num))

Comments

[hiranomo]

ありがとうございました。