weiyhcarlos / trainning

0 stars 0 forks source link

代码问题 #7

Open valensc opened 8 years ago

valensc commented 8 years ago
  1. 尽量少用module globals级别的变量
  2. 类似services.js line 25 附近的hardcode代码有点多,考虑下更优雅的方式
  3. detail_part的几个template没有必要,只用一个template即可实现
  4. template内硬编码也略多,其实很多options、links之类,可以在js内维护数据结构,tpl内循环生成
  5. machines_resources.py 73行开始的if else也是,想想怎么抽象更好
  6. 有余力的情况下,完善单元测试吧,pytest即可
weiyhcarlos commented 8 years ago

1.由于使用echart的个人开源angularjs封装项目,不能做到canvas重绘(功能不全面),需要通过页面刷新。因此需要使用一些global级别的变量。已通过整合页面减少了global级别的变量 2.通过方法封装来生成dict 3.已整合 4.整合页面之后可以共用options,大大减少controller代码 5.通过dict,用更优雅的方式重新实现 6.完成agent(test_collector.py, test_handler.py),数据接口(test_interface.py),后台(test_api_backend.py)的单元测试,由于一个repo需要放四个独立的项目部分,因此以上的py文件放在各自的目录下(需要Import相关模块,不好放在顶层目录的tests文件夹下)。通过唯一入口-tests/run.py驱动