Skip to content
This repository has been archived by the owner on Aug 13, 2022. It is now read-only.

Commit

Permalink
[#25] 피드백 반영
Browse files Browse the repository at this point in the history
- Merge branch 'feature/22' into feature/25

- `feature/22`에서구현한 페이징 계산식을 가져와서 `feature/25`에도 반영
  • Loading branch information
msugo1 committed Mar 18, 2021
2 parents d84e4dc + f5c2d95 commit 024f0c0
Show file tree
Hide file tree
Showing 4 changed files with 11 additions and 7 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -15,11 +15,12 @@ public class BlockUserListRequest {
private final int limit;

static public BlockUserListRequest create(String userId, int pageNumber, Pagination pagination) {
int pageLimit = pagination.getMaxPageBlockUser();

return BlockUserListRequest.builder()
.userId(userId)
.offset(pagination.getMaxPageBlockUser() * (Math.max(pageNumber, 1) - 1))
.limit(pagination.getMaxPageBlockUser())
.offset(pagination.calculateOffset(pageLimit, pageNumber))
.limit(pageLimit)
.build();
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -21,11 +21,12 @@ public class FriendListRequest {
Math.max(pageNumber, 1): pageNumber 가 음수로 들어와도 기본값을 1로 설정해서 맨 첫 페이지가 보이도록 설정
*/
static public FriendListRequest create(String userId, int pageNumber, Pagination pagination, FriendStatus status) {
int pageLimit = pagination.getMaxPageFriend();

return FriendListRequest.builder()
.userId(userId)
.offset(pagination.getMaxPageFriend() * (Math.max(pageNumber, 1) - 1))
.limit(pagination.getMaxPageFriend())
.offset(pagination.calculateOffset(pageLimit, pageNumber))
.limit(pageLimit)
.status(status)
.build();
}
Expand Down
3 changes: 0 additions & 3 deletions src/main/java/me/soo/helloworld/service/FriendService.java
Original file line number Diff line number Diff line change
Expand Up @@ -96,9 +96,6 @@ private void validateFriendStatusDetail(FriendStatus status) {
throw new DuplicateRequestException("이미 해당 사용자로부터 친구추가 요청을 받은 상태입니다. 받은 친구 요청을 다시 확인해주세요.");
case FRIEND:
throw new DuplicateRequestException("이미 친구로 등록된 사용자에게 다시 친구 요청을 보낼 수 없습니다.");
case NONE:
default:
break;
}
}
}
5 changes: 5 additions & 0 deletions src/main/java/me/soo/helloworld/util/Pagination.java
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,12 @@ public class Pagination {

public Pagination(@Value("${friend.max.page:30}") int maxPageFriend,
@Value("${block_user.max.page:30}") int maxPageBlockUser) {

this.maxPageFriend = maxPageFriend;
this.maxPageBlockUser = maxPageBlockUser;
}

public int calculateOffset(int pageLimit, int pageNumber) {
return pageLimit * (Math.max(pageNumber, 1) - 1);
}
}

0 comments on commit 024f0c0

Please sign in to comment.