Skip to content

Commit 718e355

Browse files
khozan0120khozan0120
authored
[refactor] 코드 주석제거 (#140)
* Update Location.java * chore: 코드 주석 제거 불필요한 주석을 삭제하여 코드 가독성을 개선했습니다. * chore: 코드 주석 제거 불필요한 주석을 삭제하여 코드 가독성을 개선했습니다. * chore: 코드 주석 제거 불필요한 주석을 삭제하여 코드 가독성을 개선했습니다. * chore: 코드 주석 제거 불필요한 주석을 삭제하여 코드 가독성을 개선했습니다. * chore: 코드 주석 제거 불필요한 주석을 삭제하여 코드 가독성을 개선했습니다. * chore: 코드 주석 제거 불필요한 주석을 삭제하여 코드 가독성을 개선했습니다. * chore: 코드 주석 제거 불필요한 주석을 삭제하여 코드 가독성을 개선했습니다. * chore: 코드 주석 제거 불필요한 주석을 삭제하여 코드 가독성을 개선했습니다. * chore: 코드 주석 제거 불필요한 주석을 삭제하여 코드 가독성을 개선했습니다. * chore: 코드 주석 제거 불필요한 주석을 삭제하여 코드 가독성을 개선했습니다. * chore: 코드 주석 제거 불필요한 주석을 삭제하여 코드 가독성을 개선했습니다. * chore: 코드 주석 제거 불필요한 주석을 삭제하여 코드 가독성을 개선했습니다. * chore: 코드 주석 제거 불필요한 주석을 삭제하여 코드 가독성을 개선했습니다. * chore: 코드 주석 제거 불필요한 주석을 삭제하여 코드 가독성을 개선했습니다. * chore: 코드 주석 제거 불필요한 주석을 삭제하여 코드 가독성을 개선했습니다. * chore: 코드 주석 제거 불필요한 주석을 삭제하여 코드 가독성을 개선했습니다. * chore: 코드 주석 제거 불필요한 주석을 삭제하여 코드 가독성을 개선했습니다. * chore: 코드 주석 제거 불필요한 주석을 삭제하여 코드 가독성을 개선했습니다. * chore: 코드 주석 제거 불필요한 주석을 삭제하여 코드 가독성을 개선했습니다. * chore: 코드 주석 제거 불필요한 주석을 삭제하여 코드 가독성을 개선했습니다. * chore: 코드 주석 제거 불필요한 주석을 삭제하여 코드 가독성을 개선했습니다. * chore: 코드 주석 제거 불필요한 주석을 삭제하여 코드 가독성을 개선했습니다. * chore: 코드 주석 제거 불필요한 주석을 삭제하여 코드 가독성을 개선했습니다. * chore: 코드 주석 제거 불필요한 주석을 삭제하여 코드 가독성을 개선했습니다. * chore: 코드 주석 제거 불필요한 주석을 삭제하여 코드 가독성을 개선했습니다. * chore: 코드 주석 제거 불필요한 주석을 삭제하여 코드 가독성을 개선했습니다. * chore: 코드 주석 제거 불필요한 주석을 삭제하여 코드 가독성을 개선했습니다. * chore: 코드 주석 제거 불필요한 주석을 삭제하여 코드 가독성을 개선했습니다. * chore: 코드 주석 제거 불필요한 주석을 삭제하여 코드 가독성을 개선했습니다. * chore: 코드 주석 제거 불필요한 주석을 삭제하여 코드 가독성을 개선했습니다. * chore: 코드 주석 제거 불필요한 주석을 삭제하여 코드 가독성을 개선했습니다. * chore: 코드 주석 제거 불필요한 주석을 삭제하여 코드 가독성을 개선했습니다. * chore: 코드 주석 제거 불필요한 주석을 삭제하여 코드 가독성을 개선했습니다. * chore: 코드 주석 제거 불필요한 주석을 삭제하여 코드 가독성을 개선했습니다. * chore: 코드 주석 제거 불필요한 주석을 삭제하여 코드 가독성을 개선했습니다. * chore: 코드 주석 제거 불필요한 주석을 삭제하여 코드 가독성을 개선했습니다. * chore: 코드 주석 제거 불필요한 주석을 삭제하여 코드 가독성을 개선했습니다. * chore: 코드 주석 제거 불필요한 주석을 삭제하여 코드 가독성을 개선했습니다. * chore: 코드 주석 제거 불필요한 주석을 삭제하여 코드 가독성을 개선했습니다. * chore: 코드 주석 제거 불필요한 주석을 삭제하여 코드 가독성을 개선했습니다. * chore: 코드 주석 제거 불필요한 주석을 삭제하여 코드 가독성을 개선했습니다. * chore: 코드 주석 제거 불필요한 주석을 삭제하여 코드 가독성을 개선했습니다. * chore: 코드 주석 제거 불필요한 주석을 삭제하여 코드 가독성을 개선했습니다. * chore: 코드 주석 제거 불필요한 주석을 삭제하여 코드 가독성을 개선했습니다. * chore: 코드 주석 제거 불필요한 주석을 삭제하여 코드 가독성을 개선했습니다. * chore: 코드 주석 제거 불필요한 주석을 삭제하여 코드 가독성을 개선했습니다. * chore: 코드 주석 제거 불필요한 주석을 삭제하여 코드 가독성을 개선했습니다. * chore: 코드 주석 제거 불필요한 주석을 삭제하여 코드 가독성을 개선했습니다. * chore: 코드 주석 제거 불필요한 주석을 삭제하여 코드 가독성을 개선했습니다.
1 parent 6e70c30 commit 718e355

50 files changed

Lines changed: 152 additions & 359 deletions

File tree

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

‎src/main/java/org/scoula/domain/location/entity/Location.java‎

Lines changed: 8 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -8,20 +8,18 @@
88
import lombok.Getter;
99
import lombok.NoArgsConstructor;
1010

11-
/**
12-
* 위치 (외국인 센터, 은행)
13-
*/
11+
1412
@Builder
1513
@Getter
1614
@AllArgsConstructor(access = AccessLevel.PRIVATE)
1715
@NoArgsConstructor(access = AccessLevel.PROTECTED)
1816
public class Location extends BaseEntity {
19-
private Long locationId; // 위치 ID
20-
private String locationName; // 위치 이름
21-
private String address; // 주소
22-
private Point point; // 위도, 경도
23-
private String homepageUrl; // 홈페이지 주소
24-
private String tel; // 전화번호
25-
private LocationType locationType; // 장소 타입 CENTER(은행), CONSULT(외국인 센터), BANK(은행)
17+
private Long locationId;
18+
private String locationName;
19+
private String address;
20+
private Point point;
21+
private String homepageUrl;
22+
private String tel;
23+
private LocationType locationType;
2624

2725
}

‎src/main/java/org/scoula/domain/location/entity/Point.java‎

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,20 +1,17 @@
11
package org.scoula.domain.location.entity;
22

33

4-
// 내부 클래스로 Point 정의
54

65
import com.fasterxml.jackson.annotation.JsonIgnore;
76

87
public record Point(Double longitude,
98
Double latitude) {
109

1110
@JsonIgnore
12-
// WKT(Well-Known Text) 형식으로 변환
1311
public String toWKT() {
1412
return String.format("POINT(%f %f)", longitude, latitude);
1513
}
1614

17-
// WKT에서 Point 객체로 변환
1815
public static Point fromWKT(String wkt) {
1916
String coordinates = wkt.replace("POINT(", "").replace(")", "");
2017
String[] parts = coordinates.split(" ");

‎src/main/java/org/scoula/domain/location/service/CsvDataProcessingService.java‎

Lines changed: 2 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -34,19 +34,16 @@ public class CsvDataProcessingService {
3434
@Transactional
3535
public void processForeignCenterCsv(HttpServletRequest request, String csvFilePath) {
3636
try {
37-
// CSV 파일 읽기
3837
List<ForeignCenterCsvDto> csvData = new CsvToBeanBuilder<ForeignCenterCsvDto>(new FileReader(csvFilePath))
3938
.withType(ForeignCenterCsvDto.class)
4039
.withIgnoreLeadingWhiteSpace(true)
4140
.build()
4241
.parse();
4342

44-
// DTO를 Location으로 변환
4543
List<Location> locations = csvData.stream()
4644
.map(this::convertToLocation)
4745
.collect(Collectors.toList());
4846

49-
// 배치로 저장 (성능 향상)
5047
if (!locations.isEmpty()) {
5148
locationMapper.insertLocationBatch(locations);
5249
}
@@ -66,21 +63,17 @@ public void processForeignCenterCsv(HttpServletRequest request, String csvFilePa
6663
}
6764

6865
private Location convertToLocation(ForeignCenterCsvDto dto) {
69-
// Point 객체 생성 (경도, 위도 순서)
7066
Point point = new Point(dto.getLongitude(), dto.getLatitude());
7167

7268

73-
// 315991700 -> 035-599-1700
74-
// 25030070 -> 025-030-070
75-
// 3180455572 -> 031-8045-5572
7669
String raw = dto.getRepresentativeTelno();
7770
StringBuilder phoneNumberBuilder = new StringBuilder(raw);
7871

7972
if (raw.startsWith("02")) {
80-
if (raw.length() == 9) { // 02-XXX-XXXX
73+
if (raw.length() == 9) {
8174
phoneNumberBuilder.insert(2, "-");
8275
phoneNumberBuilder.insert(6, "-");
83-
} else if (raw.length() == 10) { // 02-XXXX-XXXX
76+
} else if (raw.length() == 10) {
8477
phoneNumberBuilder.insert(2, "-");
8578
phoneNumberBuilder.insert(7, "-");
8679
}

‎src/main/java/org/scoula/domain/location/service/LocationService.java‎

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,6 @@ public class LocationService {
1919

2020
private final LocationMapper locationMapper;
2121

22-
// CENTER 타입만 조회하는 메서드
2322
public List<LocationResponse> getAllCenters(HttpServletRequest request) {
2423
return locationMapper.selectAllCenters()
2524
.stream()

‎src/main/java/org/scoula/domain/member/dto/MemberDTO.java‎

Lines changed: 4 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -21,9 +21,8 @@
2121
@ApiModel(description = "회원 정보")
2222
public class MemberDTO {
2323

24-
private Long remittanceInformationId; // 정기송금 정보 아이디
25-
26-
private Long remittanceGroupId; // 정기송금 그룹 아이디
24+
private Long remittanceInformationId;
25+
private Long remittanceGroupId;
2726

2827
@ApiModelProperty(value = "회원 ID", example = "1")
2928
private Long memberId;
@@ -55,11 +54,10 @@ public class MemberDTO {
5554
@ApiModelProperty(value = "설정 통화", example = "USD")
5655
private Currency currency;
5756

58-
private String connectedId; // 코데프 유저 아이디
57+
private String connectedId;
5958

60-
private String fcmToken; // firebase fcm 알림 토큰
59+
private String fcmToken;
6160

62-
// Entity -> DTO 변환
6361
public static MemberDTO from(Member member) {
6462
return MemberDTO.builder()
6563
.remittanceInformationId(member.getRemittanceInformationId())
@@ -79,7 +77,6 @@ public static MemberDTO from(Member member) {
7977
.build();
8078
}
8179

82-
// DTO -> Entity 변환
8380
public Member toEntity() {
8481
return Member.builder()
8582
.remittanceGroupId(this.remittanceGroupId)

‎src/main/java/org/scoula/domain/member/entity/Member.java‎

Lines changed: 16 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -13,30 +13,28 @@
1313
import lombok.NoArgsConstructor;
1414
import lombok.Setter;
1515

16-
/**
17-
* 회원
18-
*/
16+
1917
@Getter
2018
@Setter
2119
@NoArgsConstructor(access = AccessLevel.PROTECTED)
2220
@AllArgsConstructor(access = AccessLevel.PROTECTED)
2321
@Builder
2422
public class Member extends BaseEntity {
2523

26-
private Long memberId; // 회원 아이디 (PK)
27-
private Long remittanceInformationId; // 정기송금 정보 아이디
28-
private Long remittanceGroupId; // 정기송금 그룹 아이디
29-
private String loginId; // 로그인 아이디
30-
private String password; // 비밀번호
31-
private String passportNumber; // 여권번호
32-
private NationalityCode nationality; // 국적
33-
private String country; // 나라
34-
private LocalDate birth; // 생년월일
35-
private String name; // 성명
36-
private String phoneNumber; // 전화번호
37-
private LocalDate remainTime; // 잔여 체류기간
38-
private Currency currency; // 설정 통화
39-
private String connectedId; // 코데프 유저 아이디
40-
private String fcmToken; // firebase 유저 토큰
24+
private Long memberId;
25+
private Long remittanceInformationId;
26+
private Long remittanceGroupId;
27+
private String loginId;
28+
private String password;
29+
private String passportNumber;
30+
private NationalityCode nationality;
31+
private String country;
32+
private LocalDate birth;
33+
private String name;
34+
private String phoneNumber;
35+
private LocalDate remainTime;
36+
private Currency currency;
37+
private String connectedId;
38+
private String fcmToken;
4139

4240
}

‎src/main/java/org/scoula/domain/member/exception/MemberErrorCode.java‎

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,6 @@
1010
@AllArgsConstructor
1111
public enum MemberErrorCode implements ErrorCode {
1212

13-
// 400
1413
MEMBER_NOT_FOUND(HttpStatus.NOT_FOUND, "M-001", "존재하지 않는 회원입니다."),
1514
MEMBER_ALREADY_EXISTS(HttpStatus.BAD_REQUEST, "M-002", "이미 존재하는 회원입니다."),
1615
LOGIN_ID_ALREADY_EXISTS(HttpStatus.BAD_REQUEST, "M-003", "이미 존재하는 회원 아이디 입니다."),

‎src/main/java/org/scoula/domain/member/mapper/MemberMapper.java‎

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -13,25 +13,18 @@
1313
@Mapper
1414
public interface MemberMapper {
1515

16-
// 회원 전체 조회
1716
List<Member> selectAllMembers();
1817

19-
// 회원 상세 조회
2018
Member selectMemberById(@Param("memberId") Long memberId);
2119

22-
// 로그인 ID로 회원 조회
2320
Member selectMemberByLoginId(@Param("loginId") String loginId);
2421

25-
// 회원 등록
2622
int insertMember(Member member);
2723

28-
// 회원 정보 수정
2924
int updateMember(Member member);
3025

31-
// 회원 삭제
3226
int deleteMember(@Param("memberId") Long memberId);
3327

34-
// 로그인 ID 중복 체크
3528
int checkLoginIdDuplicate(@Param("loginId") String loginId);
3629

3730
@Update("UPDATE member SET connected_id = #{connectedId} WHERE member_id = #{memberId}")

‎src/main/java/org/scoula/domain/member/service/JoinServiceImpl.java‎

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,7 @@ public JoinResponse joinMember(JoinRequest joinRequest, HttpServletRequest reque
4646
String StayExpirationBirthDate = date.format(STAY_EXPIRATION_FORMATTER);
4747

4848
StayExpirationRequest stayExpirationRequest = StayExpirationRequest.builder()
49-
.organization("0001") // 고정값 0001
49+
.organization("0001")
5050
.birthDate(StayExpirationBirthDate)
5151
.passportNo(joinRequest.passportNumber())
5252
.nationality(joinRequest.nationality().getCode())
@@ -62,7 +62,7 @@ public JoinResponse joinMember(JoinRequest joinRequest, HttpServletRequest reque
6262
.birth(LocalDate.parse(joinRequest.birth(), DATE_FORMATTER))
6363
.nationality(joinRequest.nationality())
6464
.phoneNumber(joinRequest.phoneNumber().replaceAll("-", ""))
65-
.currency(USD) // default USD
65+
.currency(USD)
6666
.remainTime(stayExpiration)
6767
.build();
6868

@@ -85,7 +85,6 @@ private LocalDate getStayExpiration(StayExpirationRequest stayExpirationRequest,
8585
String expirationDate = stayExpirationResponse.resExpirationDate();
8686

8787
if (stayExpirationResponse.resAuthenticity().equals("0") && expirationDate == null) {
88-
// throw new CustomException(STAY_EXPIRATION, LogLevel.WARNING, null, null);
8988
return null;
9089
}
9190
return LocalDate.parse(expirationDate, DATE_FORMATTER);

‎src/main/java/org/scoula/domain/member/service/MemberAuthServiceImpl.java‎

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -45,8 +45,6 @@ public SignInResponseDto signIn(SignInRequestDto signInRequestDto, HttpServletRe
4545

4646
authenticationManagerBuilder.getObject().authenticate(authenticationToken);
4747

48-
// authenticate 메서드 내부에서 이미 존재하는지 검증이 되기 때문에 한 번 더 검증을 해주지 않았음
49-
// authenticate 에서 조회한 멤버 정보를 가져올 방법을 생각해 쿼리를 한 번 줄이면 좋을 듯
5048
MemberDTO member = memberService.getMemberByLoginId(signInRequestDto.loginId());
5149

5250
JwtToken jwtToken = jwtTokenProvider.generateToken(member.getMemberId(), member.getLoginId());

0 commit comments

Comments
 (0)