Некоторые замечания по code review

[akhtyamovpavel]

• [x] https://github.com/MelnikovIgor1/formal_languages/blob/master/machine.py#L14 - EPSILON

• [x] https://github.com/MelnikovIgor1/formal_languages/blob/master/machine.py#L90 - в Python 3.6+ есть аннотация типов

• [x] https://github.com/MelnikovIgor1/formal_languages/blob/master/machine.py#L93 - это константа или нет? Если константа, то стоит ее сделать большими буквами

• [x] https://github.com/MelnikovIgor1/formal_languages/blob/master/machine.py#L261 - лучше пользоваться subprocess, os.system небезопасный

• [x] https://github.com/MelnikovIgor1/formal_languages/blob/master/machine.py#L312-L321 - аналогично

• [ ] https://github.com/MelnikovIgor1/formal_languages/blob/master/machine.py#L341 - может быть, лучше использовать namedtuple?

• [x] https://github.com/MelnikovIgor1/formal_languages/blob/master/machine.py#L617 - лучше определять тип исключения

• [x] .pyc-файл стоит вынести в .gitignore

В остальном - все отлично! Жду исправлений, и можно использовать код!

[MelnikovIgor1]

Почти все поправил. В чем мотивация использовать namedtuple вместо tuple? tuple же вроде проще и явно быстрее.

[akhtyamovpavel]

Читабельность в разы хуже. namedtuple - структуры на минималках

On Thu, 1 Oct 2020 at 10:31, MelnikovIgor1 notifications@github.com wrote:

Почти все поправил. В чем мотивация использовать namedtuple вместо tuple? tuple же вроде проще и явно быстрее.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/MelnikovIgor1/formal_languages/issues/1#issuecomment-701947331, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABI6JMENHB7A3EHA2QNOF7TSIQV4TANCNFSM4R7NY7XQ .

[MelnikovIgor1]

Поправил