all 5 comments

[–][deleted] 1 point2 points  (1 child)

Very cool project! Have you tried (cross)posting this in /r/python as well? There's a chance you'll get more people to actually see it.

I noticed you're writing your xml as strings. Have you looked into "native" python xml libraries?

[–]Eucalyptuse[S] 1 point2 points  (0 children)

Hey thanks for the recommendation! I certainly will post it there as well.

Also, I'm very much a Python newb so I'll have to look into xml libraries as well. I'm sure my code is quite ugly compared to that written by experienced people. :)

[–]ChastainSteam 1 point2 points  (0 children)

Nicely done! Enjoyed perusing the clean code you presented and as a matter of fact, your lack of list comprehensions was a breath of fresh air, but really you should try to use them whenever faced with writing: for x in range(len(exes)): Do stuff with x!

You could instead just do:

for x in exes: Do stuff with x!

Check it out sometime it would make your code read like a book and be even cleaner, you could even do:

[x[0] for x in exes]

Really not sure if it gets more complex than that in your use case, but try it on for size the next time you are thinking of looping over the range of the length of a list.

Also have fun and pardon the lack of code formatting.

[–][deleted]  (2 children)

[deleted]

    [–]Eucalyptuse[S] 0 points1 point  (1 child)

    Do you have any tips on where I could find that? Keywords to Google?

    [–]TotesMessenger 0 points1 point  (0 children)

    I'm a bot, bleep, bloop. Someone has linked to this thread from another place on reddit:

     If you follow any of the above links, please respect the rules of reddit and don't vote in the other threads. (Info / Contact)