Как исправить ошибку из-за четвертого if подряд?

Question

[Славка]

import random
n=1
sim=""
k=0

def start():
    n=int(input("\nПорядок: "))
    sim=input("\nОперация: ")
    game(n,sim)

def game(nn, optt):
	
	s1 = int(1)
	s2 = int(10**nn)
	 
	i = random.randint(s1,s2)
	j = random.randint(s1,s2)
	 
	ans=answer(i,j,optt)
	if ans :
	
		print ("Nice")
		

	else:
	    print("Bad")
    
    
      	
	start()

def answer( i,j, optt):
	 if optt=="+":
		 print(i," ", optt," ",j," = ?")
		 a=int(input("\nОтвет: "))
		 if a == int(i)+int(j):
		     return True
		 return False

	 elif optt=="-":
		 print(i," ", optt," ",j," = ?")
		 a=int(input("\nОтвет: "))
		 if a == int(i)-int(j):
		     return True
		 return False

	 elif optt=="*":
		 print(i," ", optt," ",j," = ?")
		 a=int(input("\nОтвет: "))
		 if a == int(i)*int(j):
		     return True
		 return False
 

for i in range(100):		 
   start()

Когда пытаюсь написать в конце

elif optt=="/":
		 print(i," ", optt," ",j," = ?")
		 a=int(input("\nОтвет: "))
		 if a == float(i)/float(j):
		     return True
		 return False

он ругается, что я путаю что-то с уровнем.

Answer 1

[lega]

У вас в коде неправильные отступы - перемешаны табы и пробелы, используйте что-то одно, рекомендуется пробелы. Далее выровняйте elif по одному уровню кратно размеру отступа.

Comments

[Славка]

это так скопировалось :( хотя попробую переписать

[Илья]

в большинстве редакторов можно включать отображение непечатаемых символов, чтоб увидеть есть ли проблемы с отступами, например смешиваются ли где-то пробелы с табуляциями.

Answer 2

[iroln]

Используйте нормальный редактор, который:

- Заменяет табуляцию пробелами
- Умеет конвертировать "табуляция" <> "пробелы"

Например, Notepad++ или SublimeText.

Answer 3

[Andrey Dugin]

Почему бы вместо

if a == int(i)*int(j):
		     return True
		 return False

Не написать
return a == int(i)*int(j)
?

Кроме того, у вас переменные i и j являются int по способу создания, так что постоянно писать int(i) и int(j) не обязательно.

Answer 4

[Владислав]

import random

def game(nn, optt):	
	s1 = 1
	s2 = 10**nn	 
	i = random.randint(s1,s2)
	j = random.randint(s1,s2)
	ans=answer(i,j,optt)
	print ("Nice") if ans else print("Bad")
		
def answer( i,j, optt):
	d = {'+':(lambda x,y:x+y),'-':(lambda x,y:x-y),
		'*':(lambda x,y:x*y)}
	print(i," ", optt," ",j," = ?")
	a=int(input("\nОтвет: "))
	return a == d[optt](i,j)
		
for i in range(100):
	n=int(input("\nПорядок: "))
	sim=input("\nОперация: ")
	game(n,sim)

Советую в будущем писать более лаконично. More Python-way

Comments

[Andrey Dugin]

Функцию answer можно значительно сократить с помощью eval().

[Владислав]

@adugin хмм... буду знать. сам еще начинающий)

[Владислав]

@adugin а если случайно передадут в нее os.system(rm -rf /)? лучше уж так

[Andrey Dugin]

@Hateman31 Каким образом? Операция вводится пользователем отдельно, так что ставим элементарную проверку: if sim in "+-*/" ... - и всё. Числа проверить на то, что они числа (для этого подойдёт и текущая реализация, через int).

[Владислав]

@adugin ну ок