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












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?










share|improve this question









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


















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?










share|improve this question









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
















5












5








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?










share|improve this question









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


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









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









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




share|improve this question








edited 11 mins ago







Holyprogrammer













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 11 hours ago









HolyprogrammerHolyprogrammer

1548




1548




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.





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.






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












  • $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




















  • $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


















$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




$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




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






$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












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('{0}. {1}'.format(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









$endgroup$









  • 2




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








  • 3




    $begingroup$
    @JDG I wouldn't assume Python ≥ 3.6 if the question didn't specify it.
    $endgroup$
    – 200_success
    9 hours ago



















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









$endgroup$













    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


















    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















    Required, but never shown

























    2 Answers
    2






    active

    oldest

    votes








    2 Answers
    2






    active

    oldest

    votes









    active

    oldest

    votes






    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('{0}. {1}'.format(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









    $endgroup$









    • 2




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








    • 3




      $begingroup$
      @JDG I wouldn't assume Python ≥ 3.6 if the question didn't specify it.
      $endgroup$
      – 200_success
      9 hours ago
















    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('{0}. {1}'.format(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









    $endgroup$









    • 2




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








    • 3




      $begingroup$
      @JDG I wouldn't assume Python ≥ 3.6 if the question didn't specify it.
      $endgroup$
      – 200_success
      9 hours ago














    6












    6








    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('{0}. {1}'.format(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









    $endgroup$



    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('{0}. {1}'.format(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











    share


    share










    answered 11 hours ago









    200_success200_success

    130k16153419




    130k16153419








    • 2




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








    • 3




      $begingroup$
      @JDG I wouldn't assume Python ≥ 3.6 if the question didn't specify it.
      $endgroup$
      – 200_success
      9 hours ago














    • 2




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








    • 3




      $begingroup$
      @JDG I wouldn't assume Python ≥ 3.6 if the question didn't specify it.
      $endgroup$
      – 200_success
      9 hours ago








    2




    2




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






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






    3




    3




    $begingroup$
    @JDG I wouldn't assume Python ≥ 3.6 if the question didn't specify it.
    $endgroup$
    – 200_success
    9 hours ago




    $begingroup$
    @JDG I wouldn't assume Python ≥ 3.6 if the question didn't specify it.
    $endgroup$
    – 200_success
    9 hours ago













    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









    $endgroup$


















      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









      $endgroup$
















        3












        3








        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









        $endgroup$



        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












        share|improve this answer



        share|improve this answer










        answered 5 hours ago









        magu_magu_

        4931519




        4931519






















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










            draft saved

            draft discarded


















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













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












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
















            Thanks for contributing an answer to Code Review Stack Exchange!


            • Please be sure to answer the question. Provide details and share your research!

            But avoid



            • Asking for help, clarification, or responding to other answers.

            • Making statements based on opinion; back them up with references or personal experience.


            Use MathJax to format equations. MathJax reference.


            To learn more, see our tips on writing great answers.




            draft saved


            draft discarded














            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















            Required, but never shown





















































            Required, but never shown














            Required, but never shown












            Required, but never shown







            Required, but never shown

































            Required, but never shown














            Required, but never shown












            Required, but never shown







            Required, but never shown







            Popular posts from this blog

            Knooppunt Holsloot

            Altaar (religie)

            Gregoriusmis