Игорь Пашев - Post a comment

Feb. 18th, 2011

(Anonymous)

11:08 am

Говнокод.
1. WTF is p?
2. закомментированный код - в топку
3. PEP8 требует каждого import'а на отдельной строке.
4. ussd() и balance() - одно и то же.
5. self.write('AT+CUSD=1,' + code + ',15') - плюсы идут в жопу, ибо каждый плюс создает новый объект "строка". Use % format.
6. ln - говно название для переменной. ln - натуральный логарифм.
7. if ln - return чего-то. А в противном случае - return *что*?
8. Modem.open() не делает ничего, кроме io.open() - переместить в __init__. Рефакторить позже, по мере нужды.
9. Почему sleep(0.1), а не 0.09 и не 0.11?
10. Приватные методы сделать типа _write() - нефиг, чтобы все кто попало их вызывал.

Read Comments

Reply:

From:
(will be screened)
Identity URL: 
имя пользователя:    
Вы должны предварительно войти в LiveJournal.com
 
E-mail для ответов: 
Вы сможете оставлять комментарии, даже если не введете e-mail.
Но вы не сможете получать уведомления об ответах на ваши комментарии!
Внимание: на указанный адрес будет выслано подтверждение.
Username:
Password:
Subject:
No HTML allowed in subject
Message: