Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

我把代码整理了一下去掉了没有用到的代码 #8

Open
4ever911 opened this issue Mar 17, 2017 · 3 comments
Open

我把代码整理了一下去掉了没有用到的代码 #8

4ever911 opened this issue Mar 17, 2017 · 3 comments

Comments

@4ever911
Copy link

1.把所有没有用到的函数变量都去掉了,
2.去掉了所有重复的代码
3.把登录和取数据都整合到SinaL2里面去了
4.基本没有做代码和结构的调整,全部整到一个函数里面了。
5.之前代码会需要本地有sina.json和config.json(import dhydra就会调用), 现在通过参数传递进去,不会生成碍眼的本地文件。
6.loggin我稍微修改了一下。

另外,我个人觉得Trading-time 没必要添加到里面去,尽量保持数据的原始值。外部调用的可以自己去判断。

另外一个问题:对于全市场差不多3000只股票,我现在是可以全部推送,但是看代码里面,每个线程分配了64个,这样差不多要建50个ws连接,会不会太多。我尝试修改为300,好像也没问题。不知道你当时设置64 的考虑是什么?

SinaL2.zip

附上我整合后的代码,感觉还需要重构一下代码的逻辑。

@Emptyset110
Copy link
Owner

很感谢啊,我周末一定看了改。github上是可以直接fork出去向我递交代码的,不知道你会不会用。

你修改成300也没有问题吗?因为当时的测试情况是一旦超过64个了,那么多余的部分会被截掉,就不会被推送的。你换成300以后有检测是否遗漏吗?

@4ever911
Copy link
Author

应该谢谢你做的这个项目才对。

我还没有检测,因为现在只是盘后数据,我下周一测试看看。

@Emptyset110
Copy link
Owner

@4ever911 ……方便加一下QQ吗?21324784

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants