search

PointTeam / PointDownload

homepage
GNU General Public License v3.0
225 stars 43 forks source link

进行了一些重构，优化了底层数据通信 #9

Closed sbwtw closed 9 years ago

sbwtw commented 9 years ago

之前的任务信息都是以?:?这样的符号连接的qstring传递的，代码中要频繁的split与拼接字符串，冗余代码非常多，而且在一些状态信息的判断比如dl_downing，xware，这样的都是以string来比较，很容易因为拼写错误或者大小写出错，我构建了一个保存任务信息的类TaskInfo来完成这些任务。

之后的代码中任务信息的传递只用这个对象就可以了，可以增强结构性。

由于改动的地方非常多，可能还有很多地方有没发现的问题，希望能多测试。而且我对qml不太熟悉，所以在数据传向qml时依旧拼接成了?:?这样的字串

1dot75cm commented 9 years ago

赞，加油 看好你～

sbwtw commented 9 years ago

谢谢支持^o^

On Thu, Nov 20, 2014, 18:21 1dot75cm notifications@github.com wrote:

赞，加油 看好你～

— Reply to this email directly or view it on GitHub https://github.com/PointTeam/PointDownload/pull/9#issuecomment-63787802.

PointTeam commented 9 years ago

好的,我们尽快测试完

PointTeam commented 9 years ago

这个已经是大部分重构完了是吗？

sbwtw commented 9 years ago

这个只是任务的信息类，把以前用？；？拼接的string改为了一个对象 能够高效率一点 http下和youget我都测过 xware我这里没号账号还没测

On Thu, Nov 27, 2014, 00:17 point-team notifications@github.com wrote:

这个已经是大部分重构完了是吗？

— Reply to this email directly or view it on GitHub https://github.com/PointTeam/PointDownload/pull/9#issuecomment-64670089.

PointTeam commented 9 years ago

xware目前有问题，迅雷改了一些东西，我们正在改，你提交的没问题我们就先合并进去吧

PointTeam commented 9 years ago

迅雷接口已修复

Match-Yang commented 9 years ago

今晚测试，下载完成的记录跟丢到垃圾箱的任务记录会在程序重启后消失，我先去看看哪里有问题

sbwtw commented 9 years ago

已修复

On Sun, Nov 30, 2014, 01:17 杨万青 notifications@github.com wrote:

今晚测试，下载完成的记录跟丢到垃圾箱的任务记录会在程序重启后消失，我先去看看哪里有问题

— Reply to this email directly or view it on GitHub https://github.com/PointTeam/PointDownload/pull/9#issuecomment-64958521.

Match-Yang commented 9 years ago

点击新建按钮会导致弹出框无限循环弹出，导致系统迅速卡死，tty也基本卡死，应该是路径那里写的有点问题

sbwtw commented 9 years ago

界面上那个加号？ 我这里ok 你是release编译的吗

On Sun, Nov 30, 2014, 13:47 杨万青 notifications@github.com wrote:

点击新建按钮会导致弹出框无限循环弹出，导致系统迅速卡死，tty也基本卡死，应该是路径那里写的有点问题

— Reply to this email directly or view it on GitHub https://github.com/PointTeam/PointDownload/pull/9#issuecomment-64976541.

Match-Yang commented 9 years ago

那个卡死的问题，是因为你在修改那个路径定义的时候没有完全改完，像： const QString MAIN_PROGRAM_EXEC = "/opt/Point/PopupWindow/PointPopup"; 这是指定了执行路径的还有好几个地方没改，所以会出错

Match-Yang commented 9 years ago

你的没问题也许是因为刚开始安装到/opt/PointDownload下的文件没有删除，所以在你那里依然能正确运行，我这边是彻底删除了再测试，所以找不到指定的文件

Match-Yang commented 9 years ago

同样的，debian目录下的控制文件也没有按照你做的路径修改做相应修改。程序路径以后确定要改之后我们统一的把涉及到这些路径的一起改进修改，请暂时不要更改执行路径

sbwtw commented 9 years ago

我这边opt下面也没有安装 不过我没有使用debian那些东西，是手动部署的 我觉得可能是popup会主动启动download, 会不会download判断已经启动那里有问题

On Mon, Dec 1, 2014, 20:52 杨万青 notifications@github.com wrote:

同样的，debian目录下的控制文件也没有按照你做的路径修改做相应修改。程序路径以后确定要改之后我们统一的把涉及到这些路径的一起改进修改，请暂时不要更改执行路径

— Reply to this email directly or view it on GitHub https://github.com/PointTeam/PointDownload/pull/9#issuecomment-65060467.

Match-Yang commented 9 years ago

嗯，就是误判的问题，你可不可以先把路径修改那个patch先放放？我这边网速有点慢，测试不是很方便，分开几次来提交吧。而且最近比较忙，真的非常抱歉。我会尽快测试完合并的

1dot75cm commented 9 years ago

路径的问题，我在打包的时候修改了。 @sbwtw 可以参考我的 spec文件修改。

Match-Yang commented 9 years ago

好的，没问题

sbwtw commented 9 years ago

@1dot75cm 你打包的地址是？

sbwtw commented 9 years ago

@Match-Yang 我只是在debug下修改了源码里面那两处路径，别地方没有改，release编译的话应该还是之前的路径对才啊？

Match-Yang commented 9 years ago

@sbwtw 我已经找到问题代码了，你把PointPopup/control/datacontroler.h文件中的MAIN_PROGRAM_PATH写成了/opt/Point/PopupWindow/PointPopup 所以造成了无限自己调用自己

Match-Yang commented 9 years ago

但是你在b3028139878a78743b204aa9e9d9291d711006f6这个commit的时候是没有写错的，可能你后来改乱了吧

Match-Yang commented 9 years ago

还有下面的bug： 1、you-get任务不能暂停 2、http任务按一下暂停，实际上已经停了，xml文件记录状态为dlstate_suspend，但是按钮状态没变，再按一次才变，应该是信号处理不及时 3、把http任务暂停，关闭程序，下次启动时暂停按钮状态不对

PointTeam commented 9 years ago

上面的三个bug中，第一第二个原来就有的，第三个是@sbwtw改了之后有的

sbwtw commented 9 years ago

ok 我尽快修复

On Wed, Dec 3, 2014, 21:14 杨万青 notifications@github.com wrote:

还有下面的bug： 1、you-get任务不能暂停 2、http任务按一下暂停，实际上已经停了，xml文件记录状态为dlstate_suspend，但是按钮状态没变，再按一次才变，应该是信号处理不及时 3、把http任务暂停，关闭程序，下次启动时暂停按钮状态不对

— Reply to this email directly or view it on GitHub https://github.com/PointTeam/PointDownload/pull/9#issuecomment-65404571.

sbwtw commented 9 years ago

...又犯低级错误了╯﹏╰

On Wed, Dec 3, 2014, 20:30 杨万青 notifications@github.com wrote:

@sbwtw https://github.com/sbwtw 我已经找到问题代码了，你把PointPopup/control/datacontroler.h文件中的MAIN_PROGRAM_PATH写成了/opt/Point/PopupWindow/PointPopup 所以造成了无限自己调用自己

— Reply to this email directly or view it on GitHub https://github.com/PointTeam/PointDownload/pull/9#issuecomment-65400173.

1dot75cm commented 9 years ago

http://copr-be.cloud.fedoraproject.org/results/mosquito/myrepo/fedora-21-x86_64/pointdownload-1.2.0git20141128-1.fc21/pointdownload-1.2.0git20141128-1.fc21.src.rpm @sbwtw

PointTeam commented 9 years ago

木有关系，这些问题很正常的啦

sbwtw commented 9 years ago

@Match-Yang 3个暂停的问题已修复

choldrim commented 9 years ago

@sbwtw 你好，有一个问题，TaskInfo类中QUrl类型的rawUrl和parseUrl无法直接存储ed2k,thunder协议的Url

sbwtw commented 9 years ago

嗯，当时没考虑到这个问题，考虑从url中提取hostname所以用了qurl，稍后修改。

On Sat, Dec 6, 2014, 16:26 Choldrim notifications@github.com wrote:

@sbwtw https://github.com/sbwtw 你好，有一个问题，TaskInfo类中QUrl类型的rawUrl和parseUrl无法直接存储ed2k,thunder协议的Url

— Reply to this email directly or view it on GitHub https://github.com/PointTeam/PointDownload/pull/9#issuecomment-65889898.

PointTeam commented 9 years ago

普通迅雷账号就可以测试迅雷模块，不一定迅雷会员的 @sbwtw