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

NarouResolverの修正と改善 #609

Merged
merged 3 commits into from
Feb 14, 2021
Merged

Conversation

eai04191
Copy link
Member

@eai04191 eai04191 commented Jan 25, 2021

とデグレ

close #603

修正

  • 連載ページ(novel18.syosetu.com/n0000aa/)以外でも作者名とあらすじが取れるようになりました。

改善

  • データを小説情報(novel18.syosetu.com/novelview/infotop/ncode/n0000aa/)をからパースするようになりました。どのURLからでもタグ(キーワード)とあらすじを取得できます。
    • image

デグレ

  • 逆に単話ページ(novel18.syosetu.com/n0000aa/1)のリンクを貼っても単話のタイトルや本文(もともと対応してなかったが)が取れなくなりました。
    • これは小説情報ページにタイトルの一覧が載っていないのと、MetadataResolverは1回のメタデータ取得に1つのリクエストしか送れないためです。(正確には送れるがテストができない)
    • この点を問題と捉えるか否かは実際にこのリゾルバを使っているユーザーに聞きたいです。

WIP

  • 小説情報ページのURLもパース対象に含める

@codecov
Copy link

codecov bot commented Jan 25, 2021

Codecov Report

Merging #609 (360e107) into develop (db34e36) will increase coverage by 1.36%.
The diff coverage is 100.00%.

Impacted file tree graph

@@              Coverage Diff              @@
##             develop     #609      +/-   ##
=============================================
+ Coverage      46.07%   47.44%   +1.36%     
- Complexity       543      544       +1     
=============================================
  Files             83       83              
  Lines           1964     1975      +11     
=============================================
+ Hits             905      937      +32     
+ Misses          1059     1038      -21     
Impacted Files Coverage Δ Complexity Δ
app/MetadataResolver/MetadataResolver.php 73.52% <ø> (ø) 13.00 <0.00> (ø)
app/MetadataResolver/NarouResolver.php 100.00% <100.00%> (+100.00%) 7.00 <0.00> (+1.00)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update db34e36...360e107. Read the comment docs.

@aiwas
Copy link

aiwas commented Feb 7, 2021

単話タイトルが取れないデメリットは、どのページからでも小説情報が取れるメリットよりは相当低いと思いますので、この改修で問題ないと思います。
なろうの構造上、1話をどう区切るかは作者によって様々なので一律に単話単位で捉えるのは難しく、基本的には作品単位で考えたほうが良いかと思われ、殊更に箇所と明示したい場合までシステム側で担保する必要性は薄い(現状はみなさんノートに◯話などと書いていることと思いますが、それで十分)と考えています。

@eai04191 eai04191 marked this pull request as ready for review February 13, 2021 05:56
@shibafu528 shibafu528 merged commit b013435 into shikorism:develop Feb 14, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

NarouResolver 話数ページで作者名が取れていない
3 participants