Script that counts quarters, dimes, nickels, and penniesGroup By And Two CountsCross-platform performance and...

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

Are small insurances worth it

Create chunks from an array

I can't die. Who am I?

I encountered my boss during an on-site interview at another company. Should I bring it up when seeing him next time?

Should I use HTTPS on a domain that will only be used for redirection?

How can I conditionally format my HTML table?

Ahoy, Ye Traveler!

Quitting employee has privileged access to critical information

What is the meaning of "notice to quit at once" and "Lotty points”

Can we carry rice to Japan?

How can neutral atoms have exactly zero electric field when there is a difference in the positions of the charges?

How to get the first element while continue streaming?

If nine coins are tossed, what is the probability that the number of heads is even?

Did Amazon pay $0 in taxes last year?

Would the melodic leap of the opening phrase of Mozart's K545 be considered dissonant?

Misplaced tyre lever - alternatives?

Caulking a corner instead of taping with joint compound?

Sometimes a banana is just a banana

Formatting a table to look nice

GDAL GetGeoTransform Documentation -- Is there an oversight, or what am I misunderstanding?

PTIJ: Should I stay away from my computer?

Giving a talk in my old university, how prominently should I tell students my salary?

A bug in Excel? Conditional formatting for marking duplicates also highlights unique value



Script that counts quarters, dimes, nickels, and pennies


Group By And Two CountsCross-platform performance and statistical information scriptGrouping logs and providing countsCounting single degree vertex removal rounds (from a graph)Let's protect that CLI Python scriptIterative “counting change” implementation in mit-schemeCalculating frequencies of each obs in the dataInfinite monkey theorem demonstration in PythonReturn the amount of Quarters, Dimes, Nickels, and Pennies that you would get from the total amount of changeSimple game of hangman which counts wins and losses













2












$begingroup$


I am learning Python and I wrote a script that counts how many coins you would need for an amount in dollars. I was wondering if I could make any improvements to it.



def change():
amnt = float(input("Enter an amount in USD: "))
quarters = divmod(amnt, 0.25)
print("Quarters: ", quarters[0])
amnt = round(quarters[1], 2)
dimes = divmod(amnt, 0.10)
print("Dimes: ", dimes[0])
amnt = round(dimes[1], 2)
nickels = divmod(amnt, 0.
print("Nickels: ", nickels[0])
amnt = round(nickels[1], 2)
penny = divmod(amnt, 0.01)
print("Pennies", penny[0])
change()









share|improve this question









New contributor




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







$endgroup$

















    2












    $begingroup$


    I am learning Python and I wrote a script that counts how many coins you would need for an amount in dollars. I was wondering if I could make any improvements to it.



    def change():
    amnt = float(input("Enter an amount in USD: "))
    quarters = divmod(amnt, 0.25)
    print("Quarters: ", quarters[0])
    amnt = round(quarters[1], 2)
    dimes = divmod(amnt, 0.10)
    print("Dimes: ", dimes[0])
    amnt = round(dimes[1], 2)
    nickels = divmod(amnt, 0.
    print("Nickels: ", nickels[0])
    amnt = round(nickels[1], 2)
    penny = divmod(amnt, 0.01)
    print("Pennies", penny[0])
    change()









    share|improve this question









    New contributor




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







    $endgroup$















      2












      2








      2





      $begingroup$


      I am learning Python and I wrote a script that counts how many coins you would need for an amount in dollars. I was wondering if I could make any improvements to it.



      def change():
      amnt = float(input("Enter an amount in USD: "))
      quarters = divmod(amnt, 0.25)
      print("Quarters: ", quarters[0])
      amnt = round(quarters[1], 2)
      dimes = divmod(amnt, 0.10)
      print("Dimes: ", dimes[0])
      amnt = round(dimes[1], 2)
      nickels = divmod(amnt, 0.
      print("Nickels: ", nickels[0])
      amnt = round(nickels[1], 2)
      penny = divmod(amnt, 0.01)
      print("Pennies", penny[0])
      change()









      share|improve this question









      New contributor




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







      $endgroup$




      I am learning Python and I wrote a script that counts how many coins you would need for an amount in dollars. I was wondering if I could make any improvements to it.



      def change():
      amnt = float(input("Enter an amount in USD: "))
      quarters = divmod(amnt, 0.25)
      print("Quarters: ", quarters[0])
      amnt = round(quarters[1], 2)
      dimes = divmod(amnt, 0.10)
      print("Dimes: ", dimes[0])
      amnt = round(dimes[1], 2)
      nickels = divmod(amnt, 0.
      print("Nickels: ", nickels[0])
      amnt = round(nickels[1], 2)
      penny = divmod(amnt, 0.01)
      print("Pennies", penny[0])
      change()






      python beginner python-3.x change-making-problem






      share|improve this question









      New contributor




      Hasan Qazi 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




      Hasan Qazi 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 20 mins ago









      200_success

      130k16153417




      130k16153417






      New contributor




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









      asked 3 hours ago









      Hasan QaziHasan Qazi

      132




      132




      New contributor




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





      New contributor





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






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






















          1 Answer
          1






          active

          oldest

          votes


















          4












          $begingroup$

          Separate input from processing. If you want to test your method with a number of different values, you'll have to call change() multiple times, and enter in the value each time. Instead, change the function to accept the amnt, and you can call it many times passing in the amount of cash as an argument:



          def change(amnt):
          # ...


          Working with tuples from divmod is awkward. Python has deconstructing assignment, which will take a returned tuple an assign the members to separate variables:



          def change(amnt):
          quarters, amnt = divmod(amnt, 0.25)
          print("Quarters: ", quarters)

          dimes, amnt = divmod(round(amnt, 2), 0.10)
          print("Dimes: ", dimes)


          For the last operation, you don't use the remainder, so the "throw-away" variable _ can be used for it:



              pennies, _ = divmod(round(amnt, 2), 0.01)
          print("Pennies: ", pennies)


          If you import this script into another program, you probably don't want the script to immediately run; rather you just want the change(amnt) function to be defined so this other program can call it. This is done by adding a "guard" at the end of the script, which only runs the code if the script is invoked directly:



          if __name__ == '__main__':
          amnt = float(input("Enter an amount in USD: "))
          change(amnt)




          In addition to separating input from processing, you might want to separate the processing from the output:



          def change(amnt):
          quarters, amnt = divmod(amnt, 0.25)
          dimes, amnt = divmod(amnt, 0.10)
          nickels, amnt = divmod(amnt, 0.05)
          pennies = round(amnt / 0.01, 0)

          return list(map(int, [quarters, dimes, nickels, pennies]))

          if __name__ == '__main__':
          amnt = float(input("Enter an amount in USD: "))
          quarters, dimes, nickels, pennies = change(amnt)
          print("{} quarters, {} dimes, {} nickels, {} pennies".format(
          quarters, dimes, nickels, pennies))




          Despite attempts to fix rounding errors with things like round(amnt,2), calling change(0.85) returns [3, 0, 1, 5], showing that there wasn't quite enough change to make 2 nickels, but after removing 1 nickel, approximately 5 pennies remained. This is caused by floating point math.



          We can avoid these issues by switching to integer math, based on the number of pennies:



          def change(amnt):
          pennies = round(amnt * 100) # Convert from dollars to pennies

          quarters, pennies = divmod(pennies, 25)
          dimes, pennies = divmod(pennies, 10)
          nickels, pennies = divmod(pennies, 5)

          return quarters, dimes, nickels, pennies

          if __name__ == '__main__':
          amnt = float(input("Enter an amount in USD: "))
          quarters, dimes, nickels, pennies = change(amnt)
          print("{} quarters, {} dimes, {} nickels, {} pennies".format(
          quarters, dimes, nickels, pennies))





          share|improve this answer











          $endgroup$













          • $begingroup$
            +1 for everything except your non-PEP8 formatting, even though it does look nice here.
            $endgroup$
            – Graipher
            50 mins ago











          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
          });


          }
          });






          Hasan Qazi 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%2f214888%2fscript-that-counts-quarters-dimes-nickels-and-pennies%23new-answer', 'question_page');
          }
          );

          Post as a guest















          Required, but never shown

























          1 Answer
          1






          active

          oldest

          votes








          1 Answer
          1






          active

          oldest

          votes









          active

          oldest

          votes






          active

          oldest

          votes









          4












          $begingroup$

          Separate input from processing. If you want to test your method with a number of different values, you'll have to call change() multiple times, and enter in the value each time. Instead, change the function to accept the amnt, and you can call it many times passing in the amount of cash as an argument:



          def change(amnt):
          # ...


          Working with tuples from divmod is awkward. Python has deconstructing assignment, which will take a returned tuple an assign the members to separate variables:



          def change(amnt):
          quarters, amnt = divmod(amnt, 0.25)
          print("Quarters: ", quarters)

          dimes, amnt = divmod(round(amnt, 2), 0.10)
          print("Dimes: ", dimes)


          For the last operation, you don't use the remainder, so the "throw-away" variable _ can be used for it:



              pennies, _ = divmod(round(amnt, 2), 0.01)
          print("Pennies: ", pennies)


          If you import this script into another program, you probably don't want the script to immediately run; rather you just want the change(amnt) function to be defined so this other program can call it. This is done by adding a "guard" at the end of the script, which only runs the code if the script is invoked directly:



          if __name__ == '__main__':
          amnt = float(input("Enter an amount in USD: "))
          change(amnt)




          In addition to separating input from processing, you might want to separate the processing from the output:



          def change(amnt):
          quarters, amnt = divmod(amnt, 0.25)
          dimes, amnt = divmod(amnt, 0.10)
          nickels, amnt = divmod(amnt, 0.05)
          pennies = round(amnt / 0.01, 0)

          return list(map(int, [quarters, dimes, nickels, pennies]))

          if __name__ == '__main__':
          amnt = float(input("Enter an amount in USD: "))
          quarters, dimes, nickels, pennies = change(amnt)
          print("{} quarters, {} dimes, {} nickels, {} pennies".format(
          quarters, dimes, nickels, pennies))




          Despite attempts to fix rounding errors with things like round(amnt,2), calling change(0.85) returns [3, 0, 1, 5], showing that there wasn't quite enough change to make 2 nickels, but after removing 1 nickel, approximately 5 pennies remained. This is caused by floating point math.



          We can avoid these issues by switching to integer math, based on the number of pennies:



          def change(amnt):
          pennies = round(amnt * 100) # Convert from dollars to pennies

          quarters, pennies = divmod(pennies, 25)
          dimes, pennies = divmod(pennies, 10)
          nickels, pennies = divmod(pennies, 5)

          return quarters, dimes, nickels, pennies

          if __name__ == '__main__':
          amnt = float(input("Enter an amount in USD: "))
          quarters, dimes, nickels, pennies = change(amnt)
          print("{} quarters, {} dimes, {} nickels, {} pennies".format(
          quarters, dimes, nickels, pennies))





          share|improve this answer











          $endgroup$













          • $begingroup$
            +1 for everything except your non-PEP8 formatting, even though it does look nice here.
            $endgroup$
            – Graipher
            50 mins ago
















          4












          $begingroup$

          Separate input from processing. If you want to test your method with a number of different values, you'll have to call change() multiple times, and enter in the value each time. Instead, change the function to accept the amnt, and you can call it many times passing in the amount of cash as an argument:



          def change(amnt):
          # ...


          Working with tuples from divmod is awkward. Python has deconstructing assignment, which will take a returned tuple an assign the members to separate variables:



          def change(amnt):
          quarters, amnt = divmod(amnt, 0.25)
          print("Quarters: ", quarters)

          dimes, amnt = divmod(round(amnt, 2), 0.10)
          print("Dimes: ", dimes)


          For the last operation, you don't use the remainder, so the "throw-away" variable _ can be used for it:



              pennies, _ = divmod(round(amnt, 2), 0.01)
          print("Pennies: ", pennies)


          If you import this script into another program, you probably don't want the script to immediately run; rather you just want the change(amnt) function to be defined so this other program can call it. This is done by adding a "guard" at the end of the script, which only runs the code if the script is invoked directly:



          if __name__ == '__main__':
          amnt = float(input("Enter an amount in USD: "))
          change(amnt)




          In addition to separating input from processing, you might want to separate the processing from the output:



          def change(amnt):
          quarters, amnt = divmod(amnt, 0.25)
          dimes, amnt = divmod(amnt, 0.10)
          nickels, amnt = divmod(amnt, 0.05)
          pennies = round(amnt / 0.01, 0)

          return list(map(int, [quarters, dimes, nickels, pennies]))

          if __name__ == '__main__':
          amnt = float(input("Enter an amount in USD: "))
          quarters, dimes, nickels, pennies = change(amnt)
          print("{} quarters, {} dimes, {} nickels, {} pennies".format(
          quarters, dimes, nickels, pennies))




          Despite attempts to fix rounding errors with things like round(amnt,2), calling change(0.85) returns [3, 0, 1, 5], showing that there wasn't quite enough change to make 2 nickels, but after removing 1 nickel, approximately 5 pennies remained. This is caused by floating point math.



          We can avoid these issues by switching to integer math, based on the number of pennies:



          def change(amnt):
          pennies = round(amnt * 100) # Convert from dollars to pennies

          quarters, pennies = divmod(pennies, 25)
          dimes, pennies = divmod(pennies, 10)
          nickels, pennies = divmod(pennies, 5)

          return quarters, dimes, nickels, pennies

          if __name__ == '__main__':
          amnt = float(input("Enter an amount in USD: "))
          quarters, dimes, nickels, pennies = change(amnt)
          print("{} quarters, {} dimes, {} nickels, {} pennies".format(
          quarters, dimes, nickels, pennies))





          share|improve this answer











          $endgroup$













          • $begingroup$
            +1 for everything except your non-PEP8 formatting, even though it does look nice here.
            $endgroup$
            – Graipher
            50 mins ago














          4












          4








          4





          $begingroup$

          Separate input from processing. If you want to test your method with a number of different values, you'll have to call change() multiple times, and enter in the value each time. Instead, change the function to accept the amnt, and you can call it many times passing in the amount of cash as an argument:



          def change(amnt):
          # ...


          Working with tuples from divmod is awkward. Python has deconstructing assignment, which will take a returned tuple an assign the members to separate variables:



          def change(amnt):
          quarters, amnt = divmod(amnt, 0.25)
          print("Quarters: ", quarters)

          dimes, amnt = divmod(round(amnt, 2), 0.10)
          print("Dimes: ", dimes)


          For the last operation, you don't use the remainder, so the "throw-away" variable _ can be used for it:



              pennies, _ = divmod(round(amnt, 2), 0.01)
          print("Pennies: ", pennies)


          If you import this script into another program, you probably don't want the script to immediately run; rather you just want the change(amnt) function to be defined so this other program can call it. This is done by adding a "guard" at the end of the script, which only runs the code if the script is invoked directly:



          if __name__ == '__main__':
          amnt = float(input("Enter an amount in USD: "))
          change(amnt)




          In addition to separating input from processing, you might want to separate the processing from the output:



          def change(amnt):
          quarters, amnt = divmod(amnt, 0.25)
          dimes, amnt = divmod(amnt, 0.10)
          nickels, amnt = divmod(amnt, 0.05)
          pennies = round(amnt / 0.01, 0)

          return list(map(int, [quarters, dimes, nickels, pennies]))

          if __name__ == '__main__':
          amnt = float(input("Enter an amount in USD: "))
          quarters, dimes, nickels, pennies = change(amnt)
          print("{} quarters, {} dimes, {} nickels, {} pennies".format(
          quarters, dimes, nickels, pennies))




          Despite attempts to fix rounding errors with things like round(amnt,2), calling change(0.85) returns [3, 0, 1, 5], showing that there wasn't quite enough change to make 2 nickels, but after removing 1 nickel, approximately 5 pennies remained. This is caused by floating point math.



          We can avoid these issues by switching to integer math, based on the number of pennies:



          def change(amnt):
          pennies = round(amnt * 100) # Convert from dollars to pennies

          quarters, pennies = divmod(pennies, 25)
          dimes, pennies = divmod(pennies, 10)
          nickels, pennies = divmod(pennies, 5)

          return quarters, dimes, nickels, pennies

          if __name__ == '__main__':
          amnt = float(input("Enter an amount in USD: "))
          quarters, dimes, nickels, pennies = change(amnt)
          print("{} quarters, {} dimes, {} nickels, {} pennies".format(
          quarters, dimes, nickels, pennies))





          share|improve this answer











          $endgroup$



          Separate input from processing. If you want to test your method with a number of different values, you'll have to call change() multiple times, and enter in the value each time. Instead, change the function to accept the amnt, and you can call it many times passing in the amount of cash as an argument:



          def change(amnt):
          # ...


          Working with tuples from divmod is awkward. Python has deconstructing assignment, which will take a returned tuple an assign the members to separate variables:



          def change(amnt):
          quarters, amnt = divmod(amnt, 0.25)
          print("Quarters: ", quarters)

          dimes, amnt = divmod(round(amnt, 2), 0.10)
          print("Dimes: ", dimes)


          For the last operation, you don't use the remainder, so the "throw-away" variable _ can be used for it:



              pennies, _ = divmod(round(amnt, 2), 0.01)
          print("Pennies: ", pennies)


          If you import this script into another program, you probably don't want the script to immediately run; rather you just want the change(amnt) function to be defined so this other program can call it. This is done by adding a "guard" at the end of the script, which only runs the code if the script is invoked directly:



          if __name__ == '__main__':
          amnt = float(input("Enter an amount in USD: "))
          change(amnt)




          In addition to separating input from processing, you might want to separate the processing from the output:



          def change(amnt):
          quarters, amnt = divmod(amnt, 0.25)
          dimes, amnt = divmod(amnt, 0.10)
          nickels, amnt = divmod(amnt, 0.05)
          pennies = round(amnt / 0.01, 0)

          return list(map(int, [quarters, dimes, nickels, pennies]))

          if __name__ == '__main__':
          amnt = float(input("Enter an amount in USD: "))
          quarters, dimes, nickels, pennies = change(amnt)
          print("{} quarters, {} dimes, {} nickels, {} pennies".format(
          quarters, dimes, nickels, pennies))




          Despite attempts to fix rounding errors with things like round(amnt,2), calling change(0.85) returns [3, 0, 1, 5], showing that there wasn't quite enough change to make 2 nickels, but after removing 1 nickel, approximately 5 pennies remained. This is caused by floating point math.



          We can avoid these issues by switching to integer math, based on the number of pennies:



          def change(amnt):
          pennies = round(amnt * 100) # Convert from dollars to pennies

          quarters, pennies = divmod(pennies, 25)
          dimes, pennies = divmod(pennies, 10)
          nickels, pennies = divmod(pennies, 5)

          return quarters, dimes, nickels, pennies

          if __name__ == '__main__':
          amnt = float(input("Enter an amount in USD: "))
          quarters, dimes, nickels, pennies = change(amnt)
          print("{} quarters, {} dimes, {} nickels, {} pennies".format(
          quarters, dimes, nickels, pennies))






          share|improve this answer














          share|improve this answer



          share|improve this answer








          edited 2 hours ago

























          answered 2 hours ago









          AJNeufeldAJNeufeld

          5,7791420




          5,7791420












          • $begingroup$
            +1 for everything except your non-PEP8 formatting, even though it does look nice here.
            $endgroup$
            – Graipher
            50 mins ago


















          • $begingroup$
            +1 for everything except your non-PEP8 formatting, even though it does look nice here.
            $endgroup$
            – Graipher
            50 mins ago
















          $begingroup$
          +1 for everything except your non-PEP8 formatting, even though it does look nice here.
          $endgroup$
          – Graipher
          50 mins ago




          $begingroup$
          +1 for everything except your non-PEP8 formatting, even though it does look nice here.
          $endgroup$
          – Graipher
          50 mins ago










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










          draft saved

          draft discarded


















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













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












          Hasan Qazi 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%2f214888%2fscript-that-counts-quarters-dimes-nickels-and-pennies%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

          “%fieldName is a required field.”, in Magento2 REST API Call for GET Method Type The Next...

          How to change City field to a dropdown in Checkout step Magento 2Magento 2 : How to change UI field(s)...

          變成蝙蝠會怎樣? 參考資料 外部連結 导航菜单Thomas Nagel, "What is it like to be a...