Discussion :

[dominiquelola]

Bonjour,

lors de mon calcul de moyenne, je n'arrive pas � exclure les z�ros de mon fichiers.
voici mon code, je ne trouve pas mon erreur.

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
 
import csv
 
with open('DataToAnalyze.csv') as csv_file:
	reader = csv.reader(csv_file)
	average = 0
	sum = 0
	row_count = 0
		for row in reader:
		for column in row:
			n=float(column)
					if n != 0:
						sum += n
						row_count += 1
	average = sum / row_count
	print('La moyenne est :', average)
	csv_file.close()

merci d'avance pour une aide

[Aelurus_]

Bonjour,

D�j� reprend ton indentation car elle n'est pas fonctionnelle, c'est surprenant que tu n'aies pas de message d'erreur.

Puis essaye avec �a :

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
import csv
 
file = 'DataToAnalyze.csv' # 0;10;25;10;25
 
with open(file, 'r') as file:
    reader = csv.reader(file, delimiter=';')
    datas = [ligne for ligne in reader]
    l = [i for i in datas[0] if i != '0']
    somme = 0
    for data in datas[0]:
        somme = somme + float(data)
    moyenne = somme/len(datas[0])
    file.close()
print('la moyenne est : ',moyenne)

[marco056]

A part l'indentation, je ne vois pas d'erreur.
average = 0 ne sert � rien au d�but.

[VinsS]

Salut,

Si on comprend bien les items de valeur z�ro sont exclus du calcul de la moyenne.
Dans ce cas ton code peut �tre simplifi� comme ceci:

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

1
2
3
4
5
6
7
8
9
10
11
12
13
 
data = [[12, 50, 0, 1],
        [5, 5, 10, 0],
        [8, 0, 0, 10]]
sum_ = 0
count = 0
for row in data:
    for col in row:
        sum_ += float(col)
        count += col > 0
 
average = sum_ / count
print("Sum: %s, count: %s, average: %s" %(sum_, count, average))

[Sve@r]

Bonjour

Dans ce cas ton code peut �tre simplifi� comme ceci:

Es-tu sur qu'additionner "0" sans le tester soit une optimisation ? Surtout que juste en dessous tu testes quand-m�me si cette valeur vaut 0.
Ne vaut-il donc pas mieux la tester (puisque de toute fa�on le test reste obligatoire) et seulement si la valeur correspond alors l'additionner ? On �limine ainsi une addition parfois inutile...

Sinon passer par une liste en intension

Code python :

S�lectionner tout - Visualiser dans une fen�tre � part

1
2
d=tuple(float(c) for row in data for c in row if c != 0)
average=sum(d)/len(d)

[VinsS]

Bonjour

Es-tu sur qu'additionner "0" sans le tester soit une optimisation ? Surtout que juste en dessous tu testes quand-m�me si cette valeur vaut 0.
Ne vaut-il donc pas mieux la tester (puisque de toute fa�on le test reste obligatoire) et seulement si la valeur correspond alors l'additionner ? On �limine ainsi une addition parfois inutile...

Ce n'est pas une question d'optimisation, tout ce qui simplifie le code est bienvenu.

Par contre, v�rifier avant, �a s'appelle "Look before you leap" et ce n'est pas pythonique.

Je pr�f�re de loin mon code.

[fred1599]

Si on part d'un CSV, les regex peuvent �tre sympa pour le tri des 0.

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

1
2
3
4
5
import re
 
with open("test.csv", 'r') as f:
    numbers = re.findall("[0-9]{2}|[1-9]", f.read())
    print(sum(map(int, numbers))/len(numbers))

Peut-�tre est-ce optimis�, je sais pas...

[dominiquelola]

Merci beaucoup � tous