메소드 리팩토링

[kwongiho]

• rankDao == null 보단, Objects.isNull(rankDao); 사용을 권장드립니다.

• return null 보단 Collections.emptyList() 를 권장드립니다.

• String클래스 같은 경우는 final 클래스이기 때문에 +와 같은 연산자를 만나면 인스턴스 자체가 확장되지 못합니다.

• 그렇기 때문에 서로 .concat()메소드를 불러서 서로를 연결하는데요

• 이는 심각한 성능이슈를 발생시킵니다.

• 여러 String값들을 연결하셔야 하면, StringBuffer 혹은 StringBuilder류를 추천드립니다.

사실 자바 1.5? 부터는 +연산을 StringBuilder로 컴파일러가 바꿔준다고 하는데요.
컴파일러에게 의존하는 것 보다는 자체적으로 사용하시는걸 추천드립니다.

• 두개의 클래스의 차이점을 알고계신다면 더욱 좋을 것 같습니다.

• 또한, for 보단 for-each류를 추천드립니다.

comparator를 사용해야 하는 경우라면 for을 추천드립니다.

ColorGalliCups_Single/src/model/RankGetService.java

Lines 29 to 42 in 22835bb

	public ArrayList<String[]> getRanks() {
	if (rankDao == null) {
	System.out.println("RankGetService에서 getRanks() 메소드의 rankDao가 null");
	return null;
	}
	ArrayList<String[]> ranks = new ArrayList<>();
	ArrayList<Rank> rankD = rankDao.getRanks();
	for (int i = 0; i < rankD.size(); i++) {
	String[] rank = new String[] { "" + rankD.get(i).getRanking(), rankD.get(i).getName(),
	"" + rankD.get(i).getScore() };
	ranks.add(rank);
	}
	return ranks;
	}