인프런 커뮤니티 질문&답변

안녕하세요, 질문이 있습니다. 

 

MemberServiceTest 작동 시, 

D:\Workspace\jpashop\src\main\java\jpabook\jpashop\domain\Address.java

java: class, interface, or enum expected

라는 오류가 발생하면서, Address.java 파일이 켜집니다. 

 

이하 MemberService.java, 

MemberServiceTest.java, 

Address.java 파일과

프로젝트 구조트리입니다.

package jpabook.jpashop.service;

import jpabook.jpashop.domain.Member;
import jpabook.jpashop.repository.MemberRepository;
import lombok.RequiredArgsConstructor;
import org.springframework.beans.factory.annotation.Autowired;
import org.springframework.stereotype.Service;
import org.springframework.transaction.annotation.Transactional;

import java.util.List;

@Service
@Transactional(readOnly = true)
@RequiredArgsConstructor        //final 이 붙은 필드만 생성자 만듦
public class MemberService {

    private final MemberRepository memberRepository;

    //생성자 주입 (생성자가 하나면 @Autowired 생략 가능)
//    public MemberService(MemberRepository memberRepository) {
//        this.memberRepository = memberRepository;
//    }

    /** 
     * 회원가입
     * */
    @Transactional  //기본값은 readOnly = false
    public Long join(Member member) {
        
        validateDuplicateMember(member);    //중복 회원 검증
            memberRepository.save(member);      //영속성 컨텍스트에 persist() 되면, 그 키가 엔티티의 PK 값이다.
        return member.getId();
    }

    private void validateDuplicateMember(Member member) {
        //여기에 더해서 멀티 Thread 환경을 고려해, DB의 name 에 unique constraint 를 걸어주는 게 좋다.
        List<Member> findMembers = memberRepository.findByName(member.getName());

        //또는 세보고 크기가 0이상이면 exeption으로 해도 됨
        if(!findMembers.isEmpty()){
            throw new IllegalStateException("이미 존재하는 회원입니다.");
        }
    }

    /**
     * 회원 전체 조회
     * */
    public List<Member> findMembers() {
        return memberRepository.findAll();
    }

    public Member findOne(Long memberId) {
        return memberRepository.findOne(memberId);
    }
}

 

package jpabook.jpashop.service;

import jpabook.jpashop.domain.Member;
import jpabook.jpashop.repository.MemberRepository;
import org.junit.Test;
import org.junit.runner.RunWith;
import org.springframework.beans.factory.annotation.Autowired;
import org.springframework.boot.test.context.SpringBootTest;
import org.springframework.test.context.junit4.SpringRunner;
import org.springframework.transaction.annotation.Transactional;
import static org.junit.Assert.assertEquals;
import static org.junit.Assert.fail;
import static org.junit.Assert.*;

@RunWith(SpringRunner.class)
@SpringBootTest
@Transactional
public class MemberServiceTest {

    @Autowired MemberService memberService;
    @Autowired MemberRepository memberRepository;

    @Test
    public void 회원가입() throws Exception {
        //Given
        Member member = new Member();
        member.setName("kim");
        //When
        Long saveId = memberService.join(member);
        //Then
        assertEquals(member, memberRepository.findOne(saveId));
    }

}

 

package jpabook.jpashop.domain;

import lombok.Getter;
import lombok.Setter;

import javax.persistence.Embeddable;

@Embeddable
@Getter
public class Address {

    private String city;
    private String street;
    private String zipcode;

    /* 값타입 : 변경불가능하게 설계해야 함.
    * 따라서 @Setter 는 제거하고, 모든 값을 초기화하는 생성자를 만들어준다.
    *
    * @Embeddable : JPA 스펙상에서 이 타입은 기본생성자가 필요함(리플랙션 할 때 필요)
    *       다만 public 이면 아무데서나 가져 쓸 수 있으므로 protected 로 설정
    * */
    protected Address() {
    }
    public Address(String city, String street, String zipcode) {
        this.city = city;
        this.street = street;
        this.zipcode = zipcode;
    }
    
}

 

강의안 코드를 복붙해봐도 계속 오류가 발생해서 답답하네요. 

그래도 강의 열심히 듣고있습니다. 좋은 하루 보내세요!