Merge pull request #126 from prgrms-web-devcourse-final-project/develop

3차 메인 머지 (9/30)

Author: sso0om

back/src/main/java/com/back/domain/job/job/entity/Job.java

@@ -4,13 +4,13 @@
 import jakarta.persistence.*;
 import lombok.Getter;
 import lombok.NoArgsConstructor;
-import lombok.Setter;
 
+import java.util.ArrayList;
 import java.util.List;
 
 @Entity
 @Table(name = "job")
-@Getter @Setter
+@Getter
 @NoArgsConstructor
 public class Job extends BaseEntity {
     @Column(name = "name", nullable = false, unique = true)
@@ -20,15 +20,19 @@ public class Job extends BaseEntity {
     private String description;
 
     @OneToMany(mappedBy = "job", cascade = CascadeType.ALL)
-    private List<JobAlias> aliases;
+    private List<JobAlias> aliases = new ArrayList<>();
 
     public Job(String name, String description) {
         this.name = name;
         this.description = description;
+        this.aliases = new ArrayList<>();
     }
 
     public void addAlias(JobAlias alias) {
+        if (aliases == null) {
+            aliases = new ArrayList<>();
+        }
         aliases.add(alias);
-        alias.setJob(this);
+        alias.linkToJob(this);
     }
 }

back/src/main/java/com/back/domain/job/job/entity/JobAlias.java

@@ -4,11 +4,10 @@
 import jakarta.persistence.*;
 import lombok.Getter;
 import lombok.NoArgsConstructor;
-import lombok.Setter;
 
 @Entity
 @Table(name = "job_alias")
-@Getter @Setter
+@Getter
 @NoArgsConstructor
 public class JobAlias extends BaseEntity {
     @Column(name = "name", nullable = false, unique = true)
@@ -22,4 +21,12 @@ public JobAlias(String name) {
         this.name = name;
         this.job = null; // 기본적으로 연결된 Job이 없음 (pending 상태)
     }
+
+    public void linkToJob(Job job) {
+        this.job = job;
+    }
+
+    public boolean isPending() {
+        return this.job == null;
+    }
 }

back/src/main/java/com/back/domain/job/job/service/JobService.java

@@ -28,7 +28,7 @@ public Job create(String name, String description) {
     @Transactional
     public JobAlias createAlias(Job job, String aliasName) {
         JobAlias alias = new JobAlias(aliasName);
-        alias.setJob(job);
+        job.addAlias(alias);
         return jobAliasRepository.save(alias);
     }
 }

back/src/main/java/com/back/domain/member/member/service/MemberStorage.java

@@ -10,6 +10,8 @@
 import lombok.RequiredArgsConstructor;
 import org.springframework.stereotype.Component;
 
+import java.util.Optional;
+
 @Component
 @RequiredArgsConstructor
 public class MemberStorage {
@@ -24,12 +26,15 @@ public Mentor findMentorByMember(Member member) {
     }
 
     public Mentor findMentorByMemberId(Long memberId) {
-        return mentorRepository.findByMemberId(memberId)
+        return mentorRepository.findByMemberIdWithMember(memberId)
             .orElseThrow(() -> new ServiceException(MemberErrorCode.NOT_FOUND_MENTOR));
     }
+    public Optional<Mentor> findMentorByMemberOptional(Member member) {
+        return mentorRepository.findByMemberIdWithMember(member.getId());
+    }
 
     public Mentee findMenteeByMember(Member member) {
-        return menteeRepository.findByMemberId(member.getId())
+        return menteeRepository.findByMemberIdWithMember(member.getId())
             .orElseThrow(() -> new ServiceException(MemberErrorCode.NOT_FOUND_MENTEE));
     }
 

back/src/main/java/com/back/domain/member/mentee/dto/MenteeDto.java

@@ -6,13 +6,13 @@
 public record MenteeDto(
     @Schema(description = "멘티 ID")
     Long menteeId,
-    @Schema(description = "멘티명")
-    String name
+    @Schema(description = "멘티 닉네임")
+    String nickname
 ) {
     public static MenteeDto from(Mentee mentee) {
         return new MenteeDto(
             mentee.getId(),
-            mentee.getMember().getName()
+            mentee.getMember().getNickname()
         );
     }
 }

back/src/main/java/com/back/domain/member/mentee/repository/MenteeRepository.java

@@ -12,6 +12,9 @@ public interface MenteeRepository extends JpaRepository<Mentee, Long> {
     @Query("SELECT m FROM Mentee m WHERE m.member.id = :memberId AND m.isDeleted = false")
     Optional<Mentee> findByMemberId(@Param("memberId") Long memberId);
 
+    @Query("SELECT m FROM Mentee m JOIN FETCH m.member WHERE m.member.id = :memberId AND m.isDeleted = false")
+    Optional<Mentee> findByMemberIdWithMember(@Param("memberId") Long memberId);
+
     @Query("SELECT m FROM Mentee m WHERE m.id = :id AND m.isDeleted = false")
     Optional<Mentee> findById(@Param("id") Long id);
 

back/src/main/java/com/back/domain/member/mentor/dto/MentorDetailDto.java

@@ -6,8 +6,8 @@
 public record MentorDetailDto(
     @Schema(description = "멘토 ID")
     Long mentorId,
-    @Schema(description = "멘토명")
-    String name,
+    @Schema(description = "멘토 닉네임")
+    String nickname,
     @Schema(description = "평점")
     Double rate,
     // TODO: Job id, name
@@ -17,7 +17,7 @@ public record MentorDetailDto(
     public static MentorDetailDto from(Mentor mentor) {
         return new MentorDetailDto(
             mentor.getId(),
-            mentor.getMember().getName(),
+            mentor.getMember().getNickname(),
             mentor.getRate(),
             mentor.getCareerYears()
         );

back/src/main/java/com/back/domain/member/mentor/dto/MentorDto.java

@@ -6,13 +6,13 @@
 public record MentorDto(
     @Schema(description = "멘토 ID")
     Long mentorId,
-    @Schema(description = "멘토명")
-    String name
+    @Schema(description = "멘토 닉네임")
+    String nickname
 ) {
     public static MentorDto from(Mentor mentor) {
         return new MentorDto(
             mentor.getId(),
-            mentor.getMember().getName()
+            mentor.getMember().getNickname()
         );
     }
 }

back/src/main/java/com/back/domain/member/mentor/repository/MentorRepository.java

@@ -12,6 +12,9 @@ public interface MentorRepository extends JpaRepository<Mentor, Long> {
     @Query("SELECT m FROM Mentor m WHERE m.member.id = :memberId AND m.isDeleted = false")
     Optional<Mentor> findByMemberId(@Param("memberId") Long memberId);
 
+    @Query("SELECT m FROM Mentor m JOIN FETCH m.member WHERE m.member.id = :memberId AND m.isDeleted = false")
+    Optional<Mentor> findByMemberIdWithMember(@Param("memberId") Long memberId);
+
     @Query("SELECT m FROM Mentor m WHERE m.id = :id AND m.isDeleted = false")
     Optional<Mentor> findById(@Param("id") Long id);
 

back/src/main/java/com/back/domain/mentoring/mentoring/repository/MentoringRepositoryImpl.java

@@ -26,11 +26,11 @@ public Page<Mentoring> searchMentorings(String keyword, Pageable pageable) {
 
         BooleanBuilder builder = new BooleanBuilder();
 
-        // 제목, 멘토 이름 검색 조건
+        // 제목, 멘토 닉네임 검색 조건
         if (keyword != null && !keyword.isBlank()) {
             builder.and(
                 mentoring.title.containsIgnoreCase(keyword)
-                    .or(mentor.member.name.containsIgnoreCase(keyword))
+                    .or(mentor.member.nickname.containsIgnoreCase(keyword))
             );
         }