异常处理思考🤔

源于一行log：[2018-08-24 10:28:07,042: ERROR/MainProcess] 541137549407： update table error: TypeError: tuple indices must be integers, not str。

看到这行log... 几乎没有办法debug。

def update_product(self, item_id, product_data, where, is_zk_coupon, table_name="coupon_product_v2", update_source_id=False):
        """
        更新coupon_product_v2, 发送消息
        :param data_dict:
        :param is_zk_coupon:
        :param table_name: 更新数据表
        :return: None/coupon_id
        """
        print(json.dumps(product_data), '=' * 20)
        try:
            update_coupon_data = self.get_should_update_product_info(item_id, product_data, is_zk_coupon, update_source_id=update_source_id)
            if not update_coupon_data:
                self.logger.info('item id : {} can not get prod info, maybe start fee or price stuff,raw prod info is {}'.format(item_id, product_data))
                return

            self.ignore_admin_columns(item_id, product_data)

            self.db_write.update(update_coupon_data, where, table_name)
        except Exception as e:
            self.logger.error("{} update table error: {}".format(item_id, e))
            return
        else:
            errorcount.product_update()
            self.logger.info("{} update product success:{}".format(item_id, update_coupon_data))
            tmp_info = self.db_read.select(["coupon_id"], where="item_id={}".format(item_id), table="coupon_product_v2",
                                           result_type='dict')[0]
            coupon_id = tmp_info["coupon_id"]
            # 发送消息
            # self.send_publish('update_cache', 'product', 'u', [coupon_id], data={coupon_id: update_coupon_data})
            DbAcTracker().send_msg('product', 'u', [coupon_id], data={coupon_id: update_coupon_data})
            return coupon_id

• 在一个try里做了过多逻辑。捕获异常进行了 失败记录，后面的逻辑不清楚具体失败的原因。
• log 使用了error，但是没有使用exception，使得没有错误堆栈被打印出来。
• exception的error 没有定义清楚 到底属于哪个异常。捕获了所有异常，使得一些bug被隐藏。并且异常没有被处理。

第一步 ：去掉 包罗万象的try

[2018-08-24 10:28:07,119: ERROR/ForkPoolWorker-2] Task zion_task_savedb[4ea693cf-07ec-41e5-b3c5-c30d0004b0cd] raised unexpected: TypeError('tuple indices must be integers, not str',)
Traceback (most recent call last):
  File "/usr/local/lib/python2.7/site-packages/celery/app/trace.py", line 367, in trace_task
    R = retval = fun(*args, **kwargs)
  File "/usr/local/lib/python2.7/site-packages/celery/app/trace.py", line 622, in __protected_call__
    return self.run(*args, **kwargs)
  File "/root/wwwroot/Zion/tasks/base_zion.py", line 33, in wrapper
    res = func(self, result_info, *args, **kwargs)
  File "/root/wwwroot/Zion/tasks/task_savedb.py", line 294, in run
    coupon_id = self.store_coupon_product(coupon_row, item_value, item_id, item_info['raw_price'])
  File "/root/wwwroot/Zion/tasks/task_savedb.py", line 701, in store_coupon_product
    self.update_product(item_id, item_value, "item_id={}".format(item_id), False, update_source_id=True)
  File "/root/wwwroot/Zion/tasks/common/update_base.py", line 623, in update_product
    self.ignore_admin_columns(item_id, product_data)
  File "/root/wwwroot/Zion/tasks/common/update_base.py", line 604, in ignore_admin_columns
    if rows and (rows[0]['intv2'] >= 5 or rows[0]['improve'] or rows[0]['is_recommend'] or rows[0]['is_prior']):
TypeError: tuple indices must be integers, not str

看到熟悉的报错，顿时心里就有种莫名的熟悉

好的，经过一番斗(单)智(步)斗(调)勇(试)。原来是[图片上传失败...(image-6f65df-1544761802427)]

第二步：分析一下 需要被捕获的异常。

由于这里面的逻辑，并没有已知且正常的异常需要被处理。所以 就直接把try exception去掉了。

---

知道导致异常的原因，再去捕获特定的异常。

捕获异常的时候，只try 可能出现异常的代码。

需要考虑 异常是否需要被当即处理，还是需要被抛出。

总而言之，言而总之。就是 明白你要捕获的异常是什么再去捕获。捕获异常其实只是打一条log的话，意味着直接忽略异常，那需要考虑的可能就是 这个异常是否应该被忽略。不应该忽略的异常应该被及时的处理。

还有，有数据写入操作但是不打log的都是耍流氓呢