我只需要了解如何检测scrapy是否已保存以及蜘蛛中的物品?我从网站上提取商品,然后我就该商品提取评论。所以首先我必须保存项目,之后我会保存评论。但是当我在编写代码之后,它会给我这个错误。
save() prohibited to prevent data loss due to unsaved related object ''.
这是我的代码
def parseProductComments(self, response):
name = response.css('h1.product-name::text').extract_first()
price = response.css('span[id=offering-price] > span::text').extract_first()
node = response.xpath("//script[contains(text(),'var utagData = ')]/text()")
data = node.re('= (\{.+\})')[0] #data = xpath.re(" = (\{.+\})")
data = json.loads(data)
barcode = data['product_barcode']
objectImages = []
for imageThumDiv in response.css('div[id=productThumbnailsCarousel]'):
images = imageThumDiv.xpath('img/@data-src').extract()
for image in images:
imageQuality = image.replace('/80/', '/500/')
objectImages.append(imageQuality)
company = Company.objects.get(pk=3)
comments = []
item = ProductItem(name=name, price=price, barcode=barcode, file_urls=objectImages, product_url=response.url,product_company=company, comments = comments)
yield item
print item["pk"]
for commentUl in response.css('ul.chevron-list-container'):
url = commentUl.css('span.link-more-results::attr(href)').extract_first()
if url is not None:
for commentLi in commentUl.css('li.review-item'):
comment = commentLi.css('p::text').extract_first()
commentItem = CommentItem(comment=comment, product=item.instance)
yield commentItem
else:
yield scrapy.Request(response.urljoin(url), callback=self.parseCommentsPages, meta={'item': item.instance})
这是我的管道。
def comment_to_model(item):
model_class = getattr(item, 'Comment')
if not model_class:
raise TypeError("Item is not a `DjangoItem` or is misconfigured")
def get_comment_or_create(model):
model_class = type(model)
created = False
# Normally, we would use `get_or_create`. However, `get_or_create` would
# match all properties of an object (i.e. create a new object
# anytime it changed) rather than update an existing object.
#
# Instead, we do the two steps separately
try:
# We have no unique identifier at the moment; use the name for now.
obj = model_class.objects.get(product=model.product, comment=model.comment)
except model_class.DoesNotExist:
created = True
obj = model # DjangoItem created a model for us.
obj.save()
return (obj, created)
def get_or_create(model):
model_class = type(model)
created = False
# Normally, we would use `get_or_create`. However, `get_or_create` would
# match all properties of an object (i.e. create a new object
# anytime it changed) rather than update an existing object.
#
# Instead, we do the two steps separately
try:
# We have no unique identifier at the moment; use the name for now.
obj = model_class.objects.get(product_company=model.product_company, barcode=model.barcode)
except model_class.DoesNotExist:
created = True
obj = model # DjangoItem created a model for us.
obj.save()
return (obj, created)
def update_model(destination, source, commit=True):
pk = destination.pk
source_dict = model_to_dict(source)
for (key, value) in source_dict.items():
setattr(destination, key, value)
setattr(destination, 'pk', pk)
if commit:
destination.save()
return destination
class ProductItemPipeline(object):
def process_item(self, item, spider):
if isinstance(item, ProductItem):
item['cover_photo'] = item['files'][0]['path']
item_model = item.instance
model, created = get_or_create(item_model)
#update_model(model, item_model)
if created:
for image in item['files']:
imageItem = ProductImageItem(image=image['path'], product=item.instance)
imageItem.save()
# for comment in item['comments']:
# commentItem = CommentItem(comment=comment, product= item.instance)
# commentItem.save()
return item
if isinstance(item, CommentItem):
comment_to_model = item.instance
model, created = get_comment_or_create(comment_to_model)
if created:
print model
else:
print created
return item
答案 0 :(得分:1)
您的代码的很大一部分似乎处理了get_or_create
的明显弱点# Normally, we would use `get_or_create`. However, `get_or_create` would
# match all properties of an object (i.e. create a new object
# anytime it changed) rather than update an existing object.
幸运的是,这种明显的短缺可以很容易地克服。感谢get_or_create
的默认参数传递给get_or_create()的任何关键字参数 - 除了可选项 一个名为defaults - 将用于get()调用。如果一个对象是 find,get_or_create()返回该对象的元组,返回False。如果 找到多个对象,get_or_create引发 MultipleObjectsReturned。如果找不到对象,则get_or_create() 将实例化并保存一个新对象,返回一个新元组 对象和真实。
仍然不相信get_or_create是这项工作的合适人选吗?我也不是。还有更好的东西。 update_or_create !!
使用给定的kwargs更新对象的便捷方法, 必要时创建一个新的。默认值是字典 (字段,值)对用于更新对象。
但我不打算详细介绍update_or_create的用户,因为代码中尝试更新模型的行已被注释掉,而您还没有明确说明要更新的内容。
使用标准API方法,包含管道的模块只会缩减为ProductItemPipeline类。这可以修改
class ProductItemPipeline(object):
def process_item(self, item, spider):
if isinstance(item, ProductItem):
item['cover_photo'] = item['files'][0]['path']
model, created = ProductItem.get_or_create(product_company=item['product_company'], barcode=item['bar_code'],
defaults={'Other_field1': value1, 'Other_field2': value2})
if created:
for image in item['files']:
imageItem = ProductImageItem(image=image['path'], product=item.instance)
imageItem.save()
return item
if isinstance(item, CommentItem):
model, created = CommentItem.get_or_create(field1=value1, defaults={ other fields go in here'})
if created:
print model
else:
print created
return item
我确实认为这是存在错误的地方。
obj = model_class.objects.get(product=model.product, comment=model.comment)
现在我们没有使用它,所以bug应该消失。如果您仍有问题,请粘贴完整追溯。