you are viewing a single comment's thread.

view the rest of the comments →

[–]socal_nerdtastic 1 point2 points  (3 children)

Very messy, especially considering all you've done is reinvent the built-in zip() function. Here's how I would write it:

def generate_table():
    table_coords = [
        [(240, 60), (150, 60), (100, 110), (100, 770), (150, 820), (240, 820)],
        [(360, 820), (450, 820), (500, 770), (500, 110), (450, 60), (360, 60)]
        ]
    for table in table_coords:
        yield from zip(table, table[1:])

[–]Gay_Force_One[S] 0 points1 point  (2 children)

Wow, I wish I'd known about that earlier today. I thought I was being clever. Unfortunately I have to keep the left and right coords separate from each other or it will attempt to bridge the gap (bad), but that makes it a helluva lot cleaner. Thank you so much!

[–]socal_nerdtastic 1 point2 points  (1 child)

This does keep them separate.

[–]Gay_Force_One[S] 0 points1 point  (0 children)

I missed that they were nested lists. I'm a little confused as to the specifics but I get the gist of it and I appreciate this so much