while文のエラー

Question

while文で配列の値の条件分岐ができなくてエラーが起きました。

def A(引数):
 ・
 ・
  return array

def B(引数):
 ・
 ・
 return array
 

def C(引数):
 ・
 ・
 return array

def D(引数):
 ・
 ・
 return array

def X(引数):
 aa =A(引数)
 bb =B(引数)
 cc =C(引数)
 dd =D(引数)
 del 配列を削除

while len(array) !=0:
   X()
print(bb[0])

というコードを書いて実行したらwhile文が３回（３回がwhileが回るトータル）
回る途中時にXメソッドの中のBメソッドでlen(array) が０なのでエラーが起き、while文でもエラーが起きました。
Xメソッドで定義している通り、A~Dのメソッドはセットで実行したいです。しかし、３回目まで回った時にメソッドBを実行した後にwhile文を抜け一番下のprint文でメソッドBの結果の配列を出力する方法がわかりません。
cc =C(引数)からlen(array) ==0 の条件を満たしてしまうので。

count=0

とwhile文の外に置いておいてwhile文が何回回ったかを計測して３回目だけ処理を変えるという方法はあると思うのですが、このアルゴリズムをいろいろな配列に対応させたいのでその方法はないとします。
どのようにやりたいことを書けますか？

Comments

[can110]

第三者が再現可能なコードを提示ください。

[mattn]

過去に質問された内容から判断させて頂くと、この文法は理解できるはずと推測致します。何度も言いますがまず基本構文を学ばれた方が良いと思います。答えを貰ってご自分の知識として習得できていないのなら、質問者も回答者も無駄な時間を過ごす事になります。

[can110]

NameError: name 'array' is not defined で実行できません。実行できるようコード修正ください。

Answer 1

本当にコードは合っていますか？

while len(array) != 0:
    X()
print(array[0])

arrayの長さがゼロにならないとwhileを抜けないのに、次の行でprint(array[0])って矛盾していませんか？
array（グローバル変数？）の長さを変えている場所も見当たりませんし。

X番目に処理を変えたいのであればwhileではなくfor i, x in enumerate(array)のようにenumrateを使った方が良いと思います。

---

コメントを受けてソースを変えました。
そもそもwhileで配列を削除(list.pop(-1)を使う？)していくコードはバグが入りやすく、理解もしにくいと思います。
このような処理ならforを使うのが順当かと思います。

def X(引数):
   aa =A(引数)
   bb =B(引数)
   if len(bb) != 0:
       print(bb)
   cc =C(引数)
   dd =D(引数)

for a in array:
    X(a)

bbの長さがゼロならprintしたいという事ですが、こういう事でしょうか・・・？

Comments

[退会済みユーザー]

ありがとうございます。情報を修正・追加しました。 bb =B(引数)で空の配列が取得できるので、bbのlenが０かどうか判定し該当するので
print文を呼び出したいです。なのでenumrateを使ってX番目に処理を変えたいわけではないです。もし何かお分かりでしたらお願いします。

[pashango2]

一応コメントは書きましたが、もっと大きなくくりでrrrrrrrrさんが大きな勘違いをしているような・・・
客観的に見て、提示されたコードは何がしたいのか分かりません、おそらく一般的な解決方法はもっと別な形になるような気がします。

Answer 2

A関数実行後に条件文書いてreturnすればいいのでは。

return を使うと、式リスト (または None) を戻り値として、現在の関数呼び出しから抜け出します。

7. 単純文 (simple statement) — Python 3.6.1 ドキュメント

質問が変更になったので追記

Bメソッドでlen(array) が０なのでエラー

↑これと↓には矛盾がありますよね。

メソッドBを実行した後にwhile文を抜け一番下のprint文でメソッドBの結果の配列を出力

そうしたいなら、len(array) が0の時にエラーになるB関数の仕様が間違っているかと思います。

Answer 3

今頭が回っていないので、わかりませんが、

私なら

C++とかなら bool にあたるもの ( C言語だと存在しないので int で代用とか ) を返して、

この戻り値が true ( true相当の値 ) のとき、次のメソッド実行っていう感じにしてはどうでしょうか?

私はC/C++でやっているので C++風に書くと

if( funcA() ){
       if( funcB() ){
            if( funcC() ){
                 return true;
            }
       }
}else{
       return false;
}

みたいにするとか。

それか、

Pythonはやってないので適当ですが、

def func( 本来の引数, 直前のメソッドの値 ):
    if 直前のメソッドの値 == true:
         # 実行
         return true
    else:
         return false

みたいに

直前のメソッドを受け取って その値が true とかみたいに "成功" を示すものであれば実行, そうじゃなければ falseを返すとか。

で、メソッドに渡す前に if文でチェック。
false状態なら 質問でいう Xメソッド が return する。

次に3回だけ行うとか、3回目のとき...とかなら、

私なら 別の変数 counter とかみたいなものを用意して、

ループするたびに インクリメント ( C++ とかだと counter++; で、VB系だと counter = counter + 1 ってやつ。 )して、

これが 3回目のときにbreak に相当するものでループを抜けるとか。

そうすれば 配列の型に関係なくできると思います。

---

他の方から「上記の方法はレガシーコードだ」と指摘を受けました。

その方からは「例外処理」を提示されました。
確かに例外処理もあります。

なので、

メソッドA の内部で処理するときに 指定の処理が失敗したら例外を投げる。
メソッドB でも同様。
メソッドC も...

という風にしておきます。

Pythonはよくわからないので調べてみると、
throw ではなく raise らしいですね。

なので、

def methodA():
    # なんらかの処理
    # ここで処理失敗したとする
    raise Exception( "methodA失敗" )

def methodB():
    # なんらかの処理
    # ここで処理失敗したとする
    raise Exception( "methodB失敗" )

def methodC():
    # なんらかの処理
    # ここで処理失敗したとする
    raise Exception( "methodC失敗" )

としておけば、

def methodX():
    try:
         methodA()
         methodB()
         methodC()
    except Exception, e:
        print e.value

みたいにできます。

で、もしmethodAで失敗すれば 例外が投げられるので methodB以降は処理しない状態です。

なので、用件にあっているかと。

例外を投げる

Comments

[LouiS0616]

これはバッドコードじゃないでしょうか？
関数の成功／失敗を返り値のboolで返すのはいわゆるレガシーコードかと思います。
C++にもPythonにも例外処理があります。

また、戻り値を順に判定するような処理を書くにしても、ネストではなく
return funcA() && funcB() && funcC(); で充分なのでは？

[BeatStar]

LouiS0616さん

あー、そういえば例外処理がありましたね。
編集しておきます。