Почему не проходит проверка значений?

Question

[Denis]

Добрый день!

Собственно, в коде все описано. Основная проблема в том, что значения "а" и "с" некорректны. Мне кажется, что проблема в проверке.
Подскажите, пожалуйста: что делать и как это исправить?

path = 'D:\\For_Denis\\Projects\\nginx_log_parser\\nginx.txt'
path_1 = 'D:\\For_Denis\\Projects\\nginx_log_parser\\nginx_1.txt'
log = open(path, 'r')
logs = open(path_1, 'r')

lines = 0 

for line in log:
    lines = lines + 1 

print(lines) #выдает верное значение
    
a = 0 #количество POST
c = 0 #количество GET
b = 0 #количество пройденных строк

while b < lines:
    check = str(logs.readline())
    if 'POST' in check:
        a = a + 1
        b = b + 1 
    elif 'GET' in check:
        c = c + 1
        b = b + 1 
    else:
        break
 
print(a, c, b, sep =',') #выдает "1, 4, 5". Скорее всего проблема в проверке им значений

Answer 1

[FeNUMe]

У вас первый цикл for, вычитывает весь файл до конца и в log.readline() уже пусто.

path = 'D:\\For_Denis\\Projects\\nginx_log_parser\\nginx.txt'

a = 0 #количество GET
c = 0 #количество POST
b = 0 #количество пройденных строк

with open(path) as f:
    for line in f:
        b += 1
        if 'GET' in line:
            a +=1
        elif 'POST' in line:
            c +=1
        else:
            break
        
print(a,",", c)

Comments

[Denis]

Большое спасибо!
Если я сделаю такой же файл, но с другим именем, пропишу до него путь и тд., то мой код должен заработать?

[Denis]

FeNUMe, проверил Ваш код.
Некорректно работает, к сожалению :(

[Denis]

подредачил до такого состояния и все равно некорректно работает:

path = 'D:\\For_Denis\\Projects\\nginx_log_parser\\nginx.txt'
path_1 = 'D:\\For_Denis\\Projects\\nginx_log_parser\\nginx_1.txt'
log = open(path, 'r')
logs = open(path_1, 'r')

lines = 0 

for line in log:
    lines = lines + 1 

print(lines) #выдает верное значение

check = str(logs.readline())
    
a = 0 #количество GET
c = 0 #количество POST
b = 0 #количество пройденных строк

while b < lines:
    if check.find("POST"):
        a = a + 1
        b = b + 1 
    elif check.find("GET"):
        c = c + 1
        b = b + 1 
    else:
        break
 
print(a, c, b, sep =',') #выдает "8, 0, 8". Скорее всего проблема в проверке им значений

[FeNUMe]

Выложите ваш nginx.txt, чтобы можно было увидеть в чем проблема. Со стандартным access.log nginxа мой код работает.
Хотите чтобы заработал ваш изначальный код - добавьте log.seek(0) перед check = str(logs.readline())

[Denis]

Нашел проблему. В некоторых строках есть значения помимо 'GET' и 'POST'.
На этом работа программы завершалась, поэтому выдавались некорректные значения.

Answer 2

[Denis]

Вот как была решена задача:

path = 'D:\\For_Denis\\Projects\\nginx_log_parser\\big_log.txt'
path_1 = 'D:\\For_Denis\\Projects\\nginx_log_parser\\big_log_1.txt'
log = open(path, 'r')
logs = open(path_1, 'r')

lines = 0 

for line in log:
    lines = lines + 1 
    
a = 0 #количество POST
b = 0 #количество пройденных строк

while b < lines:
    check = str(logs.readline())
    if 'GET' in check: 
        if '200' in check:
            a += 1
            b += 1
        else:
            b += 1       
    else:
        b += 1
    
print(a)