-
Notifications
You must be signed in to change notification settings - Fork 220
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
意见反馈 #22
Comments
价格爬爬就好了 |
@gouyuwang 这个已经在考虑中了哈,由于目前房协网是没有显示价格的,只能从第三方网站上爬取,现在正在思考怎么更优雅的整合这两个数据,🤔🤔。 |
爬虫代码在本repository吗? |
@lefter 是的,在这个目录下: src/nodeuii/utils/spiderHelper.ts |
我的意思是爬内容的代码,应该是用python写的脚本吧?node也能做爬虫吗? |
@lefter 项目中的爬虫的确是用 node 写的。而且能做爬虫的不是只有 python 而已,php、java、node 等都可以。只要是能够发起 http 请求,接收 http 响应,就可以用来做爬虫,甚至直接写 shell + curl 都行。 |
cdfangxie 有价格吧?只是在 pdf 里。可能不是很方便解析。 |
@lambdaq 谢谢提醒,读取解析 pdf 目前没找到合适的库,后面应该会爬取摇号助手里的价格😏😏 |
我跑起来本地开发环境有两个地方的代码因为路径原因导致的报错,一个是koa-static那里 |
@guojintao32 你好,我这边确实是用的 mac,后面我用 windows 试试,估计和操作系统的路径 path |
项目的ts有生效吗,因为没看到有用到ts-loader,ts在这里感觉没有起到作用 |
你好,项目中使用 babel 编译 ts,和 tsc 编译 ts 文件是一个道理。这里使用 babel 是为了统一项目的编译器。 @DavidWong9785 |
为啥要封装一个init方法 https://github.com/lmjben/cdfang-spider/blob/master/src/nodeuii/controllers/index.ts#L66-L68 ?我或许会直接在入口 |
@GengPeng951 你好,这段代码 |
👏👏 欢迎大家提出宝贵的意见 👏👏
The text was updated successfully, but these errors were encountered: