drivegameのマップを作成中ですが、実行結果がＮｏｎｅと出ます。どの箇所をどう直せば良いのでしょうか。

python
1コード
2from random import randint
3
4def create_map_line(ginfo):
5    if randint(0, 99) < 10:
6        ginfo["dir"] = randint(-1, 1)
7    cx, sz, gdir = (ginfo["cx"], ginfo["size"], ginfo["dir"])
8    if (gdir == -1 and cx <= 1):
9        ginfo["dir"] *=-1
10    if (gdir == 1 and (cx + sz) >= (ginfo["cols"] - 1)):
11        ginfo["dir"] *= -1
12        ginfo["cx"] += ginfo["dir"]
13        line = [1] * ginfo["cols"]
14        for i in range(ginfo["size"]):
15            line[i + ginfo["cx"]] = 0
16        return line
17def create_map(ginfo):
18    map_data = []
19    for i in range(ginfo["rows"]):
20        line = create_map_line(ginfo)
21        map_data.insert(0, line)
22    return map_data
23
24if __name__ == "__main__":
25    ginfo = {
26        "rows": 20,
27        "cols": 30,
28        "dir": 0,
29        "cx": 10,
30        "size": 8
31    }
32    map_data = create_map(ginfo)
33    for row in map_data: print(row)```
34コード

hayataka2049

2019/02/12 10:04

インデントが潰れて読めません。https://teratail.com/tour　を参考に質問文を再編集し、シンタックスハイライト・マークダウンでコードを囲んでください。問題のもう少し詳しい説明、実行結果、それを受けて試したことなどを書くと回答が得られやすいです。

proTiku

2019/02/12 10:20

お待たせしました。問題については0,1の数字を用いて0が走行可能部分で1が走行不可部分の道を作成したいのですが、実行結果はＮｏｎｅが羅列されるだけでした。試したことはデバッカーを使用して、map_data　＝　[]にＮｏｎｅと表示されることを突き止めました。

行動規範の内容に同意します

回答1件

どんな結果になればいいのかまではわからないので、Noneとなる原因だけ指摘します。

create_map_lineの最後の以下の部分

Python
1    if (gdir == 1 and (cx + sz) >= (ginfo["cols"] - 1)):
2        ginfo["dir"] *= -1
3        ginfo["cx"] += ginfo["dir"]
4        line = [1] * ginfo["cols"]
5        for i in range(ginfo["size"]):
6            line[i + ginfo["cx"]] = 0
7        return line

returnがif文の中に入っています。
これではif文がTrueの時しかlineの値をreturnしないので、
if文の条件がFalseだった場合はNoneが返ってしまいます。

つまりは

Python
1if (gdir == 1 and (cx + sz) >= (ginfo["cols"] - 1)):

がFalseになるのでNoneが返っているのが現状です。

そこの条件がFalseの場合はどうすれば良いか、処理を追加すれば対策できるかと思います。

投稿2019/02/12 10:31

dice142

総合スコア5158

proTiku

2019/02/12 13:35

色々、試行していると以下の実行結果が出ました。Ｎｏｎｅの箇所を全て数値化させるには如何すれば良いのでしょうか。 None [1, 1, 1, 1, 0, 0, 0, 0, 0, 0, 0, 0, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1] None [1, 1, 1, 1, 1, 0, 0, 0, 0, 0, 0, 0, 0, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1] None None None None None [1, 1, 1, 1, 1, 1, 0, 0, 0, 0, 0, 0, 0, 0, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1] None None None [1, 1, 1, 1, 1, 1, 1, 0, 0, 0, 0, 0, 0, 0, 0, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1] None None [1, 1, 1, 1, 1, 1, 1, 1, 0, 0, 0, 0, 0, 0, 0, 0, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1] [1, 1, 1, 1, 1, 1, 1, 1, 1, 0, 0, 0, 0, 0, 0, 0, 0, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1] None None

dice142

2019/02/13 02:09

> Ｎｏｎｅの箇所を全て数値化させるには如何すれば良いのでしょうか。 returnされないパターンがあることが原因だとおもいます。 returnがif文の内側にしか無いのであれば、全パターンを網羅しているか確認しましょう。あるいはどのif文にもあてはまらなかったときのために、一番最後に必ず何かをreturnするという手もあります。

行動規範の内容に同意します