File tree Expand file tree Collapse file tree
code-review-practice/solutions Expand file tree Collapse file tree Original file line number Diff line number Diff line change 4747 долго; стоит корректно ловить ` KeyboardInterrupt ` /SIGTERM, чтобы не оставить
4848 недописанный ` report.csv ` . Тут же смыкается с п.2 (незакрытые дескрипторы при выходе).
4949
50- ** Итого:** тихое строковое сложение — не заметить его на ревью стыдно ; плюс несколько
50+ ** Итого:** тихое строковое сложение легко пропустить — тем оно и коварно ; плюс несколько
5151проблем надёжности парсинга. Мораль: тип данных из внешнего источника всегда под подозрением.
Original file line number Diff line number Diff line change 2020
21215 . ** ` re.match ` уже якорит по началу** — а паттерн начинается с ` .* ` , что делает
2222 regex медленнее и допускает бэктрекинг. Плюс ` re.match(pattern, line) ` при уже
23- скомпилированном ` pattern ` стоит звать как ` pattern.match(line) ` (мелочь, но показывает,
24- что автор не разобрался ).
23+ скомпилированном ` pattern ` идиоматичнее звать как ` pattern.match(line) ` (мелочь, но
24+ характерный сигнал, что с API ` re ` стоит свериться ).
2525
26266 . Строка регэкспа ` '.* (\d{3}) \d+ ".*"' ` — не raw-строка (` r'...' ` ). Сейчас работает,
2727 но ` \d ` в обычной строке — повод для ` DeprecationWarning ` /` SyntaxWarning ` в свежих Python.
You can’t perform that action at this time.
0 commit comments