関数を使ったコードの簡略化

import random
def decision():
  number1 = int(input("Enter in a 3 digit password (numbers only): "))
  number2 = int(input("Enter in a 4 digit password (numbers only): "))
  number3 = int(input("Enter in a 5 digit password (numbers only): "))
  number4 = int(input("Enter in a 6 digit password (numbers only): "))
  numax = int(input("Enter in max iterations: "))
  leng1= len(str(number1))
  leng2= len(str(number2))
  leng3= len(str(number3))
  leng4= len(str(number4))
  counts1 = []
  for i in range(10):
    guess1 = 0
    count1 = 0
    while (guess1 != number1):
            guess1 = random.randint(10**(leng1-1),(10**leng1-1))
            count1 += 1
            if count1 == numax:
              break
    counts1.append(count1)
    counts2 = []
  for i in range(10):
    guess2 = 0
    count2= 0
    while (guess2 != number2):
            guess2 = random.randint(10**(leng2-1),(10**leng2-1))
            count2 += 1
            if count2 == numax:
              break
    counts2.append(count2)
    counts3 = []
  for i in range(10):
    guess3 = 0
    count3 = 0
    while (guess3 != number3):
            guess3 = random.randint(10**(leng3-1),(10**leng3-1))
            count3 += 1
            if count3 == numax:
              break
    counts3.append(count3)
    counts4 = []
  for i in range(10):
    guess4 = 0
    count4 = 0
    while (guess4 != number4):
            guess4 = random.randint(10**(leng4-1),(10**leng4-1))
            count4 += 1
            if count4 == numax:
              break
    counts4.append(count4)
  result1 = sum(counts1)/len(counts1)
  result2 = sum(counts2)/len(counts2)
  result3 = sum(counts3)/len(counts3)
  result4 = sum(counts4)/len(counts4)
  counts1 = []
  counts2 = []
  counts3 = []
  counts4 = []
  print("FOR 10 TESTS, the average number of iterations to find password are: ")
  print("3 DIGIT PASSWORD: " + str(result1) + " iterations")
  print("4 DIGIT PASSWORD: " + str(result2) + " iterations")
  print("5 DIGIT PASSWORD: " + str(result3) + " iterations")
  print("6 DIGIT PASSWORD: " + str(result4) + " iterations")

簡単な質問ですいません。明日提出の課題なのでできればサンプルコードを見せていただけるとありがたいです。functionを使ってコードを短く簡略化したいのですが、どのようにすればよいでしょうか？

hoshi-takanori

2021/08/22 18:50

同じパターンが繰り返し現れるところを関数にすればいいと思いますが、課題は自分でやらないと意味がありませんよ。

gentaro

2021/08/22 21:43

ルール確認して出直してこよう https://teratail.com/help/avoid-asking コードをください・デバッグしてください等の丸投げの質問何かを作りたいのでコードを書いてほしい、学校の課題を解いてほしい等の質問は、具体的にプログラミングで困っている質問ではないと考え、推奨していません。

hentaiman

2021/08/23 04:29

サンプルコードがたくさんあるサイト https://docs.python.org/ja/3.9/

meg_

2021/08/23 05:35

課題とのことですが、正しく動作するのであれば多少冗長でも良いのではないですか？また、コードの内容を簡単には説明すべきではないでしょうか？

kazuma-s

2021/08/23 12:43

質問のコードに与えるテストデータをいくつか提示してもらえませんか？

BeatStar

2021/08/24 00:52 編集

(問題発言のため、削除)

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

回答2件

関数を使った簡略化ではなく、リストと for文を使って簡略化してみました。

Python
1import random
2
3def decision():
4	data = []
5	for i in range(4):
6		data.append(input("Enter in a " + str(i+3) + " digit password (numbers only): "))
7	
8	numax = int(input("Enter in max iterations: "))
9
10	result = []
11	for s in data:
12		number = int(s)
13		leng = len(s)
14		counts = []
15		for i in range(10):
16			guess = count = 0
17			while guess != number:
18				guess = random.randint(10**(leng-1),(10**leng-1))
19				count += 1
20				if count == numax:
21					break
22			counts.append(count)
23		result.append(sum(counts)/len(counts))
24
25	print("FOR 10 TESTS, the average number of iterations to find password are: ")
26	for i in range(4):
27		print(str(i+3) + " DIGIT PASSWORD: " + str(result[i]) + " iterations")
28
29decision()
30

希望通りの簡略化でなくてすみませんが、このコードについてコメントをお願いします。

投稿2021/08/23 13:05

編集2021/08/23 13:14

kazuma-s

総合スコア8224

繰り返しでてくる部分を関数にする

python
1  counts1 = []
2  for i in range(10):
3    guess1 = 0
4    count1 = 0
5    while (guess1 != number1):
6            guess1 = random.randint(10**(leng1-1),(10**leng1-1))
7            count1 += 1
8            if count1 == numax:
9              break
10    counts1.append(count1)