python プログラム

Question

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文を関数の外に出した形にしたいです。わかる方、力を貸して欲しいです。

Comments

[hayataka2049]

質問文を再編集して、ちゃんとコードブロックを閉じてください（print("ゲーム終了")の行の下に```と入れれば良いと思います。あと二行目の「コード」も消しましょう）

[mather]

「このプログラム」とは dicegame のことですか？

[takahahi929]

そうです

[takahahi929]

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

Answer 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 ドキュメント

Answer 2

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 の返り値を何にしたいか、という指定がなかったので結果の値だけを返すようにしました。

Answer 3

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

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


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


５回、繰り返す
　関数２を呼び出す

終了メッセージを表示

"""

Answer 4

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