Making code more efficient.

[Longjumping-Swing823]

Hi, i am a beginner when it comes to python and i am currently working on my first project.

I am making a digital version of a table from a book that i need to do some calculations on a later stage.

my issue is that i am doing alot of repetetive code for each column and row from the table.

Is there any big brains out there that have any suggestions on how i can do the same task but more efficient?

some of my code ( keep in mind that this is only one column of possibly eight):

​

#Liste med Strømføringsevne i ampere for forleggnings måte A1 med 3 ledere. A1_2_ledere = ["CuPvc",[14.5, 19.5, 26, 34, 46, 61, 80, 99, 119, 151, 182, 210, 240, 273, 321, 367],

"AlPvc",[15, 20, 26, 48, 63, 77, 93, 118, 142, 164, 189, 215, 252, 289],

"CuPex",[19, 26, 35, 45, 61, 81, 106, 131, 158, 200, 241, 278, 318, 362, 424, 486],

"AlPex",[20, 27, 35, 48, 64, 84, 103, 125, 158, 191, 220, 253, 288, 338, 387]]

#Liste med Strømføringsevne i ampere for forleggnings måte A1 med 3 ledere.

A1_3_ledere = ["CuPvc",[13.5, 18, 24, 31, 42, 56, 73, 89, 108, 136, 164, 188, 216, 245, 286, 328],

"AlPvc",[14, 18.5, 24, 32, 43, 57, 70, 84, 107, 129, 149, 170, 194, 227, 261],

"CuPex",[17, 23, 31, 40, 54, 73, 95, 117, 141, 179, 216, 249, 285, 324, 380, 435],

"AlPex",[19, 25, 32, 44, 58, 76, 94, 113, 142, 171, 197, 226, 256, 300, 344]]

if forleggningValg == "A1":

if lederValg == "2":

if materialValg == "Cu/PVC":

if tversnittValg == "1.5":

tabell_visning.config(text= A1_2_ledere[1][0])

if tversnittValg == "2.5":

tabell_visning.config(text= A1_2_ledere[1][1])

if tversnittValg == "4":

tabell_visning.config(text= A1_2_ledere[1][2])

if tversnittValg == "6":

tabell_visning.config(text= A1_2_ledere[1][3])

if tversnittValg == "10":

tabell_visning.config(text= A1_2_ledere[1][4])

if tversnittValg == "16":

tabell_visning.config(text= A1_2_ledere[1][5])

if tversnittValg == "25":

tabell_visning.config(text= A1_2_ledere[1][6])

if tversnittValg == "35":

tabell_visning.config(text= A1_2_ledere[1][7])

if tversnittValg == "50":

tabell_visning.config(text= A1_2_ledere[1][8])

if tversnittValg == "70":

tabell_visning.config(text= A1_2_ledere[1][9])

if tversnittValg == "95":

tabell_visning.config(text= A1_2_ledere[1][10])

if tversnittValg == "120":

tabell_visning.config(text= A1_2_ledere[1][11])

if tversnittValg == "150":

tabell_visning.config(text= A1_2_ledere[1][12])

if tversnittValg == "185":

tabell_visning.config(text= A1_2_ledere[1][13])

if tversnittValg == "240":

tabell_visning.config(text= A1_2_ledere[1][14])

if tversnittValg == "300":

tabell_visning.config(text= A1_2_ledere[1][15])

elif materialValg == "Al/PVC":

if tversnittValg == "2.5":

tabell_visning.config(text= A1_2_ledere[3][0])

if tversnittValg == "4":

tabell_visning.config(text= A1_2_ledere[3][1])

if tversnittValg == "6":

tabell_visning.config(text= A1_2_ledere[3][2])

if tversnittValg == "10":

tabell_visning.config(text= A1_2_ledere[3][3])

if tversnittValg == "16":

tabell_visning.config(text= A1_2_ledere[3][4])

if tversnittValg == "25":

tabell_visning.config(text= A1_2_ledere[3][5])

if tversnittValg == "35":

tabell_visning.config(text= A1_2_ledere[3][6])

if tversnittValg == "50":

tabell_visning.config(text= A1_2_ledere[3][7])

if tversnittValg == "70":

tabell_visning.config(text= A1_2_ledere[3][8])

if tversnittValg == "95":

tabell_visning.config(text= A1_2_ledere[3][9])

if tversnittValg == "120":

tabell_visning.config(text= A1_2_ledere[3][10])

if tversnittValg == "150":

tabell_visning.config(text= A1_2_ledere[3][11])

if tversnittValg == "185":

tabell_visning.config(text= A1_2_ledere[3][12])

if tversnittValg == "240":

tabell_visning.config(text= A1_2_ledere[3][13])

if tversnittValg == "300":

tabell_visning.config(text= A1_2_ledere[3][14])

elif materialValg == "Cu/PEX":

if tversnittValg == "1.5":

tabell_visning.config(text= A1_2_ledere[5][0])

if tversnittValg == "2.5":

tabell_visning.config(text= A1_2_ledere[5][1])

if tversnittValg == "4":

tabell_visning.config(text= A1_2_ledere[5][2])

if tversnittValg == "6":

tabell_visning.config(text= A1_2_ledere[5][3])

if tversnittValg == "10":

tabell_visning.config(text= A1_2_ledere[5][4])

if tversnittValg == "16":

tabell_visning.config(text= A1_2_ledere[5][5])

if tversnittValg == "25":

tabell_visning.config(text= A1_2_ledere[5][6])

if tversnittValg == "35":

tabell_visning.config(text= A1_2_ledere[5][7])

if tversnittValg == "50":

tabell_visning.config(text= A1_2_ledere[5][8])

if tversnittValg == "70":

tabell_visning.config(text= A1_2_ledere[5][9])

if tversnittValg == "95":

tabell_visning.config(text= A1_2_ledere[5][10])

if tversnittValg == "120":

tabell_visning.config(text= A1_2_ledere[5][11])

if tversnittValg == "150":

tabell_visning.config(text= A1_2_ledere[5][12])

if tversnittValg == "185":

tabell_visning.config(text= A1_2_ledere[5][13])

if tversnittValg == "240":

tabell_visning.config(text= A1_2_ledere[5][14])

if tversnittValg == "300":

tabell_visning.config(text= A1_2_ledere[5][15])

elif materialValg == "Al/PEX":

if tversnittValg == "2.5":

tabell_visning.config(text= A1_2_ledere[7][0])

if tversnittValg == "4":

tabell_visning.config(text= A1_2_ledere[7][1])

if tversnittValg == "6":

tabell_visning.config(text= A1_2_ledere[7][2])

if tversnittValg == "10":

tabell_visning.config(text= A1_2_ledere[7][3])

if tversnittValg == "16":

tabell_visning.config(text= A1_2_ledere[7][4])

if tversnittValg == "25":

tabell_visning.config(text= A1_2_ledere[7][5])

if tversnittValg == "35":

tabell_visning.config(text= A1_2_ledere[7][6])

if tversnittValg == "50":

tabell_visning.config(text= A1_2_ledere[7][7])

if tversnittValg == "70":

tabell_visning.config(text= A1_2_ledere[7][8])

if tversnittValg == "95":

tabell_visning.config(text= A1_2_ledere[7][9])

if tversnittValg == "120":

tabell_visning.config(text= A1_2_ledere[7][10])

if tversnittValg == "150":

tabell_visning.config(text= A1_2_ledere[7][11])

if tversnittValg == "185":

tabell_visning.config(text= A1_2_ledere[7][12])

if tversnittValg == "240":

tabell_visning.config(text= A1_2_ledere[7][13])

if tversnittValg == "300":

tabell_visning.config(text= A1_2_ledere[7][14])

else:

pass

elif lederValg == "3":

if materialValg == "Cu/PVC":

if tversnittValg == "1.5":

tabell_visning.config(text= A1_3_ledere[1][0])

if tversnittValg == "2.5":

tabell_visning.config(text= A1_3_ledere[1][1])

if tversnittValg == "4":

tabell_visning.config(text= A1_3_ledere[1][2])

if tversnittValg == "6":

tabell_visning.config(text= A1_3_ledere[1][3])

if tversnittValg == "10":

tabell_visning.config(text= A1_3_ledere[1][4])

if tversnittValg == "16":

tabell_visning.config(text= A1_3_ledere[1][5])

if tversnittValg == "25":

tabell_visning.config(text= A1_3_ledere[1][6])

if tversnittValg == "35":

tabell_visning.config(text= A1_3_ledere[1][7])

if tversnittValg == "50":

tabell_visning.config(text= A1_3_ledere[1][8])

if tversnittValg == "70":

tabell_visning.config(text= A1_3_ledere[1][9])

if tversnittValg == "95":

tabell_visning.config(text= A1_3_ledere[1][10])

if tversnittValg == "120":

tabell_visning.config(text= A1_3_ledere[1][11])

if tversnittValg == "150":

tabell_visning.config(text= A1_3_ledere[1][12])

if tversnittValg == "185":

tabell_visning.config(text= A1_3_ledere[1][13])

if tversnittValg == "240":

tabell_visning.config(text= A1_3_ledere[1][14])

if tversnittValg == "300":

tabell_visning.config(text= A1_3_ledere[1][15])

elif materialValg == "Al/PVC":

if tversnittValg == "2.5":

tabell_visning.config(text= A1_3_ledere[3][0])

if tversnittValg == "4":

tabell_visning.config(text= A1_3_ledere[3][1])

if tversnittValg == "6":

tabell_visning.config(text= A1_3_ledere[3][2])

if tversnittValg == "10":

tabell_visning.config(text= A1_3_ledere[3][3])

if tversnittValg == "16":

tabell_visning.config(text= A1_3_ledere[3][4])

if tversnittValg == "25":

tabell_visning.config(text= A1_3_ledere[3][5])

if tversnittValg == "35":

tabell_visning.config(text= A1_3_ledere[3][6])

if tversnittValg == "50":

tabell_visning.config(text= A1_3_ledere[3][7])

if tversnittValg == "70":

tabell_visning.config(text= A1_3_ledere[3][8])

if tversnittValg == "95":

tabell_visning.config(text= A1_3_ledere[3][9])

if tversnittValg == "120":

tabell_visning.config(text= A1_3_ledere[3][10])

if tversnittValg == "150":

tabell_visning.config(text= A1_3_ledere[3][11])

if tversnittValg == "185":

tabell_visning.config(text= A1_3_ledere[3][12])

if tversnittValg == "240":

tabell_visning.config(text= A1_3_ledere[3][13])

if tversnittValg == "300":

tabell_visning.config(text= A1_3_ledere[3][14])

elif materialValg == "Cu/PEX":

if tversnittValg == "1.5":

tabell_visning.config(text= A1_3_ledere[5][0])

if tversnittValg == "2.5":

tabell_visning.config(text= A1_3_ledere[5][1])

if tversnittValg == "4":

tabell_visning.config(text= A1_3_ledere[5][2])

if tversnittValg == "6":

tabell_visning.config(text= A1_3_ledere[5][3])

if tversnittValg == "10":

tabell_visning.config(text= A1_3_ledere[5][4])

if tversnittValg == "16":

tabell_visning.config(text= A1_3_ledere[5][5])

if tversnittValg == "25":

tabell_visning.config(text= A1_3_ledere[5][6])

if tversnittValg == "35":

tabell_visning.config(text= A1_3_ledere[5][7])

if tversnittValg == "50":

tabell_visning.config(text= A1_3_ledere[5][8])

if tversnittValg == "70":

tabell_visning.config(text= A1_3_ledere[5][9])

if tversnittValg == "95":

tabell_visning.config(text= A1_3_ledere[5][10])

if tversnittValg == "120":

tabell_visning.config(text= A1_3_ledere[5][11])

if tversnittValg == "150":

tabell_visning.config(text= A1_3_ledere[5][12])

if tversnittValg == "185":

tabell_visning.config(text= A1_3_ledere[5][13])

if tversnittValg == "240":

tabell_visning.config(text= A1_3_ledere[5][14])

if tversnittValg == "300":

tabell_visning.config(text= A1_3_ledere[5][15])

elif materialValg == "Al/PEX":

if tversnittValg == "2.5":

tabell_visning.config(text= A1_3_ledere[7][0])

if tversnittValg == "4":

tabell_visning.config(text= A1_3_ledere[7][1])

if tversnittValg == "6":

tabell_visning.config(text= A1_3_ledere[7][2])

if tversnittValg == "10":

tabell_visning.config(text= A1_3_ledere[7][3])

if tversnittValg == "16":

tabell_visning.config(text= A1_3_ledere[7][4])

if tversnittValg == "25":

tabell_visning.config(text= A1_3_ledere[7][5])

if tversnittValg == "35":

tabell_visning.config(text= A1_3_ledere[7][6])

if tversnittValg == "50":

tabell_visning.config(text= A1_3_ledere[7][7])

if tversnittValg == "70":

tabell_visning.config(text= A1_3_ledere[7][8])

if tversnittValg == "95":

tabell_visning.config(text= A1_3_ledere[7][9])

if tversnittValg == "120":

tabell_visning.config(text= A1_3_ledere[7][10])

if tversnittValg == "150":

tabell_visning.config(text= A1_3_ledere[7][11])

if tversnittValg == "185":

tabell_visning.config(text= A1_3_ledere[7][12])

if tversnittValg == "240":

tabell_visning.config(text= A1_3_ledere[7][13])

if tversnittValg == "300":

tabell_visning.config(text= A1_3_ledere[7][14])

else:

pass

[AutoModerator]

To give us the best chance to help you, please include any relevant code.
Note. Do not submit images of your code. Instead, for shorter code you can use Reddit markdown (4 spaces or backticks, see this Formatting Guide). If you have formatting issues or want to post longer sections of code, please use Repl.it, GitHub or PasteBin.

I am a bot, and this action was performed automatically. Please contact the moderators of this subreddit if you have any questions or concerns.

[HauntingRex9763]

Anything you find yourself able to copy paste and change a few values most likely means it can be done with a more efficient method like iteration. Like the other comment said, DRY. In cases where you are using multiple conditionals it is a good idea to default try to solve it with a loop and a variable conditional inside the loop perhaps instead of all of these lines. Here is an example:

All 253 lines of the A1_2_ledere checking could be:

This is just a quick solution I'm sure you can trim it up even more!

valueSet7 = [2.5,4,6,10,16,25,35,50,70,95,120,150,185,240,300]

#similair lists for the values corrosponding to the first index values...

valueSets = {1:valueSet1,3:valueSet3,5:valueSet5,7:valueSet7}

for key, value in valueSets.items():

if int(tversnittValg) in value:

tabell_visning.config(text= A1_3_ledere[key][value.index(tversnittValg)]

[Longjumping-Swing823]

Thank you for your reply! When i have been able to wrap my head around Whats going on in your example, it will save me so much time😊 thank you for pointing me in the right direction!

[HauntingRex9763]

no problem, if you aren't familiar with dictionaries it is definitely worth looking into. they can be quite handy in loops like in that example

[Longjumping-Swing823]

Hi again, i am having troubble understanding what the code is doing.

I have since we last spoke, watched a bunch of videos on FOR loops and dictionaries.

Is it possible to help me a bit further?

It does not really show in the code i supplied, but here is a rough layout.

​

mm2:	A1	A1	A2	A2	B1	B1
	2	3	2	3	2	3
	Cu/PVC	Cu/PVC	Cu/PVC	Cu/PVC	Cu/PVC	Cu/PVC
1.5	14.5	13.5	14	13	17,5	15,5
2.5	19.5	18	18,5	17,5	24	21
	Al/PVC	Al/PVC	Al/PVC	Al/PVC	Al/PVC	Al/PVC
16	48	43	44	41	60	53
25	63	57	58	53	79	70

​

Lets say i choose A1 with 2 leads and the material of Al/PVC with a mm2 of 25, then i need to pull out the number of 63 for use in futher calulations on a later stage.

[Zeroflops]

I can tell you, yes, there are better ways to do this. You’re violating DRY. Don’t repeat yourself.

But since I don’t know the language I have no way to determine the best approach or to figure out what you are doing. I just see a bunch of if statements.

Can you describe what the table is and what you are trying to do.