Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

use local file path instead of temp file path when loadRes #77

Closed
wants to merge 1 commit into from

Conversation

PPpro
Copy link

@PPpro PPpro commented Mar 7, 2019

loadRes 第一次加载远程资源,是直接返回 临时路径的,而临时路径不可靠
有用户反馈播放的音频错误,原因是 audio 引用了临时路径

应该等资源存储到本地后,再用本地的资源路径

@zilongshanren
Copy link

zilongshanren commented Mar 11, 2019

把 nextPipe移动到cacheAssets里面去了,不会导致 pipe的时间变长么?是不是会导致之前做的性能优化失效? refer: #72

@zilongshanren
Copy link

zilongshanren commented Mar 11, 2019

image

downloadFile直接有个参数可以直接下载到指定地址,但是要注意兼容性。

@PPpro
Copy link
Author

PPpro commented Mar 11, 2019

downloadFile直接有个参数可以直接下载到指定地址,但是要注意兼容性。

多谢提醒,我这边更新下看看,目前的实现是会延长管线加载的时间的

@zilongshanren
Copy link

@umbrellaPP 这个filePath可以直接用,你们多测试一下,不用saveFile,直接downloadFile速度更快。

@PPpro
Copy link
Author

PPpro commented Mar 11, 2019

@zilongshanren
跟 panda 他们讨论了下,目前不用 filePath 有几个考虑
最主要的原因还是没办法很好地处理下载失败的情况,一旦下载失败,会导致重试的时候无法判断是否要重新下载某个资源。还不一定能获得下载失败的回调,比如用户退微信,或者断电关机

目前的下载流程可以保障缓存中一定是正确的文件

@zilongshanren
Copy link

可以在读取的时候,如果文件出错就删除并重新下载吧? 现在把pipe放在cacheAssets里面,感觉不是很好。

@PPpro
Copy link
Author

PPpro commented Mar 12, 2019

可以在读取的时候,如果文件出错就删除并重新下载吧?

目前还没办法做到判断文件是否出错,

由于 audio 引用了临时路径造成的问题,理论上应该算是微信环境的问题,具体原因还需要再排查下
现在的修复有点绕过问题了, 这个 pr 就先关了吧

@PPpro PPpro closed this Mar 12, 2019
@PPpro
Copy link
Author

PPpro commented Mar 12, 2019

1
好吧,saveFile 之后 tempFilePath 就无效了,之后会有另外提交的修复 @zilongshanren
感谢大大指导

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants