Ptt-official-app / Ptt-backend

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

修正 get classes #282

Closed nickyanggg closed 3 years ago

nickyanggg commented 3 years ago

👏 解決掉的 issue / Resolved Issues

⛏ 變更內容 / Details of Changes

codecov-commenter commented 3 years ago

Codecov Report

Merging #282 (887555a) into development (cb8fb31) will decrease coverage by 0.36%. The diff coverage is 20.00%.

Impacted file tree graph

@@               Coverage Diff               @@
##           development     #282      +/-   ##
===============================================
- Coverage        44.41%   44.04%   -0.37%     
===============================================
  Files               29       29              
  Lines             1664     1712      +48     
===============================================
+ Hits               739      754      +15     
- Misses             822      848      +26     
- Partials           103      110       +7     
Impacted Files Coverage Δ
internal/delivery/http/route_classes.go 36.95% <20.00%> (-3.52%) :arrow_down:
internal/repository/user.go 44.77% <0.00%> (-2.85%) :arrow_down:
internal/repository/article.go 0.00% <0.00%> (ø)
internal/delivery/http/route_users.go 37.69% <0.00%> (ø)
internal/usecase/board.go 33.33% <0.00%> (+2.46%) :arrow_up:
internal/usecase/article.go 34.28% <0.00%> (+7.45%) :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 cb8fb31...887555a. Read the comment docs.

PichuChen commented 3 years ago

想確定一下,這樣修改的話用 Class ID 取回 Class 的時候能正確被取到嗎?

nickyanggg commented 3 years ago

想確定一下,這樣修改的話用 Class ID 取回 Class 的時候能正確被取到嗎?

其實我不確定原本在抓 /classes/1 時利用 .BRD 的 offset 來生成 ID 是有對應到屬於這分類的 BoardHeader 裡的 gid 嗎?

PichuChen commented 3 years ago

應該是有?

nickyanggg commented 3 years ago

因為原本的實作方式回傳回來的目錄中的第一個類別,用它的 id 去抓的結果會永遠跟 /classes/1 一樣,即使加上了一個 base,主目錄回傳的第一個類別的 id 變為 1+base,當我們在抓 /classes/1+base,扣掉了 base 的結果還是一樣在抓 /classes/1,所以才想說會不會原本的對應方式就是錯的?因為只有第二筆分類開始才有可能會抓到屬於它的看板,而第一個分類的結果永遠只是在抓分類主目錄

PichuChen commented 3 years ago

我知道你在說什麼了,分類主目錄印象中他並沒有在.BRD檔案中,你不能設定分類主目錄的標題

nickyanggg commented 3 years ago

@PichuChen 所以 Gid = 1 的 BoardHeader 代表說為分類主目錄中的其中某一類,那 Gid = 2 的 BoardHeader 所屬分類應該是分類主目錄中的第一筆分類還是第二筆分類呢?因為原本的實作方式好像會是屬於第二筆,也就是說分類主目錄中的第一筆分類不會有自己的看板,因為 Gid = 1 的 BoardHeader 應該為 class。不知道我的理解有沒有錯誤。

PichuChen commented 3 years ago

@PichuChen 所以 Gid = 1 的 BoardHeader 代表說為分類主目錄中的其中某一類,那 Gid = 2 的 BoardHeader 所屬分類應該是分類主目錄中的第一筆分類還是第二筆分類呢?因為原本的實作方式好像會是屬於第二筆,也就是說分類主目錄中的第一筆分類不會有自己的看板,因為 Gid = 1 的 BoardHeader 應該為 class。不知道我的理解有沒有錯誤。

你現在指的 Gid 是指 ClassID 嗎?

nickyanggg commented 3 years ago

你現在指的 Gid 是指 ClassID 嗎?

是的

PichuChen commented 3 years ago

API傳入 ClassID 的時候要扣掉 Base的部分好像沒做到?

nickyanggg commented 3 years ago

API傳入 ClassID 的時候要扣掉 Base的部分好像沒做到?

恩我上面其實就是想討論傳入 ClassID 是否要扣掉 Base,不然我先加上去好了

PichuChen commented 3 years ago

應該是要扣,不然我月底測試應該會不過

nickyanggg commented 3 years ago

@PichuChen 已修正,再麻煩幫忙看一下