Codebreaker 2bc08752a0894eb2c7afb345286e391d

‘너 어느 별에서 왔니’의 소스코드 분석

http://heygom.com/blogthings/planet/

index.html, result.php 파일 두개로 이루어져있다. result.php는 $GET[planet] 값에 따라서 적절한 페이지를 출력해주는 일을 하고, 출신 행성--?은 index.html에서 submit을 하기 전에 자바스크립트 process() 함수에 의해서 미리 정해진다.

process() 함수의 내부는 크게 4부분으로 나누어져있다. 1. 변수 선언부 2. 사용자의 입력값을 종합하는 부분 3. 정렬하는 부분 4. 현재 페이지의 location.href 값을 바꾸는 부분(결과 페이지로 이동)

총 문항 수는 6개, 문항당 보기의 수는 10개이다. 각 보기는 차례대로 sun, moon, mercury, venus, mars, jupiter, saturn, uranus, neptune, pluto 에 대응된다. 예를 들면, 사용자가 첫 문항에서 두번째 보기를 선택하면 moon 의 값을 1 증가시키는 방식이다.

이렇게 종합된 값을 가지고 정렬을 하는데, 맙소사[!] 정렬 방식이 매우... (좋게 표현해서) 비효율적이다. 반복문 쓰면 3~4줄이면 될것을 if문을 10개 써서 구현해놨다.

이 프로그램의 논리적 오류: 1. 행성 변수(sum, moon 등..)의 값이 같으면 위에 서술한 순서대로(sun, moon, mercury...) 높은 우선순위를 가지게 된다.

개선할 점: 1. 기존의 구조를 그대로 유지할 경우 1) 체크된 변수를 찾는 부분 break문을 써주면 약간의 성능향상 및 시스템 자원 절약 효과를 기대할 수 있음. (그렇다고 인간이 느낄 수 있는 차이는 아니다;ㅋㅋ) 2) 행성 변수의 값을 증가시키는 부분 if문 10번 대신 switch문을 쓰는것을 추천.. 3) 정렬 만든 사람의 노가다에 경의를 표하며-_-; for문을 쓸 것을 권장함. 2. 새로 만들 경우 one.value 를 쓰면 반복문 쓸 필요도 없는데.. one[i].checked 로 모든 값을 검사하는건 다중 선택이 허용될때나 그렇게 하는것이다. 배열을 적절히 사용한다면 위의 2, 3번 과정은 만들 필요도 없다.

소스코드 보기가 너무 불편해서 내가 들여쓰기 스타일을 좀 바꿨다.

function process()
{
    var sun = 0;
    var moon = 0;
    var mercury = 0;
    var venus = 0;
    var mars = 0;
    var jupiter = 0;
    var saturn = 0;
    var uranus = 0;
    var neptune = 0;
    var pluto = 0;
    var f = document.f;
    var i = 0;


    for (i = 0; i < f.one.length; i++)
        if (f.one[i].checked)
            value = f.one[i].value;
    if (value == "1") { sun++; }
    if (value == "2") { moon++; }
    if (value == "3") { mercury++; }
    if (value == "4") { venus++; }
    if (value == "5") { mars++; }
    if (value == "6") { jupiter++; }
    if (value == "7") { saturn++; }
    if (value == "8") { uranus++; }
    if (value == "9") { neptune++; }
    if (value == "10") { pluto++; }


    for (i = 0; i < f.two.length; i++)
        if (f.two[i].checked)
            value = f.two[i].value;
    if (value == "1") { sun++; }
    if (value == "2") { moon++; }
    if (value == "3") { mercury++; }
    if (value == "4") { venus++; }
    if (value == "5") { mars++; }
    if (value == "6") { jupiter++; }
    if (value == "7") { saturn++; }
    if (value == "8") { uranus++; }
    if (value == "9") { neptune++; }
    if (value == "10") { pluto++; }


    for (i = 0; i < f.three.length; i++)
        if (f.three[i].checked)
            value = f.three[i].value;
    if (value == "1") { sun++; }
    if (value == "2") { moon++; }
    if (value == "3") { mercury++; }
    if (value == "4") { venus++; }
    if (value == "5") { mars++; }
    if (value == "6") { jupiter++; }
    if (value == "7") { saturn++; }
    if (value == "8") { uranus++; }
    if (value == "9") { neptune++; }
    if (value == "10") { pluto++; }


    for (i = 0; i < f.four.length; i++)
        if (f.four[i].checked)
            value = f.four[i].value;
    if (value == "1") { sun++; }
    if (value == "2") { moon++; }
    if (value == "3") { mercury++; }
    if (value == "4") { venus++; }
    if (value == "5") { mars++; }
    if (value == "6") { jupiter++; }
    if (value == "7") { saturn++; }
    if (value == "8") { uranus++; }
    if (value == "9") { neptune++; }
    if (value == "10") { pluto++; }


    for (i = 0; i < f.five.length; i++)
        if (f.five[i].checked)
            value = f.five[i].value;
    if (value == "1") { sun++; }
    if (value == "2") { moon++; }
    if (value == "3") { mercury++; }
    if (value == "4") { venus++; }
    if (value == "5") { mars++; }
    if (value == "6") { jupiter++; }
    if (value == "7") { saturn++; }
    if (value == "8") { uranus++; }
    if (value == "9") { neptune++; }
    if (value == "10") { pluto++; }


    for (i = 0; i < f.six.length; i++)
        if (f.six[i].checked)
            value = f.six[i].value;
    if (value == "1") { sun++; }
    if (value == "2") { moon++; }
    if (value == "3") { mercury++; }
    if (value == "4") { venus++; }
    if (value == "5") { mars++; }
    if (value == "6") { jupiter++; }
    if (value == "7") { saturn++; }
    if (value == "8") { uranus++; }
    if (value == "9") { neptune++; }
    if (value == "10") { pluto++; }


    var out = "sun";
    i = sun;
    if (moon > i) {
        out = "moon";
        i = moon;
    }
    if (mercury > i) {
        out = "mercury";
        i = mercury;
    }
    if (venus > i) {
        out = "venus";
        i = venus;
    }
    if (mars > i) {
        out = "mars";
        i = mars;
    }
    if (jupiter > i) {
        out = "jupiter";
        i = jupiter;
    }
    if (saturn > i) {
        out = "saturn";
        i = saturn;
    }
    if (uranus > i) {
        out = "uranus";
        i = uranus;
    }
    if (neptune > i) {
        out = "neptune";
        i = neptune;
    }
    if (pluto > i) {
        out = "neptune";
        i = neptune;
    }


    location.href = "./result.php?planet=" + out;
}