Игорь Пашев - Скрипт для USB модема

Feb. 17th, 2011

12:38 pm - Скрипт для USB модема

Previous Entry Add to Memories Tell A Friend Next Entry
(4 комментария | Оставить комментарий)

Comments:

[User Picture]
From:[info]igorpashev
Date:February 18th, 2011 - 12:25 pm

Спасибо :-)

(Link)
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())

(Reply to this) (Parent) (Thread)
[User Picture]
From:[info]igorpashev
Date:February 18th, 2011 - 05:23 pm

Re: Спасибо :-)

(Link)
Асло по седьмому пункту:

7. if ln - return чего-то. А в противном случае - return *что*?

противного случая на самом деле не будет, ибо читается поток
(Reply to this) (Parent)