Как сократить строку?

Question

[RimMirK]

path = glob.glob(get_script_directory()+"\\gifs\\"+' '.join(msg.text.split(' ')[1:]).strip()+".*")[0]

вот есть такая строка. Она отвечает за поиск файла. Как можно ее упростить? Меня скорее волнует часть ' '.join(msg.text.split(' ')[1:]).strip()

Comments

[soremix]

Главный вопрос - а зачем?

Answer 1

[mayton2019]

По количеству информации на квадратный метр - эта строка вполне себе плотная и насыщенная действиями.

Чтоб делать какой-то интересный рефакторинг - надо делать не одно-строчники а функции.

Тоесть

def composePath(....) -> string:
  return ....

Вот. Функции можно покрывать тестами и таким образом гарантировать стабильность от изменений.

Answer 2

[Dmitrii]

Выигрыл в читаемости чуть-чуть и в скорости также должен быть прирост.

text = "Hello world hi mir"
new_text = ' '.join(text.split(' ')[1:]).strip()
print(f'"{new_text}"') # "world hi mir"

another_new_text = text[text.find(' ')+1:]
print(f'"{another_new_text}"') # "world hi mir"

А итоговый код можно записать так:

path = glob.glob(fr'{get_script_directory()}\gifs\{msg.text[text.find(" ")+1:]}".*"')[0])

Comments

[RimMirK]

- path = glob.glob(get_script_directory()+"\\gifs\\"+' '.join(msg.text.split(' ')[1:]).strip()+".*")[0]
+ path = glob.glob(fr'{get_script_directory()}\gifs\{msg.text[msg.text.find(" ")+1:]}.*')

почти то же самое ¯\_(ツ)_/¯

[RimMirK]

только есть проблема. Как мне теперь отлавливать NotFound?
раньше IndexError вылетал если файл не найден. а тут теперь как?

[RimMirK]

RimMirK, наверное, прошлый вариант лучше был (