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

feat: add new api support #123

Merged
merged 1 commit into from
Jan 27, 2025
Merged

feat: add new api support #123

merged 1 commit into from
Jan 27, 2025

Conversation

dyphire
Copy link
Collaborator

@dyphire dyphire commented Jan 26, 2025

基于 #121

实现 #114 (comment) 中的部分想法

通过在搜索关键词后追加|ds, |dy|dm等额外关键词触发新的 API 弹幕搜索(对应电视剧、电影、动漫)
它会获取弹弹play 的 API 支持解析的视频网站的对应播放链接,然后再通过调用弹弹play 的 API 来获取弹幕
PS:动漫不推荐使用它,弹弹play 的 api 要更强大(国漫的话可以使用它,更全)

还有些小细节待完善

功能示意:

test.mp4

API 文档参考:https://www.eruyi.cn/thread-11578-1-1.html

@dyphire dyphire force-pushed the extra branch 2 times, most recently from bc4be91 to 713e551 Compare January 26, 2025 13:52
@dyphire dyphire force-pushed the extra branch 6 times, most recently from f737273 to 3c9824f Compare January 27, 2025 05:29
@dyphire dyphire marked this pull request as ready for review January 27, 2025 05:30
@dyphire dyphire changed the title [WIP] feat: add new api support feat: add new api support Jan 27, 2025
@dyphire dyphire requested a review from Tony15246 January 27, 2025 05:30
@dyphire
Copy link
Collaborator Author

dyphire commented Jan 27, 2025

@Tony15246 代码已完成,可以审查了

@dyphire dyphire force-pushed the extra branch 2 times, most recently from 34f090a to b8581b4 Compare January 27, 2025 05:56
@dyphire
Copy link
Collaborator Author

dyphire commented Jan 27, 2025

记录:对于各大视频网站的弹幕解析还有个替代方案 https://github.com/lyz05/danmaku

不过 dandanplay 的 api 接口已足够强大,暂只做记录

@Tony15246
Copy link
Owner

@Tony15246 代码已完成,可以审查了

OK

我现在在城里的医院体检,晚上到家我review一下

@MiKoto-Railgun
Copy link
Contributor

MiKoto-Railgun commented Jan 27, 2025

记录:对于各大视频网站的弹幕解析还有个替代方案 https://github.com/lyz05/danmaku

好家伙,我之前下弹幕也用过它 主流视频网站弹幕文件解析接口

@dyphire dyphire force-pushed the extra branch 4 times, most recently from 24cc067 to 5317d2a Compare January 27, 2025 07:06
Brings support for non-animated danmaku
Copy link
Owner

@Tony15246 Tony15246 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Tony15246
Copy link
Owner

大佬太强了,光看代码就看了我半个多小时

@Tony15246
Copy link
Owner

我有一个小地方不明白,danmaku.source这个变量是用来干什么的?有赋值和存值,但好像从来没有取用过。

@dyphire
Copy link
Collaborator Author

dyphire commented Jan 27, 2025

我有一个小地方不明白,danmaku.source这个变量是用来干什么的?有赋值和存值,但好像从来没有取用过。

在代码中并不使用它,主要是为了用户查看 json 文件时能直观的看出弹幕来源
以后会不会用到不好说

@Tony15246
Copy link
Owner

ok,那我合并代码了

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

Successfully merging this pull request may close these issues.

3 participants