Tsuk1ko / pxder

🖼 Download illusts from pixiv.net P站插画批量下载器
https://moe.best/projects/pxder.html
GNU General Public License v3.0
742 stars 78 forks source link

Pxrepo #61

Closed EIGHTfs closed 2 years ago

Tsuk1ko commented 2 years ago

@EIGHTfs 有件难以启齿的事情,我看了下你的改动,实在是太多了,导致我不敢合进去

你建仓库是直接复制粘贴了我所有的代码而不是从我这里 fork 过去,因此我完全看不到历史提交记录,没办法知道你是从哪个 commit 开始改动的,而且我有一些新功能也出现在你的 Initial commit 之后,说明你后面同步了我的代码也是通过复制粘贴,我完全不知道你的仓库相对我的仓库落后了多少领先了多少😢

EIGHTfs commented 2 years ago

@EIGHTfs 有件难以启齿的事情,我看了下你的改动,实在是太多了,导致我不敢合进去

你建仓库是直接复制粘贴了我所有的代码而不是从我这里 fork 过去,因此我完全看不到历史提交记录,没办法知道你是从哪个 commit 开始改动的,而且我有一些新功能也出现在你的 Initial commit 之后,说明你后面同步了我的代码也是通过复制粘贴,我完全不知道你的仓库相对我的仓库落后了多少领先了多少😢

额,确实是这样,复制粘贴的,但其实看上去改得多,其实是用来格式化文档,不如我直接说我思路吧,因为改得倒是没啥技术含量就是有些自己想法。 其实改动的文件就几个 downloader,js改了一点,其实就是改了缓存目录,用你原来的不会出问题 下载的缓存目录位置名字改了,然后不会一启动就删除一遍这个文件夹,因为其实不影响使用,但也加了个命令专门遍历这个目录所有文件删除

pixiv-api-client-mod.js 改了一点点 简单来说就是原来你是用的正则判断有没有Rate Limt,实际用发现出现这个问题并不能正则匹配到。我就改成了JSON.stringify(msg).search('Rate Limit') != -1

plugins 在src文件夹里新增的这个文件夹,主要就是把原来的tools.js移过去,当然顺带的其他引用这个js文件的文件也就改了一下

最后 bin 文件夹里的那个文件其实是改的最多的,但主要就是原来是获取了就下载,现在单独的,而且公开和私人获取都是两个单独的命令,改成了一个命令传Boolean参数决定。对了原来的isPrivate我都对应改成isPublic了,,,,然后自己一些加的功能也在里面,,

如果还有什么修改的地方无非就是运行经常写json文件,,,免得突然中断白获取了半天

其实你不用合并我的,我的本来就烂,但是有两个地方你确实可以加上 1.经常会一直不动,其实是有几张图片 data.on('end', () => { console.timeEnd(filename) reslove(response); }); data.on('error', reject); 这两种情况都没有触发?再后面加一句就可以 setTimeout(()=>{ //console.warn(Promise time out:${errorTimeout}) reject('Promise time out'); }, errorTimeout);

2.就是判断有没有Rate Limit错误那里 JSON.stringify(msg).search('Rate Limit') != -1

Tsuk1ko commented 2 years ago