Как правильно добавить if в скрипт?

Question

[EAleg]

interface = input("Enter interface number: ")
vlan = input("Enter vlan number: ")
desc = input("Enter description of interface: ")
operation = input(""""Please choose switch mode on interface: 
	Access 				Trunk
	   1				  2
			""")

access_mode = ["interface GigabitEthernet {}",
				"switchport mode access",
				"switchport access vlan {}",
				"Description {}",
				"No shutdown"]

trunk_mode = ["interface GigabitEthernet {}",
				"switchport mode trunk",
				"switchport trunk allowed vlan {}",
				"Description {}",
				"No shutdown"]


if operation == "Access" or operation == "1":
	print("-" * 35)
	print("\n".join(access_mode).format(interface, vlan, desc))
	print("-" * 35)


if operation == "Trunk" or operation == "2":
	print("-" * 35)
	print("\n".join(trunk_mode).format(interface, vlan, desc))
	print("-" * 35)

Куда нужно вставить if, чтобы максимальное значение, которое можно было задать для Vlan было 4096?
И ещё, нужно ли сюда добавлять функцию, или она здесь не нужна?
П.с. только учусь писать, поэтому и сам скрипт выглядит через 5ю точку.
Спасибо за внимание, ответ и понимание!
Всем хорошего дня (=

Comments

[User]

vlan = ''
while len(vlan) < 1 or len(vlan) > 4096:
    vlan = input("Enter vlan number: ")
print(vlan)

Затестите

Answer 1

[nochkin]

Для начала надо "vlan" перевести в int, что бы можно было сравнить с 4096. А потом уже делать сравнение.
Что-то типа такого:

try:
  vlan = int(vlan)
except ValueError:
  print("vlan must be int")
  return

После этого можно проверять:

if vlan > 4096:
  print("vlan can not exceed 4096")
  return

P.S.: если есть проверка верхнего уровня, то, пожалуй, есть смысл проверить нижний так же. Если так, то проверка будет примерно такая (проверяет что бы vlan не был меньше 1 и больше 4096):

if not vlan in range(1, 4096):
  print("vlan must be between 1 and 4096")
  return

Comments

[EAleg]

Добрый день, спасибо за ответ! А скажи пожалуйста, как правильно это написать,т.е, чтобы при неверном вводе значения, оно написало, что значение неверно, и вернуло обратно в поле для ввода влан?

[nochkin]

Например, можно завернуть в while цикл. Что-то типа такого:

while True:
  input...
  input...
  input...

  if ... :
    print("incorrect input")
    continue

  if ... :
    print("good input")
    break

Мои "return" надо будет заменить на "continue".
То есть если результат верный, то достаточно сделать "break" что бы while закончил повторять.

[EAleg]

nochkin, Добрый вечер, только что попробовал один из ваших ответов!
Вот код

access_mode = ["interface GigabitEthernet {}",
				"switchport mode access",
				"switchport access vlan {}",
				"Description {}",
				"No shutdown"]

trunk_mode = ["interface GigabitEthernet {}",
				"switchport mode trunk",
				"switchport trunk allowed vlan {}",
				"Description {}",
				"No shutdown"]

interface = input("Enter interface number: ")
vlan = int(input("Enter vlan number: "))
desc = input("Enter description of interface: ")
operation = input(""""Please choose switch mode on interface: 
	Access 				Trunk
	   1				  2
			""")

if vlan > int(4096):
	print("Вы указали неверный номер влана!")
	return
if operation == "Access" or operation == "1":
	print("-" * 35)
	print("\n".join(access_mode).format(interface, vlan, desc))
	print("-" * 35)


if operation == "Trunk" or operation == "2":
	print("-" * 35)
	print("\n".join(trunk_mode).format(interface, vlan, desc))
	print("-" * 35)

Я вставил return, но оно пишет вот такую ошибку:


Хотя странно, причём тут функция?

[nochkin]

"return" работает внутри функций только. Я добавил return как пример, так как там надо остановиться и я предположил, что код оформлен внутри функции (ну, это было бы логично так делать).

Во втором комментарии я предложил завернуть это в "while True" и заменить "return" на "continue" и добавить "break" где надо.

P.S.: к коду это не имеет отношения, но лучше либо все сообщения на английском (пусть и ломанном) или все на русском. Но когда мешать языки, то выглядит очень неопрятно даже для самого себя.