Python： 'count'を使って行数を制限しようとすると "print"が出力されます

Question

私はシステムディレクトリを歩き回り、そのディレクトリ内のファイルサイズを取得するスクリプトを持っています。ファイルサイズ（降順）でソートし、2つの引数をとります。最初のargはシステムパスで、2番目のargは出力をその整数カウントに制限する整数です。

私は印刷出力を制限するためにwhileループを使用していますが、そのループは無限ループに陥っています....明らかに、何かが間違っていますが、私はそれを見ません。

#!/usr/bin/python 

import sys 
import os 

#Grab arguments 
mydir = sys.argv[1] 
mycnt = sys.argv[2] 

print mydir 
print mycnt 
#set base dir 
root_path = mydir 


fileSize = 0 
rootdir = mydir 
filedict = dict() 
count = 0 

for root, dirs, files in os.walk(rootdir): 
    for file in files: 
     filepath = os.path.join(root,file) 
     filesize = fileSize + os.path.getsize(filepath) 
     filedict[filepath] = filesize 


for key, value in sorted(filedict.iteritems(), key=lambda (key,val): (val,key), reverse=True): 
    while (count <= mycnt): 
     print "(%8s)" "  (%-8s)" % (value, key) 
     count += 1 
    else: 
     sys.exit() 

Answer 1

mycntが文字列ではなく、（sys.argvから直接読んだとき、それはある）の整数である場合は、あなたのループが終了することはありません。

Answer 2

whileの代わりにifが必要です。

あなたはcount <= mycnt場合、プログラムを終了します。

Answer 3

else句をループのwhileに追加しました。 whileループが実行されない場合にのみ実行されます。

あなたwhileループがmycnt過去count増加すると終了し、forループのanoter反復が実行されます。あなたはただそれに気づかないのです。あなたのforループが長くかかるかもしれません。

Answer 4

FWIW、reprモジュールには、出力行数を制限しながらリストを表示するツールがあります。

Answer 5

Larry Lustig has already hit the nail on the headについては、mycntですが、上限に達したかどうかの確認も間違っています（Hamish points outなど）。

if (count <= mycnt):ではなく、[:maximum]のようにsliceを使用して、count変数を削除する必要があります。変数について言えば、より良い名前の変数の恩恵を受けることをお勧めします。 ビズ：

#!/usr/bin/env python 

import sys 
import os 

rootdir = sys.argv[1] 
maximum = int(sys.argv[2]) 

print 'Printing the biggest ', maximum, 'files in', rootdir, '...' 

filedict = {} 
for root, _, files in os.walk(rootdir): 
    for filename in files: 
     filepath = os.path.join(root, filename) 
     filesize = os.path.getsize(filepath) 
     filedict[filepath] = filesize 

sorted_by_size = sorted(filedict.iteritems(), key=lambda(path, size): (size, path), reverse=True) 
for path, size in sorted_by_size[:maximum]: 
    print "(%8s)  (%-8s)" % (size, path)