проверяй длину

И в самом деле. А sys.argv это строка?

>>> type(sys.argv)
<class 'list'>

Лист строк.

Почему-то даже при нулевой длине лезет в ветку ненулевой длины.

У argv не может быть нулевой длины, argv[0] - имя скрипта.

Что-то как с листами не получается с ним работать.

Блин, точно ведь. Опять туплю.

Может, есть способ поизящнее, чем проверять длину?

А чем способ берсерка не изящен?

if len(sys.argv) > 1:

Может, есть способ поизящнее, чем проверять длину?

Судя по тому, что именно ты там проверяешь, тебе лучше argparse заюзать, а не городить велосипед. Вот это и будет самый изящный способ.

Может, есть способ поизящнее, чем проверять длину?

Лови исключение! :)

лучше argparse

поддерживаю

предлагаю не останавливаться на этом и реализовать на исключениях всю логику

def izyashniy_sposob(lst, idx):
    try:
        return lst[idx]
    except IndexError:
        return None

if izyashniy_sposob(sys.argv, 1) == 'add':

Если не нравится argparse, посмотри https://github.com/docopt/docopt .

А так вообще у тебя первая проверка if sys.argv[1] лишняя по сути. Нет, юзер, конечно, может передать "", но проверяешь ты вряд ли это, а узнать, есть ли там вообще аргумент или нет — if len(sys.argv) > 1:. Вместо этого, а не вместе.

Больше говнокода в тред!

if "add" in sys.argv[:2]:

if sys.argv[1:]: blabla

или

for arg in sys.argv[1:]: do_something_with(arg)

Неохота библиотечные парсеры использовать. Напрасно в python нет case.

Может, есть способ поизящнее, чем проверять длину?

docopt, argparse, etc.

Ну или так, лол:

if (sys.argv[1:2]+[None])[0]:
    if sys.argv[1] == 'add':
        ...

https://github.com/neithere/argh вот ниче ещё вроде

И в стандартной библиотеке есть getopt

предлагаю не останавливаться на этом и реализовать на исключениях всю логику

Ну итераторы же на них и реализовали.

И даже вот такое http://code.activestate.com/recipes/474088/

Раз уж тут советуют библиотеки, то рекомендую попробовать ещё и cleo. Только это не парсер аргументов, а целый фреймворк для создания консольных приложений. Inspired by Symfony/Console.

Может, есть способ поизящнее, чем проверять длину?

да, argparse

try: except IndexError:

Я делаю так во всяких мелких скриптах, если может быть только один аргумент.

if sys.argv[-1] == 'add':
    ...

не занимайся онанизмом и других на это не подстрекай. optparse

Скорее всего у него после add идет имя файла или ещё что-нибудь такое.

Ну тогда argparse или docopt, иначе ручной парсинг быстро превратится в боль при добавлении еще пары опций.

except IndexError же

А разве добавление пары опций в ручной парсинг будет сложнее, чем добавление пары опций через библиотечный парсер?

Почему так нельзя сделать?

потому что питон. Хочешь красоты - мирись с сложностью:

...
   if $ARGV[0] eq 'add';

Зависит от количества опций и наличия [необязательных] аргументов, флагов (в короткой и длинной форме). Уверен, ты не захочешь парсить вручную все варианты:

script.py some_file.txt
script.py -a SOME_NAME --some_long_arg some_file.txt
script.py -a SOME_NAME -d -g optional_arg some_file.txt
script.py -gd some_file.txt
... etc

Без фабрик это решение будет неполным.

экспешоном дорого такие вещи проверять, да и профит с этого не совсем понятно какой.

from __future__ import print_function

import timeit
import sys


def foo_exc(*args):
    try:
        if args[0] == "add":
            print("lol", file=sys.stderr)
        else:
            print("unknown command", file=sys.stderr)
    except IndexError:
        print("no args given", file=sys.stderr)

def foo_noexc(*args):
    if args:
        if args[0] == "add":
            print("lol", file=sys.stderr)
        else:
            print("unknown command", file=sys.stderr)
    else:
        print("no args given", file=sys.stderr)

if "__main__" == __name__:
    print(timeit.timeit("foo_exc('add')", setup="from __main__ import foo_exc"))
    print(timeit.timeit("foo_exc()", setup="from __main__ import foo_exc"))
    print(timeit.timeit("foo_noexc('add')", setup="from __main__ import foo_noexc"))
    print(timeit.timeit("foo_noexc()", setup="from __main__ import foo_noexc"))

(17:16) novel@yrly:~ %> python args.py  2>/dev/null
2.69668388367
5.61795711517
2.69180679321
2.34060001373
(17:17) novel@yrly:~ %> 

Ошибка, если sys.argv[1] нет:

кстати, раз уж предлагают стремные решения, и я предложу (это не значит что им стоит пользоваться)

if (sys.argv + [None] * index)[index] == 'add'

А ты попробуй оба варианта и сравни.

Argparse не предлагать, он мне не нравится.

Тогда тебе лучше не писать на питоне.

Как тогда нужно это делать?

см. выше.

Тогда тебе лучше не писать на питоне

А я вот возьму и буду.

А я что, запрещаю? Это так, рекомендация. Только вот код, основанный на «мне так не нравится», а не на каких-то веских причинах, в серьезные проекты принимать не будут :)

Причём веские причины «это им в серьезном проекте так нравится».

http://click.pocoo.org/5/