csvファイルからn個のデータの代表値を求めるー複数行でのやり方 python3

Question

###前提・実現したいこと
Python3で

12.5,13.4,12.9,・・・, 12.5

のようにn個の小数点がならんでいるときにそれを20個の代表データにしたい。
具体的には
n/20づつの平均値を計算して
a,b,d,d,・・・,t (20個の小数点が並んだデータにしたい)

###発生している問題・エラーメッセージ

1行ならばできるようになったのですが複数行で行おうとするとエラーがでます。

エラーメッセージ

ValueError Traceback (most recent call last)
<ipython-input-7-6ed550d9bc09> in <module>()
6 columns = row.rstrip().split(',')
7 for column in columns:
----> 8 n=float(column)
9 list.append(n)
10 repeat = len(list)//20

ValueError: could not convert string to float:

###該当のソースコード

list = []
new_list=[]
new_list_2=[]
with open ('test5.csv', encoding ='utf-8') as f:
for row in f:
columns = row.rstrip().split(',')
for column in columns:
n=float(column)
list.append(n)
repeat = len(list)//20
for t in range (20):
for num in list[trepeat:trepeat+repeat]:
new_list.append(num)
new_value=(sum(new_list)/len(new_list))
new_list_2.append(new_value)
new_list=[]
print (new_list_2)

###試したこと
コードも素人丸出しで申し訳ないです

###補足情報(言語/FW/ツール等のバージョンなど)
複数行ある場合はなぜエラーが出るのでしょうか？

Comments

[LouiS0616]

slashさんの回答のようにコードを見やすく表示する機能があります。そのために、コードをバッククオート三つで括ってください。よくわからない場合は、マークダウン記法について調べてみてください。特にPythonの場合、インデントが崩れるとコードの意味が変わってきます。

Answer 1

問題を分割して考えるようにしてください。

n個の小数がならんでいるとき

同じような変数をまとめて扱うときは、どのようにすればよいでしょうか？
Pythonで数値を扱う場合は、ほぼ一択なので調べれば出ますよ。

n/20づつ

分割する方法を調べましょう。試しにググったら1ページ目に出ました。

平均値を計算

平均値を求める方法を調べましょう。試しにググったらやはり1ページ目に出ました。

---

ここまで考えましたか？
思考に慣れが必要なのは言うまでもないですが、

• どうしたものかわかりません。
• かけないです。
• わかりません。

と**『出来ない』**とただ主張されても、解決策が見えないです。

追記

1行ならばできるようになったのですが複数行で行おうとするとエラーがでます。

色々と情報が不足していますが、問題点に心当たりがあります。
質問者様のプログラムは、このような構造になっていますね。(ところどころ端折ってます)

with open(filename) as f:
    for row in f:
        columns = row.rstrip().split(',')
        for column in columns:
            n=float(column)

期待している通りに値が取り扱われているか、次のように確かめることが出来ます。

with open(filename) as f:
    for row in f:
        print('row', row)
        columns = row.rstrip().split(',')
        print('columns', columns)
        for column in columns:
            print(column)
            n=float(column)

ところどころにprint関数を挟んだだけです。俗にprintデバッグと呼びます。
非常にシンプルな手法ですが、強力な方法です。

さて、ひょっとしてcsvファイルは、このような形式ではないですか？

1.3, 5.3, 5.2,
3.4, 3.2, 4.1

これを読み込んで先ほどのprintデバッグをすると、次のように出力されます。

row 1.3, 5.3, 5.2,
columns ['1.3', ' 5.3', ' 5.2', '']
1.3
 5.3
 5.2

Traceback (most recent call last):
  File "testest.py", line 8, in <module>
    n = float(column)
ValueError: could not convert string to float:

columnsに変なのがまじっていませんか...？

結論を言うと、末尾のコンマを除去していないのが原因です。

>>> 'a, b, c, d'.split(',')
['a', ' b', ' c', ' d']
>>> 'a, b, c, d,'.split(',')
['a', ' b', ' c', ' d', '']
>>>

>>> float(' 3.4 ')
3.4
>>> float('')
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
ValueError: could not convert string to float:
>>>

---

そうなると問題は、末尾のコンマの取り除き方に切り替わります。
一般的には、文字列の右端を切り取るのにはスライスを用いることが出来ます。

>>> 'a, b, c, d,'[:-1]
'a, b, c, d'
>>>

しかし、この方法は最終行の最後の文字も削ってしまい、ここでは不適です。
rstripでコンマを除去すると良いでしょう。

>>> 'a, b, c, d,'.rstrip(',')
'a, b, c, d'
>>> 'a, b, c, d'.rstrip(',')
'a, b, c, d'
>>>

これなら上手くいきそうですね。

---

コードに関しましても、マークダウン表記をしていただければ、簡単にレビューします。
ただ一つだけ先に言っておくと、インスタンスにlistと名付けるのは絶対に避けてください。

>>> list('abc')
['a', 'b', 'c']
>>> list = []
>>> list('abc')
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
TypeError: 'list' object is not callable
>>>

Answer 2

# -*- coding: utf-8 -*-
import random
import numpy as np
list = []
max = 20
for num in range(0,max +1):
    list = list + [np.round(random.random(),1)]

print(np.array(list)/max )