テキストファイルから座標を読み込む際に、先頭の要素だけ抜けてしまう問題

Question

テキストファイルから座標を読み込むプログラムを作りました。このプログラムは以下のような仕事をします。

1.テキストファイルを読み込み、行を要素とするリストを作る
2.整形用の空白文字を取り除く
例)['','','1','','21','288']→['1','21','288']
3.strからfloat型に変換する。変換できなかったものはヘッダーと見なして削除する

以下、コードと対象のテキストファイルになります。テキストは長いので省略しています。

def str2float(cities):
    """ [[str,str],[str]] → [[float,float],[float]] """
    
    for i in range(len(cities)):
        try:
            for j in range(len(cities[i])):
                cities[i][j] = float(cities[i][j])
        except:
            print(f"I deleted {cities[0]}")
            del cities[0]
            continue
    

def remove_blank(cities):
    for i in range(len(cities)):
        for j in range(len(cities[i])):
            try:
                cities[i].remove('')
            except:
                continue
    if cities[-1] == ['EOF']:
        del cities[-1]


with open("ali535.tsp","r") as fin:
    cities = [data.split(' ') for data in fin.read().splitlines()]
    
    remove_blank(cities)
    str2float(cities)

    
print(cities,len(cities))

NAME: ali535
TYPE: TSP
COMMENT: 535 Airports around the globe (Padberg/Rinaldi)
DIMENSION: 535
EDGE_WEIGHT_TYPE: GEO
DISPLAY_DATA_TYPE: COORD_DISPLAY
NODE_COORD_SECTION
1  36.49  7.49
2  57.06  9.51
3  30.22  48.14
4  5.15 -3.56
5  34.59 -106.37
...
...
...
532 -6.13  39.13
533  47.28  8.33
534  51.33  0.14
535  24.58  91.53
EOF

こちらのテキストはhttp://comopt.ifi.uni-heidelberg.de/software/TSPLIB95/tsp/からダウンロードできます。
以下、実行結果になります。

I deleted ['NAME:', 'ali535']
I deleted ['TYPE:', 'TSP']
I deleted ['COMMENT:', '535', 'Airports', 'around', 'the', 'globe', '(Padberg/Rinaldi)']
I deleted ['DIMENSION:', '535']
I deleted ['EDGE_WEIGHT_TYPE:', 'GEO']
I deleted ['DISPLAY_DATA_TYPE:', 'COORD_DISPLAY']
I deleted ['NODE_COORD_SECTION']
I deleted ['1', '36.49', '7.49']
[[2.0, 57.06, 9.51], [3.0, 30.22, 48.14], [4.0, 5.15, -3.56], [5.0, 34.59, -106.37], [6.0, 57.12, -2.12], [7.0, 16.45, -99.45], [8.0, 5.36, -0.1], [9.0, 28.56, -13.36],
...
...
...
 [529.0, 44.06, 15.21], [530.0, 45.45, 16.04], [531.0, 41.4, -1.03], [532.0, -6.13, 39.13], [533.0, 47.28, 8.33], [534.0, 51.33, 0.14], [535.0, 24.58, 91.53]] 534

以上になります。解決方法などありましたら回答よろしくお願いします。

Answer 1

for文でリストにアクセスしながらそのリストのメンバーを削除するのは筋が悪いです。
削除してしまうと、ループで次にアクセスする場所が想定のものでなくなる場合があります。

そういう場合は、始めに空のリストを用意して、条件に合うものだけリストに追加するということをするのがいいでしょう。

---

ちなみに、以下のところは意図しているのかどうかわかりませんが、おかしな処理です。

            print(f"I deleted {cities[0]}")
            del cities[0]
            continue

このときチェックしているのはi番目の行のはずですが、かならず0番目を消しています。
削除しているので、詰ってしまうからこうしないとうまく動かなかったのかなと思っていますが、そうだとすると、正常に動かない原因を表わしています。

Comments

[old.exe]

回答ありがとうございます。空リストを用意して、ループ内のtryで型変換&空リストに追加、expectでデリートせずにコンティニューしてみます。

[old.exe]

解決できました。改めて回答していただきありがとうございました。

Answer 2

解決できたのでプログラムを共有します。

def str2float(cities):
    """ [[str,str],[str]] → [[float,float],[float]] """
    
    data = [[0]]*len(cities)
    
    for i in range(len(cities)):
        city = [0]*len(cities[i])
        data[i] = city
        try:
            for j in range(len(cities[i])):
                data[i][j] = float(cities[i][j])
        except:
            data[i]*=0
            continue
    
    data2 = list(filter(None,data))
    
    return data2
    

def remove_blank(cities):
    for i in range(len(cities)):
        for j in range(len(cities[i])):
            try:
                cities[i].remove('')
            except:
                continue
    

with open("ali535.tsp","r") as fin:
    cities = [data.split(' ') for data in fin.read().splitlines()]
    
    remove_blank(cities)
    data = str2float(cities)

    
print(data,len(data))