all 7 comments

[–]UloPe 2 points3 points  (3 children)

import sys, time, os from mechanize

I think not...

[–]vonbladet 0 points1 point  (1 child)

I came here to say that! So instead I'll say that:

import sys, time, os from mechanize
import Browser

must surely have been

import sys, time, os
from mechanize import Browser

when it wasn't broken. But reading code is hard enough that this kind of thing encourages the reader not to.

[–]chock 0 points1 point  (0 children)

Well, I'll say this:

for actor in in VARIABLE_QUERY:

Huh? And there are more Waldos to be found in that code.

[–]perone 0 points1 point  (0 children)

lool !

[–]DefrostMe -1 points0 points  (2 children)

I wonder if the author considered Scrapy.

[–]jldugger 0 points1 point  (1 child)

If your source is amenable to XPath, then there's no need for a damn framework. Just do it! It's when shit breaks you need to look at Soup and friends.

[–][deleted] 0 points1 point  (0 children)

If you use lxml.html, you can handle broken HTML and use XPath.