明確にしてください

Question

ハローのpythonに新しいイムやドキュメント上で読んでされており、ここではargparse

とのトラブルを抱えていますが、私はシンプル志向の人のために、それはonsimple説明してください 感謝しようとしていますコードですこれでイムではなく、多くのELIFの駅のすべてのオプションをコーディングするためのより良い（以上ニシキヘビ）の方法があるが、事前に

#!/usr/bin/env python 
import argparse 
from string import ascii_lowercase 
from string import ascii_uppercase 
from string import digits 
from string import punctuation 

def options(): 
    parser = argparse.ArgumentParser(description="Create a list of characters") 
    parser.add_argument('-c:') 
    if (args.c):# == "-c:": 
     if "-c" + ":" + "caps": 
      charset = ascii_uppercase 
     elif "-c:" + "small": 
      charset = ascii_lowercase 
     elif "-c:" + "digits": 
      charset = digits 
     elif "-c:" + "punc": 
      charset = punctuation + " " 
     elif "-c:" + "space": 
      charset = " " 
     elif "-c:" + "all": 
      charset = ascii_lowercase + ascii_uppercase + digits + punctuation + space 
     else: 
      print("when using -c you must include an option! (caps, small, digits, punc, space, all)") 

def filename(): 
    filename = open('C:\\Users\\MSec\\Desktop\\WordLists\\'+sys.argv[1], 'w') 
    return 

if __name__ == "__main__": 
    filename() 
    options() 

再びイム新しいdidntのはかなり

は私が推測する文書を理解しますテンション？

ありがとうございました

Answer 1

すべてのifステートメントは常にtrueです。

if "-c" + ":" + "caps": 

手段

if "-c:caps": 

文字列 "-c：キャップは" 非falseです。類似のif文の残りの部分についても同様です。

おそらく

if args.c == 'caps': 

を意味そして、いや、elifsの長いリストそれは良い方法がしばしばありません。他の方法もありますが、これは最も簡単に書いて理解することができ、したがって最高です。

CHAR_OPTIONS = {'caps': ascii_uppercase, 
       'small': ascii_lowercase, 
       etc.... 
       } 

そしてそうのようにそれを使用する：この場合

、しかし、あなたは辞書作ることができます理にかなって

charset = CHAR_OPTIONS[args.c]