Q&A
『もっとも大きくかつ奇数』は普通に読むと、「最大の奇数」と異なると思うのですが、それは大丈夫ですか？
「３つの正の変数x,y,zを調べ、その中でもっとも大きくかつ奇数を表示するプログラミング（ただし、ない場合はnoneを出力）」を書きたいのですが、
自分は
１、maxを選ぶ。
２、それが奇数かどうかの判定
はい→それがmaxで、操作は終了。
いいえ→その数を省いて、再度操作を行う。
３、もし、文字が全て無くなれば、その時点で終了し、noneを出力する。
python
1def max_odd(x,y,z) 2 while （x,y,zがある時）: 3 a = max[x,y,z] 4 if a % 2 == 1: 5 answer = a 6 else: 7 (その数を省く） 8 else: 9 answer = none 10 print(answer)
と整理してみたのですが、「x,y,zがある時」と「省く」という操作を記述するにはどうすればよいでしょうか？またこの操作はあっているのでしょうか？
回答2件
0
ベストアンサー
max_oddの引数をタプルで取得するようにすれば良いです。
手前味噌で恐縮ですが、こちらが参考になるかと。 Qiita - Python3.xのアスタリスク逆引き
タプルのままにしておくと要素が変更できないので、引数を受け取ってからリストに変換します。
「x,y,zがある時」という操作の記述
『x,y,zがある時』を『リストが空でないとき』と換言できます。
「省く」という操作を記述
list.removeを使えば良いです。
個人的には
タプルを降順にソートしたものを巡回し、最初の奇数を返せばいいと思います。
投稿2018/04/18 13:45
編集2018/04/18 13:48
総合スコア35660
def max_odd(x,y,z):
a = [x,y,z]
while len(a) != 0:
b = max(a)
if b % 2 ==1:
answer = b
break
else:
a.remove(b)
else:
answer = "none"
print(answer)
このように書いたらanswerが定義されていないと出たのですが、なぜでしょう？
すいません、インテンドできていませんでした...
・while len(a) != 0 ではなく、単に while a で充分だったりします。
・普通『noneを返す』と言われたら、 文字列"none" じゃなくて None かと思います。
> このように書いたらanswerが定義されていないと出たのですが、なぜでしょう？
コメント欄では行頭の空白が効かないので、空白文字の代わりに.や_などを使うか、質問本文に追記する形にしてください。
わかりました。ありがとうございます。
answerに関するエラーは、インデントを直したら解消したのでしょうか？
解消しました。インテンドで適切に処理するところを蔑ろにしていた為に、「定義されていない」というエラーが出ていたみたいです。皆様ありがとうございました。
0
まずwhile文の構文が間違ってませんか？
最後の「else:」に対応するifがないですね。
それから「３つの正の変数」、この３つというのは不変、固定でしょうか。
そうであれば３つの整数をまずソートした後に、リストに保持。
リストのサイズをループ条件としたfor文でもって処理を回す。
この処理の方が効率いいコードになると思います。
投稿2018/04/18 13:59
総合スコア1195
疑似コードなので文法的におかしい点は他にもありますよ。
a = max[x,y,z] とか answer = none とか。
それに、最後のelseはwhileに対応しているので問題ないです。
Pythonにはif-elseのほかにfor-else, while-elseがあります。
LouiS0616様
はじめまして。いつも回答を見させていただいております。
先の回答への補足コメントにも
「『もっとも大きくかつ奇数』は普通に読むと、「最大の奇数」と異なると思うのですが、それは大丈夫ですか？」
とありました。
文法的なこと、出題の問題の定義、条件ともに”揺れ”、あいまいさが多いですね。
少し様子見します。
参考書の練習題をそのまま載せると
「３つの正の変数x、y、zを調べ、その中でもっとも大きい奇数を表示するプログラムを書け。ただし、奇数がない場合はその旨を結果に表示せよ。」です。
はじめまして。夜遅くにコメントありがとうございます。
LouiS0616様の回答は、Python的な書き方です。そちらを参考にして下さい。
あなたの回答
tips
プレビュー
