Dtjytyk

Learning to quickly identify valid fingering for piano?

Why would the IRS ask for birth certificates or even audit a small tax return?

What does "rhumatis" mean?

Sundering Titan and basic normal lands and snow lands

I've given my players a lot of magic items. Is it reasonable for me to give them harder encounters?

Is this nominative case or accusative case?

What is the oldest European royal house?

Does the in-code argument passing conventions used on PDP-11's have a name?

What is the purpose of a disclaimer like "this is not legal advice"?

Align equations with text before one of them

Is there a way to find out the age of climbing ropes?

Why do we call complex numbers “numbers” but we don’t consider 2 vectors numbers?

Why are special aircraft used for the carriers in the United States Navy?

Are there other characters in the Star Wars universe who had damaged bodies and needed to wear an outfit like Darth Vader?

How spaceships determine each other's mass in space?

Create chunks from an array

Is there a math equivalent to the conditional ternary operator?

How do you make a gun that shoots melee weapons and/or swords?

How to write a chaotic neutral protagonist and prevent my readers from thinking they are evil?

Was it really inappropriate to write a pull request for the company I interviewed with?

Rationale to prefer local variables over instance variables?

The need of reserving one's ability in job interviews

Can you run a ground wire from stove directly to ground pole in the ground

Linear Combination of Atomic Orbitals

Python 3.6+ function to ask for a multiple-choice answer

Simple multiple choice quizPython bot to answer mathematical questions for a remote serverRun multiple python codes and Check with answer dataImproved version of “Let's read a random Goodreads book…”Bruteforce hashes using Python => 3.6 (Update)Lovely Lucky LAMBsPython 3.6 Rock-Paper-Scissors gameSort an iterable in Python 3.6First Hangman game in Python 3.6Python 3.6 Dice Histogram: Using random.seed effectively

5

$begingroup$

# Standard multi choice question template
def multiChoiceQuestion(options: list):
 while True:
 print("nEnter the number of your choice - ")
 for x in range(len(options)):
 print(str((x + 1)) + ". " + options[x])
 print("n")
 try:
 answer = int(input())
 except ValueError:
 print("Doesn't seem like a number! Try again!")
 continue
 if answer < 1 or answer > len(options):
 print("That option does not exist! Try again!")
 continue
 return answer

I created a template to ask a multi choice question in python. The loop will never reach it's end, since there is always a continue or a return statement. Is the while True condition appropriate for it?

python python-3.x validation

share|improve this question

edited 23 mins ago

Holyprogrammer

asked 12 hours ago

HolyprogrammerHolyprogrammer

1568

New contributor

Holyprogrammer is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
Check out our Code of Conduct.

$endgroup$

• 
  
  
  

  
  

  
  
  
  
  
  

  
  $begingroup$
  "The loop will never reach it's end ... Is the while True condition appropriate for it?" That depends on whether that is the intended behaviour. Is it?
  $endgroup$
  – Mast
  11 hours ago
  

  
  
  
  


• 
  
  
  

  
  1
  

  
  
  
  
  
  

  
  $begingroup$
  Side note: for x in len(options): will produce an error as Python doesn't allow iteration over an integer. 200_success's approach is the way to go here, but for future reference, use for x in range(len(options)): if you really need to loop a certain number of times. This takes the len(options) integer and creates an interable out of it.
  $endgroup$
  – JDG
  9 hours ago
  
  

  
  
  
  

add a comment | 

5

$begingroup$

# Standard multi choice question template
def multiChoiceQuestion(options: list):
 while True:
 print("nEnter the number of your choice - ")
 for x in range(len(options)):
 print(str((x + 1)) + ". " + options[x])
 print("n")
 try:
 answer = int(input())
 except ValueError:
 print("Doesn't seem like a number! Try again!")
 continue
 if answer < 1 or answer > len(options):
 print("That option does not exist! Try again!")
 continue
 return answer

I created a template to ask a multi choice question in python. The loop will never reach it's end, since there is always a continue or a return statement. Is the while True condition appropriate for it?

python python-3.x validation

share|improve this question

edited 23 mins ago

Holyprogrammer

asked 12 hours ago

HolyprogrammerHolyprogrammer

1568

New contributor

Holyprogrammer is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
Check out our Code of Conduct.

$endgroup$

• 
  
  
  

  
  

  
  
  
  
  
  

  
  $begingroup$
  "The loop will never reach it's end ... Is the while True condition appropriate for it?" That depends on whether that is the intended behaviour. Is it?
  $endgroup$
  – Mast
  11 hours ago
  

  
  
  
  


• 
  
  
  

  
  1
  

  
  
  
  
  
  

  
  $begingroup$
  Side note: for x in len(options): will produce an error as Python doesn't allow iteration over an integer. 200_success's approach is the way to go here, but for future reference, use for x in range(len(options)): if you really need to loop a certain number of times. This takes the len(options) integer and creates an interable out of it.
  $endgroup$
  – JDG
  9 hours ago
  
  

  
  
  
  

add a comment | 

$begingroup$

# Standard multi choice question template
def multiChoiceQuestion(options: list):
 while True:
 print("nEnter the number of your choice - ")
 for x in range(len(options)):
 print(str((x + 1)) + ". " + options[x])
 print("n")
 try:
 answer = int(input())
 except ValueError:
 print("Doesn't seem like a number! Try again!")
 continue
 if answer < 1 or answer > len(options):
 print("That option does not exist! Try again!")
 continue
 return answer

I created a template to ask a multi choice question in python. The loop will never reach it's end, since there is always a continue or a return statement. Is the while True condition appropriate for it?

python python-3.x validation

share|improve this question

edited 23 mins ago

Holyprogrammer

asked 12 hours ago

HolyprogrammerHolyprogrammer

1568

New contributor

Holyprogrammer is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
Check out our Code of Conduct.

$endgroup$

# Standard multi choice question template
def multiChoiceQuestion(options: list):
 while True:
 print("nEnter the number of your choice - ")
 for x in range(len(options)):
 print(str((x + 1)) + ". " + options[x])
 print("n")
 try:
 answer = int(input())
 except ValueError:
 print("Doesn't seem like a number! Try again!")
 continue
 if answer < 1 or answer > len(options):
 print("That option does not exist! Try again!")
 continue
 return answer

share|improve this question

edited 23 mins ago

Holyprogrammer

asked 12 hours ago

HolyprogrammerHolyprogrammer

1568

New contributor

Holyprogrammer is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
Check out our Code of Conduct.

share|improve this question

edited 23 mins ago

Holyprogrammer

asked 12 hours ago

HolyprogrammerHolyprogrammer

1568

New contributor

Holyprogrammer is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
Check out our Code of Conduct.

asked 12 hours ago

HolyprogrammerHolyprogrammer

1568

New contributor

Holyprogrammer is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
Check out our Code of Conduct.

asked 12 hours ago

HolyprogrammerHolyprogrammer

1568

2 Answers
2

active

oldest

votes

6

$begingroup$

The while True is fine, and is probably the best way to do it. However, the rest of the flow control is a bit clumsy. By rearranging a few statements, you can eliminate the continues.

PEP 8, the official Python style guide, recommends lowercase_with_underscores for function names unless you have a good reason to deviate.

The loop to print the numbered menu would be better written using enumerate(). Also, Python supports double-ended comparisons for validating that the answer is in range.

def multi_choice_question(options: list):
 while True:
 print("nEnter the number of your choice - ")
 for i, option in enumerate(options, 1):
 print(f'i. option')
 print("n")
 try:
 answer = int(input())
 if 1 <= answer <= len(options):
 return answer
 print("That option does not exist! Try again!")
 except ValueError:
 print("Doesn't seem like a number! Try again!")

share|improve this answer

edited 10 mins ago

Holyprogrammer

1568

answered 11 hours ago

200_success200_success

130k16153419

$endgroup$

• 
  
  
  

  
  2
  

  
  
  
  
  
  

  
  $begingroup$
  Is it standard now in python 3.x to use f-strings? print(f'i. option')
  $endgroup$
  – JDG
  9 hours ago
  
  

  
  
  
  

add a comment | 

3

$begingroup$

I think that 200_success already covered most points. I would however like to add an alternative idea for the printing part:

print("Enter the number of your choice -",
 *(f'i. opt' for i, opt in enumerate(options, 1)),
 sep='n', end='nn')

Explanation:
from the docs we see that following signature for the print function:

 print(*objects, sep=' ', end='n', file=sys.stdout, flush=False)

we can therefore print everything with a single print call instead of three individual ones. I leave it up to you which one you perceive easier to use.

share|improve this answer

answered 5 hours ago

magu_magu_

4931519

$endgroup$

add a comment | 

Your Answer

StackExchange.ifUsing("editor", function ()
return StackExchange.using("mathjaxEditing", function ()
StackExchange.MarkdownEditor.creationCallbacks.add(function (editor, postfix)
StackExchange.mathjaxEditing.prepareWmdForMathJax(editor, postfix, [["\$", "\$"]]);
);
);
, "mathjax-editing");

StackExchange.ifUsing("editor", function ()
StackExchange.using("externalEditor", function ()
StackExchange.using("snippets", function ()
StackExchange.snippets.init();
);
);
, "code-snippets");

StackExchange.ready(function()
var channelOptions =
tags: "".split(" "),
id: "196"
;
initTagRenderer("".split(" "), "".split(" "), channelOptions);

StackExchange.using("externalEditor", function()
// Have to fire editor after snippets, if snippets enabled
if (StackExchange.settings.snippets.snippetsEnabled)
StackExchange.using("snippets", function()
createEditor();
);

else
createEditor();

);

function createEditor()
StackExchange.prepareEditor(
heartbeatType: 'answer',
autoActivateHeartbeat: false,
convertImagesToLinks: false,
noModals: true,
showLowRepImageUploadWarning: true,
reputationToPostImages: null,
bindNavPrevention: true,
postfix: "",
imageUploader:
brandingHtml: "Powered by u003ca class="icon-imgur-white" href="https://imgur.com/"u003eu003c/au003e",
contentPolicyHtml: "User contributions licensed under u003ca href="https://creativecommons.org/licenses/by-sa/3.0/"u003ecc by-sa 3.0 with attribution requiredu003c/au003e u003ca href="https://stackoverflow.com/legal/content-policy"u003e(content policy)u003c/au003e",
allowUrls: true
,
onDemand: true,
discardSelector: ".discard-answer"
,immediatelyShowMarkdownHelp:true
);



);

Holyprogrammer is a new contributor. Be nice, and check out our Code of Conduct.

draft saved

draft discarded

Sign up or log in

StackExchange.ready(function ()
StackExchange.helpers.onClickDraftSave('#login-link');
);

Sign up using Google

Sign up using Facebook

Sign up using Email and Password

Post as a guest

Name

Email

Required, but never shown

StackExchange.ready(
function ()
StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fcodereview.stackexchange.com%2fquestions%2f214935%2fpython-3-6-function-to-ask-for-a-multiple-choice-answer%23new-answer', 'question_page');

);

Post as a guest

Name

Email

Required, but never shown

6

$begingroup$

The while True is fine, and is probably the best way to do it. However, the rest of the flow control is a bit clumsy. By rearranging a few statements, you can eliminate the continues.

PEP 8, the official Python style guide, recommends lowercase_with_underscores for function names unless you have a good reason to deviate.

The loop to print the numbered menu would be better written using enumerate(). Also, Python supports double-ended comparisons for validating that the answer is in range.

def multi_choice_question(options: list):
 while True:
 print("nEnter the number of your choice - ")
 for i, option in enumerate(options, 1):
 print(f'i. option')
 print("n")
 try:
 answer = int(input())
 if 1 <= answer <= len(options):
 return answer
 print("That option does not exist! Try again!")
 except ValueError:
 print("Doesn't seem like a number! Try again!")

share|improve this answer

edited 10 mins ago

Holyprogrammer

1568

answered 11 hours ago

200_success200_success

130k16153419

$endgroup$

• 
  
  
  

  
  2
  

  
  
  
  
  
  

  
  $begingroup$
  Is it standard now in python 3.x to use f-strings? print(f'i. option')
  $endgroup$
  – JDG
  9 hours ago
  
  

  
  
  
  

add a comment | 

6

$begingroup$

The while True is fine, and is probably the best way to do it. However, the rest of the flow control is a bit clumsy. By rearranging a few statements, you can eliminate the continues.

PEP 8, the official Python style guide, recommends lowercase_with_underscores for function names unless you have a good reason to deviate.

The loop to print the numbered menu would be better written using enumerate(). Also, Python supports double-ended comparisons for validating that the answer is in range.

def multi_choice_question(options: list):
 while True:
 print("nEnter the number of your choice - ")
 for i, option in enumerate(options, 1):
 print(f'i. option')
 print("n")
 try:
 answer = int(input())
 if 1 <= answer <= len(options):
 return answer
 print("That option does not exist! Try again!")
 except ValueError:
 print("Doesn't seem like a number! Try again!")

share|improve this answer

edited 10 mins ago

Holyprogrammer

1568

answered 11 hours ago

200_success200_success

130k16153419

$endgroup$

• 
  
  
  

  
  2
  

  
  
  
  
  
  

  
  $begingroup$
  Is it standard now in python 3.x to use f-strings? print(f'i. option')
  $endgroup$
  – JDG
  9 hours ago
  
  

  
  
  
  

add a comment | 

$begingroup$

The while True is fine, and is probably the best way to do it. However, the rest of the flow control is a bit clumsy. By rearranging a few statements, you can eliminate the continues.

PEP 8, the official Python style guide, recommends lowercase_with_underscores for function names unless you have a good reason to deviate.

The loop to print the numbered menu would be better written using enumerate(). Also, Python supports double-ended comparisons for validating that the answer is in range.

def multi_choice_question(options: list):
 while True:
 print("nEnter the number of your choice - ")
 for i, option in enumerate(options, 1):
 print(f'i. option')
 print("n")
 try:
 answer = int(input())
 if 1 <= answer <= len(options):
 return answer
 print("That option does not exist! Try again!")
 except ValueError:
 print("Doesn't seem like a number! Try again!")

share|improve this answer

edited 10 mins ago

Holyprogrammer

1568

answered 11 hours ago

200_success200_success

130k16153419

$endgroup$

def multi_choice_question(options: list):
 while True:
 print("nEnter the number of your choice - ")
 for i, option in enumerate(options, 1):
 print(f'i. option')
 print("n")
 try:
 answer = int(input())
 if 1 <= answer <= len(options):
 return answer
 print("That option does not exist! Try again!")
 except ValueError:
 print("Doesn't seem like a number! Try again!")

share|improve this answer

edited 10 mins ago

Holyprogrammer

1568

answered 11 hours ago

200_success200_success

130k16153419

edited 10 mins ago

Holyprogrammer

1568

edited 10 mins ago

Holyprogrammer

1568

answered 11 hours ago

200_success200_success

130k16153419

answered 11 hours ago

200_success200_success

130k16153419

3

$begingroup$

I think that 200_success already covered most points. I would however like to add an alternative idea for the printing part:

print("Enter the number of your choice -",
 *(f'i. opt' for i, opt in enumerate(options, 1)),
 sep='n', end='nn')

Explanation:
from the docs we see that following signature for the print function:

 print(*objects, sep=' ', end='n', file=sys.stdout, flush=False)

we can therefore print everything with a single print call instead of three individual ones. I leave it up to you which one you perceive easier to use.

share|improve this answer

answered 5 hours ago

magu_magu_

4931519

$endgroup$

add a comment | 

3

$begingroup$

I think that 200_success already covered most points. I would however like to add an alternative idea for the printing part:

print("Enter the number of your choice -",
 *(f'i. opt' for i, opt in enumerate(options, 1)),
 sep='n', end='nn')

Explanation:
from the docs we see that following signature for the print function:

 print(*objects, sep=' ', end='n', file=sys.stdout, flush=False)

we can therefore print everything with a single print call instead of three individual ones. I leave it up to you which one you perceive easier to use.

share|improve this answer

answered 5 hours ago

magu_magu_

4931519

$endgroup$

add a comment | 

$begingroup$

I think that 200_success already covered most points. I would however like to add an alternative idea for the printing part:

print("Enter the number of your choice -",
 *(f'i. opt' for i, opt in enumerate(options, 1)),
 sep='n', end='nn')

Explanation:
from the docs we see that following signature for the print function:

 print(*objects, sep=' ', end='n', file=sys.stdout, flush=False)

we can therefore print everything with a single print call instead of three individual ones. I leave it up to you which one you perceive easier to use.

share|improve this answer

answered 5 hours ago

magu_magu_

4931519

$endgroup$

print("Enter the number of your choice -",
 *(f'i. opt' for i, opt in enumerate(options, 1)),
 sep='n', end='nn')

 print(*objects, sep=' ', end='n', file=sys.stdout, flush=False)

share|improve this answer

answered 5 hours ago

magu_magu_

4931519

answered 5 hours ago

magu_magu_

4931519

answered 5 hours ago

magu_magu_

4931519