0

現在、ユーザーが指定したディレクトリ内のファイルを検索できるプログラムを作成しています。tryしかし、私のステートメントとステートメントを要約するのに苦労していexceptます。私の現在のコードは次のとおりです。

if os.path.exists(file_path)!= True:
        print('\n******* Path does not exist. *******\n')       

    else:
        while True:
            aa = '''\nWhich search characteristics would you like to use? \n 1. Search by name: N \n 2. Search by extension: E \n 3. Search by size: S
                \n    Please enter your choice: '''

            answer = input(aa).strip()

            if answer not in 'NnEeSs' or answer=='':
                print('\n***** Invalid choice. *****')

            elif answer in 'Nn':
                while True:
                    try:
                        name = input ('\nEnter file name: ')
                        rr = search_by_name(name, file_path)
                        if not rr:
                            print('\n***** File not found *****\n')
                        else:
                            break

                    except WindowsError:
                        print('\n***** Oops! Access denied.*****\n')
                        continue


            elif answer in 'Ee':
                while True:
                    try:
                        ending = input ('\nEnter the file extension: ')
                        rr = search_by_extention(ending, file_path)
                        if not rr:
                            print('\n***** No File(s) found *****\n')
                        else:
                            break

                    except WindowsError:
                        print('\n***** Oops! Access denied. *****\n')
                        continue              

            elif answer in 'Ss':
                while True:
                    try:
                        size = int(input('\nPlease enter file size: '))
                        rr = search_by_size(size, file_path)
                        if not rr:
                            print('\n***** No file(s) found *****\n')
                        else:
                            break

                    except ValueError:
                        print('\n***** Enter an numeric value. *****\n')
                        continue

                    except WindowsError:
                        print('\n***** Oops! Access denied. *****\n')
                        continue

これらのケースのそれぞれまたは全体として、上記のコードを書くためのより良い凝縮された方法はありますか? 各検索関数は、コードの結果を含むリストを返します。

4

1 に答える 1

1

いくつかの提案があります。

1.インデントを改善します。

ファイルの上部には次のものがあります。

if os.path.exists(file_path)!= True:
    print('\n******* Path does not exist. *******\n')       

    else:

ステートメントには、元のelseと同じインデント値が必要ifです。

2. 改行を削除

スペースを取ることが心配なので、すべての改行を削除してください。コードが読みにくくなる可能性があるため、これはお勧めしませんが、スペースにこだわる場合は、どうぞ。11 個の改行を数えることができるので、それらを削除すると、ファイルの5分の1が削除されたことになります。

3. パイソンになる

コードが占める行が気になる場合は、将来のレビュー/編集のために、各行についてできるだけPythonicになるようにしてください。または かどうかをif x in 'Ss':確認する代わりに、次のようにします。xsSif x.lower() == 's':

4.フォーマットtry... except

あなたtry... exceptの はあなたのコードのほとんどを網羅しています。エラーが発生する可能性のある行だけに制限するようにしてください。次に例を示します。

try:
    var = int(raw_input())
    x = float(var)/100.0
    if x == 1:
        print 'YAY'
    else:
        print 'BOO!'
    for k in range(1, x):
        print 'this has looped %d times' %(k)
except ValueError:
    print 'Your input is not in a number format!'

このコードでは、入力が数値であることを確認します。ただし、それは最初の行でしか確認できず、それ以外はすべてゴミです。したがって、最初の行だけを囲み、残りはそのままにしておきます。

ノート：

から必要なコンポーネントを削除しないでくださいtry... except。何らかのエラーが発生する可能性があります。

5.whileループを縮小する

次のコードを実行する代わりに:

while True:
    try:
        size = int(input('\nPlease enter file size: '))
        rr = search_by_size(size, file_path)
        if not rr:
            print('\n***** No file(s) found *****\n')
        else:
            break

    except ValueError:
        print('\n***** Enter an numeric value. *****\n')
        continue

できるよ

size = input('\nPlease enter file size: ')
while size.isdigit() == False or not rr = search_by_size(size, file_path):
    size = input('\nPlease enter file size: ')

編集したコード:オリジナル: 59 行; 編集: 33 行:

if os.path.exists(file_path)!= True:
    print('\n******* Path does not exist. *******\n')
else:
    while True:
        aa = '''\nWhich search characteristics would you like to use? \n 1. Search by name: N \n 2. Search by extension: E \n 3. Search by size: S
            \n    Please enter your choice: '''
        answer = input(aa).strip()
        if answer not in 'NnEeSs' or answer=='':
            print('\n***** Invalid choice. *****')
        elif answer.lower() == 'n':
            name = input ('\nEnter file name: ')
            try:
                while not search_by_name(name, file_path):
                    name = input ('\nEnter file name: ')
            except WindowsError:
                print('\n***** Oops! Access denied.*****\n')
                continue
        elif answer .lower() == 'e':
            ending = input ('\nEnter the file extension: ')
            try:
                while not search_by_extention(ending, file_path):
                    ending = input ('\nEnter the file extension: ')
            except WindowsError:
                print('\n***** Oops! Access denied. *****\n')
                continue
        elif answer.lower() == 's':
            size = input('\nPlease enter file size: ')
            try:
                while size.isdigit() == False or not search_by_size(size, file_path):
                    size = input('\nPlease enter file size: ')
            except WindowsError:
                print('\n***** Oops! Access denied. *****\n')
                continue
于 2014-04-17T02:49:22.317 に答える