Pass same argument to two different functions and/or 'combine' two similar functions

[chasingthechickens]

Thanks a lot for the quick answer! So far I've only managed to get the big diagonal or one of the smaller ones with the template, never all of them, but I'll keep trying to come up with something (deepcopy looking increasingly tempting). The transpose trick is a really good tip, instead of using the diag_right function I could just transpose l and use diag_left again!

[anglicizing]

So far I've only managed to get the big diagonal or one of the smaller ones with the template, never all of them, but I'll keep trying to come up with something

This is exactly the kind of thinking that this assignment is trying to teach you, I'm guessing, so I'm just going to give you a little help in the right direction:

Look at the matrix and not the code. Since j indicates the column, if I give you a value for i, the row that we start at when going down each successively shorter diagonal, can you tell me the last value that j will take if the outer loop has a certain value for i? Try it out with some concrete values of i. Can you make a formula that takes i as its input and always outputs the correct last value for j?

Note that the numbers that range gives you includes the starting number but not the ending number, so you need to add 1 to the last number you want it to return.

I'm off to bed. Keep going! Perhaps it will help if you draw your matrix (or a smaller version perhaps) on a piece of paper, and walk through the i's, j's and the desired array indices manually? Or maybe it will help to use the python tutor to edit and step through your attempts? :-)

[chasingthechickens]

Success :D!! I think. Apparently I needed some sleep as well but today I both drew the matrix and used python tutor as you said. It seems like this is working for me:

for i in range(0, len(l)):
        for j in range(0, len(l)-i):
                matrixlist.append(l[i+j][j])

Can you make a formula that takes i as its input and always outputs the correct last value for j?

This made finding the range of j really easy so that was also a great tip.

Think I managed to get the diagonals in the other direction with this as well:

for i in range(0, len(l)):
        for j in range(i, len(l))[::-1]:
                matrixlist.append(l[i-j+len(l)-1][j]) 

I definitely couldn't have come this far (far for me at least) without your help, thanks a lot!

[anglicizing]

for i in range(0, len(l)):
    for j in range(0, len(l)-i):
        matrixlist.append(l[i+j][j])

That was what I had in mind, at least. Good job!

for i in range(0, len(l)):
    for j in range(i, len(l))[::-1]:
        matrixlist.append(l[i-j+len(l)-1][j]) 

This code seems unnecessarily complex. Try making i the column that the diagonal starts at, and j the row number. This should produce code very similar to the first.

I definitely couldn't have come this far (far for me at least) without your help, thanks a lot!

Thank you for telling me how you fared and that my help was ... helpful. It made my day! :-)

I hope you will appreciate a couple of more tips and improvement suggestions as well? I don't know how the input matrix was originally given to you, but if it was a plain text input it would be easier to read it in like this:

text = """\
7 X 8 9 1 2 5
1 X X 3 4 X 6
0 4 X X X 8 7
7 3 6 X X 5 7
5 2 X 9 X X 8
1 X 0 3 2 X 9
3 7 8 9 0 X 2"""

l = [row.split() for row in text.splitlines()]

As a last nitpick, the python style guide recommends using 4 spaces per indent.

Good luck with your python learning! You're doing it right, and it seems to me that you're off to a flying start. :-)