Java 코드리뷰...1 forEach

public class 도서관_도서대여_관리프로그램 {
	private static Scanner sc = new Scanner(System.in);
	private static BookManager bookMgr = new BookManager();
	private static MemberManager memberMgr = new MemberManager();

• 기존과는 다르게 출력 부분에 해당하는 곳의 길이를 좀 적게 출력하고자 Manager이라는 클래스를 생성하고 객체를 통해 메서드를 불러온다.

private static void 회원삭제() {
		System.out.println("삭제할 id 입력>> ");
		String id = sc.nextLine();
		boolean b = memberMgr.remove(id);
		
		if( b ) {
			System.out.println("삭제 성공!");
		}else {
			System.out.println("삭제 실패!");
		}
		
	}

• 회원 삭제 부분은 클래스를 만들어 boolean 타입으로 객체를 생성해내서 if를 통해 출력했고

private static void delBook() {
		System.out.println("삭제할 책 isbn 검색");
		int isbn = sc.nextInt();
		sc.nextLine();
		try {
			BookM.findBook(isbn);
			bookM.remove(BookM.findBook(isbn));
			System.out.println("삭제완료");
		} catch (Exception e) {
			System.out.println("해당 도서가 없습니다");
		}

• 나는 try catch를 사용하여 출력을 해냈다. 기능상 큰 차이는 없다.

public static Book findBook(String title) {
	bol.forEach(c -> bo2 = (c.getTitle().equals(title)) ? (Book) c : null);
	return bo2;
}

public static Book findBook(int isbn) {
	bol.forEach(c -> bo2 = (c.getIsbn() == isbn) ? (Book) c : null);
	return bo2;
}

• 목적에 맞게끔 위의 코드처럼 매니저 클래스 내에 메서드를 오버로딩하여 사용이 가능하다.
• 람다식을 계속 듣기만 했고 기본 형태만 봤던 때는 쉽게만 생각을 했었는데 그걸 조건식과 묶는게 조금 까다로워 그걸 생각하는데 시간을 좀 쏟았었다. 결국은 삼항식으로 해결을 했다! ㅎㅎ

BookM.getBol().forEach(c -> System.out.println(c.toString()));

• 위의 코드 같은 경우에는 전체 출력하는 코드인데 get(i)한 이후 sysout을 할 필요없이 이렇게 한 줄로도 완성이 된다. 아무래도 삼항식 이용하다 보니 전보다는 조금 작성하는데 편했다.

li.forEach(System.out::println);
li.forEach(c->System.out:println(c));

• 위의 코드가 동일하다고 하니 한 번 사용해보도록 하자.
• Collection은 Iterator를 사용해서 출력을 했으나 이제는 주로 forEach로 출력을 한다고 한다.
• 다음 포스트는 Iterator와 다른 Collection인 Set과 Map을 다루고 Hash를 다룰예정이다.

(List)Arrays.asList();

• forEach의 경우에는 배열도 쓸 수 있을 거라 생각했지만 forEach는 Collection에서만 사용가능한 걸 명심!
• 타입변환을 통해 forEach를 사용하자.

 

•  아래의 형태랑 forEach문과는 큰 차이는 없지만 코드의 길이를 조금이라도 줄일 수 있다.
• 향상된 for문

public static Book findBook(String title) {
	for(Book i: bol){
		if(i.getTitle().equals(title))
        	return i;
        }
        return null;
}
		
public static Book findBook(int isbn) {
	for(Book i: bol){
		if(i.getIsbn==isbn)
	        return i;
        }
        return null;
}

 

• for 문

public static Book findBook(String title) {

	for(int i =0;i<bol.size();i++){
    	if(bol.get(i).getTitle().equals(title))
        return bol.get(i);
    }
        return null;
}
		
public static Book findBook(int isbn) {
	for(int i =0;i<bol.size();i++){
    	if(bol.get(i).getIsbn()==isbn)
        return bol.get(i);
        }
        return null;
}

• return 형이 어떤 것인지 잘 파악하고 있으면 코드 작성하는데 좀 수월하다.

 

결론

1. 클래스를 따로 만들어 메서드를 호줄하는 식으로 이용하는 것이 꽤 깔끔하게 만들어 지는 것 같다.

2. 반복문을 사용할때 forEach를 이용한 람다식으로 구성을 해보자