tkinter csvログ渡すと 処理が勝手に動いてしまう。

Question

import sys
import tkinter as tk
import tkinter.filedialog as tkfile
from tkinter import filedialog
from tkinter import ttk
import pandas as pd
from sklearn.model_selection import train_test_split
import os, tkinter, tkinter.filedialog, tkinter.messagebox


class Application(tk.Frame):
    def __init__(self, master):
        super().__init__(master)
        self.master = master
        self.master.title('ana_ver2')
        self.pack()
        self.create_widgets() #widget作る時に必要コード


    def create_widgets(self):
        self.input_box = tkinter.Entry(width=40)
        self.input_box.place(x=10, y=100)

            #ラベルの作成
        self.input_label = tkinter.Label(text="結果")
        self.input_label.place(x=10, y=70)

            #ボタンの作成
        self.button = tkinter.Button(text="参照",command=file_select)
        self.button.place(x=10, y=130)




        self.button2 = tkinter.Button(text="ana",command=ana_do)
        self.button2.place(x=10, y=160)

        self.input_box2 = tkinter.Entry(width=40)
        self.input_box2.place(x=10, y=180)




#ボタンがクリックされたら実行
def file_select():
    idir = 'C:\python_test' #初期フォルダ
    filetype = [("","*")] #拡張子の選択
    file_path = tkinter.filedialog.askopenfilename(filetypes = filetype, initialdir = idir)
    app.input_box.insert(tkinter.END, file_path) #結果を表示

    ana_do(file_path)


def ana_do(FP):

    df_a = pd.read_csv(FP)


    del df_a['default']
    del df_a['id']

    #文字列のカラムを数値に置き換えて最終的にfloat変換（自動化完了）
    #------------------------------------------------------------------------------
    col_name=df_a.columns#カラム名取得

    #カラム名数処理繰り返し
    for i in range(len(col_name)):

        #カラムobjectの時の処理　　要素抽出→置き換え
        if df_a[col_name[i]].dtypes==object:
            list_a = df_a[col_name[i]].unique()

            for u in range(len(list_a)):
                df_a[col_name[i]] = df_a[col_name[i]].replace(list_a[u], str(u))

    #カラム毎にfloat変換
    df_a[col_name[i]] =df_a[col_name[i]].astype(float)

    #------------------------------------------------------------------------------



    X =df_a[["age","job","marital","education","balance","housing","loan","contact","duration","campaign","pdays","previous","poutcome"]]
    y =df_a["y"]


    #学習データとtestデータに分ける
    from sklearn.model_selection import train_test_split
    # train_test_splitを使って、データセットをトレーニングデータとテストデータに分割する。　７：３
    X_train, X_test, y_train, y_test = train_test_split(X, y, test_size=0.3,random_state=0)

    #線形回帰モデル読み込み
    from sklearn.linear_model import LinearRegression
    lr=LinearRegression()

    lr.fit(X_train,y_train)
    lr.predict(X_test)

    #print(lr.coef_)#重み
    #print(lr.score(X_train,y_train)) #決定係数 0.22全然だめ
    #print(lr.score(X_test,y_test)) #決定係数


    #モデル読み込み
    from sklearn.linear_model import LogisticRegression
    Lr=LogisticRegression()

    #モデルに適合
    Lr.fit(X_train,y_train)
    #テストデータ予測
    y_a =Lr.predict(X_test)

    #誤判定件数確認
    from sklearn.metrics import confusion_matrix
    #print(confusion_matrix(y_test,y))

    out_1 = ('ロジスティック回帰　誤分類の個数：',  (y_a !=  y_test).sum())

    #正答率を確認
    from sklearn.metrics import accuracy_score
    out_2 = ('ロジスティック回帰　正答率：' , accuracy_score(y_test,y_a))

    app.input_box2.insert(tkinter.END, out_1)


root = tk.Tk()
root.title(u"ana_ver3")
root.geometry("500x600")
app = Application(master=root)
app.mainloop()

tkinterにて ボタン1 →　該当csvログ取得　→　ログを渡す　→　ボタン2 →　csvログ読み込み、解析処理
と動くコードを書きたいと思っているのですが、うまくいっておらず困っております。
下記コードを書いてみたのですが、ボタン1を押しただけで解析（def_ana_do(FP)まで勝手に動いてしまいます。
ログを渡したら自動で動いてしまうのかなと思ってはいるのですが、直し方が分かりません。
素人な質問で申し訳ありませんが分かる方助けて頂きたいです。

Comments

[y_waiwai]

このままではコードが読めないので、質問を編集し、<code>ボタンを押し、出てくる’’’の枠の中にコードを貼り付けてください

[x98000]

ソースコードは、<code>を押して「コード」と書かれた部分を上書きする形で貼り付けてください。
特にPythonはインデントが消えると不正なコードになりますので、まず修正をお願いします。

[masayam4]

申し訳ありません。修正させて頂きました。

Answer 1

下記コードを書いてみたのですが、ボタン1を押しただけで解析（def_ana_do(FP)まで勝手に動いてしまいます。

理由は単純で、ファイルを選択するときに実行される file_select関数 の最後の行で ana_do関数を呼んでいるからです。

def file_select():
    idir = 'C:\python_test' #初期フォルダ
    filetype = [("","*")] #拡張子の選択
    file_path = tkinter.filedialog.askopenfilename(filetypes = filetype, initialdir = idir)
    app.input_box.insert(tkinter.END, file_path) #結果を表示

    ana_do(file_path) # <- これを呼ぶと当然、解析処理が走る

ですのでこの部分を削除すると良いかと思います。

ただし現状のコードでは ana ボタンを押しても動作しませんので、この部分も編集する必要があります。（ana_do関数にはファイルパスの引数を渡す必要があるが、ボタンを押した時にこの引数を渡していないため ）

とりあえず、この部分は

        self.button2 = tkinter.Button(text="ana", state='disable', command = ana_do)

の行を

        self.button2 = tkinter.Button(text="ana", state='disable', command = lambda : ana_do(self.input_box.get()))

のように修正して、引数にファイルパスを渡すようにしてください。

Comments

[masayam4]

回答ありがとうございます！！
無事狙い通り動きました！！

Answer 2

プログラムのインデントは大丈夫ですか？pythonはC++やjavascriptなどと違い、{}がない分、インデントが重要になっています。
「該当csvログ取得」の関数や、「ログを渡す」関数の中に一緒になってしまって、解析処理が一度に行われてしまっているのかもしれません。