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

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

ただいまの
回答率

87.58%

python プログラム

解決済

回答 4

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 1,066

score 13

from random import randint
def dice():
     num=randint(1,6)
     return num
def dicegame():
     dice1=dice()
     dice2=dice()
     sum=dice1+dice2
     if sum%2==0:
          print(f"合計{sum}、偶数")
     else:
          print(f"合計{sum}、奇数")
for i in range(5):
    dicegame()
print("ゲーム終了")


このプログラムを戻り値のある関数にしてprint文を関数の外に出した形にしたいです。わかる方、力を貸して欲しいです。

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

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

  • takahahi929

    2018/10/17 18:10

    そうです

    キャンセル

  • takahahi929

    2018/10/17 18:26

    すみません、書き損じがありました。print文を関数の外に出した形にしたいです。

    キャンセル

  • 退会済みユーザー

    2018/10/17 20:24

    複数のユーザーから「意図的に内容が抹消された質問」という意見がありました
    解決後に編集機能を用いて質問内容を改変し関係のない内容にしたり、内容を削除する行為は禁止しています。
    投稿していただいた質問は、後に他の誰かが困ったときに助けになる情報資産になると考えるからです。
    「質問を編集する」ボタンから編集を行い、他のユーザにも質問内容が見えるように修正してください。

回答 4

checkベストアンサー

+1

汎用性を持たせつつ、dicegame()と同様の機能を返り値で実現できるような関数にするとしたら、

  • 偶数か奇数かという情報
  • 結果の値

の2つを返せば良さそうに思えます。
(更に推し進めてmatherさんの回答のように関数の外でifを書く、という手もありますが)

def dicegame():
    dice1 = dice()
    dice2 = dice()
    s = dice1 + dice2
    if s%2 == 0:
        return "偶数", s
    else:
        return "奇数", s

受け取る側では、

even_or_odd, s = dicegame()

あとは文字列を組み立て表示すれば良いです。

最終的なコード。

from random import randint

def dice():
    num = randint(1,6)
    return num

def dicegame():
    dice1 = dice()
    dice2 = dice()
    s = dice1 + dice2
    if s%2 == 0:
        return "偶数", s
    else:
        return "奇数", s

for i in range(5):
     even_or_odd, s = dicegame()
     print(f"合計{s}{even_or_odd}")
print("ゲーム終了")

質問文のコードにコメント。

sumという変数名は、組み込み関数sum()を上書きしてしまうので、避けたほうがバグを防げます。

また、コーディングスタイルに改善の余地があるようです。

  • 代入や演算子の前後にはスペースを入れた方が読みやすいです。
  • 半角スペース6つのインデントは変則的です。pythonでは一般的に半角スペース4つが好まれます。

参考:
はじめに — pep8-ja 1.0 ドキュメント

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

0

printの代わりにreturnを使いましょう。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

0

dice は dict(辞書)と似てたのでdaisuに変えてます。
詳しくは下記のURL読んで下さい。

Python チュートリアル
https://docs.python.jp/3/tutorial/

# sum() 合計を求める
l=[2,3]
print(sum(l))



import random
def daisu():
    return random.randint(1,6)# 整数のらんだむ値

def dicegame():

    users=[daisu(),daisu()]

    if sum(users)%2==0:
        print("合計"+str(sum(users))+"偶数")
        #数値と文字列は足せないので、数値を文字列(str)として扱ってます
    else:
        print("合計"+str(sum(users))+"奇数")

for i in range(5):
    dicegame()
print("ゲーム終了")


"""
関数1
1から6までの乱数を作成する関数 *1

関数2
ユーザーの値を格納する配列(users)を作成
*1を呼び出して、配列に登録


偶数か奇数かをif文で判定
 配列に入ってる合計数値を求める。(sum)
 表示する時に数値を文字列として扱う。(str)


5回、繰り返す
 関数2を呼び出す

終了メッセージを表示

"""

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

0

def dicegame():
      dice1 = dice()
      dice2 = dice()
      return dice1 + dice2

for i in range(5):
     result = dicegame()
     if result % 2 == 0:
           print(f"合計{result}、偶数")
     else:
           print(f"合計{result}、奇数")

dicegame の返り値を何にしたいか、という指定がなかったので結果の値だけを返すようにしました。

投稿

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

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

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

関連した質問

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