Pythonでif文でのbreakの使い方

Question

前提・実現したいこと

Pythonのコード内で特定のフォルダーにあるcsvをglob.glob()で習得し、for文とlen()でcsvファイルの長さを判別し、100行
以外の時はそのCSVのファイル名をprint()させるというコードの実装を目指しています。
下記の写真の「A.csv」と「D.csv」が100行以上(以下)あるcsvファイルです。

該当のソースコード

import pandas as pd
import glob
import os
import numpy as np

def identification_filetype(path):
    _, ext = os.path.splitext(path)
    if ext == ".csv":
        return pd.read_csv(path, encoding="shift-jis")
    elif ext == ".txt":
        return pd.read_csv(path, encoding="shift-jis", sep="\t")

os.chdir("C:\Users\For  Programming\Documents\Python Scripts\Lencheck\")

csv_files_side=("C:\Users\For  Programming\Documents\Python Scripts\Lencheck\*.csv")

csv_files_side_glob = glob.glob(csv_files_side)

list_csv_side = []
for f in csv_files_side_glob:
    basename = os.path.basename(f)
    a = identification_filetype(f)
    if len(a) != 100:
        print(f"{basename}")
        break
    elif len(a) == 100:
        list_csv_side.append(identification_filetype(f))
CSV_version_DataFrame_side= pd.concat(list_csv_side)

実現上の課題、解決したい事

実際の用途として、glob.glob()で習得したcsvファイルの中に、100行以外のデータセットがある場合は、elif以下の動作に進まず、該当するcsvファイルを表示させたいのですが、現時点での上記のコードではそれが上手くできません。

list_csv_side = []
for f in csv_files_side_glob:
    basename = os.path.basename(f)
.
.
.
A.csv

というように、100行以外のファイルが実際は2つ(A.csvとD.csv)ある中で、1つ（A.csv)しか表示されないので、該当するファイルを全て表示させたいです。

恐らく、break文の使い方が間違っていると思うのですが、自分で調べてみた範囲では見つけられませんでした。

補足情報（FW/ツールのバージョンなど）

Python3.7
Jupyter NoteBook
お忙しいとは思いますが、よろしくお願いいたします。
情報に不足がありましたら、ご指摘お願いいたします。

Comments

[meg_]

「その時点でfor文の処理を止めて」と「該当するファイルを全て表示させたい」が矛盾しているように思えるのですが違いますでしょうか？

[Danrussia]

すいません矛盾していました。書き直すと
「全ファイルをチェックして、100行以外のファイルがある」→「elifの動作に進まず、100行以外のファイルをprintする」という動作です。

[meg_]

上記の文章の通り、処理を分けるのが良いと思います。全ファイルの行数をチェックして100行以外のファイルがなければ後の処理を実行します。
※ファイルの1行目はヘッダとして扱いその下のデータ行が100行、ということで良いのですよね？

[Danrussia]

処理を分割する方法で考えてみます。(※以下の認識で大丈夫です)

[Danrussia]

for f in csv_files_side_glob:
basename = os.path.basename(f)
a = identification_filetype(f)
if len(a) != 100:
print(f"{basename}")

for f in csv_files_side_glob:
basename = os.path.basename(f)
a = identification_filetype(f)
if len(a) != 100:
break

こういう形で2つに区切って考えてみたのですが、どうですか?
(動作する上では恐らく問題ないのですが、もしもう少しスマートなものが何かあれば...?)

Answer 1

Data Frame a の行数を取得する場合、len(a.index) です。
len(a) だと列数になります。
また break するとその時点で for 文を抜けてしまうので、break する必要はないと思います。

以下のように変更してはどうでしょうか。

    if len(a.index) != 100:
        # 100行でない場合
        print(f"{basename}")
    else:
        # 100行の場合
        list_csv_side.append(identification_filetype(f))

Comments

[Danrussia]

すいません。素で列数と行数の意味を間違えました...
breakした後の行程をいくらか端折ってしまったのですが、
「全ファイルをチェックして、100列以外のファイルがある」→「elifの動作に進まず、100列以外のファイルをprintする」→「printされたcsvファイルを確認して、修正を加える(or取り除く)」→「再度for分で統合を試みる」
という過程を踏ませたいので、100列以外のファイルがあった時点でbreakで動作をストップさせたいというのも意図でした。

[tiitoi]

break で for 文を抜けたら同じ for 文に戻ってくることはできません。
100列以外のファイルに対して行う修正というのはすべて共通ですか？
共通であれば、if len(a) != 100 の場合にその処理を行ってから append すればいいのではないでしょうか。

[Danrussia]

返信が遅くなってしまい申し訳ありません。
100列以外のファイルに行う処理は共通ではないです。

[tiitoi]

ファイルごとに異なるのであれば、100列でないファイルは予め Excel 等で編集して調整して保存してから、質問のスクリプトで読み込むではダメなのでしょうか？