ディレクトリ内のトラフファイルを反復する

Question

新しいファイルが変更されて画面に出力されるまで、実行中に最後に変更されたファイルを取得して印刷することになっていました。 TypeError：Unicodeに強制する：必要な文字列またはバッファー、intが見つかりました。

import os, sys, re, time 
from datetime import date, timedelta, datetime 
from time import localtime 

files = os.listdir('dir_path') 

files = [f for f in files if re.search('.csv', f, re.I)] 
files.sort 

d = datetime.now() - timedelta(days = 30) 
d = d.timetuple() 

oldfiles = 0 
newfiles = 0 
for file in files: 
    filetimesecs = os.path.getmtime('dir_path' + file) 
    filetime = localtime(filetimesecs) 

if filetime < d: 
    oldfiles += 1 
if filetime > d: 
    newfiles += open(files, 'r') 
    for k in newfiles: 
     sys.stderr.write(k) 
    while True: 
     time.sleep(2) 
     print"new: %s" % newfiles 

Answer 1

コード内には、奇妙に見えるものがいくつかあります。例：

files = [f for f in files if re.search('.csv', f, re.I)] 

これは本当に必要なものですか？このドットは、改行を除く任意の文字に一致します。あなたはおそらく、望ましい動作を得るためにそれをエスケープする必要がありますか、代わりにf.endswith('.csv')でテストすることができます。

files.sort 

これは関数呼び出しではありません。ディレクトリ・パスとファイル名を結合するためにos.path.join()を使用することをお勧めしfiles.sort()

filetimesecs = os.path.getmtime('dir_path' + file) 

でなければなりません。

newfiles += open(files, 'r') 

files変数はリストであり文字列ではありません。この行に誤りはありませんか？

Answer 2

私たちはいくつかのタイプのエラーがあるようですが、私はこれをできる限り上にクリアしようとします。

oldfiles = 0 
# if newfiles is an integer, I'm not sure that it will be 
# as helpful to you, I think you'd really rather have a list 
newfiles = list() 
for file in files: 
    filetimesecs = os.path.getmtime('C:/tmp/' + file) 
    filetime = localtime(filetimesecs) 
    if filetime < d: 
     oldfiles += 1 
    if filetime > d: 
     newfiles.append(open('C:/tmp/' + file, 'r')) 
     for k in newfiles: 
      # read the entire file and write it to standard error 
      sys.stderr.write(k.read()) 

# I believe you want the while out here, after everything is complete 
# it would seem that you would want the newfiles loop within the first 
# for loop 
while True: 
    time.sleep(2) 
    print "new: %s" % newfiles 

希望の値です。