2010-11-06 10 views
8

Piszę serwer zapytań narzędzia, a ja mam trochę kodu do analizowania argumentów na samej górze:Python argparse: nargs + lub * w zależności od wcześniejszego argumentu

# Parse arguments 
p = argparse.ArgumentParser() 
g = p.add_mutually_exclusive_group(required=True) 
g.add_argument('--odam', dest='query_type', action='store_const', 
     const='odam', help="Odamex Master query.") 
g.add_argument('--odas', dest='query_type', action='store_const', 
     const='odas', help="Odamex Server query.") 
p.add_argument('address', nargs='*') 
args = p.parse_args() 

# Default master server arguments. 
if args.query_type == 'odam' and not args.address: 
    args.address = [ 
      'master1.odamex.net:15000', 
      'master2.odamex.net:15000', 
      ] 

# If we don't have any addresses by now, we can't go on. 
if not args.address: 
    print "If you are making a server query, you must pass an address." 
    sys.exit(1) 

Czy istnieje ładniejszy sposób to zrobić, najlepiej wszystko w parserze? Ten ostatni błąd wygląda trochę nie na miejscu i byłoby miło, gdybym mógł zrobić nargs na adres, zależnie od tego, czy --odam czy --- odas zostanie przekazany. Mógłbym utworzyć parsektor, ale to sprawiłoby, że pomoc wyglądałaby trochę dziwnie, ponieważ zostawiłaby część adresu polecenia.

Odpowiedz

12

Można to zrobić ze zwyczajem argparse.Action:

import argparse 
import sys 

class AddressAction(argparse.Action): 
    def __call__(self, parser, args, values, option = None): 
     args.address=values 
     if args.query_type=='odam' and not args.address: 
      args.address=[ 
       'master1.odamex.net:15000', 
       'master2.odamex.net:15000', 
       ]   
     if not args.address: 
      parser.error("If you are making a server query, you must pass an address.") 

p = argparse.ArgumentParser() 
g = p.add_mutually_exclusive_group(required=True) 
g.add_argument('--odam', dest='query_type', action='store_const', 
     const='odam', help="Odamex Master query.") 
g.add_argument('--odas', dest='query_type', action='store_const', 
     const='odas', help="Odamex Server query.") 
p.add_argument('address', nargs='*', action=AddressAction) 
args = p.parse_args() 

daje

% test.py --odas 
If you are making a server query, you must pass an address. 
% test.py --odam 
Namespace(address=['master1.odamex.net:15000', 'master2.odamex.net:15000'], query_type='odam') 
% test.py --odam 1 2 3 
Namespace(address=['1', '2', '3'], query_type='odam') 
+0

Dokładnie to, czego szukasz! – AlexMax

+0

Prawdopodobnie bardziej sensowne wydaje się używanie [błędu parsera] (http://docs.python.org/library/argparse.html#argparse.ArgumentParser.error) zamiast 'sys.exit()' – Christoph

+0

@ Christophoph: True; dzięki! – unutbu

Powiązane problemy