리팩토링(refactoring)

2017-11-26 01:32:25

소스코드

변경사항

김승현어제

comparison operator비교연산자->boolean불리언->conditional statements조건문->refactoring중복제거개선

중복의 제거
반복되는 것을 변수로 지정한다
var target = ;
id 대신에 this를 사용한다.
if(this.value==='')

문법적으로는 이렇게 사용하는거구나라고 생각이들지만 직접 활용하려고 하니 아직 이해가 되지 않는다 ㅠㅠ 많은 예를 접해볼 필요가 있을 것 같다.

답글
295122일 전

921

답글
Paul Yoon9일 전

ㅇㅂㅇ....우와.. this를 쓰면 이렇게 편리해지다니...

답글
H-mechanic15일 전

20.09.08

답글
허우룩19일 전

2020.09.04

답글
chapter544721일 전

완료하였습니다

답글
andrew07012425일 전

변수를 많이 써야겠군요 ㄷ

답글
banaba25일 전

8/29 와우 확줄여지니까 속이 시원해요,,

답글
난지단지26일 전

완료!

답글
CODING29일 전

완료

답글
김요한한 달 전

2020.08.20 완료.

답글
JaanMun한 달 전

2020. 08. 18 완료

답글
wonki한 달 전

this var

답글
joas한 달 전

16.08.20 완료

답글
smr0417한 달 전

2020.08.16 완료

답글
buffybuff한 달 전

대박.... this 라는 거는 어디서 들어보긴 했는데 이렇게 쓸 수 있다니...
오늘도 어썸한 강의 ㄳ함다

답글
Jay1025한 달 전

2020.08.06 완료

답글
장정민한 달 전

즐코즐코

답글
따릉이덕후한 달 전

2020.08.04 완료

답글
switpotato2개월 전

20200725 완료

답글
dongmi952개월 전

20.07.20 완료

답글
Itsyng2개월 전

코딩 잘하는법 : 중복을 제거한다

답글
yjk2개월 전

완료!

답글
Gyeong Min Lee2개월 전

0713 완료

답글
뽀짝이2개월 전

완료

답글
rosydarling2개월 전

7/9 완료

답글
코딩지망생2개월 전

Atom 에디터 자동 줄바꿈을 하는 방법은 뭔가요?

답글
pupu2개월 전

2020/07/08 완료

답글
똑똑한_코딩2개월 전

20200707완료!!!!

답글
김정원2개월 전

2020/07/05 완료

답글
김혜린3개월 전

6월 22일 완료! 코드가 점점 간단해지는 데서 오는 짜릿함이 있네요 ㅎㅎ

답글
psyless3개월 전

20200620

답글
김광현3개월 전

6/19오ㅓㄴ료

답글
GLORIA3개월 전

완료! 감사합니다 :)

답글
guswn3개월 전

20200616

답글
christinelee3개월 전

06122020오늘은 여기까지~

답글
christinelee3개월 전

저도 궁금했는데 감사합니다:)
대화보기
답글
김호균3개월 전

저도 궁금했는데 감사합니다 ㅎㅎ
대화보기
답글
Cyanine3개월 전

다중 선택이 너무 궁금했는데 Ctrl + D 네요 ㅎㅎ 잘보고 있습니다.

답글
지현3개월 전

20200609

답글
Nam3개월 전

20200609

답글
psyless3개월 전

20200607

답글
장지윤3개월 전

완료

답글
cheonsu923개월 전

'white'뒤에 세미콜론(;) 안하셔서 그런거 아니에요 ?
대화보기
답글
최준영3개월 전

atom 에서 저는 제 나름 편의로
{document.querySelector('body').style.backgroundColor='black';document.querySelector('body').style.color='white' this.value='day';}
이런 한 항목을 한줄에다 놓고 했었는데요
이때까지는 이고잉님과 줄바꿈이 달라도 문제가 없었는데
이번 파트에서는
this.value='day'; 부분이 윗줄에 있으면(다른 코드들과 한줄에 있으면) 반영이 안됩니다
다른 것 아무것도 수정한 것 없이 this.value='day'; 만 따로 줄바꿈을 해주면 반영이 되고요
어떤 문제일까요?

답글
choicode20203개월 전

Ddfsdfe Gdeef 이해가 안되는 부분이 있어서요 변수로 바꾼다 해도 결국 타겟으로 전부 바꿔야 하는거 아닌가요?(1억개가 있다 치면)
어떤 부분에서 효과적이란 건지 이해를 못했어요 ㅠㅠ JS는 거의 처음이라 ㅠㅠ ㅎㅎㅎ 아시는분 있으면
답변 부탁드립니다 / 5월 28일

-> 이 말은 아마 모든 document.querySelecor('body')가 target으로 쓰여져 있고, 이 상황에서 document.querySelector('.body')를 다른 것으로 바꿔야 할 때 document.querySelecor('body')로 써져 있는 1억개 모두를 다른 무언가(예:<html>)로 바꾸는 대신 target만 다른 무언가((예:<html>)로 바꾸기만 하면 된다는 것 같아요

예)
<input type="button" value="night" onclick="
if (this.value === 'night'){
document.querySelector('body').style.backgroundColor = 'black';
document.querySelector('body').style.color = 'white';
this.value = 'day';
} else {
document.querySelector('body').style.backgroundColor = 'white';
document.querySelector('body').style.color = 'black'
this.value = 'night';
}
">
에서 document.querySelector('body')를 target으로 바꾸면
var target = document.querySelector('body'); //document.querySelector('body')가 target
if (this.value === 'night'){
target.style.backgroundColor = 'black';
target.style.color = 'white';
this.value = 'day';
} else {
target.style.backgroundColor = 'white';
target.style.color = 'black'
this.value = 'night';
}
">
로 만약 body를 html로 바꾸려 한다면
var target = document.querySelector('body');를 var target = document.querySelector('html');로 바꾸기만 하면 되용

답글
choicode20203개월 전

5/31 완료

답글
HyeonHui Jeong3개월 전

5/28

답글
SONSHARK3개월 전

음 한줄 한줄
document.querySelector('body') 부분을 수정하는걸
target 으로 설정 해두면
var target = ~~~ 부분만 수정하면 되니까
1억개의 줄의 document.querySelector('body') 부분을 바꾸는것보다 편하다 라는거 같습니다!!
대화보기
답글
Ddfsdfe Gdeef3개월 전

이해가 안되는 부분이 있어서요 변수로 바꾼다 해도 결국 타겟으로 전부 바꿔야 하는거 아닌가요?(1억개가 있다 치면)
어떤 부분에서 효과적이란 건지 이해를 못했어요 ㅠㅠ JS는 거의 처음이라 ㅠㅠ ㅎㅎㅎ 아시는분 있으면
답변 부탁드립니다

답글