if分の条件の単純化

Question

pythonでの疑問というより知恵を貸してほしい質問になります

配列para[5]にキーボード入力で0から30までの好きな整数を入力し、その数字の組み合わせによって結果を表示したいのです

そこでpara[1]とpara[3]が同程度(差が3で考えています)でかつpara[0] para[2] para[4]が同程度(差が3)でBDペアがACEに対して相対的に高い(差が６程度)組み合わせの時に「Ｍ型」という結果を表示したいとします

今は、配列を並び替えて最大と最小の差とすべての数字の大小関係、並び替えから1番大きいものと2番目がpara[1]para[3]にあるのかどうかといった条件をすべてifの条件に書いているんですが

あまりに条件が長く多くなってしまっており。それをシンプルにする方法があれば教えてほしいのです。こういった条件でもいけるや、もっと単純な方法で判断できるなど教えてください

Comments

[can110]

現状のソースコードを提示ください。

Answer 1

まず「同程度」と「相対的に高い」を明確にする必要があります。
ご自身でもよく分からない、うまく説明できない場合は、シンプルな具体例をとっかかりにして考えてみましょう。
たとえばACE=[1,3,5]は「同程度」か？また、それは何故か？を考えます。
この「何故か？」が分からない、説明できないとコードに落とすことはできません。

これらが明確になれば、次に処理を分割して箇条書きにしてみます。
具体的には

• BDとACEについて、各「同程度」度を算出する。
• 各「同程度」度が範囲外ならダメ。
• BDとACEについて、各「高い」度を算出する。
• BDとACEの「高い」度を比較する。

といった感じでしょうか。
これらを踏まえ、ややエスパーして組んでみました。
判定処理を別関数にし、合致しない場合は早めにreturnするとコードの見通しが良くなります。

def isM( para):

    bd = [para[1],para[3]]
    ace= [para[0],para[2],para[4]]

    # 「同程度」か
    if max(bd)-min(bd) > 3:
        return False
    if max(ace)-min(ace) > 3: # 差分値で判断（1,4,7を同程度とする）なら 3 * 2(ギャップ数)
        return False

    # BDがACEに対して「相対的に高い」か
    # とりあえず平均値で判定する
    if (sum(bd)/len(bd)) - (sum(ace)/len(ace)) < 6:
        return False

    return True

print(isM([1,4,2,5,3]))
print(isM([1,8,2,9,3]))

Comments

[12tenten31]

なるほど、言葉足らずなものにお答えしていただき感謝します。回答を見ながらもう一度じっくり考えます

Answer 2

修正依頼ですが、コメント欄に収まりきらないので回答に書きます。

配列para[5]にキーボード入力で0から30までの好きな整数を入力し、その数字の組み合わせによって結果を表示したいのです

こう書かれると、**配列para[5]**がまず何を意味する表現なのか悩みます。要素数5のリストですか？

そこでpara[1]とpara[3]が同程度(差が3で考えています)でかつpara[0] para[2] para[4]が同程度(差が3)でBDペアがACEに対して相対的に高い(差が６程度)組み合わせの時に「Ｍ型」という結果を表示したいとします

BDペアとACEについて説明をしてください（推測はできるけど）。
相対的に高いとは、どうやって測るのですか？　平均とか？

Answer 3

例えば、こんな感じですかね？

abs(para[1]-para[3])<=3 and max(para[0],para[2],para[4])-min(para[0],para[2],para[4])<=3 and min(para[1],para[3])>=max(para[0],para[2],para[4])+6