Skip to content
This repository has been archived by the owner on Sep 12, 2023. It is now read-only.

使用Ehviewer-Overhauled直接浏览zip压缩包中本子,页面顺序错乱 #427

Closed
4 of 5 tasks
VVaraiotoko opened this issue Nov 19, 2022 · 5 comments · Fixed by #544
Closed
4 of 5 tasks

使用Ehviewer-Overhauled直接浏览zip压缩包中本子,页面顺序错乱 #427

VVaraiotoko opened this issue Nov 19, 2022 · 5 comments · Fixed by #544
Labels
bug Something isn't working

Comments

@VVaraiotoko
Copy link

复现步骤 / Step to reproduce

  1. 不解压zip,直接选择Ehviewer-Overhauled浏览其中本子文件
  2. 页面顺序错乱,无法按照页码顺序甚至文件默认排列顺序翻页

预期行为 / Expected behaviour

它应该按页码顺序翻页

实际行为 / Actual Behaviour

Screenshot_20221119-221547
在ZArchiver中,zip文件内图片顺序正常
Screenshot_20221119-221854
在原版EhViewer中,第1页为封面,图片顺序正常
Screenshot_20221119-222339
在Ehviewer-Overhauled中,第1页为实际的第10页,图片顺序异常

备注 / Addition details

No response

EhViewer 版本号 / EhViewer version code

1.8.3.0-beta02

EhViewer CI 版本 / EhViewer CI Version

https://github.com/Ehviewer-Overhauled/Ehviewer/actions/runs/3489725580

Android 系统版本 / Android version

Android 12, Sony Stock 58.2.A.7.93

设备型号 / Device model

Sony Xperia 1 II

SoC 型号 / Soc model

骁龙865

应用日志 / App logs

log-2022-11-19-22-34-22-671.zip

自查步骤 / Verify steps

  • 如果您有足够的时间和能力,并愿意为此提交 PR ,请勾上此复选框 / Pull request is welcome. Check this if you want to start a pull request
  • 您已仔细查看并知情 Q&A 中的内容 / You have checked Q&A carefully
  • 您已搜索过 Issue Tracker,没有找到类似内容 / I have searched on Issue Tracker, No duplicate or related open issue has been found
  • 您确保这个 Issue 只提及一个问题。如果您有多个问题报告,烦请发起多个 Issue / Ensure there is only one bug report in this issue. Please make mutiply issue for mutiply bugs
  • 您确保已使用最新 CI 版本测试,并且该问题在最新 CI 版本中并未解决 / This bug have not solved in latest CI version
@VVaraiotoko VVaraiotoko added the bug Something isn't working label Nov 19, 2022
@VVaraiotoko
Copy link
Author

这应该是对原版的修改造成的bug,可否修复一下呢?

@asuka-mio
Copy link
Member

这应该是对原版的修改造成的bug,可否修复一下呢?

这是预期行为 我们从p7zip实现迁移到了libarchive实现 遵循流式读取 出现这种情况的原因是图片文件在压缩包中数据的先后顺序和页码并不是一个顺序
至于以后要不要实现按文件名排序功能我再考虑 因为现在本身应用的页码请求机制就不是面向流式优化的 我得先重构这块

@asuka-mio
Copy link
Member

#240 这个或许可以帮助到你

@VVaraiotoko
Copy link
Author

VVaraiotoko commented Nov 22, 2022

这应该是对原版的修改造成的bug,可否修复一下呢?

这是预期行为 我们从p7zip实现迁移到了libarchive实现 遵循流式读取 出现这种情况的原因是图片文件在压缩包中数据的先后顺序和页码并不是一个顺序 至于以后要不要实现按文件名排序功能我再考虑 因为现在本身应用的页码请求机制就不是面向流式优化的 我得先重构这块

实现的迁移肯定是为了提高应用运行效率,但还是希望能按文件名/页码顺序浏览zip中图片吧,毕竟这样是符合用户直觉的,谢谢。

@asuka-mio
Copy link
Member

这应该是对原版的修改造成的bug,可否修复一下呢?

这是预期行为 我们从p7zip实现迁移到了libarchive实现 遵循流式读取 出现这种情况的原因是图片文件在压缩包中数据的先后顺序和页码并不是一个顺序 至于以后要不要实现按文件名排序功能我再考虑 因为现在本身应用的页码请求机制就不是面向流式优化的 我得先重构这块

实现的迁移肯定是为了提高应用运行效率,但还是希望能实现按文件名/页码顺序浏览吧,毕竟这样是符合用户直觉的,谢谢。

其实主要是看p7zip不顺眼(

FooIbar referenced this issue in FooIbar/EhViewer Dec 17, 2022
FooIbar referenced this issue in FooIbar/EhViewer Dec 17, 2022
asuka-mio pushed a commit that referenced this issue Dec 17, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
2 participants