.iniファイルから取得した文字列をparseして数値リストを作るソースコードの可読性を上げたい

Question

前提・実現したいこと

前提

1. ユーザーがconfig.iniに数値および数値の範囲を入力
2. 数値の範囲の範囲内にある数値を補完してリスト化
3. 数値とリスト化した数値の範囲を単一のリストにする

####実現したいこと

• 可読性のよいきれいなコードにしたい
• 冗長な部分をきれいにまとめたい（split多い、空のリスト、if等）

発生している問題

ソースコードが冗長な感じがする

該当のソースコード

[FILESETTING]
#数値および数値範囲(3-5など)を指定
DeleteLines = 1,2,3-5,8

import configparser as cp
config = cp.ConfigParser()
config.read(r'config.ini', encoding='UTF-8')

"""
'-'を含むものを別処理にして、splitで分割
分割した数値をrangeに入力し、範囲内の数値を補完し再度listにする
""" 
delete_lines_list = []
for string in config['FILESETTING']['DeleteLines'].split(','):
    if '-' in string:
        range_list = [int(s) for s in string.split('-')]
        delete_lines_list.extend([ _ for _ in range(range_list[0],range_list[-1]+1)])
    else:
        delete_lines_list.append(int(string))

delete_lines_list
>>> [1, 2, 3, 4, 5, 8]

Answer 1

やってる事は提示のコードとほとんど同じなんですが、変数の名前を変えてみました。どうでしょう？

ranges = '1,2,3-5,8'
lines = []
for range_ in ranges.split(','):
    try:
        from_, to = map(int, range_.split('-'))
    except ValueError:
        lines.append(int(range_))
    else:
        lines.extend(range(from_, to + 1))

print(lines)

Comments

[snuow]

こうしてみると、いかに自分が冗長なコードを書いているかよくわかりました。
ありがとうございます。