Python-сообщество

Rodegast
Например если я захочу проверять начало файла на “server”, то достаточно дописать:

  
s[6:] if s[:6] == "server" else "" [:-3]

А в случае одного среза эта возможность становится не очевидной.

  
>>> s = 'server123.sh'
>>> s[6:] if s[:6] == "server" else "" [:-3]
'123.sh'
>>>

houey
Если кто знает С++ - я хочу написать полный аналог данной проги на С++, но уже на Python.

  
>>> lst = ['server1.sh', 'server3.sh', 'client1.sh', 'server2.sh', '1.sh', '2.sh']
>>> 
>>> out = [i for i in lst if i.startswith('server')]
>>> out
['server1.sh', 'server3.sh', 'server2.sh']
>>>

  
>>> def filt(seq, pat):
...     return [i for i in seq if i.startswith(pat)]
... 
>>> lst = ['server1.sh', 'server3.sh', 'client1.sh', 'server2.sh', '1.sh', '2.sh']
>>> 
>>> out = filt(lst, 'server')
>>> out
['server1.sh', 'server3.sh', 'server2.sh']
>>>

  
>>> def filt(seq, pat):
...     return [i for i in seq if not i.startswith(pat)]
... 
>>> lst = ['server1.sh', 'server3.sh', 'client1.sh', 'server2.sh', '1.sh', '2.sh']
>>> 
>>> out = filt(lst, 'server')
>>> out
['client1.sh', '1.sh', '2.sh']
>>>

Отредактировано py.user.next (Май 31, 2017 03:14:20)

В итоге пришел к такому варианту:

 import re
def Match(item, pattern):
  result = item.find(pattern)
  return result >= 0
        
        
def KeyFunction(item):
  pattern = r'(\d+)\.sh$'
  result = re.search(pattern, item);
  if result is None:
    return 0
  return int(result.group(1))
  
  
items = ['server1.sh', 'server3.sh', 'client1.sh', 'client2.sh', 'server2.sh', 'server.sh', 'client.sh']
clients = [it for it in items if Match(it, "client")]
servers = [it for it in items if Match(it, "server")]
clients = sorted(clients, key=KeyFunction)
servers = sorted(servers, key=KeyFunction)
print (clients)
print (servers)

py.user.next
Спасибо большое за ваши советы!
Отписался в теме, не посмотрев новые ответы, в итоге пришел примерно к тому, о чём Вы говорили!

Чуть позже более внимательно перечитаю ваш пост, как появится свободная минутка спасибо!

Отредактировано houey (Май 30, 2017 08:50:46)

> Потом сходу (за секунду) непонятно, что это за выражение;

Как раз для этого и существуют комментарии.

> Перед отправкой ты его не проверил, он выдаёт совсем не то, что должен.

Да, точно скобочки забыл Но на суть это не влияет, это была просто демонстрация того что между этими срезами можно ещё что-то вставлять.

Rodegast
Как раз для этого и существуют комментарии.

Rodegast
это была просто демонстрация того что между этими срезами можно ещё что-то вставлять

houey
Буду рад услышать советы по улучшению программы.

  
def Match(item, pattern):
  result = item.find(pattern)
  return result >= 0

  
def match(item, pattern):
    return pattern in item

Отредактировано py.user.next (Май 30, 2017 15:48:33)

> Короче, алгоритм комментируется обычно, а не код

Комментируется то что может быть не понятным или не очевидным. По этому если что-то в коде может быть не очевидно, то это может быть прокомментировано. Это норма.

> Демонстрация-то понятна, но ошибку-то ты допустил.

Это меня отвлекли, вот скобок я и не поставил. И не в стиле проблема. Вот например у тебя тремя сообщениями выше есть такой код:

 >>> def filt(seq, pat):
...     return [i for i in lst if i.startswith(pat)]

Rodegast
Это ошибка, хотя код и выглядит вполне разборчивым.

  
>>> def filt(seq, pat):
...     return [i for i in lst if i.startswith(pat)]
... 
>>> def f():
...     lst = ['server1.sh', 'server3.sh', 'client1.sh', 'server2.sh', '1.sh', '2.sh']
...     out = filt(lst, 'server')
...     return out
... 
>>> f()
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "<stdin>", line 3, in f
  File "<stdin>", line 2, in filt
NameError: global name 'lst' is not defined
>>>

Rodegast
По этому если что-то в коде может быть не очевидно, то это может быть прокомментировано.

Отредактировано py.user.next (Май 31, 2017 03:24:54)

> Это очень слабая ошибка, потому что её запуск пропустил. Такое бывает раз на миллион…В реальном коде она бы сразу всплыла, потому что там бы lst не было в глобальном пространстве…Вот использование кода с моей ошибкой

Не оправдывайся

> Иными словами, если у тебя код надо описывать комментариями, значит он неясный.

Для меня ясный, для кого-то не очень, а кто-то любой без комментариев не понимает. Это демагогия.

Rodegast
Для меня ясный, для кого-то не очень, а кто-то любой без комментариев не понимает. Это демагогия.

Отредактировано py.user.next (Май 31, 2017 11:28:44)