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