ディレクトリに複数あるファイルを移動させたい。

Question

前提

あるディレクトリにファイルが複数入っています。
例）0123456-01製品.xlsx,0123456-01図面.pdf, 1234546-a0製品.xlsx,123456-a0図面.pdfというように
数字7桁-[0-9a-z]{2}のパターンに製品と図面が複数入っているディレクトリがあります。
このディレクトリに、数字7桁-[0-9a-z]{2}+’sample'というディレクトリを作り、共通するファイルを入れていきたいです。

上記の例でいうと
0123456-01sample-0123456-01製品.xlsx,0123456-01図面.pdf

1234546-a0sample-1234546-a0製品.xlsx,123456-a0図面.pdf
が入る形になります。

うまくいかないので、どうすればよいでｙそうか？

実現したいこと

ここに実現したいことを箇条書きで書いてください。

• ▲▲機能を動作するようにする

発生している問題・エラーメッセージ

ディレクトリは作られるのですが、ファイルが入りません。

#ディレクトリ作成

def make_target_dir(filelists):
    for file in filelists:
        filepath = os.path.split(file)[0]
        basename = os.path.split(file)[1]
        name,ext = os.path.splitext(basename)
        foldername = re.search(r'[0-9]{7}-[0-9A-z]{2}',name).group()
        new_folder_path = os.path.join(filepath,foldername+'sample')
        if not os.path.exists(new_folder_path):
            os.makedirs(new_folder_path)

#ディレクトリとファイルを判定
def filter_dir(filelists):
    dir_ = []
    for file in filelists:
        if os.path.isdir(file):
            dir_.append(file)
    return dir_

def filter_file(filelists):
    filelist = []
    for file in dirs:
        if os.path.isfile(file):
            filelist.append(file)
    return filelist



#同じディレクトリにあるものをフォルダ名とファイル名が一致したらフォルダに入れる
def insert_file(dirs,filelist):
    for target_dir in dirs:
        basename = os.path.basename(target_dir)
        basename_code = re.match(r'[0-9]{7}-[a-z0-9]{2}',basename).group()
        for file in filelist:
            if re.search(r'[0-9]{7}-[a-z0-9]{2}',file).group() in basename_code:
                shutil.move(file,target_dir)

if __name__ == '__main__':
    file_path_lists =glob.glob(r'ファイルパス/*')
    make_target_dir(file_path_lists)
    dirs = filter_dir(file_path_lists)
    filelist =filter_dir(dirs)
    insert_file(dirs,filelist)

試したこと

ここに問題に対して試したことを記載してください。

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

ここにより詳細な情報を記載してください。

Answer 1

とりあえず、修正しました。
修正箇所は　#修正　のコメントが入っています。

#ディレクトリ作成

def make_target_dir(filelists):
    dirs = []
    for file in filelists:
        filepath = os.path.split(file)[0]
        basename = os.path.split(file)[1]
        name,ext = os.path.splitext(basename)
#修正開始
        match = re.search(r'[0-9]{7}-[0-9A-z]{2}',name)
        if match != None:
            foldername = match.group()
            new_folder_path = os.path.join(filepath,foldername+'sample')
            if new_folder_path not in dirs:
                dirs.append(new_folder_path)
            if not os.path.exists(new_folder_path):
                os.makedirs(new_folder_path)
    return dirs
#修正終了

#ディレクトリとファイルを判定
def filter_dir(filelists):
    dir_ = []
    for file in filelists:
        if os.path.isdir(file):
            dir_.append(file)
    return dir_

def filter_file(filelists):
    filelist = []
    for file in filelists:              #修正
        if os.path.isfile(file):
            filelist.append(file)
    return filelist



#同じディレクトリにあるものをフォルダ名とファイル名が一致したらフォルダに入れる
def insert_file(dirs,filelist):
    for target_dir in dirs:
        basename = os.path.basename(target_dir)
        basename_code = re.match(r'[0-9]{7}-[a-z0-9]{2}',basename).group()
        for file in filelist:
#修正開始
            match = re.search(r'[0-9]{7}-[a-z0-9]{2}',file)
            if match != None:
                if match.group() in basename_code:
                    shutil.move(file,target_dir)
#修正終了

if __name__ == '__main__':
    file_path_lists =glob.glob(r'ファイルパス/*')
#修正開始
    dirs = make_target_dir(file_path_lists)
    filelist =filter_file(file_path_lists)
#修正終了
    insert_file(dirs,filelist)

Comments

[oobutamaru]

ありがとうございます。
やりたかったことが出来ました。
debugをして、コードの流れを追って、自分のどこがいけなかったのかを調べてみます。

Answer 2

            shutil.move(file,target_dir)

このときに、fileとtarget_dir になにが入ってるのか、printするなどして確認してみては