Python

Python RSS

Repetitive code problem

Please support our Python advertiser: Programming Forums - DaniWeb Sister Site

Thread Solved

•

Join Date: Sep 2009

Posts: 112

Reputation: AutoPython is an unknown quantity at this point

Solved Threads: 9

AutoPython AutoPython is offline

Offline

Junior Poster

Repetitive code problem

Oct 11th, 2009

I'm having a problem sizing down this block of code:

 Help with Code Tags 
 Python Syntax (Toggle Plain Text) 
   if POS == 1:
        POS1 = "->"
    else:
        POS1 = "  "
 
    if POS == 2:
        POS2 = "->"
    else:
        POS2 = "  "
 
    if POS == 3:
        POS3 = "->"
    else:
        POS3 = "  "
 
    if POS == 4:
        POS4 = "->"
    else:
        POS4 = "  "
 
    if POS == 5:
        POS5 = "->"
    else:
        POS5 = "  "
 
    if POS == 6:
        POS6 = "->"
    else:
        POS6 = "  "
 
    if POS == 7:
        POS7 = "->"
    else:
        POS7 = "  "
 
    if POS == 8:
        POS8 = "->"
    else:
        POS8 = "  "
 
    if POS == 9:
        POS9 = "->"
    else:
        POS9 = "  "
 
    if POS == 1:
        POS1 = "->"
    else:
        POS1 = "  "
   if POS == 1:
        POS1 = "->"
    else:
        POS1 = "  "

    if POS == 2:
        POS2 = "->"
    else:
        POS2 = "  "
        
    if POS == 3:
        POS3 = "->"
    else:
        POS3 = "  "

    if POS == 4:
        POS4 = "->"
    else:
        POS4 = "  "

    if POS == 5:
        POS5 = "->"
    else:
        POS5 = "  "

    if POS == 6:
        POS6 = "->"
    else:
        POS6 = "  "

    if POS == 7:
        POS7 = "->"
    else:
        POS7 = "  "

    if POS == 8:
        POS8 = "->"
    else:
        POS8 = "  "

    if POS == 9:
        POS9 = "->"
    else:
        POS9 = "  "
        
    if POS == 1:
        POS1 = "->"
    else:
        POS1 = "  "

The reason why I can't figure it out is mainly because the sequence I would use a for loop with is in a variable. Any suggestions would be appreciated.

•

Join Date: Feb 2007

Posts: 1,606

Reputation: scru has a spectacular aura about

Solved Threads: 130

scru

Offline

Posting Virtuoso

-1

Oct 11th, 2009

 Help with Code Tags 
 python Syntax (Toggle Plain Text) 
locals_dict = locals()
for i in range(1, 10):
    var_name = "POS" + str(i)
    if i == POS:
        locals_dict[var_name] = "->"
    else:
        locals_dict[var_name] = " "
locals_dict = locals()
for i in range(1, 10):
    var_name = "POS" + str(i)
    if i == POS:
        locals_dict[var_name] = "->"
    else:
        locals_dict[var_name] = " "

discalimer: That code uses hacks and the purists may have your head for it.

Last edited by scru; Oct 11th, 2009 at 8:47 am.

•

Join Date: Sep 2009

Posts: 112

Reputation: AutoPython is an unknown quantity at this point

Solved Threads: 9

AutoPython AutoPython is offline

Offline

Junior Poster

Oct 11th, 2009

Yes, I see how that would be considered a hack. Will it be fairly reliable? Or will it crash most of the time.

•

Join Date: Oct 2007

Posts: 149

Reputation: jice is on a distinguished road

Solved Threads: 38

jice

Offline

Junior Poster

Oct 11th, 2009

Why do you want to have vars named POSn.
I wouldn't do that...
Very complicated. It's far easier to have one list variable where you assign pos[n]

 Help with Code Tags 
 Python Syntax (Toggle Plain Text) 
pos=['  ' for i in range(10)] # defines a list inited with ['  ', '  ',... ten times]
pos[POS-1]='->'
pos=['  ' for i in range(10)] # defines a list inited with ['  ', '  ',... ten times]
pos[POS-1]='->'

Last edited by jice; Oct 11th, 2009 at 4:10 pm.

•

Join Date: Jul 2005

Posts: 1,221

Reputation: bumsfeld will become famous soon enough

Solved Threads: 137

bumsfeld

Offline

Nearly a Posting Virtuoso

Oct 11th, 2009

Similar to jice's code:

 Help with Code Tags 
 python Syntax (Toggle Plain Text) 
# simply ignore index zero
pos = ['  '] * 11
print( pos )
 
POS = 1
 
pos[POS] = '->'
print( pos )
 
"""my display -->
['  ', '  ', '  ', '  ', '  ', '  ', '  ', '  ', '  ', '  ', '  ']
['  ', '->', '  ', '  ', '  ', '  ', '  ', '  ', '  ', '  ', '  ']
"""
# simply ignore index zero
pos = ['  '] * 11
print( pos )

POS = 1

pos[POS] = '->'
print( pos )

"""my display -->
['  ', '  ', '  ', '  ', '  ', '  ', '  ', '  ', '  ', '  ', '  ']
['  ', '->', '  ', '  ', '  ', '  ', '  ', '  ', '  ', '  ', '  ']
"""

Should you find Irony, you can keep her!

•

Join Date: Sep 2009

Posts: 112

Reputation: AutoPython is an unknown quantity at this point

Solved Threads: 9

AutoPython AutoPython is offline

Offline

Junior Poster

Oct 11th, 2009

If I knew how the 2nd line of Jiccess code I could make it work. But I forgot to mention that each POS variable is in a certain place. I won't work to only have 1 variable. I also forgot to mention that scru's code didn't work either, it said that POS1 wasn't defined.

•

Join Date: Oct 2007

Posts: 149

Reputation: jice is on a distinguished road

Solved Threads: 38

jice

Offline

Junior Poster

Oct 11th, 2009

•

Originally Posted by AutoPython

If I knew how the 2nd line of Jiccess code I could make it work. But I forgot to mention that each POS variable is in a certain place. I won't work to only have 1 variable. I also forgot to mention that scru's code didn't work either, it said that POS1 wasn't defined.

Sorry but I don't understand what you mean.
What do you mean with "each POS variable is in a certain place" ?

If I use one variable, it contains the tenth values.
Instead of using POS1, you'll use pos[0], POS5 will become pos[4]...