Constructive Criticism

[jeremiahv...@gmail.com]

Hi, hows it going I've been self teaching myself python, and i typed up this small script now i know its not the best the coding is not the best but i would like to know of ways to make a small script like this better so all constructive critisim is Welcome.



Here is the link to the code

" http://pastebin.com/5uCFR2pz "

[Paul Pittlerson]

I think the only winning move is not to play.

[Ben Finney]

jeremiahv...@gmail.com writes:

> Hi, hows it going I've been self teaching myself python

Welcome to Python, and to this discussion forum!

> and i typed up this small script now i know its not the best the
> coding is not the best but i would like to know of ways to make a
> small script like this better so all constructive critisim is Welcome.

Some constructive criticism:

* Please make an effort to write readable English here. A run-on
sentence like the above makes it seem as though you don't care enough
for our help to make your message easily readable.

* Please paste the code in your actual message, rather than directing to
an ephemeral website. This is so that the discussion will always have
the code for context when later readers view it.

Once you re-post your program and describe its purpose, I'm sure there
will be some responses giving feedback.

--
\ “My girlfriend has a queen sized bed; I have a court jester |
`\ sized bed. It's red and green and has bells on it, and the ends |
_o__) curl up.” —Steven Wright |
Ben Finney

[jeremiah valerio]

yea sorry about this screw up "i know its not the best the coding is not the best but" and i'm new to forums never posted so i will take all your input and re-post maybe later thank you.

[Mark Lawrence]

On 09/01/2014 06:16, jeremiah valerio wrote:
> On Thursday, January 9, 2014 12:09:17 AM UTC-6, Ben Finney wrote:
>> jeremiahv...@gmail.com writes:
>>
>>
>>
>>> Hi, hows it going I've been self teaching myself python
>>
>>
>>
>> Welcome to Python, and to this discussion forum!
>>
>>
>>
>>> and i typed up this small script now i know its not the best the
>>
>>> coding is not the best but i would like to know of ways to make a
>>
>>> small script like this better so all constructive critisim is Welcome.
>>
>>
>>
>> Some constructive criticism:
>>
>>
>>
>> * Please make an effort to write readable English here. A run-on
>>
>> sentence like the above makes it seem as though you don't care enough
>>
>> for our help to make your message easily readable.
>>
>>
>>
>> * Please paste the code in your actual message, rather than directing to
>>
>> an ephemeral website. This is so that the discussion will always have
>>
>> the code for context when later readers view it.
>>
>>
>>
>> Once you re-post your program and describe its purpose, I'm sure there
>>
>> will be some responses giving feedback.
>>
>>
>>
>> --
>>

>> \ �My girlfriend has a queen sized bed; I have a court jester |

>>
>> `\ sized bed. It's red and green and has bells on it, and the ends |
>>

>> _o__) curl up.� �Steven Wright |

>>
>> Ben Finney
>
> yea sorry about this screw up "i know its not the best the coding is not the best but" and i'm new to forums never posted so i will take all your input and re-post maybe later thank you.
>

Before you repost would you please read and action this
https://wiki.python.org/moin/GoogleGroupsPython to prevent us seeing the
double line spacing above, thanks.

--
My fellow Pythonistas, ask not what our language can do for you, ask
what you can do for our language.

Mark Lawrence

[Peter Otten]

> time.sleep(1)
> import time
> print("Closing in 9 ")
> time.sleep(1)
> import time
> print("Closing in 8 ")

- You should import modules just once, at the beginning of your script.

- Repetetive tasks are best handled with a for-loop, e. g.:

>>> import time
>>> for seconds_left in reversed(range(1, 10)):
... print("Closing in", seconds_left, "seconds")
... time.sleep(1)
...
Closing in 9 seconds
Closing in 8 seconds
Closing in 7 seconds
Closing in 6 seconds
Closing in 5 seconds
Closing in 4 seconds
Closing in 3 seconds
Closing in 2 seconds
Closing in 1 seconds

> user_input = input("\nWhos your favorite Football team? \n 1.Arizona
> Cardinals\n 2.Atlanta Falcons\n 3.Baltimore Ravens\n 4.Buffalo Bills\n
> 5.Miami Dolphins\n 6.Minnesota Vikings \n 7.New England Patriots \n
> 8.New Orleans Saints \n 9.Carolina
[snip]

Python offers triple-quoted strings which may include newline literals:

user_input = input("""
Who's your favorite Football team?
1. Arizona Cardinals
2. Atlanta Falcons
...
""")

> if user_input == "1" :
> print("\nThey suck! BYE!")
>
> elif user_input == "2" :
> print("\nThey suck! BYE!")
>
> elif user_input == "3" :
> print("\nThey suck!BYE!")
[snip]

Ignoring the typos you are taking the same action for all inputs but "17".
So:

if user_input != "17":
print()
print("They suck! BYE!")

You should give some thought how unexpected user input like "", "123",
"whatever" should be handled.

> elif user_input == "no" :
> print("\nAlrighty bye have a nice day! :)\n\nClosing in 10.")
> import time
> time.sleep(1)
> import time
> print("Closing in 9 ")
> time.sleep(1)
> import time
> print("Closing in 8 ")
> time.sleep(1)
> import time

OK, you are doing the count-down thing twice -- time to write a function,
say countdown(), that you can put where you need a count-down instead of the
repetetive code.

[jeremiah valerio]

Thanks so much,exactly what i was looking for
thanks for taking the time.

[Christopher Welborn]

I'm not sure if someone already pointed this out, but imports only need
to be done once. Usually at the beginning of the file, but not always.
In your case I would say yes, at the beginning.

import sys
import time

def countdown(seconds):'
# start at 'seconds' and count down with a for-loop
for i in range(seconds, 0, -1):
# print the current second (i)
print('closing in {} seconds.'.format(i))
# sleep for one second (no need to import time again).
time.sleep(1)

# Example usage:
print('hello')
# Prints the countdown.
countdown(10)
sys.exit(0)
--

- Christopher Welborn <cjwe...@live.com>
http://welbornprod.com

[jeremiah valerio]

Mr.Peter Otten did

"- You should import modules just once, at the beginning of your script. "

-Peter Otten
With his help this is what i have now

def countdown():

import time
for seconds_left in reversed(range(1, 10)):

print("Closing in", seconds_left, "seconds")

time.sleep(1)
exit()

if user_input == "yes" :
user_input = input("\nGreat what should we talk about?\nSports\nWeather")

elif user_input == "no" :
print("\nAlrighty bye have a nice day! :)\n\nClosing in 10.")

countdown()

[Alister]

you could improve your countdown function further by adding an optional
count vaule

def countdown(count=10):
for timeleft in reversed(1,count):
print ("Shutting down in {} Seconds".format(timeleft))
time.sleep(1)
--
Most people can't understand how others can blow their noses differently
than they do.
-- Turgenev

[jeremiah valerio]

So always think of the if's and possibility's that
other people might do, thanks for you input.

[Chris Angelico]

On Sat, Jan 11, 2014 at 7:26 AM, jeremiah valerio
<jeremiahv...@gmail.com> wrote:
> So always think of the if's and possibility's that
> other people might do, thanks for you input.

Also think of the possibility that someone will read your post and its
quoted text. Please get off Google Groups, or if you must keep using
it, clean up your quotes - they're all coming out double-spaced.

You may find the mailing list easier to use:

https://mail.python.org/mailman/listinfo/python-list

ChrisA

[Mark Lawrence]

On 10/01/2014 20:26, jeremiah valerio wrote:

For the second time of asking would you please read and action this

https://wiki.python.org/moin/GoogleGroupsPython to prevent us seeing

double line spacing, thanks. Failing that, please arm yourself with a
semi-decent email client, there are umpteen to choose from.