Accountant who does not count

Question

Hello I did the following program:

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

import datetime
import os
import sys

ref_arquivo = open("trata.txt","r")

for linha in ref_arquivo:
    valor = linha.split(" ")
    recebido0=0
    send0=0
    recebido1=0
    send1=1
    if (valor[0] == "0"):
        if ( valor[1] == "RECEBIDO" ):
            recebido0 = recebido0 + 1
            print ("RECEBIDO 00000")
        elif ( valor[1] == "SENDING" ):
            send0 = send0 + 1
            print ("SEND 00000")

    else:
        if (valor[0] == "1"):
            if ( valor[1] == "RECEBIDO" ):
                recebido1 = recebido1 + 1
                print ("RECEBIDO 11111")
            elif ( valor[1] == "SENDING" ):
                send1 = send1 + 1
                print ("SEND 1111")

print ("=========================")
print ('Recebidos [0] =', recebido0)
print ('Enviados [0] =', send0)
print ("=========================")
print ('Recebidos [1] =', recebido1)
print ('Enviados [1] =', send1)

ref_arquivo.close()

This program takes the line of a txt file, removes the first character (0 or 1) and makes the condition (IF). Regardless of the return of the condition, it enters a second condition to verify whether it is SEND or RECEIVED. Following the prints, it’s working, but when I check the counters, it’s not incremented. At the level of clarification, there are 9 RECEIVED, 5 from zero and 4 from 1 and 33 SENDING, and after execution the counters have

=========================
Recebidos [0] = 0
Enviados [0] = 0
=========================
Recebidos [1] = 0
Enviados [1] = 1

Can someone help me?

Answer 1

was taking a look at your code. You should perform 2 adjustments, the variables of counters should be out of the loop and should remove the \n of the value variable.

import datetime
import os
import sys

ref_arquivo = open("trata.txt","r")

recebido0=0
send0=0
recebido1=0
send1=1

for linha in ref_arquivo:
    valor = linha.split(" ")
    valor[1] = valor[1].replace('\n','')

    if (valor[0] == "0"):
        if ( valor[1] == "RECEBIDO" ):
            recebido0 = recebido0 + 1
            print ("RECEBIDO 00000")
        elif ( valor[1] == "SENDING" ):
            send0 = send0 + 1
            print ("SEND 00000")

    else:
        if (valor[0] == "1"):
            if ( valor[1] == "RECEBIDO" ):
                recebido1 = recebido1 + 1
                print ("RECEBIDO 11111")
            elif ( valor[1] == "SENDING" ):
                send1 = send1 + 1
                print ("SEND 1111")

print ("=========================")
print ('Recebidos [0] =', recebido0)
print ('Enviados [0] =', send0)
print ("=========================")
print ('Recebidos [1] =', recebido1)
print ('Enviados [1] =', send1)

ref_arquivo.close()

Variable result value of the old code:

Result of updated code:

Answer 2

As commented, the counters are being initialized at each loop with zero value, as they are "participating" in the loop.

To solve, it is enough that they are initialized only once, being outside the loop.

Code with the same corrected example:

import datetime
import os
import sys

ref_arquivo = open("trata.txt","r")

valor = linha.split(" ")
recebido0=0
recebido1=0
send1=1

for linha in ref_arquivo:
    if (valor[0] == "0"):
        if ( valor[1] == "RECEBIDO" ):
            recebido0 = recebido0 + 1
            print ("RECEBIDO 00000")
        elif ( valor[1] == "SENDING" ):
            send0 = send0 + 1
            print ("SEND 00000")

    else:
        if (valor[0] == "1"):
            if ( valor[1] == "RECEBIDO" ):
                recebido1 = recebido1 + 1
                print ("RECEBIDO 11111")
            elif ( valor[1] == "SENDING" ):
                send1 = send1 + 1
                print ("SEND 1111")

print ("=========================")
print ('Recebidos [0] =', recebido0)
print ('Enviados [0] =', send0)
print ("=========================")
print ('Recebidos [1] =', recebido1)
print ('Enviados [1] =', send1)

ref_arquivo.close()