Q&A
そのまま実行できるコードとデータ
できれば今回の質問で触れられていないfr,f2,name,truncを取り除いて
簡略化したコードを提示していただけると回答得られやすいと思います。
Python3で書かれたコートの中で一部理解出来ない表現があります。
以下のコードは別ファイルから引数として渡された情報の中から遷移元id（ind_row）と選択肢(ind_row2/テキスト情報)の２種類のカラム情報を処理しています。
その中で以下の一部のコードについて自分の解釈が合っているか確認頂けたら幸いです。
単純なPythonの実力不足だとは思うのですが、少しでも理解して行きたくご質問させて頂きたいです。
①８行目：if x!=""：xの要素が空欄出ないなら＝存在したら以降の処理を実行 ②１１行目：if len(row_new1)>1:row_new1(row_newをsplitした物)の要素が１以上である場合 ③１２〜１４行目： if len(row_new1)!=len(b1): if len(b1)==1: c = row_new1[0] row_new1(選択肢)とb1(id)の数が一致しない場合でb1(id)の要素数が１つの場合は、 変数cをind_row１の0番目の要素と定義する ④１９行目：c = row_new1[b1.index(x)]：←ここが一番理解出なかったです。 前にb1(id)をfor文でxという変数としてループ処理しているので、以下の形になるのかと予想しました。 ---------- key|index←b1.index(x)?? １ 1000 ２ 2000 ３ 3000 --------- このrow_new1ごとの上記indexを取得している？
function.py def _id_choice(fr, ind_row, ind_row2): len0 = 0 for row in fr: if len0>0: a, b = row[0], row[ind_row] b1 = b.split("／") for x in b1: if x!="": row_new = _replace(row[ind_row2]) row_new1 = row_new.split("／") ### check later! ### if len(row_new1)>1: if len(row_new1)!=len(b1): if len(b1)==1: c = row_new1[0] ### choose one of choices! ### c = c[:trunc] ### truncation! ### f2.write("%s/%s\t%s\r\n" % (x,a,c)) # check later else: print("different numbers of items at ID: %s(%s)" % (row[0],name)) c = row_new1[b1.index(x)] # check later c = c[:trunc] ### truncation! ### f2.write("%s/%s\t%s\r\n" % (x,a,c)) # check later else: c = row_new1[b1.index(x)] c = c[:trunc] ### truncation! ### f2.write("%s/%s\t%s\r\n" % (x,a,c)) else: c = row_new1[0] c = c[:trunc] ### truncation! ### f2.write("%s/%s\t%s\r\n" % (x,a,c)) len0 += 1 return len0
deal_csv.py if switch == 1: f0, fr0 = _load_file(file0) file2 = "./id_choice.tsv" f2 = codecs.open(file2, "w", "utf-8") f2.write("%s\t%s\r\n" % ("ID_bef/aft","choice")) trunc = 100 # trancation length len2 = _id_choice(fr0,2,3) # depends on csv file f0.close() f2.close()
csvファイルカラム構造 id|message|遷移元id|選択肢| １ xxxxxx 30/31/32 選択肢１ ２ xxxxxx 40/41/42 選択肢２
回答1件
0
ベストアンサー
④１９行目：c = row_new1[b1.index(x)]：←ここが一番理解出なかったです。
列2: b1=["30","31",32"]
列3: row_new1=["選択肢1.1","選択肢1.2","選択肢1.3"]
という状態のときに、xは30→31→32と変化していく訳ですが、index()は今処理しているxがリストの何番目に来るか返す関数ですので、0→1→2と変化します。そのためcはrow_new1[0]→row_new1[1]→row_new1[2]と変化し、結果として選択肢1.1→選択肢1.2→選択肢1.3と変化することになります。
ソースコードを解析する際のコツとしては
デバッガを使え！
でしょうか。VSCodeやPycharmなどのエディタでは、ソースコードを1行ずつ追えて変数をウォッチ可能なデバッガが使えます。最初はデバッガの準備が面倒かも知れませんが、このようなソースコードを解析する作業が多いのであれば、デバッガの使い方を習得する方が近道です。
単純なPythonの実力不足だとは思うのですが
いえ、これはソースコードの方も難解です。ソースコードが読みにくい理由は
- インデントが深い
- 意味のない代入が多い(a, b = row[0], row[ind_row]やc = row_new1[0])
- 変数名から内容が予測できない
- 同じ概念を持つ変数名が全く異なる変数名で定義されている(b1とrow_new1)
- 対になっていない(ind_rowとind_row2)
- 不要なif分が多い、処理をまとめていない
- ループの外で実施すべきものがループの中に入っている
参考までにほとんど等価なコードを作ってみました(スラッシュは全角から半角にしています)。
python
1def _id_choice(fr, ind_row1, ind_row2): 2 len0 = 1 3 for row in fr[1:]: 4 b1 = row[ind_row1].split("/") 5 b2 = _replace(row[ind_row2]).split("/") 6 if len(b1) != len(b2) and len(b1) != 1 and len(b2) > 1: 7 print("different numbers of items at ID: %s(%s)" % (row[0], name)) 8 for x in b1: 9 if x == "": 10 continue 11 if len(b2) > 1: 12 f2.write("%s/%s\t%s\r\n" % (x, row[0], b2[b1.index(x)][:trunc])) 13 else: 14 f2.write("%s/%s\t%s\r\n" % (x, row[0], b2[0][:trunc])) 15 len0 += 1 16 return len0
投稿2020/07/05 12:02
編集2020/07/05 12:04
総合スコア1615
頂いたコードも読ませて頂きまして理解しました。すごく読みやすい。。。。
そのように処理されているのですね。頭で理解したとしても頂いたご説明のように言語化するのも私だと頭の中でこんがらがりそうです（笑）。とても参考になりました！Pycharmのデバッガはやはり重要ですね。Pycharmを使用してみます。
比較的難しいコードでもスキルのある方はこのように書き換えまでできるのですね。驚きました。
書き換えて頂いた方も参考にさせて頂きます！！
ありがとうございました！！
あなたの回答
tips
プレビュー
