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

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

新規登録して質問してみよう
ただいま回答率
85.48%
Python

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

Q&A

解決済

2回答

2029閲覧

pythonで乱数生成による無限ループ脱出

ETOLEA

総合スコア15

Python

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

0グッド

0クリップ

投稿2018/09/14 13:23

前提・実現したいこと

初めての質問です。よろしくお願いします
pythonでヌメロンっていうゲームを作ろうと思った際に行き詰まりました
簡単にいうと2人で対戦し、お互い3桁の数字を決め、当て合うゲームです。
相手をコンピューターにしようとしています。
これは、相手側の数字3桁をランダムで生成しようとしています
条件としては、同じ数字を使わないことです
あとで重複を確認(duplication_check)するためにリストに直しています
桁が足りないときは頭にランダムで足すようにしています
ですが、このコードを実行するとプレイヤー側が数値を入力後、
無限ループ?っぽくなります。エラーメッセージは出ず、おそらく生成しないまま
ぐるぐる回っているかと思われます。
無駄まみれでわかりづらいかとは思いますが、ご助力お願いいたします

発生している問題・エラーメッセージ

#出ません

該当のソースコード

python

1import random 2def duplication_check(num): 3 return len(num) == len(set(num)) and len(num) == 3 4while True: 5 try: 6 random_number = str(random.randint(1,1000)) 7 if len(random_number) == 3: 8 computer_number = list(random_number) 9 if deplication_check(computer_number): 10 break; 11 else: 12 pass 13 elif len(random_number) == 2: 14 computer_number = list(random_number) 15 computer_number.insert(0, "0") 16 if deplication_check(computer_number): 17 break; 18 else: 19 pass 20 except: 21 pass``` 22 23### 試したこと 24breakの位置か個数かと思いました 25色々やったら余計わからなくなりました

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

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

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

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答2

0

ベストアンサー

try-exceptが例外を潰してしまっているのが問題です。
これを外して実行してみましょう。

Python

1while True: 2 random_number = str(random.randint(1,1000)) 3 if len(random_number) == 3: 4 computer_number = list(random_number) 5 if deplication_check(computer_number): 6 break; 7 else: 8 pass 9 elif len(random_number) == 2: 10 computer_number = list(random_number) 11 computer_number.insert(0, "0") 12 if deplication_check(computer_number): 13 break; 14 else: 15 pass

実行結果

plain

1Traceback (most recent call last): 2 File "xxx.py", line 11, in <module> 3 if deplication_check(computer_number): 4NameError: name 'deplication_check' is not defined

タイプミスをキャッチしちゃってますね。
こういう事態が起きるので、例外処理は充分注意して扱う必要があります。

例外を安易に握りつぶさなければ、簡単なエラーメッセージを捌くだけで済んだのです。


無駄まみれでわかりづらいかとは思います

random.sampleの出番です。

Python

1>>> import random 2>>> 3>>> num_chars = '1234567890' 4>>> while True: 5... ans = random.sample(num_chars, 3) 6... if ans[0] != '0': 7... break 8... 9>>> ans 10['1', '6', '8']

投稿2018/09/14 13:31

編集2018/09/14 13:39
LouiS0616

総合スコア35660

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

ETOLEA

2018/09/14 13:35

なるほど!tryやexpectは実はネットから引っ張って来て書き換えたもので、そのものの働きを私自身よく理解していなかったのがいけませんね。 タイプミスはお恥ずかしいですが、おかげさまで問題なく実行できました! ありがとうございます!!!
LouiS0616

2018/09/14 13:38 編集

try: ~ except: pass と書いてあるコードは絶対に参考にしない方が良いです。 例外処理は強力ですが、使い方を誤ると自らの首を絞めます。
guest

0

コードを読んでわからに場合は、デバッグしてみるとよいです。

参考情報

  • PythonデバッグTips

https://qiita.com/TakesxiSximada/items/45b6d71a44f2706798ed

質問文にあるコードを pde でステップ実行させてみました。
イメージ説明

'deplication_check' is not defined のエラーが発生していることがわかります。

投稿2018/09/14 14:16

katoy

総合スコア22324

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.48%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問