Mogara / QSanguosha-v2

Main Repository of QSanguosha V2
Other
360 stars 192 forks source link

加入UPNP、列表服务器功能,提高编译速度 #47

Closed popkc closed 9 years ago

popkc commented 9 years ago

开启了预编译头文件 减少了头文件间的相互包含 加入UPNP端口映射功能 加入查找服务器功能,可方便找到正在运行的服务器。 加入加入列表服务器功能,方便让人找到。

popkc commented 9 years ago

默认的列表服务器忘了改,请设成http://fairsgs-fairsgs.stor.sinaapp.com/

Fsu0413 commented 9 years ago

…………谢谢这位程序员的努力,不过这个Pull Request我不准备合入。 我来客观的说一下情况吧。

1、这个repo是旧框架,既然被称作旧框架,就肯定会有一些历史遗留问题(比如已经完全不能用的Debug),已经不准备进行功能上的更新,对代码进行大张旗鼓的修改我们肯定是不会同意的。 2、现在我们正在发愁新框架没人干活,组长有事暂时不能进行更新,现在只有代理组长和一些组员在努力的弄代码。除了代理组长之外,其他写代码的人都在弄旧框架。我们也正在寻找可以进入新框架开发的人才。 3、新框架准备开发“大厅”功能,届时将会有可能(我只能说有可能,真正如何实现我也不清楚)需要这份代码,具体如何弄可以询问代理组长 @takashiro

综上,我认为这个Pull Request已经没有合入的必要。 如有疑问,可以在这里回复,也可以加入开发群,咱们一起讨论。

-- Mogara-旧框架负责人 Fsu0413

Fsu0413 commented 9 years ago

…………额外说一句,现在项目发布时,只有Mac版使用Qt Creator进行编译,而在Win上是使用VS进行编译的,在Qt 项目中加入PCH就相当于废了VS,这种修改我们是不可能接受的。

popkc commented 9 years ago

新框架在哪 ?

popkc commented 9 years ago

加入PCH等于废了VS我不能理解,QT工程的编译方式不都是一样的么(先生成MAKEFILE,然后MOC,然后才是编译器编译),和IDE有什么关系?

popkc commented 9 years ago

本来我也是有意向要给太阳神三国杀加入大厅功能的(我还打算尝试加入房间选择和观战功能),如果已经有人在做了,那么一起做正好。

popkc commented 9 years ago

@dragon100a 是啊。

Fsu0413 commented 9 years ago

VS是使用他自己的环境进行编译的,要求每个cpp都要include那个预编译头文件对应的.h。不是使用qmake生成makefile进行编译的。所以加了pch肯定废了VS。 新框架是QSanguosha工程,不过现在暂时处于基本没人干的阶段……

Fsu0413 commented 9 years ago

https://github.com/Mogara/QSanguosha/tree/dev 这是新项目的地址,只有dev分支是新框架,master分支是旧框架

popkc commented 9 years ago

那就给每个cpp都加呗,确实能提高编译速度啊。嫌麻烦的话让我来做,我处理头文件相互包含的问题时比这麻烦多了。

popkc commented 9 years ago

新框架使用的是QML。。。而且这个项目基本是空的。。。 如果要支持手机、平板的话,我觉得用QML做界面就好,内核部分完全没必要改吧。

popkc commented 9 years ago

http://stackoverflow.com/questions/13710243/how-to-use-precompiled-headers-in-qt-project

sunnyadn commented 9 years ago

内核用了5年了,之前的开发也没有统一标准,导致现在非常难维护。另外新的架构有新的设计,从Cardirector的submodule上就能看出些端倪。 项目风格可以参考一下这个 https://github.com/Mogara/QSanguosha-For-Hegemony/wiki/Coding-Style

Fsu0413 commented 9 years ago

内核的确要重做了……从已经废了的debug就能看出来。新框架现时确实是空的,不过旧框架不也是从空项目一点一点搭起来的么。

popkc commented 9 years ago

我用VS2010实际测试了一下,PCH并没有问题,并不需要每个文件手动INCLUDE。 哎,可能你们有自己的想法吧,但我觉得我的这些改动是好的,如果主项目不接纳的话,我只能继续在分支项目继续下去了。

Fsu0413 commented 9 years ago

这样吧,加一下开发群258201399,加入时注明身份,和大家探讨下新框架的开发事宜,怎样