複数のtxtファイルを作成済みのコードに当てはめ1つずつ処理したいです

Question

前提・実現したいこと

Python使い始めて約一か月の初心者なので力を貸してください

txtファイル2th-scan_BLT1.txt～2th-scan_BLTn.txt(nは数)を下記コードで1つずつ処理をしたいです。
下記コードは「input(入力データ)」の部分に手入力で『2th-scan_BLT1.txt』のように入力すれば処理されます。
複数のファイルを与えられたとき一つ一つ入力が大変なので自動処理できるよう変更すべき点を教えて頂きたいです。
色々調べたのですがtxtだったりskiprowsだったりx,yで必要な値を抜き出しているようなものがなかったので、、

コードを記載していただけると助かります、、
必要な情報があればコメント頂ければ記載します！

該当のソースコード

from scipy.optimize import curve_fit
import matplotlib.pyplot as plt
import matplotlib.cm as cm
import pandas as pd
import numpy as np
from scipy import signal
import os
from scipy.signal import find_peaks


datafile=input("入力データ")

x, y = np.loadtxt(datafile, skiprows=29, unpack=True)

試したこと

https://ja.stackoverflow.com/questions/45204
https://masaeng.hatenablog.com/entry/2020/06/28/233020
などのサイトも確認しましたがx,yの壁が立ちはだかりチンプンカンプンでした

Comments

[1T2R3M4]

泣txtファイル
泣コード
とはどんなものでしょうか。
あと最後の()内も意味がわかりません。

[Nari9113]

繋がっていたので消しておきました

Answer 1

まずは、目標として、

data_file_list = <対象ファイルのリスト>

for data_file in data_file_list:
    process(data_file)

のように処理できるようにするのがいいと思います。

ここでの対象ファイルのリストは、どこかのディレクトリ配下のファイル一覧を取得するでも、スペース区切りで入力されたものを分割したリストでもいいでしょう。

process()関数は、現在のファイルでトップレベルで記述されている処理を関数化して作ります。
「x, y = np.loadtxt(datafile, skiprows=29, unpack=True)」 の行と、「#初期値のリストを作成」以下の部分をそっくり 「def rocess():」の中に入れてしまえばだいたい動くんじゃないでしょうか。
ただ、「detect_peak()」はx, yをglobalするのを止めて、引数で受け取るようにしたほうがいいですね。 ほかにもあるかもしれませんが見てません。

まず最初の目標は、

datafile=input("入力データ")
process(datafile)

として、これまでどおりに動くようにすることになりますね。

---

追記

process()を作るというのはだいたいこんな感じで、detect_peak関数などを中に入れるのではなくて、同じレベルに記述します。

def process(datafile):
    x, y = np.loadtxt(datafile, skiprows=29, unpack=True)
    
    #初期値のリストを作成
    #[amp,ctr,wid]

    # x1, y1 = detect_peak() ##
    np_x, np_y = detect_peak() ##
・・省略・・
    csv_datafile=input("保存データ名")
    df_h_in.to_csv(csv_datafile, sep=",",index=False)

これを、もとは「#初期値のリストを作成」のところに記述することになります。

そして、最後に、

datafile=input("入力データ")
process(datafile)

と書いて、もとのファイルと同じどうさをするようになればOK

Comments

[Nari9113]

変更部分を追記で載せさせていただきました
process関数の使い方をあまり理解できておらずすみません
入力の流れは変更追記のコードのようになるのでしょうか、？

[TakaiY]

違います。 それでは回答に書いたとおりではありません。
回答をちょっと追記しておきます。

[Nari9113]

あ、該当コードをprocess()という1つの大きな関数にして、最後のでこのデータをprocess()で処理してくれーとなるのですね！

その際def detect_peakは入れ込めなくなるということですか？

[TakaiY]

そういうことです。

> その際def detect_peakは入れ込めなくなるということですか？
いいえ。入れ込むこともできますが、関数の外で定義した関数も問題なく 呼べるので、外に出してあったほうが管理が楽です。
今の作りでは、外に出しておく利点はありませんが、将来大きなものを作るとき、特定の機能を持った処理=関数を適切に定義して作ることで設計も管理もしやすくなります。

そして、もういちど書いておきますが、「detect_peak()」はx, yをglobalするのを止めて、引数で受け取るようにしたほうがいいです。

[Nari9113]

globalをやめて、
datafile=input("入力データ")
process(datafile)
でできるようになりました！！このコードを
data_file_list = <対象ファイルのリスト>

for data_file in data_file_list:
process(data_file)
に変えて、
<対象ファイルのリスト>は[]内にファイル名打ち込む形式でいいのでしょうか？

[TakaiY]

> <対象ファイルのリスト>は[]内にファイル名打ち込む形式でいいのでしょうか？

はい。まずはそれでいいんじゃないでしょうか。

今後のことを考えたら、たとえば入力時にリストで渡せるようにするとか、ディレクトリ内にあるすべてのファイルのリストを取得してそれを処理するみたいなこともやっていけばいいのではないかと。
ディレクトリ内のリストの取得は、globモジュールのglobや、pathlibのglob(個人的にはこっち)を使うのがいいでしょう。

[Nari9113]

分かりました！
細かく教えて頂きありがとうございました！！