修复一个线上1万6千次错误的bug

def get_root(self): try: self.photo except: return key = "photo:root:%s:" % self.photo.id root = cache.get(key) if root is None: try: root = self.category in [1, 9] and \ self or Message.objects.get(photo__id=self.photo_id, category__in=[1, 9], status__in=[0, 6])[0] except Message.DoesNotExist: pass if root: cache.set(key, root, 3600 * 2) return root and root.status != 5 and root or None

?里面的get其实存在大于1这种情况的。修改为：

?

   if root is None:            try:                root = self.category in [1, 9] and \                    self or Message.objects.filter(photo__id=self.photo_id, category__in=[1, 9], status__in=[0, 6])[0]            except Message.DoesNotExist:                pass            except IndexError:                pass

?

?

小结：

? ? 对于这种业务上的唯一性，一定要通过数据库保证唯一性约束

?

?

?

?

?

?

?