Rapidshare to Megaupload script

MRAB google at mrabarnett.plus.com
Sun Feb 15 14:10:42 EST 2009


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)



More information about the Python-list mailing list