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

Root: Fix parsing bbsmenu for 2ch.sc and next2ch.net #196

Merged
merged 1 commit into from
Mar 1, 2020

Conversation

ma8ma
Copy link
Collaborator

@ma8ma ma8ma commented Mar 1, 2020

#195 (comment) より

話がちょっとそれますが

2. `next2ch.net`はサイトの`http://next2ch.net/bbsmenu.html`を読み込むとクラッシュするため別サイトを使った
3. `2ch.sc`はサイトの`https://menu.2ch.sc/bbsmenu.html`を読み込むとクラッシュするため外部板として登録した

中略

// 現在の仕様では HTML > BODY > font[size="2"] の子要素が対象

この辺で死んでます。クラッシュするサイトのソースを見ると、他のにはある<font size=2> というのがこれらのサイトに無い為、html.getElementsByTagName( "font" )[0]というのがnullptrになって死んでいるようです。

2ch.scとnext2ch.netのbbsmenu.htmlではfontタグが使われていないためfontタグが見つからない場合は別のタグ(small, body)を取得して解析するようにします。
それでも見つからないときはエラーメッセージを出して解析処理を中止します。

2ch.scとnext2ch.netのbbsmenu.htmlではfontタグが使われていないため
fontタグが見つからない場合は別のタグ(small, body)を取得して解析する
ようにする。それでも見つからないときはエラーメッセージを出して
解析処理を中止する。
@ma8ma ma8ma added the bug バグの追跡 label Mar 1, 2020
@ma8ma ma8ma mentioned this pull request Mar 1, 2020
@ma8ma ma8ma marked this pull request as ready for review March 1, 2020 13:21
@ma8ma ma8ma merged commit e58626e into JDimproved:master Mar 1, 2020
@ma8ma ma8ma deleted the fix-crash-parsing-bbsmenu branch March 1, 2020 13:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug バグの追跡
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant