修改代码格式

[jiminHuang]

• stock_price.py
  • 行7 不必要的空行
  • 行39-44
  • 不抛出异常的语句可以不用try
  • 判断值不合理毫无必要 因为不合理的值不会循环
  • 行50-61 使用for循环就可以了
  • 行35-38 这样明显要更好

number_of_total_page_ustr_list = \
    response.css('.page_info').xpath('./text()').extract()

• test_stock_price.py
  • 行25-28 缩进的问题
  • 行57-58 同样这样可读性比较差，可以这样写

    fake_item['stock_market_link'] = '{0}/{1}'.format(
        'http://q.10jqka.com.cn/stock/thshy',
        fake_data['hycode']
    )

• test_database.py
  • 行24
  • 使用mysqldb比较好
  • 你这样写不还是暴露了数据库连接的全部信息
  • 这里明显过长
  • 行33 打log而不是print
• data.py
  • 行31 为什么还要rollback
  • 行36-40 为什么不直接使用update
  • 行41 这里也可以with session
  • 行44 raise e比较好
• test_data.py
  • 行3 不必要的空行
  • 行4 这里Mock从未被使用
  • 行6 不必要的空行
  • 行56 使用已有数据做测试 这样测试不依赖上一个样例的结果

[jiminHuang]

• test_database.py
  • 行27-35 这里更简洁的写法应该是直接mock掉config的返回值
• data.py
  • 行27-33 这里也不要这样写 这样可能更好看一点

return (
    u"<ItemTable(stock_market='{0}',"
    u"stock_market_link='{1}',"
    u"scrape_time=='{2}')>"
).format(
   self.stock_market,
   self.stock_market_link,
   self.scrape_time
)

- 行47-52 既然exception会往上抛 你可以转换一下思路 在insert外用一个方法try住再处理 不是更好么
- 行63-64 没必要弄出一个变量出来