search

GeneralLibrary / GeneralUpdate

My ship looks old, who can help me upgrade it.
https://www.justerzhu.cn/
MIT License
304 stars 33 forks source link

Feat: 错误修复和性能优化 #57

Closed textGamex closed 2 months ago

textGamex commented 3 months ago

@JusterZhu 可以审查了

textGamex commented 2 months ago

每次自动升级完成程序就退出了，你这里是通过什么手段发现会有泄露问题的能分享一下吗。

仅仅通过没有及时释放资源

JusterZhu commented 2 months ago
  1. if (hashAlgorithm.ComputeHash(oldFile) == hashAlgorithm.ComputeHash(newFile)) hash字符串判断是否相等，建议用equals。
  2. 这里其实还有一个问题，如果“当文件没有发生改变时也会生成补丁文件”那么你应该会发现你所有的文件都会被生成一个差分文件而并不是部分文件会出现这个问题。符合您说的“当文件没有发生改变时也会生成补丁文件”的文件能分享一下是什么文件内容吗？如果真的一模一样出现了这个现象我想差分算法出现了bug，仅判断两个文件的hash值是否相等可能也只是修复了表面问题。
JusterZhu commented 2 months ago

PR里的其他内容看起来没有什么问题，主要是“当文件没有发生改变时也会生成补丁文件”这个问题我想搞明白。

textGamex commented 2 months ago
  1. if (hashAlgorithm.ComputeHash(oldFile) == hashAlgorithm.ComputeHash(newFile)) hash字符串判断是否相等，建议用equals。
  2. 这里其实还有一个问题，如果“当文件没有发生改变时也会生成补丁文件”那么你应该会发现你所有的文件都会被生成一个差分文件而并不是部分文件会出现这个问题。符合您说的“当文件没有发生改变时也会生成补丁文件”的文件能分享一下是什么文件内容吗？如果真的一模一样出现了这个现象我想差分算法出现了bug，仅判断两个文件的hash值是否相等可能也只是修复了表面问题。

1我改了, 2我发你邮箱了

JusterZhu commented 2 months ago
  1. if (hashAlgorithm.ComputeHash(oldFile) == hashAlgorithm.ComputeHash(newFile)) hash字符串判断是否相等，建议用equals。
  2. 这里其实还有一个问题，如果“当文件没有发生改变时也会生成补丁文件”那么你应该会发现你所有的文件都会被生成一个差分文件而并不是部分文件会出现这个问题。符合您说的“当文件没有发生改变时也会生成补丁文件”的文件能分享一下是什么文件内容吗？如果真的一模一样出现了这个现象我想差分算法出现了bug，仅判断两个文件的hash值是否相等可能也只是修复了表面问题。

1我改了, 2我发你邮箱了

1.单发一个文件我没办法比对差异，提供一下当时一起比对的文件。 2.只有DryIoC.dll 这一个文件出现“当文件没有发生改变时也会生成补丁文件”这个问题吗。

textGamex commented 2 months ago
  1. if (hashAlgorithm.ComputeHash(oldFile) == hashAlgorithm.ComputeHash(newFile)) hash字符串判断是否相等，建议用equals。
  2. 这里其实还有一个问题，如果“当文件没有发生改变时也会生成补丁文件”那么你应该会发现你所有的文件都会被生成一个差分文件而并不是部分文件会出现这个问题。符合您说的“当文件没有发生改变时也会生成补丁文件”的文件能分享一下是什么文件内容吗？如果真的一模一样出现了这个现象我想差分算法出现了bug，仅判断两个文件的hash值是否相等可能也只是修复了表面问题。

1我改了, 2我发你邮箱了

1.单发一个文件我没办法比对差异，提供一下当时一起比对的文件。 2.只有DryIoC.dll 这一个文件出现“当文件没有发生改变时也会生成补丁文件”这个问题吗。

DryIoC.dll再复制一份对比就可以了, 不止这一个,不过问题应该是一样的