質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

90.83%

  • Python

    6429questions

    Pythonは、コードの読みやすさが特徴的なプログラミング言語の1つです。 強い型付け、動的型付けに対応しており、後方互換性がないバージョン2系とバージョン3系が使用されています。 商用製品の開発にも無料で使用でき、OSだけでなく仮想環境にも対応。Unicodeによる文字列操作をサポートしているため、日本語処理も標準で可能です。

変数を無作為に選ぶという操作について

解決済

回答 2

投稿

  • 評価
  • クリップ 0
  • VIEW 157

eme

score 17

「3つの正の変数x,y,zを調べ、その中でもっとも大きくかつ奇数を表示するプログラミング(ただし、ない場合はnoneを出力)」を書きたいのですが、
自分は
1、maxを選ぶ。
2、それが奇数かどうかの判定
はい→それがmaxで、操作は終了。
いいえ→その数を省いて、再度操作を行う。
3、もし、文字が全て無くなれば、その時点で終了し、noneを出力する。

def max_odd(x,y,z)
    while (x,y,zがある時):
        a = max[x,y,z]
        if a % 2 == 1:
           answer = a
        else:
           (その数を省く)
    else:
        answer = none
 print(answer)

と整理してみたのですが、「x,y,zがある時」と「省く」という操作を記述するにはどうすればよいでしょうか?またこの操作はあっているのでしょうか?

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • Lhankor_Mhy

    2018/04/18 22:56

    『もっとも大きくかつ奇数』は普通に読むと、「最大の奇数」と異なると思うのですが、それは大丈夫ですか?

    キャンセル

  • eme

    2018/04/18 23:00

    そうですね。最大の奇数が正しいです。

    キャンセル

回答 2

checkベストアンサー

+1

max_oddの引数をタプルで取得するようにすれば良いです。
手前味噌で恐縮ですが、こちらが参考になるかと。 Qiita - Python3.xのアスタリスク逆引き

タプルのままにしておくと要素が変更できないので、引数を受け取ってからリストに変換します。

「x,y,zがある時」という操作の記述
『x,y,zがある時』を『リストが空でないとき』と換言できます。

「省く」という操作を記述
list.removeを使えば良いです。

個人的には

タプルを降順にソートしたものを巡回し、最初の奇数を返せばいいと思います。

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/04/18 23:42

    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が定義されていないと出たのですが、なぜでしょう?

    キャンセル

  • 2018/04/18 23:43

    すいません、インテンドできていませんでした...

    キャンセル

  • 2018/04/18 23:47

    ・while len(a) != 0 ではなく、単に while a で充分だったりします。
    ・普通『noneを返す』と言われたら、 文字列"none" じゃなくて None かと思います。

    > このように書いたらanswerが定義されていないと出たのですが、なぜでしょう?
    コメント欄では行頭の空白が効かないので、空白文字の代わりに.や_などを使うか、質問本文に追記する形にしてください。

    キャンセル

  • 2018/04/19 00:21

    わかりました。ありがとうございます。

    キャンセル

  • 2018/04/19 00:22

    answerに関するエラーは、インデントを直したら解消したのでしょうか?

    キャンセル

  • 2018/04/19 09:26

    解消しました。インテンドで適切に処理するところを蔑ろにしていた為に、「定義されていない」というエラーが出ていたみたいです。皆様ありがとうございました。

    キャンセル

0

まずwhile文の構文が間違ってませんか? 
最後の「else:」に対応するifがないですね。

それから「3つの正の変数」、この3つというのは不変、固定でしょうか。
そうであれば3つの整数をまずソートした後に、リストに保持。
リストのサイズをループ条件としたfor文でもって処理を回す。
この処理の方が効率いいコードになると思います。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2018/04/18 23:02

    疑似コードなので文法的におかしい点は他にもありますよ。
    a = max[x,y,z] とか answer = none とか。

    キャンセル

  • 2018/04/18 23:06 編集

    それに、最後のelseはwhileに対応しているので問題ないです。
    Pythonにはif-elseのほかにfor-else, while-elseがあります。

    キャンセル

  • 2018/04/18 23:08

    LouiS0616様
    はじめまして。いつも回答を見させていただいております。

    先の回答への補足コメントにも

    「『もっとも大きくかつ奇数』は普通に読むと、「最大の奇数」と異なると思うのですが、それは大丈夫ですか?」

    とありました。
    文法的なこと、出題の問題の定義、条件ともに”揺れ”、あいまいさが多いですね。

    少し様子見します。

    キャンセル

  • 2018/04/19 00:23

    参考書の練習題をそのまま載せると
    「3つの正の変数x、y、zを調べ、その中でもっとも大きい奇数を表示するプログラムを書け。ただし、奇数がない場合はその旨を結果に表示せよ。」です。

    キャンセル

  • 2018/04/19 00:35

    はじめまして。夜遅くにコメントありがとうございます。
    LouiS0616様の回答は、Python的な書き方です。そちらを参考にして下さい。

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 90.83%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

  • 解決済

    (Python)構文間違い

    構文間違い(SyntaxError: invalid syntax)と怒られましたけど、 どこが間違っているのかを教えてください。 コード: class michael:

  • 受付中

    大量のCSVデータの処理方法

    100個ほどCSVファイルがあり、 それぞれのファイルで、30×30のセルに数値が入力されています。 この100個のCSVファイルに対して、同じセル座標について最大値最小値を計算し

  • 解決済

    Pythonの階層クラスタを構成する葉を取得したい。

    Pythonの scipy.cluster.hierarchyで階層クラスタリングをすると、以下のようなリンケージ行列だけが返ってきますが、簡単にクラスタを構成する葉を取ってくるこ

  • 解決済

    Pythonでのリスト操作 | リストを分割したい

    実現したいこと Pythonでのリスト操作について質問です [1,2,3,4,5,1,2,3,4,1,2,1,2,3,4,5,6] のようなリストを1から始まる要素で分割し

  • 解決済

    Pythonのリスト内の辞書方について

    前提・実現したいこと 下記のリストをマージさせ、List1に追加したいと思ってます。 for文で回せばできるのでしょうが、python独自のやり方で簡単に出来る方法がないか 探

  • 解決済

    Python 3.x 辞書のキー値によって変換する場合の高速化

    Pythonにて、辞書(dict({key,value})を使って、list型の全要素をValue値に変換する際の、 高速化が可能かどうかをご教授いただきたいです。 dict1

  • 解決済

    pygameを用いたMIDIの制御

    実現したい内容 pygameを用いてMIDIデバイスの入力を受け取り、リアルタイムでSin波で演奏 複数音の出力および、ベロシティの対応  発生してる問題 キーを押した時と離し

  • 受付中

    python実行が正常に動作したりしなかったりする

     前提・実現したいこと ここに質問の内容を詳しく書いてください。 (例)PHP(CakePHP)で●●なシステムを作っています。 pythonからRを呼び出してコードを実行したいが

同じタグがついた質問を見る

  • Python

    6429questions

    Pythonは、コードの読みやすさが特徴的なプログラミング言語の1つです。 強い型付け、動的型付けに対応しており、後方互換性がないバージョン2系とバージョン3系が使用されています。 商用製品の開発にも無料で使用でき、OSだけでなく仮想環境にも対応。Unicodeによる文字列操作をサポートしているため、日本語処理も標準で可能です。