xfangfang / borealis

Hardware accelerated, Nintendo Switch inspired UI library for PC, Android, iOS, PSV, PS4 and Nintendo Switch
Apache License 2.0
31 stars 21 forks source link

use ShellExecute to open url #5

Closed dragonflylee closed 1 year ago

dragonflylee commented 1 year ago
xfangfang commented 1 year ago

@dragonflylee 感谢提交修补

我今天没回家,所以得等明天才能测试合并。

关于使用新的方法打开url,因为我对windows下的各个api不是特别熟悉,尤其涉及到执行shell命令,有时候会出现几个问题:

  1. 在某些系统上存在调用时会弹出一闪而过的终端页面。
  2. url中有 & 或者其他特殊字符可能会造成命令执行失败,或者打开了错误的url。
  3. 是同步调用的,会阻塞线程。

第三点比较好检测。
第二点可以在wiliwili的番剧页点击番剧标题,逐个测试其中各项网址。
第一点因为没办法找到所有环境,可以广泛搜索看看有没有其他人遇到过这样的问题。

改起来代码确实简洁了,但是同时也增加了后续万一出现问题时的工作量。所以如果要改动这里,需要广泛测试看看。