nodequant
nodequant copied to clipboard
提一些小建议
- 建议添加 eslint , editorconfig 等代码风格 config , 比较方便其他人提交代码
- 可以开一个 slack group 吗?感觉在 qq 群不能贴代码,slack 或者 gitter 比较适合
- 看到很多变量放到了 global 变量里,其实是不是应该直接新建一个单例暴露出来会比较好
- 感觉如果把接收的逻辑分离出来会比较好,一个是可以避免策略逻辑复杂阻塞 tick 行情的推送,一个是可以支持回测。
感谢建议,以下简单回复 1.@Elric-pp editorconfig风格的config?你是说修改一下userconfig配置文件的风格吗?其他人提交代码是? 2.因为现在好像量化圈子还比较小,暂时没有开其他讨论组,个人精力有限管理 3.global变量只是用了几个,就是主程序,主引擎,策略引擎,事件分发器 4.接收的逻辑分离?具体是指?由于node.js是一个事件队列,策略耗时长是可能阻塞着tick排队在事件队列中,如果策略逻辑耗时长建议用tagg2等多线程库。接收tick逻辑现在是基于事件的会直接由strategyEngine推送到策略
- 就是一些代码风格的配置,比如说缩进,tab 或者 空格,还有 js 的风格检测,这样子像我贡献代码的时候可以保证整个项目的代码风格保持一致
- 其实我可以开一个:D,不过这个不是特别重要啦,方便项目管理而已。
- 是的,但是模块化的方案会比较可读些
- 就是 CTP client 直接推送数据然后存到 redis 里,和策略逻辑分离开来,不然如果要怎么实现历史数据回测呢?
刚放完假,懒洋洋的。关于第四点建议,redis数据库的数据是存在内存的,交易系统用它是非常好的。行情数据太多了,要用其他数据库做回测。回测想用ssdb数据库+cluster多进程+网页显示回测结果图形,各方面差不多准备好了,差点时间把它写了。@Elric-pp
用eslint做代码风格的规范统一还是有必要的,另外建议升级到node 8,这样基本支持所有的es6特性了
eslint风格之后会用工具统一,node 8会升级,因为现在node.js官网node 8还不是LTS版本,10月31日才是LTS版本,所以会在10月31日之后升级到node 8 @yuu2lee4
建议添加 eslint , editorconfig 等代码风格 config , 比较方便其他人提交
我对这个很感兴趣,如果作者不介意,我愿意完成这项任务。