scrapy 中使用 MongoDB 数据库存储抓取到的 IP，判断新抓的 IP 在数据库中是否已经存在，如果存在则不再添加，不存在则插入这条数据，但是执行结果总是不对？

pipelines.py 中的关键代码为：
if db.mycollection.find({"ip":item["ip"]}):
print "{} has already existed.".format(item["ip"])
print db.mycollection.find()
else:
db.mycollection.insert_one(data)
print db.mycollection.count({})
执行的结果总是：
120.25.164.134 has already existed.
<pymongo.cursor.Cursor object at 0x0000000005B330F0>
……
提示我抓取到的每一条数据都是存在的，我的 mycollection 聚集中最开始是空的，为什么会出现这种情况呢？

existed

item

already

14 条回复 2017-12-06 09:03:17 +08:00

zyy1245367562

2017-12-03 14:48:52 +08:00

怎么没有前辈来指教一下呢？

tosexxx

2017-12-03 15:09:52 +08:00

大佬都在度假，没空 8 小时之外看 v 站

SO647898

2017-12-03 15:16:03 +08:00 via Android

大佬都在度假，没空 8 小时之外看 v 站

kenzh

2017-12-03 15:27:51 +08:00

看了下文档，find()总是返回的是 Cursor，猜测应该用 db.mycollection.find({"ip":item["ip"]}).count() 吧。

golmic

2017-12-03 15:29:52 +08:00 via Android

用 upsert

swulling

2017-12-03 16:19:44 +08:00 via iPad

https://docs.mongodb.com/manual/reference/method/db.collection.find/

brickyang

2017-12-03 19:38:17 +08:00 via iPhone

.find() 返回的是 cursor，用 find().toArray() 返回的是查找结果的数组，如果不存在就是个空数组。

你也可以试试 .findOneAndUpdate() 并设置 upsert: true，这样如果找到记录则更新（更新内容为空），找不到则新建一条记录。

fds

2017-12-03 20:00:39 +08:00

返回的是 cursor，肯定为 True 呀，你得调用.next()才可能获得一个空结果吧。
另外，这种约束条件一般是这样实现：在 ip 上建立一个 unique 的索引，然后每次都直接插入；如果已有，则会报错 duplicate，忽略即可。你这种 ifelse 不是“原子”操作，如果有多个进程同时工作，可能插入多条相同 ip 的。

livexia

2017-12-03 21:32:11 +08:00

find_one

lihongjie0209

2017-12-03 21:38:37 +08:00

动态语言就是有这个问题
如果你需要集合为空:
那么使用 if collection.isEmpty 或者是 if collection.size() == 0

如果你需要集合为 null/none:
那么使用 if collection== null/none

这样写代码的时候意图清楚, 看代码的人也轻松.

zyy1245367562

2017-12-05 19:26:28 +08:00

@kenzh 你的建议可以实现。谢谢。

zyy1245367562

2017-12-05 19:29:16 +08:00

@fds 嗯，你说的对，这一点确实没有考虑到。

zyy1245367562

2017-12-05 19:30:01 +08:00

@lihongjie0209 嗯，受教了。谢谢前辈。

toono

2017-12-06 09:03:17 +08:00

find 方法返回的都是集合对象。

下面我的实际代码，是需要先把查询回来的结果对象使用 count 方法去查看具体数量

def process_item(self, item, spider):
result = self.db[self.mongo_collection].find({'source_url': item['source_url']})
if result.count() != 0:
raise DropItem("Duplicate item found: %s" % item)
else:
return item