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












4












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


















4












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
















4












4








4





$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 2 hours 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 14 hours ago









HolyprogrammerHolyprogrammer

1568




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.





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
    13 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
    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
    13 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
    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
13 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
13 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
11 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
11 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(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











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













      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(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











      $endgroup$


















        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











        $endgroup$
















          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(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











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














          share|improve this answer



          share|improve this answer








          edited 2 hours ago









          Holyprogrammer

          1568




          1568










          answered 13 hours ago









          200_success200_success

          130k16153419




          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









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

                      日野市

                      GameSpot

                      Tu-95轟炸機