Python - Fishing Simulator The 2019 Stack Overflow Developer Survey Results Are In Announcing the arrival of Valued Associate #679: Cesar Manara Planned maintenance scheduled April 17/18, 2019 at 00:00UTC (8:00pm US/Eastern)Turn-based battle simulatorBattle simulator RPGScoring and grading answers against an answer keyC++ Quiz game w/questions in random orderPython PID simulator controller outputRandom MP3 Selector and PlayerMonty Hall Simulator - Python“The Story of a Tree” solved using depth-first searchPython CS GO Case SimulatorGame of Life simulator, Python 3

Are spiders unable to hurt humans, especially very small spiders?

Windows 10: How to Lock (not sleep) laptop on lid close?

Loose spokes after only a few rides

Presidential Pardon

The following signatures were invalid: EXPKEYSIG 1397BC53640DB551

Can the DM override racial traits?

Was credit for the black hole image misappropriated?

What aspect of planet Earth must be changed to prevent the industrial revolution?

Identify 80s or 90s comics with ripped creatures (not dwarves)

Is this wall load bearing? Blueprints and photos attached

Is it ethical to upload a automatically generated paper to a non peer-reviewed site as part of a larger research?

What can I do if neighbor is blocking my solar panels intentionally?

Would an alien lifeform be able to achieve space travel if lacking in vision?

Homework question about an engine pulling a train

Is there a writing software that you can sort scenes like slides in PowerPoint?

Do warforged have souls?

What information about me do stores get via my credit card?

One-dimensional Japanese puzzle

Do working physicists consider Newtonian mechanics to be "falsified"?

Does Parliament need to approve the new Brexit delay to 31 October 2019?

Do I have Disadvantage attacking with an off-hand weapon?

Button changing its text & action. Good or terrible?

should truth entail possible truth

"is" operation returns false even though two objects have same id



Python - Fishing Simulator



The 2019 Stack Overflow Developer Survey Results Are In
Announcing the arrival of Valued Associate #679: Cesar Manara
Planned maintenance scheduled April 17/18, 2019 at 00:00UTC (8:00pm US/Eastern)Turn-based battle simulatorBattle simulator RPGScoring and grading answers against an answer keyC++ Quiz game w/questions in random orderPython PID simulator controller outputRandom MP3 Selector and PlayerMonty Hall Simulator - Python“The Story of a Tree” solved using depth-first searchPython CS GO Case SimulatorGame of Life simulator, Python 3



.everyoneloves__top-leaderboard:empty,.everyoneloves__mid-leaderboard:empty,.everyoneloves__bot-mid-leaderboard:empty margin-bottom:0;








2












$begingroup$


I'm new to Python, for a school project I created a "fishing simulator". Basically, use of random. I know that my code is repetitive towards the end but I don't know how to simplify it.



import time
import random
fishing = True
a = b = c = d = e = 0 #define multiple variables as same thing
print ("~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~")
print ("Welcome to Lake Tocowaga")
print ("~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~")
time.sleep(1)
name = input("What is your name fisherman?")
answer = input("Would you like to go fishing, " + name + "?")
if answer.lower() == "no":
fishing == False
while fishing == True:
time.sleep(1)
answer = input("Throw out your line, or go home?")
if answer == "go home":
fishing = False
er = float(e / (a + b + c + d))
print("~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~")
print("Thanks for playing " + name + "!")
print("You caught:", str(a), "cod, ", str(b), "salmon, ", str(c), "shark, ", str(d), "wildfish. nEfficiency Rate: ", str(er), ".")
else:
t = random.randrange(1, 7)
if t == 1:
a += 1
print("You caught a cod!")
elif t == 2:
b += 1
print("You caught a salmon!")
elif t == 3:
c += 1
print("You caught a shark!")
elif t == 4:
d += 1
print("You caught a wildfish!")
elif t >= 5:
e += 1
print("You caught nothing!")









share|improve this question









New contributor




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







$endgroup$











  • $begingroup$
    Why did you tag this question as python-2.x? I'm not convinced that it would work correctly in Python 2.
    $endgroup$
    – 200_success
    1 hour ago










  • $begingroup$
    One other thing not in either of the answers so far, would be to consider using a dictionary for your fishes, eg. fishes = 'cod': 0, 'salmon': 0,..., then it becomes far more readable when adding (fishes['cod'] += 1) and dumping (for fish, count in fishes.items():nt print("fish -> count".format(**'fish': fish, 'count': count))) values related fish caught.
    $endgroup$
    – S0AndS0
    35 mins ago


















2












$begingroup$


I'm new to Python, for a school project I created a "fishing simulator". Basically, use of random. I know that my code is repetitive towards the end but I don't know how to simplify it.



import time
import random
fishing = True
a = b = c = d = e = 0 #define multiple variables as same thing
print ("~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~")
print ("Welcome to Lake Tocowaga")
print ("~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~")
time.sleep(1)
name = input("What is your name fisherman?")
answer = input("Would you like to go fishing, " + name + "?")
if answer.lower() == "no":
fishing == False
while fishing == True:
time.sleep(1)
answer = input("Throw out your line, or go home?")
if answer == "go home":
fishing = False
er = float(e / (a + b + c + d))
print("~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~")
print("Thanks for playing " + name + "!")
print("You caught:", str(a), "cod, ", str(b), "salmon, ", str(c), "shark, ", str(d), "wildfish. nEfficiency Rate: ", str(er), ".")
else:
t = random.randrange(1, 7)
if t == 1:
a += 1
print("You caught a cod!")
elif t == 2:
b += 1
print("You caught a salmon!")
elif t == 3:
c += 1
print("You caught a shark!")
elif t == 4:
d += 1
print("You caught a wildfish!")
elif t >= 5:
e += 1
print("You caught nothing!")









share|improve this question









New contributor




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







$endgroup$











  • $begingroup$
    Why did you tag this question as python-2.x? I'm not convinced that it would work correctly in Python 2.
    $endgroup$
    – 200_success
    1 hour ago










  • $begingroup$
    One other thing not in either of the answers so far, would be to consider using a dictionary for your fishes, eg. fishes = 'cod': 0, 'salmon': 0,..., then it becomes far more readable when adding (fishes['cod'] += 1) and dumping (for fish, count in fishes.items():nt print("fish -> count".format(**'fish': fish, 'count': count))) values related fish caught.
    $endgroup$
    – S0AndS0
    35 mins ago














2












2








2





$begingroup$


I'm new to Python, for a school project I created a "fishing simulator". Basically, use of random. I know that my code is repetitive towards the end but I don't know how to simplify it.



import time
import random
fishing = True
a = b = c = d = e = 0 #define multiple variables as same thing
print ("~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~")
print ("Welcome to Lake Tocowaga")
print ("~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~")
time.sleep(1)
name = input("What is your name fisherman?")
answer = input("Would you like to go fishing, " + name + "?")
if answer.lower() == "no":
fishing == False
while fishing == True:
time.sleep(1)
answer = input("Throw out your line, or go home?")
if answer == "go home":
fishing = False
er = float(e / (a + b + c + d))
print("~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~")
print("Thanks for playing " + name + "!")
print("You caught:", str(a), "cod, ", str(b), "salmon, ", str(c), "shark, ", str(d), "wildfish. nEfficiency Rate: ", str(er), ".")
else:
t = random.randrange(1, 7)
if t == 1:
a += 1
print("You caught a cod!")
elif t == 2:
b += 1
print("You caught a salmon!")
elif t == 3:
c += 1
print("You caught a shark!")
elif t == 4:
d += 1
print("You caught a wildfish!")
elif t >= 5:
e += 1
print("You caught nothing!")









share|improve this question









New contributor




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







$endgroup$




I'm new to Python, for a school project I created a "fishing simulator". Basically, use of random. I know that my code is repetitive towards the end but I don't know how to simplify it.



import time
import random
fishing = True
a = b = c = d = e = 0 #define multiple variables as same thing
print ("~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~")
print ("Welcome to Lake Tocowaga")
print ("~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~")
time.sleep(1)
name = input("What is your name fisherman?")
answer = input("Would you like to go fishing, " + name + "?")
if answer.lower() == "no":
fishing == False
while fishing == True:
time.sleep(1)
answer = input("Throw out your line, or go home?")
if answer == "go home":
fishing = False
er = float(e / (a + b + c + d))
print("~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~")
print("Thanks for playing " + name + "!")
print("You caught:", str(a), "cod, ", str(b), "salmon, ", str(c), "shark, ", str(d), "wildfish. nEfficiency Rate: ", str(er), ".")
else:
t = random.randrange(1, 7)
if t == 1:
a += 1
print("You caught a cod!")
elif t == 2:
b += 1
print("You caught a salmon!")
elif t == 3:
c += 1
print("You caught a shark!")
elif t == 4:
d += 1
print("You caught a wildfish!")
elif t >= 5:
e += 1
print("You caught nothing!")






python beginner python-3.x






share|improve this question









New contributor




Mattthecommie 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




Mattthecommie 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 1 hour ago









Austin Hastings

7,7671236




7,7671236






New contributor




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









asked 2 hours ago









MattthecommieMattthecommie

464




464




New contributor




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





New contributor





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






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











  • $begingroup$
    Why did you tag this question as python-2.x? I'm not convinced that it would work correctly in Python 2.
    $endgroup$
    – 200_success
    1 hour ago










  • $begingroup$
    One other thing not in either of the answers so far, would be to consider using a dictionary for your fishes, eg. fishes = 'cod': 0, 'salmon': 0,..., then it becomes far more readable when adding (fishes['cod'] += 1) and dumping (for fish, count in fishes.items():nt print("fish -> count".format(**'fish': fish, 'count': count))) values related fish caught.
    $endgroup$
    – S0AndS0
    35 mins ago

















  • $begingroup$
    Why did you tag this question as python-2.x? I'm not convinced that it would work correctly in Python 2.
    $endgroup$
    – 200_success
    1 hour ago










  • $begingroup$
    One other thing not in either of the answers so far, would be to consider using a dictionary for your fishes, eg. fishes = 'cod': 0, 'salmon': 0,..., then it becomes far more readable when adding (fishes['cod'] += 1) and dumping (for fish, count in fishes.items():nt print("fish -> count".format(**'fish': fish, 'count': count))) values related fish caught.
    $endgroup$
    – S0AndS0
    35 mins ago
















$begingroup$
Why did you tag this question as python-2.x? I'm not convinced that it would work correctly in Python 2.
$endgroup$
– 200_success
1 hour ago




$begingroup$
Why did you tag this question as python-2.x? I'm not convinced that it would work correctly in Python 2.
$endgroup$
– 200_success
1 hour ago












$begingroup$
One other thing not in either of the answers so far, would be to consider using a dictionary for your fishes, eg. fishes = 'cod': 0, 'salmon': 0,..., then it becomes far more readable when adding (fishes['cod'] += 1) and dumping (for fish, count in fishes.items():nt print("fish -> count".format(**'fish': fish, 'count': count))) values related fish caught.
$endgroup$
– S0AndS0
35 mins ago





$begingroup$
One other thing not in either of the answers so far, would be to consider using a dictionary for your fishes, eg. fishes = 'cod': 0, 'salmon': 0,..., then it becomes far more readable when adding (fishes['cod'] += 1) and dumping (for fish, count in fishes.items():nt print("fish -> count".format(**'fish': fish, 'count': count))) values related fish caught.
$endgroup$
– S0AndS0
35 mins ago











2 Answers
2






active

oldest

votes


















2












$begingroup$

A few simple things.




a = b = c = d = e = 0


This is bad for two reasons:



  • Those are all non-descriptive, overly simple names. There's no way to tell what they represent just by looking at them.


  • You're shoving their declarations/definitions all on one line. This is generally regarded as poor practice. Say I'm looking for where c is defined. It's much easier to find it when I can be sure that I'm looking for exactly c = ... somewhere. It's harder to find though when it's declared half way through a line.


In both cases, you're sacrificing readability for brevity. Avoid doing this unless you're code golfing. Readability take precedence over nearly everything else.




fishing = True is the third line in your file, yet you don't use it until later. Unless it's a constant, it's better to declare variables near where they're first used in many cases. When someone's reading your code and want to see the definition of fishing, it's more efficient if they only have to look up a line or two instead of needing to scroll to the top of the file.




while fishing == True: can simply be written as while fishing:, which reads nicer anyways.




You actually have a bug. fishing == False should be fishing = False.




if answer.lower() == "no": could be written to be more "tolerant" (but less exact) by only checking the first letter:



if answer.lower()[0] == "n":


Now input like "nope" will work as well. Whether or not you want this behavior is another story. If you had other answers that require "n" as the first letter, obviously this would break things.






share|improve this answer











$endgroup$




















    1












    $begingroup$

    Welcome to CodeReview. It's never too early to develop good coding habits, and reviewing your code is about the best way to do so.



    First, congratulations on writing a clean, straightforward program. While you do have some issues (below), they're not major, and your program seems appropriate for its level.



    Now, for the issues ;-)



    Use whitespace



    Python requires you to use horizontal whitespace. But you should also use vertical whitespace (aka "blank lines") to organize the different parts of your code into paragraphs.



    This huge block:



    import time
    import random
    fishing = True
    a = b = c = d = e = 0 #define multiple variables as same thing
    print ("~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~")
    print ("Welcome to Lake Tocowaga")
    print ("~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~")
    time.sleep(1)
    name = input("What is your name fisherman?")
    answer = input("Would you like to go fishing, " + name + "?")
    if answer.lower() == "no":
    fishing == False
    while fishing == True:


    would read better if it were broken up like so:



    import time
    import random

    fishing = True
    a = b = c = d = e = 0 #define multiple variables as same thing

    print ("~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~")
    print ("Welcome to Lake Tocowaga")
    print ("~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~")
    time.sleep(1)

    name = input("What is your name fisherman?")
    answer = input("Would you like to go fishing, " + name + "?")

    if answer.lower() == "no":
    fishing == False

    while fishing == True:


    All I did was add a few blank lines, but I was trying to show that "these things go together" and "these things are in sequence but not related".



    Use meaningful names:



    Which one of these is the shark?



    a = b = c = d = e = 0


    I have no idea. But if you named them appropriately:



    cod = shark = wildfish = salmon = nothing = 0


    I would know for sure!



    Use named constants



    This line appears three times:



    print ("~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~")


    It's probably hard to get the right number of tilde characters, unless you are copy/pasting it. And if you're doing that, it's probably a pain. Instead, create a name for the tildes. By convention, constants are spelled in uppercase. (It's not really a constant, but since constants are spelled in upper case, if you name it in upper case you'll know not to modify it.)



    H_LINE = "~" * 32

    print(H_LINE)
    print("Welcome to Lake Tocowaga")
    print(H_LINE)


    Put last things last



    There's a place for everything. And everything should be in its place. The place for printing a summary would be at the bottom.



    You had a good idea with your while fishing: loop. But instead of immediately printing the summary when you respond to the user input, just change the variable and let the loop fail, then print the summary at the bottom. It's more "natural" (and it makes your loops easier to read!).



    while fishing == True: 
    time.sleep(1)
    answer = input("Throw out your line, or go home?")
    if answer == "go home":
    fishing = False
    er = float(e / (a + b + c + d))
    print("~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~")
    print("Thanks for playing " + name + "!")
    print("You caught:", str(a), "cod, ", str(b), "salmon, ", str(c), "shark, ", str(d), "wildfish. nEfficiency Rate: ", str(er), ".")
    else:
    ...


    Becomes:



    while fishing == True: 
    time.sleep(1)
    answer = input("Throw out your line, or go home?")
    if answer == "go home":
    fishing = False
    else:
    ...

    er = float(e / (a + b + c + d))
    print(H_LINE)
    print("Thanks for playing " + name + "!")
    print("You caught:", str(a), "cod, ", str(b), "salmon, ", str(c), "shark, ", str(d), "wildfish. nEfficiency Rate: ", str(er), ".")


    Let the built-in functions do their job



    You are calling functions that you don't need to call. The result of "true" division between integers is a float. You don't need to call float(e / (a + b + c + d)). And if you did need to call it, you'd be calling it too late!



    Likewise, print knows how to handle integers and floating point numbers. You don't need to print(..., str(a), ...) when you can just do: print(..., a, ...).






    share|improve this answer









    $endgroup$













      Your Answer






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



      );






      Mattthecommie 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%2f217357%2fpython-fishing-simulator%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









      2












      $begingroup$

      A few simple things.




      a = b = c = d = e = 0


      This is bad for two reasons:



      • Those are all non-descriptive, overly simple names. There's no way to tell what they represent just by looking at them.


      • You're shoving their declarations/definitions all on one line. This is generally regarded as poor practice. Say I'm looking for where c is defined. It's much easier to find it when I can be sure that I'm looking for exactly c = ... somewhere. It's harder to find though when it's declared half way through a line.


      In both cases, you're sacrificing readability for brevity. Avoid doing this unless you're code golfing. Readability take precedence over nearly everything else.




      fishing = True is the third line in your file, yet you don't use it until later. Unless it's a constant, it's better to declare variables near where they're first used in many cases. When someone's reading your code and want to see the definition of fishing, it's more efficient if they only have to look up a line or two instead of needing to scroll to the top of the file.




      while fishing == True: can simply be written as while fishing:, which reads nicer anyways.




      You actually have a bug. fishing == False should be fishing = False.




      if answer.lower() == "no": could be written to be more "tolerant" (but less exact) by only checking the first letter:



      if answer.lower()[0] == "n":


      Now input like "nope" will work as well. Whether or not you want this behavior is another story. If you had other answers that require "n" as the first letter, obviously this would break things.






      share|improve this answer











      $endgroup$

















        2












        $begingroup$

        A few simple things.




        a = b = c = d = e = 0


        This is bad for two reasons:



        • Those are all non-descriptive, overly simple names. There's no way to tell what they represent just by looking at them.


        • You're shoving their declarations/definitions all on one line. This is generally regarded as poor practice. Say I'm looking for where c is defined. It's much easier to find it when I can be sure that I'm looking for exactly c = ... somewhere. It's harder to find though when it's declared half way through a line.


        In both cases, you're sacrificing readability for brevity. Avoid doing this unless you're code golfing. Readability take precedence over nearly everything else.




        fishing = True is the third line in your file, yet you don't use it until later. Unless it's a constant, it's better to declare variables near where they're first used in many cases. When someone's reading your code and want to see the definition of fishing, it's more efficient if they only have to look up a line or two instead of needing to scroll to the top of the file.




        while fishing == True: can simply be written as while fishing:, which reads nicer anyways.




        You actually have a bug. fishing == False should be fishing = False.




        if answer.lower() == "no": could be written to be more "tolerant" (but less exact) by only checking the first letter:



        if answer.lower()[0] == "n":


        Now input like "nope" will work as well. Whether or not you want this behavior is another story. If you had other answers that require "n" as the first letter, obviously this would break things.






        share|improve this answer











        $endgroup$















          2












          2








          2





          $begingroup$

          A few simple things.




          a = b = c = d = e = 0


          This is bad for two reasons:



          • Those are all non-descriptive, overly simple names. There's no way to tell what they represent just by looking at them.


          • You're shoving their declarations/definitions all on one line. This is generally regarded as poor practice. Say I'm looking for where c is defined. It's much easier to find it when I can be sure that I'm looking for exactly c = ... somewhere. It's harder to find though when it's declared half way through a line.


          In both cases, you're sacrificing readability for brevity. Avoid doing this unless you're code golfing. Readability take precedence over nearly everything else.




          fishing = True is the third line in your file, yet you don't use it until later. Unless it's a constant, it's better to declare variables near where they're first used in many cases. When someone's reading your code and want to see the definition of fishing, it's more efficient if they only have to look up a line or two instead of needing to scroll to the top of the file.




          while fishing == True: can simply be written as while fishing:, which reads nicer anyways.




          You actually have a bug. fishing == False should be fishing = False.




          if answer.lower() == "no": could be written to be more "tolerant" (but less exact) by only checking the first letter:



          if answer.lower()[0] == "n":


          Now input like "nope" will work as well. Whether or not you want this behavior is another story. If you had other answers that require "n" as the first letter, obviously this would break things.






          share|improve this answer











          $endgroup$



          A few simple things.




          a = b = c = d = e = 0


          This is bad for two reasons:



          • Those are all non-descriptive, overly simple names. There's no way to tell what they represent just by looking at them.


          • You're shoving their declarations/definitions all on one line. This is generally regarded as poor practice. Say I'm looking for where c is defined. It's much easier to find it when I can be sure that I'm looking for exactly c = ... somewhere. It's harder to find though when it's declared half way through a line.


          In both cases, you're sacrificing readability for brevity. Avoid doing this unless you're code golfing. Readability take precedence over nearly everything else.




          fishing = True is the third line in your file, yet you don't use it until later. Unless it's a constant, it's better to declare variables near where they're first used in many cases. When someone's reading your code and want to see the definition of fishing, it's more efficient if they only have to look up a line or two instead of needing to scroll to the top of the file.




          while fishing == True: can simply be written as while fishing:, which reads nicer anyways.




          You actually have a bug. fishing == False should be fishing = False.




          if answer.lower() == "no": could be written to be more "tolerant" (but less exact) by only checking the first letter:



          if answer.lower()[0] == "n":


          Now input like "nope" will work as well. Whether or not you want this behavior is another story. If you had other answers that require "n" as the first letter, obviously this would break things.







          share|improve this answer














          share|improve this answer



          share|improve this answer








          edited 49 mins ago

























          answered 56 mins ago









          CarcigenicateCarcigenicate

          4,10411633




          4,10411633























              1












              $begingroup$

              Welcome to CodeReview. It's never too early to develop good coding habits, and reviewing your code is about the best way to do so.



              First, congratulations on writing a clean, straightforward program. While you do have some issues (below), they're not major, and your program seems appropriate for its level.



              Now, for the issues ;-)



              Use whitespace



              Python requires you to use horizontal whitespace. But you should also use vertical whitespace (aka "blank lines") to organize the different parts of your code into paragraphs.



              This huge block:



              import time
              import random
              fishing = True
              a = b = c = d = e = 0 #define multiple variables as same thing
              print ("~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~")
              print ("Welcome to Lake Tocowaga")
              print ("~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~")
              time.sleep(1)
              name = input("What is your name fisherman?")
              answer = input("Would you like to go fishing, " + name + "?")
              if answer.lower() == "no":
              fishing == False
              while fishing == True:


              would read better if it were broken up like so:



              import time
              import random

              fishing = True
              a = b = c = d = e = 0 #define multiple variables as same thing

              print ("~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~")
              print ("Welcome to Lake Tocowaga")
              print ("~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~")
              time.sleep(1)

              name = input("What is your name fisherman?")
              answer = input("Would you like to go fishing, " + name + "?")

              if answer.lower() == "no":
              fishing == False

              while fishing == True:


              All I did was add a few blank lines, but I was trying to show that "these things go together" and "these things are in sequence but not related".



              Use meaningful names:



              Which one of these is the shark?



              a = b = c = d = e = 0


              I have no idea. But if you named them appropriately:



              cod = shark = wildfish = salmon = nothing = 0


              I would know for sure!



              Use named constants



              This line appears three times:



              print ("~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~")


              It's probably hard to get the right number of tilde characters, unless you are copy/pasting it. And if you're doing that, it's probably a pain. Instead, create a name for the tildes. By convention, constants are spelled in uppercase. (It's not really a constant, but since constants are spelled in upper case, if you name it in upper case you'll know not to modify it.)



              H_LINE = "~" * 32

              print(H_LINE)
              print("Welcome to Lake Tocowaga")
              print(H_LINE)


              Put last things last



              There's a place for everything. And everything should be in its place. The place for printing a summary would be at the bottom.



              You had a good idea with your while fishing: loop. But instead of immediately printing the summary when you respond to the user input, just change the variable and let the loop fail, then print the summary at the bottom. It's more "natural" (and it makes your loops easier to read!).



              while fishing == True: 
              time.sleep(1)
              answer = input("Throw out your line, or go home?")
              if answer == "go home":
              fishing = False
              er = float(e / (a + b + c + d))
              print("~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~")
              print("Thanks for playing " + name + "!")
              print("You caught:", str(a), "cod, ", str(b), "salmon, ", str(c), "shark, ", str(d), "wildfish. nEfficiency Rate: ", str(er), ".")
              else:
              ...


              Becomes:



              while fishing == True: 
              time.sleep(1)
              answer = input("Throw out your line, or go home?")
              if answer == "go home":
              fishing = False
              else:
              ...

              er = float(e / (a + b + c + d))
              print(H_LINE)
              print("Thanks for playing " + name + "!")
              print("You caught:", str(a), "cod, ", str(b), "salmon, ", str(c), "shark, ", str(d), "wildfish. nEfficiency Rate: ", str(er), ".")


              Let the built-in functions do their job



              You are calling functions that you don't need to call. The result of "true" division between integers is a float. You don't need to call float(e / (a + b + c + d)). And if you did need to call it, you'd be calling it too late!



              Likewise, print knows how to handle integers and floating point numbers. You don't need to print(..., str(a), ...) when you can just do: print(..., a, ...).






              share|improve this answer









              $endgroup$

















                1












                $begingroup$

                Welcome to CodeReview. It's never too early to develop good coding habits, and reviewing your code is about the best way to do so.



                First, congratulations on writing a clean, straightforward program. While you do have some issues (below), they're not major, and your program seems appropriate for its level.



                Now, for the issues ;-)



                Use whitespace



                Python requires you to use horizontal whitespace. But you should also use vertical whitespace (aka "blank lines") to organize the different parts of your code into paragraphs.



                This huge block:



                import time
                import random
                fishing = True
                a = b = c = d = e = 0 #define multiple variables as same thing
                print ("~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~")
                print ("Welcome to Lake Tocowaga")
                print ("~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~")
                time.sleep(1)
                name = input("What is your name fisherman?")
                answer = input("Would you like to go fishing, " + name + "?")
                if answer.lower() == "no":
                fishing == False
                while fishing == True:


                would read better if it were broken up like so:



                import time
                import random

                fishing = True
                a = b = c = d = e = 0 #define multiple variables as same thing

                print ("~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~")
                print ("Welcome to Lake Tocowaga")
                print ("~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~")
                time.sleep(1)

                name = input("What is your name fisherman?")
                answer = input("Would you like to go fishing, " + name + "?")

                if answer.lower() == "no":
                fishing == False

                while fishing == True:


                All I did was add a few blank lines, but I was trying to show that "these things go together" and "these things are in sequence but not related".



                Use meaningful names:



                Which one of these is the shark?



                a = b = c = d = e = 0


                I have no idea. But if you named them appropriately:



                cod = shark = wildfish = salmon = nothing = 0


                I would know for sure!



                Use named constants



                This line appears three times:



                print ("~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~")


                It's probably hard to get the right number of tilde characters, unless you are copy/pasting it. And if you're doing that, it's probably a pain. Instead, create a name for the tildes. By convention, constants are spelled in uppercase. (It's not really a constant, but since constants are spelled in upper case, if you name it in upper case you'll know not to modify it.)



                H_LINE = "~" * 32

                print(H_LINE)
                print("Welcome to Lake Tocowaga")
                print(H_LINE)


                Put last things last



                There's a place for everything. And everything should be in its place. The place for printing a summary would be at the bottom.



                You had a good idea with your while fishing: loop. But instead of immediately printing the summary when you respond to the user input, just change the variable and let the loop fail, then print the summary at the bottom. It's more "natural" (and it makes your loops easier to read!).



                while fishing == True: 
                time.sleep(1)
                answer = input("Throw out your line, or go home?")
                if answer == "go home":
                fishing = False
                er = float(e / (a + b + c + d))
                print("~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~")
                print("Thanks for playing " + name + "!")
                print("You caught:", str(a), "cod, ", str(b), "salmon, ", str(c), "shark, ", str(d), "wildfish. nEfficiency Rate: ", str(er), ".")
                else:
                ...


                Becomes:



                while fishing == True: 
                time.sleep(1)
                answer = input("Throw out your line, or go home?")
                if answer == "go home":
                fishing = False
                else:
                ...

                er = float(e / (a + b + c + d))
                print(H_LINE)
                print("Thanks for playing " + name + "!")
                print("You caught:", str(a), "cod, ", str(b), "salmon, ", str(c), "shark, ", str(d), "wildfish. nEfficiency Rate: ", str(er), ".")


                Let the built-in functions do their job



                You are calling functions that you don't need to call. The result of "true" division between integers is a float. You don't need to call float(e / (a + b + c + d)). And if you did need to call it, you'd be calling it too late!



                Likewise, print knows how to handle integers and floating point numbers. You don't need to print(..., str(a), ...) when you can just do: print(..., a, ...).






                share|improve this answer









                $endgroup$















                  1












                  1








                  1





                  $begingroup$

                  Welcome to CodeReview. It's never too early to develop good coding habits, and reviewing your code is about the best way to do so.



                  First, congratulations on writing a clean, straightforward program. While you do have some issues (below), they're not major, and your program seems appropriate for its level.



                  Now, for the issues ;-)



                  Use whitespace



                  Python requires you to use horizontal whitespace. But you should also use vertical whitespace (aka "blank lines") to organize the different parts of your code into paragraphs.



                  This huge block:



                  import time
                  import random
                  fishing = True
                  a = b = c = d = e = 0 #define multiple variables as same thing
                  print ("~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~")
                  print ("Welcome to Lake Tocowaga")
                  print ("~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~")
                  time.sleep(1)
                  name = input("What is your name fisherman?")
                  answer = input("Would you like to go fishing, " + name + "?")
                  if answer.lower() == "no":
                  fishing == False
                  while fishing == True:


                  would read better if it were broken up like so:



                  import time
                  import random

                  fishing = True
                  a = b = c = d = e = 0 #define multiple variables as same thing

                  print ("~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~")
                  print ("Welcome to Lake Tocowaga")
                  print ("~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~")
                  time.sleep(1)

                  name = input("What is your name fisherman?")
                  answer = input("Would you like to go fishing, " + name + "?")

                  if answer.lower() == "no":
                  fishing == False

                  while fishing == True:


                  All I did was add a few blank lines, but I was trying to show that "these things go together" and "these things are in sequence but not related".



                  Use meaningful names:



                  Which one of these is the shark?



                  a = b = c = d = e = 0


                  I have no idea. But if you named them appropriately:



                  cod = shark = wildfish = salmon = nothing = 0


                  I would know for sure!



                  Use named constants



                  This line appears three times:



                  print ("~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~")


                  It's probably hard to get the right number of tilde characters, unless you are copy/pasting it. And if you're doing that, it's probably a pain. Instead, create a name for the tildes. By convention, constants are spelled in uppercase. (It's not really a constant, but since constants are spelled in upper case, if you name it in upper case you'll know not to modify it.)



                  H_LINE = "~" * 32

                  print(H_LINE)
                  print("Welcome to Lake Tocowaga")
                  print(H_LINE)


                  Put last things last



                  There's a place for everything. And everything should be in its place. The place for printing a summary would be at the bottom.



                  You had a good idea with your while fishing: loop. But instead of immediately printing the summary when you respond to the user input, just change the variable and let the loop fail, then print the summary at the bottom. It's more "natural" (and it makes your loops easier to read!).



                  while fishing == True: 
                  time.sleep(1)
                  answer = input("Throw out your line, or go home?")
                  if answer == "go home":
                  fishing = False
                  er = float(e / (a + b + c + d))
                  print("~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~")
                  print("Thanks for playing " + name + "!")
                  print("You caught:", str(a), "cod, ", str(b), "salmon, ", str(c), "shark, ", str(d), "wildfish. nEfficiency Rate: ", str(er), ".")
                  else:
                  ...


                  Becomes:



                  while fishing == True: 
                  time.sleep(1)
                  answer = input("Throw out your line, or go home?")
                  if answer == "go home":
                  fishing = False
                  else:
                  ...

                  er = float(e / (a + b + c + d))
                  print(H_LINE)
                  print("Thanks for playing " + name + "!")
                  print("You caught:", str(a), "cod, ", str(b), "salmon, ", str(c), "shark, ", str(d), "wildfish. nEfficiency Rate: ", str(er), ".")


                  Let the built-in functions do their job



                  You are calling functions that you don't need to call. The result of "true" division between integers is a float. You don't need to call float(e / (a + b + c + d)). And if you did need to call it, you'd be calling it too late!



                  Likewise, print knows how to handle integers and floating point numbers. You don't need to print(..., str(a), ...) when you can just do: print(..., a, ...).






                  share|improve this answer









                  $endgroup$



                  Welcome to CodeReview. It's never too early to develop good coding habits, and reviewing your code is about the best way to do so.



                  First, congratulations on writing a clean, straightforward program. While you do have some issues (below), they're not major, and your program seems appropriate for its level.



                  Now, for the issues ;-)



                  Use whitespace



                  Python requires you to use horizontal whitespace. But you should also use vertical whitespace (aka "blank lines") to organize the different parts of your code into paragraphs.



                  This huge block:



                  import time
                  import random
                  fishing = True
                  a = b = c = d = e = 0 #define multiple variables as same thing
                  print ("~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~")
                  print ("Welcome to Lake Tocowaga")
                  print ("~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~")
                  time.sleep(1)
                  name = input("What is your name fisherman?")
                  answer = input("Would you like to go fishing, " + name + "?")
                  if answer.lower() == "no":
                  fishing == False
                  while fishing == True:


                  would read better if it were broken up like so:



                  import time
                  import random

                  fishing = True
                  a = b = c = d = e = 0 #define multiple variables as same thing

                  print ("~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~")
                  print ("Welcome to Lake Tocowaga")
                  print ("~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~")
                  time.sleep(1)

                  name = input("What is your name fisherman?")
                  answer = input("Would you like to go fishing, " + name + "?")

                  if answer.lower() == "no":
                  fishing == False

                  while fishing == True:


                  All I did was add a few blank lines, but I was trying to show that "these things go together" and "these things are in sequence but not related".



                  Use meaningful names:



                  Which one of these is the shark?



                  a = b = c = d = e = 0


                  I have no idea. But if you named them appropriately:



                  cod = shark = wildfish = salmon = nothing = 0


                  I would know for sure!



                  Use named constants



                  This line appears three times:



                  print ("~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~")


                  It's probably hard to get the right number of tilde characters, unless you are copy/pasting it. And if you're doing that, it's probably a pain. Instead, create a name for the tildes. By convention, constants are spelled in uppercase. (It's not really a constant, but since constants are spelled in upper case, if you name it in upper case you'll know not to modify it.)



                  H_LINE = "~" * 32

                  print(H_LINE)
                  print("Welcome to Lake Tocowaga")
                  print(H_LINE)


                  Put last things last



                  There's a place for everything. And everything should be in its place. The place for printing a summary would be at the bottom.



                  You had a good idea with your while fishing: loop. But instead of immediately printing the summary when you respond to the user input, just change the variable and let the loop fail, then print the summary at the bottom. It's more "natural" (and it makes your loops easier to read!).



                  while fishing == True: 
                  time.sleep(1)
                  answer = input("Throw out your line, or go home?")
                  if answer == "go home":
                  fishing = False
                  er = float(e / (a + b + c + d))
                  print("~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~")
                  print("Thanks for playing " + name + "!")
                  print("You caught:", str(a), "cod, ", str(b), "salmon, ", str(c), "shark, ", str(d), "wildfish. nEfficiency Rate: ", str(er), ".")
                  else:
                  ...


                  Becomes:



                  while fishing == True: 
                  time.sleep(1)
                  answer = input("Throw out your line, or go home?")
                  if answer == "go home":
                  fishing = False
                  else:
                  ...

                  er = float(e / (a + b + c + d))
                  print(H_LINE)
                  print("Thanks for playing " + name + "!")
                  print("You caught:", str(a), "cod, ", str(b), "salmon, ", str(c), "shark, ", str(d), "wildfish. nEfficiency Rate: ", str(er), ".")


                  Let the built-in functions do their job



                  You are calling functions that you don't need to call. The result of "true" division between integers is a float. You don't need to call float(e / (a + b + c + d)). And if you did need to call it, you'd be calling it too late!



                  Likewise, print knows how to handle integers and floating point numbers. You don't need to print(..., str(a), ...) when you can just do: print(..., a, ...).







                  share|improve this answer












                  share|improve this answer



                  share|improve this answer










                  answered 43 mins ago









                  Austin HastingsAustin Hastings

                  7,7671236




                  7,7671236




















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









                      draft saved

                      draft discarded


















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












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











                      Mattthecommie 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%2f217357%2fpython-fishing-simulator%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

                      Can not update quote_id field of “quote_item” table magento 2Magento 2.1 - We can't remove the item. (Shopping Cart doesnt allow us to remove items before becomes empty)Add value for custom quote item attribute using REST apiREST API endpoint v1/carts/cartId/items always returns error messageCorrect way to save entries to databaseHow to remove all associated quote objects of a customer completelyMagento 2 - Save value from custom input field to quote_itemGet quote_item data using quote id and product id filter in Magento 2How to set additional data to quote_item table from controller in Magento 2?What is the purpose of additional_data column in quote_item table in magento2Set Custom Price to Quote item magento2 from controller

                      Magento 2 disable Secret Key on URL's from terminal The Next CEO of Stack OverflowMagento 2 Shortcut/GUI tool to perform commandline tasks for windowsIn menu add configuration linkMagento oAuth : Generating access token and access secretMagento 2 security key issue in Third-Party API redirect URIPublic actions in admin controllersHow to Disable Cache in Custom WidgetURL Key not changing in Magento 2Product URL Key gets deleted when importing custom options - Magento 2Problem with reindex terminalMagento 2 - bin/magento Commands not working in Cpanel Terminal

                      Aasi (pallopeli) Navigointivalikko