search

Ptt-official-app / Ptt-backend

PTT APP 的後端
BSD 3-Clause "New" or "Revised" License
208 stars 67 forks source link

Added UpdateUsefuleness in usecase #268

Closed kyho4515 closed 3 years ago

kyho4515 commented 3 years ago

👏 解決掉的 issue / Resolved Issues

📝 相關的 issue / Related Issues

⛏ 變更內容 / Details of Changes

codecov-commenter commented 3 years ago

Codecov Report

Merging #268 (fcf6a69) into development (db1d8a2) will decrease coverage by 0.16%. The diff coverage is 30.95%.

Impacted file tree graph

@@               Coverage Diff               @@
##           development     #268      +/-   ##
===============================================
- Coverage        44.41%   44.24%   -0.17%     
===============================================
  Files               29       29              
  Lines             1664     1704      +40     
===============================================
+ Hits               739      754      +15     
- Misses             822      841      +19     
- Partials           103      109       +6
Impacted Files Coverage Δ
internal/repository/article.go 0.00% <0.00%> (ø)
internal/usecase/article.go 34.28% <43.33%> (+7.45%) :arrow_up:
internal/usecase/board.go 33.33% <0.00%> (+2.46%) :arrow_up:

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 db1d8a2...fcf6a69. Read the comment docs.

kyho4515 commented 3 years ago

test function我不確定怎麼寫比較好, 如果有什麼建議都可以提出來

y2468101216 commented 3 years ago

LGTM

nickyanggg commented 3 years ago

另外想問一下，目前的實作方式是不是無法避免其他使用者的刻意留言？也就是說假如有人的留言帶有其他使者的名字以及推噓文符號，是不是那位被提到的使用者在推噓文的時候就會計算到別人的刻意留言？

kyho4515 commented 3 years ago

另外想問一下，目前的實作方式是不是無法避免其他使用者的刻意留言？也就是說假如有人的留言帶有其他使者的名字以及推噓文符號，是不是那位被提到的使用者在推噓文的時候就會計算到別人的刻意留言？

目前看起來是這樣, 如果多加個檢查上下箭頭前一個rune是否為 \n 不知道能不能處理這個問題？

y2468101216 commented 3 years ago

另外想問一下，目前的實作方式是不是無法避免其他使用者的刻意留言？也就是說假如有人的留言帶有其他使者的名字以及推噓文符號，是不是那位被提到的使用者在推噓文的時候就會計算到別人的刻意留言？

目前看起來是這樣, 如果多加個檢查上下箭頭前一個rune是否為 \n 不知道能不能處理這個問題？

我個人認為依照 ptt 現行做法怎樣都會被有心人操弄，因為畢竟寫入推文的時候我們不會刻意去檢查推文是否有含上下箭頭 多 + \n 也只是避免一般使用者誤碰而已，不過我覺得這個想法很好，能避免一般使用者誤觸就夠了，剩下的靠版主處理

kyho4515 commented 3 years ago

另外想問一下，目前的實作方式是不是無法避免其他使用者的刻意留言？也就是說假如有人的留言帶有其他使者的名字以及推噓文符號，是不是那位被提到的使用者在推噓文的時候就會計算到別人的刻意留言？

目前看起來是這樣, 如果多加個檢查上下箭頭前一個rune是否為 \n 不知道能不能處理這個問題？

我個人認為依照 ptt 現行做法怎樣都會被有心人操弄，因為畢竟寫入推文的時候我們不會刻意去檢查推文是否有含上下箭頭 多 + \n 也只是避免一般使用者誤碰而已，不過我覺得這個想法很好，能避免一般使用者誤觸就夠了，剩下的靠版主處理

我想的是在寫入的時候, \n(換行)之後加個→之類的做區別, 所以推文時就是\n↑, 使用者使用換行的話就是\n→ 我以為平常在ppt上看到的換行看起來就像換行後有個右箭頭?