aiwarrior wrote:
Thanks a lot for your input i really needed because i realized these
are minor flaws but even so define whether its good or bad code and i
really need to improve that. I already implemented the changes you
suggested and this one,
cookie = dict(x.split("=") for x in cookie)
for me is just very good and really is elegant. An aspect you didn't
mention was the
urls.append("http://rapidshare.com/files/"; + retrieved_data[0] +
"/" + retrieved_data[1])
I think its very hackish and crude but urlparser doesnt seem to
accept more than one argument at a time and doing it in a loop seem
worse than the current solution. What would you do?
re.findall() returns a list of tuples, so retrieved_data is a tuple and
the following will work:
urls.append("http://rapidshare.com/files/%s/%s"; % retrieved_data)
--
http://mail.python.org/mailman/listinfo/python-list
