Python-сообщество

Shaman
лучше учесть все случаи

Отредактировано py.user.next (Авг. 22, 2015 20:22:47)

ну пока поменять алгоритм создания я не могу - приходится юзать то что есть

py.user.next
Функция данной регулярки заключается в выборе полей, она не должна заниматься чисткой данных. Чисткой данных должен заниматься специальный участок, чья функция именно в этом и заключается.

Shaman
эта регулярка вообще ничего не чистит

py.user.next
Как не чистит, учитывает наличие \r, которого не должно быть.

py.user.next
совпадение с одним симоволом будет работать быстрее на каждой записи.

Shaman
На сколько микропроцентов?

[guest@localhost ~]$ python3 -mtimeit -s "import re" "re.findall(r'\n', 'abc\n' * 10)"
100000 loops, best of 3: 3.65 usec per loop
[guest@localhost ~]$ python3 -mtimeit -s "import re" "re.findall(r'\s', 'abc\n' * 10)"
100000 loops, best of 3: 5.59 usec per loop
[guest@localhost ~]$ python3 -mtimeit -s "import re" "re.findall(r'\s+', 'abc\n' * 10)"
100000 loops, best of 3: 6.41 usec per loop
[guest@localhost ~]$

py.user.next
Тут в два раза, но это самый простой случай. А там полей несколько, можно и замерить для там.

with open ("somefile.txt", "r") as myfile:
    data=myfile.read()
    data = re.sub('\r','',data)
    data = re.sub('\n\n','\n',data)

Отредактировано Shaman (Авг. 23, 2015 09:46:24)

Shaman
Для полноты так же нужно посчитать затраты на “(\d+)” - вдруг от них тоже стоит отказаться?

Shaman
А теперь то же самое для

py.user.next
С чего это вдруг? \d+ - необходимое условие, иначе как цифры накапливать?

py.user.next
Да надо просто файл через sed снаружи прогнать, если невозможно исправить алгоритм, который порождает его (со смешанными концами строк).

py.user.next
Кстати, если есть этот костыль (re.sub()), то твоя регулярка не только будет дольше искать, но и работать впустую.

py.user.next
Ты же привязал таким образом регулярку к побочному содержимому, которое ещё может быть удалено каким-нибудь способом.

«Если у вас была одна проблема, и вы решили использовать регулярные выражения, то теперь у вас есть две проблемы»

Конечные автоматы гораздо быстрее и надежнее

out =[]
pos = -1
with open('d:\\info.txt') as f:
    for line in f:
        if line[0] == '-':
            pos = 0
        elif pos == 0:
            mt = int(line.split(':')[1].strip().split(' ')[0])
            pos = 1
        elif pos == 1:
            mf = int(line.split(':')[1].strip().split(' ')[0])
            pos = 2
        elif pos == 2:
            buf = int(line.split(':')[1].strip().split(' ')[0])
            pos = -1
            out.append({'MemTotal':mt,'MemFree':mf,'Buffers':buf})