java 두가지 코드 성능 비교 보시고 의견 부탁드립니다.

Question

A 라는 배열을 가지고 작업을 수행할 때A 배열에 있는 객체를 꺼내와서 getter 로 해당 값을 조회 후 Map 에 넣는 형태입니다.제가 2가지 방식으로 나눠 코딩 하였는데, 어떤 로직을 택하는게 좋은 코드일지 궁금합니다.의견 나눠주시면 감사드리겠습니다.첫번째 로직은 if ~ else if 를 나열해 가면서 10가지 객체 정도 로 조회하는 방식입니다.Object[] obj = { Obj1, Obj2, Obj3 Obj4, Obj5 }; // 각 객체 필드에는 값이 정해져 있습니다. Map map = new LinkedHashMap<>(); for (Object arg : obj) { if (arg instance of Obj1) { Obj1 obj = (Obj1) arg map.put("test", obj.getTest()); map.put("test2", obj.getTest2()); } else if (arg instance of Obj2) { // 위 로직과 동일합니다. } }두번째 방식은 매서드 내 에서 파라미터로 Object 를 받아서 처리하는 방식입니다.Object[] obj = { Obj1, Obj2, Obj3, Obj4, Obj5 }; // 각 객체 필드에는 값이 정해져 있습니다. for (Object arg : obj) { if (arg instance of Obj1) { Obj1 obj = (Obj1) arg map.put("test", obj.getTest()); map.put("test2", obj.getTest2()); } else { extracted(arg); } } private void extracted(Object param) throws Exception { Map map = new LinkedHashMap<>(); String test = ""; String test2 = ""; Method[] method = ArrayUtils.addAll(param.getClass().getDeclaredMethods(), param.getClass().getSuperclass().getDeclaredMethods()); Method getTest = Arrays.stream(method).filter(o -> o.getName().equals("getTest")).findFirst().get(); Method getTest2 = Arrays.stream(method).filter(o -> o.getName().equals("getTest2")).findFirst().get(); test = (String) getTest.invoke(param); test2 = (String) getTest2.invoke(param); map.put("test", test); map.put("test2", test2 ); }정리하면 배열에 들어있는 오브젝트의 타입은 클래스 이며, 해당 클래스들은 모두 getTest, getTest2 매서드를 가지고 있습니다.차이점 은if ~ else if 로 모든 객체를 확인하면서 동일한 객체를 찾는 것객체 클래스가 가 가지는 모든 매서드를 가져온 후 매칭되는 매서드를 stream 형태로 찾아와 값을 불러오는 것이며, 두 로직중 어떤 것이 더 나은 것일까 하는 궁금증이 생기네요

김포포 · Accepted Answer

일단 코드 자체에 오류가 너무 많아서 그거 부터 정리 좀 하고 다시 코드 올려보세요. 둘 다 문제가 많아 보이며, 폴리모피즘이 모르면 그거 부터 공부해서 그냥 if 문 없이 동일 메소드 호출 만 하면 되도록 변경하셈 (인터페이스 추가) 그 정도로 안되면 조금 더 공부하셔서 더블디스패치로~ (Chain of Responsibility, Visitor 패턴도 공부해보시고)

감독님테스트가하고싶어요 · Answer

instance of쓰지마세요다형성 이용하면 if else없이 그냥 처리할 수 있지 않을까요? Obj1, Obj2, Obj3 Obj4, Obj5들을 하나의 인터페이스로 묶어서 getTest(), getTest2() 구현하세요.

양근당파 · Answer

자린이 진짜 몰라서 여쭤보는건데 이거 obj3 obj4 쉼표 의도된건가여?Object[] obj = { Obj1, Obj2, Obj3 Obj4, Obj5 };

adduci · Answer

앞에 인터페이스 하나 두고 aop대상이 되는 모든 객체들이 인터페이스를 implements하도록 하시고 인터페이스 안에 get메소드를 선언하셔서 쓰시는 방법도 있습니다그럼 캐스팅도 필요없겠죠

java 두가지 코드 성능 비교 보시고 의견 부탁드립니다.

4개의 답변

Q&A 목록