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

Feb. 18th, 2011

[info]igorpashev

12:25 pm - Спасибо :-)

1. WTF is p?
2. закомментированный код - в топку
3. PEP8 требует каждого import'а на отдельной строке.

4. ussd() и balance() - одно и то же.
Семантика разная

5. self.write('AT+CUSD=1,' + code + ',15') - плюсы идут в жопу, ибо каждый плюс создает новый объект "строка". Use % format.
Алсо, это Питон 3

6. ln - говно название для переменной. ln - натуральный логарифм.
Алсо, у буржуев log

7. if ln - return чего-то. А в противном случае - return *что*?
8. Modem.open() не делает ничего, кроме io.open() - переместить в __init__. Рефакторить позже, по мере нужды.

9. Почему sleep(0.1), а не 0.09 и не 0.11?
Пока работает и без задержки

10. Приватные методы сделать типа _write() - нефиг, чтобы все кто попало их вызывал.


#!/usr/bin/env python3
#coding: utf8

import io

class Modem:
    def __init__(self, dev='/dev/ttyUSB1'):
        self.stream = io.open(dev, 'w+b', 0)

    def __write(self, s):
        self.stream.write(str.encode('{}\r\n'.format(s)))

    def close(self):
        self.stream.close()

    def ussd(self, code):
        self.__write('AT+CPBS="SM"')
        self.__write('AT+CPMS="SM","SM",""')
        self.__write('AT+ZSNT=0,0,2')
        self.__write('AT+CUSD=1,{},15'.format(code))

        for l in self.stream:
            if l.startswith(b'+CUSD'):
                msg = l[10:l.rfind(b'"')].decode('ascii')
                return bytes.fromhex(msg).decode('utf-16-be')
        return ''

    def balance(self, code='*100#'):
        return self.ussd(code)

modem = Modem()
print(modem.balance())

Read Comments

Reply:

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