#359 refactoring onclick event

[MoojinChae]

359 solved.

[shaynekang]

확인했습니다~

1. 자바스크립트는 굉장히 결함이 많은 언어이기 때문에(예전에 잠시 보여드렸던 호이스팅 이라던가...) 문법을 명시적으로 작성하는 걸 추천합니다. if문 같은 경우도 한 줄로 작성할 수 있지만, 가능하면 중괄호를 사용하는 걸 권장합니다. 코드가 길어지는 단점이 있지만 보다 더 안전하죠. ㅎㅎ
2. jQuery의 최신버전은 click(function(){}); 보다 on("click", function(){});을 권장하고 있습니다. click과 on("click")의 차이점은 이 아티클을 참고해주세요.
3. assets/javascripts/admin/notices.js의 56번째 라인에 alert(id); 코드가 있네요. 디버깅용 코드 맞죠? 나중에 지워주셔민 될 것 같습니다. ㅎㅎ
4. this.id.split("-")[3]; 와 같은 코드는 [3] 값에 어떤 내용이 들어있는지 알 수 없죠. 코드만 보고서는 작동 원리를 이해하기 어렵습니다. 이런 경우는 HTML5의 _data_ attribute를 활용해보세요. <button type="button" class="open-memo-btn" data-memo-id="1">메모</button> 등으로 해결 가능합니다.

수고하셨습니다. 시간 날 때 수정 부탁드릴께요. ㅎㅎ

[MoojinChae]

말씀하신 부분 수정 했습니다!

[yhoonkim]

확인

[angdev]

확인했습니다.

[minhyeok4dev]

확인

[shaynekang]

ㅇㅋ 저도 확인했습니다. 수고하셨습니다~