Почему код не работает?

Question

[Maximer2016]

Я крч нуб нубов, не смейтесь, но помогите создать систему пароля, тип вводишь пароль и тебе доступно всё, а если нет, то нет. (

password = input(Pass: ")
bool1 = Welcome
bool2 = access_denied

result = bool2

if password == "555":
    result = bool1
else:
    print(bool2)

if result is not bool2:
    print("Password:", result)

Comments

[Евгений]

Вам надо начинать с этого: https://docs.python.org/3/tutorial/index.html
Думаю, что русский перевод можно легко найти.

Answer 1

[Anatoly Scherbakov]

• Строковые переменные не в кавычках
  
• Не надо сравнивать result с bool2, ведь вы только что туда записали значение. Достаточно только одного оператора if. Попробуйте :)
  

Comments

[javedimka]

Да и вообще можно в одну строчку уложиться, с принтом в две.

[Anatoly Scherbakov]

javedimka: ну зачем в одну. Мне нравится так:

print {
    True: "Launching missiles.",
    False: "Access denied."
}[input("Password: ") == "123456"]

https://repl.it/IP9p/0 Но не стоит новичкам предлагать так писать :)

[Влад Григорьев]

Зачем вызывать конструктор dict, когда можно использовать тернарную операцию

print("Access denied." if input("Password: ") == "123456" else "Launching missiles.")

[Anatoly Scherbakov]

Влад Григорьев: можно, но эта операция очень длинна и легко выйдет за пределы разрешённой длины строки в 80 символов. Можно использовать скобки, чтоб разбить на строки, но всё равно как-то не то.

Субъективно мне кажется, что dict читается лучше. Кроме того, часто бывает, что вариантов больше одного - его преимущество становится ещё более очевидным. Такое слабое подобие оператора match в Scala.

[Влад Григорьев]

Если условий больше то да, а если 2 то совершенно нет смысла вызывать лишнии конструкторы. и увеличивать расход ресурсов.

Answer 2

[Ivanq]

password = input(Pass: ") # Кавычку перед Pass: забыли
bool1 = Welcome # В кавычки
bool2 = access_denied # В кавычки

result = bool2 # Лучше перенести после else:

if password == "555":
    result = bool1
else:
    print(bool2)

if result is not bool2: # Лучше result != bool2
    print("Password:", result)

А я бы написал так:

Передумал

Answer 3

[Alex F]

Писать доступ по паролю на скриптовом языке - моветон.

Comments

[Anatoly Scherbakov]

Человек просто изучает язык, чего придираетесь :)

[Alex F]

Anatoly Scherbakov: Отнюдь, не придираюсь.
Когда я взялся за Delphi(после ассма) - была необходимость делать формы для отчетов из БД. Потом вернулся в ассм, ибо была необходимость работать RT на устройствах. Когда взялся за Python была необходимость автоматизировать некие процессы в системе(да и живой интерес, пожалуй). Когда надо было сделать небольшую систему безопасности, написал ее опять таки на ассме(правда защита была примитивной - по XOR).

Ну а теперь о "начинает изучать"... Ну что, правда нет других задач, кроме проверки пароля? Поэтому написал мнение.

P.S: Я еще и бейсик "Микрон" знаю ;)