二次元配列をwhileでループ

Question

python初心者です。
pythonを対話モードで使用しています。数字がすべて0の10000×10000の二次元配列にwhileを使って数字を上書きしたいです。
e=np.loadtxt('a.csv',delimiter=',',skiprows=1,usecols=[0])
f=np.loadtxt('a.csv',delimiter=',',skiprows=1,usecols=[1])
i=0
j=0
k=0
g=np.zeros((10000,10000))
while i<(len(e)-80):
---if e[i]<e[i+10] and e[i+10]<e[i+20] and e[i+20]<e[i+30] and e[i+30]<e[i+40] and e[i+40]>e[i+50] and e[i+50]>e[i+60] and e[i+60]>e[i+70] and e[i+70]>e[i+80]:
------print((e[i+40]),(f[i+40]),i+40)
------------g[j][k]=e[i+40]
------------k+=1
------else:
------------print('b')
------------j+=1
------------k=0
------i+=1

e,fにはそれぞれexcelの数値が入っていて、whileで一番下の数値までループを回しています。ifの条件に合った時にg[j][k]=e[i+40]で0を上書きしk+=1で次の列へ、条件に合わなかったときj+=1で次の行に移りk=0で列を始めに戻そうと考えています。

流れは↑の画像のようなものをイメージしています

こちらが現在のエラーです

改善法を教えていただきたいです。よろしくお願いします。

Answer 1

      print((e[i+40]),(f[i+40]),i+40)
            g[j][k]=e[i+40]

は同じインデントでなければなりません。

            print((e[i+40]),(f[i+40]),i+40)
            g[j][k]=e[i+40]

に変更しましょう。

Comments

[chario1230]

回答ありがとうございます。インデントを変更したところ作動しました。
重ねて質問で申し訳ないのですが、whileでループした後にgを出力すると配列の中身がすべて０になってしまうのですが、なぜか教えていただけませんでしょうか。

[ppaul]

デバッグは以下のように行います。
(1) データのサイズを小さくして再現することを確認する
(2) 要所要所にprint文を入れるか対話型デバッガを使って、どのルートを通っているかを確認する
(3) 期待するルートを通っていない場合はwhile文やif文の条件式の条件をprint文か対話型デバッガで確認する。
taretailでの質問のしかたです。
質問する場合はソースコード、エラーコード、実行結果をコピペ可能(つまりテキスト)で書く。画像で出すのは「回答者が手で打ち込んで検索したり実行するは当然」という意思表示ですよ。
質問した回答に次々に追加質問するのは、「プログラミング能力を上げたいのではなく作ってもらいたいだけという意思表示ですよ。

[ppaul]

ある質問をした質問者が、次の質問では一段階成長して、その成長の結果が見える質問をしてきた場合、多くの回答者は、喜んでその質問に回答するでしょう。
成長する質問者になって、次の質問をしてくださることを期待しています。