JSP)) 이거 간단하게 코딩 어떻게 하나요?

if (result.getSpell1()==1) {
spell_1=spell_1+1;
}

if (result.getSpell1()==2) {
spell_2=spell_2+1;
}

if (result.getSpell1()==3) {
spell_3=spell_3+1;
}

if (result.getSpell1()==4) {
spell_5=spell_5+1;
}

if (result.getSpell1()==5) {
spell_5=spell_5+1;
}

if (result.getSpell1()==6) {
spell_6=spell_6+1;
}

if (result.getSpell1()==7) {
spell_7=spell_7+1;
}

spell_N 에다가 몇 번 이용되었는지 넣어주려고 하는데요
무식하게 저렇게 하는거 말고 뭔가 깔끔하게 될거같은데 쪼렙이라...ㅠㅠ

잡담 | 1404명이 읽었어요. 3.22.27.45

비공개 손님 … 2015-01-27 20:32:35

switch case ??;;;

비공개 손님 … 2015-01-27 20:34:27

spell_N을 배열로 만들면 안되나요? (JSP 한번도 안해봄)

비공개 손님 … 2015-01-27 21:18:52

map 을 이용하심이 어떨까요

대충

key = "spell_" + result.getSpell1();
if (map.contains(key)) map.put(key, map.get(key) + 1);
else map.put(key, 1);

비공개 손님 … 2015-01-27 22:01:07

JSP면 (서버단 언어)
프레임 워크단에서 Attribute에 speelNum=숫자 넣으시고

spell_${speelNum}=spell_${speelNum}+1;
하면 될겁니다.

비공개 손님 … 2015-01-27 22:27:20

답 2 혹은 3을 채택하면 될듯
근데 나라면 만약 이코드대로 저장할 대상이 7개뿐이라면 그냥 배열쓸듯

비공개 손님 … 2015-01-27 22:28:16

저건 서버로직이지 클라 렌더링로직이 아니라서 답5같이 EL 쓰면 컴파일이 안됩니다.

비공개 손님 … 2015-01-27 22:43:34

유지보수 할일도 없고
하더라도 간단한 변경만 있을거잖아요
저같으면 직관력 있게 switch case 보기만좋게 다듬겠습니다

비공개 손님 … 2015-01-27 22:48:52

라인수 줄인다고 보기좋은건
아니라고 생각합니당..

비공개 손님 … 2015-01-27 22:53:00

다시 생각해봤는데
그냥 취향같아요...
(내 소스에서 저런경우 맵으로 처리한거 보고왔단투로)

비공개 손님 … 2015-01-27 23:47:44

맵사용하는게 제일좋을거같네요

비공개 손님 … 2015-01-28 05:59:17

감사합니다!

비공개 손님 … 2015-02-18 12:28:44

9// 좋은 프로그램의 가중 중요한 요소 중 하나가 중복을 없애는 일입니다.

댓글을 작성하실 수 없습니다.
(권한이 없는 회원레벨)

목록으로

△ 이전글: 요새 의욕 안돋아요... [10]

▽ 다음글: 좆같아서 회사 때려쳐야겠네요 [3]

TOP arrow_upward