このPythonコードがランダムに動作する理由を理解する

Question

私は、サウンドファイルからメタデータを取得し、目的の値を持つ文字列を作成する小さなスクリプトをコーディングしています。私は何か間違っていることを知っていますが、私はなぜそれがわからないのですが、おそらく私がifを繰り返す方法です。私はコードを実行すると：

import os, mutagen 

XPATH= "/home/xavier/Code/autotube/tree/def" 

DPATH="/home/xavier/Code/autotube/tree/down" 


def get_meta(): 
    for dirpath, directories,files in os.walk(XPATH): 
     for sound_file in files : 
      if sound_file.endswith('.flac'): 
       from mutagen.flac import FLAC 
       metadata = mutagen.flac.Open(os.path.join(dirpath,sound_file)) 
       for (key, value) in metadata.items(): 
        #print (key,value) 
        if key.startswith('date'): 
         date = value 
         print(date[0]) 

        if key.startswith('artist'): 
         artist = value 
         #print(artist[0]) 
        if key.startswith('album'): 
         album = value 
         #print(album[0]) 
        if key.startswith('title'): 
         title = value 
         #print(title[0]) 
         build_name(artist,album,title) # UnboundLocalError gets raised here 


def build_name(artist,album,title): 
    print(artist[0],album[0],title[0]) 

を私はランダムに、望ましい結果やエラーが表示されます。

結果：

1967 Ravi Shankar & Yehudi Menuhin West Meets East Raga: Puriya Kalyan 

がERROR：

Traceback (most recent call last): 
    File "<stdin>", line 39, in <module> 
    File "<stdin>", line 31, in get_meta 
    build_name(artist,album,title) 
UnboundLocalError: local variable 'album' referenced before assignment 

Answer 1

を"title"が前に来る場合メタデータに"album"がある場合、albumは決して初期化されません。 "album"がまったく存在しないことがあります。

トラックは以前"album"が"album"を定義していない次のトラックが前のトラックの値を使用します、次に定義されていた場合は、各トラックのalbumの値を空白ではないそうであるように。

各トラックに空白の値を付けます（それが妥当である場合）。 build_nameで値を見ると

は、文字列のリストなので、デフォルトでは['']次のようになります。

for sound_file in files: 
    artist = album = title = [''] 

メタデータは、故障している場合しかし、あなたはまだbuild_nameを呼び出す前の値を取得することはできません。

あなたはループの外build_name(artist, album, title)を移動する必要があります。

for (key, value) in metadata.items(): 
    ... # searching metadata 
build_name(artist, album, title)