calcul moyenne en python à partir fichier CSV

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:

---

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

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:

---

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)

---

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

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:

---

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))

---

Bonjour

Citation:

---

Envoyé par VinsS

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:

---

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

---

Citation:

---

Envoyé par Sve@r

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.

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

Code:

---

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...

Merci beaucoup à tous