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

关于变更的部分 #40

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

关于变更的部分 #40

wants to merge 4 commits into from

Conversation

MtDesert
Copy link

把连续的4个空格都换成了TAB
删除了许多IS_SERVER的判断,主要的观点是服务端不一定要和客户端的代码完全一样,不然就显得非常庞大,鄙人认为客户端和服务端主要是通讯,所以应该在协议上下功夫
在创建主场景的时候,本来是从外部传递的登陆状态,变成了在构造函数内部直接判断套接字管理器中的登陆状态,这样省了一些不必要的代码
增加了下载模块,可以依次下载文件,目前是进入游戏就下载,如有必要我们可以以后改成手动
希望能提一些意见或者建议

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

Successfully merging this pull request may close these issues.

1 participant