WEB2 - JavaScript

리팩토링(refactoring)

소스코드

변경사항

 

댓글

댓글 본문
  1. 김승현
    comparison operator비교연산자->boolean불리언->conditional statements조건문->refactoring중복제거개선

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

    문법적으로는 이렇게 사용하는거구나라고 생각이들지만 직접 활용하려고 하니 아직 이해가 되지 않는다 ㅠㅠ 많은 예를 접해볼 필요가 있을 것 같다.
  2. 29512
    921
  3. Paul Yoon
    ㅇㅂㅇ....우와.. this를 쓰면 이렇게 편리해지다니...
  4. H-mechanic
    20.09.08
  5. 허우룩
    2020.09.04
  6. chapter5447
    완료하였습니다
  7. andrew070124
    변수를 많이 써야겠군요 ㄷ
  8. banaba
    8/29 와우 확줄여지니까 속이 시원해요,,
  9. 난지단지
    완료!
  10. CODING
    완료
  11. 김요한
    2020.08.20 완료.
  12. JaanMun
    2020. 08. 18 완료
  13. this var
  14. 16.08.20 완료
  15. smr0417
    2020.08.16 완료
  16. buffybuff
    대박.... this 라는 거는 어디서 들어보긴 했는데 이렇게 쓸 수 있다니...
    오늘도 어썸한 강의 ㄳ함다
  17. Jay1025
    2020.08.06 완료
  18. 장정민
    즐코즐코
  19. 따릉이덕후
    2020.08.04 완료
  20. switpotato
    20200725 완료
  21. dongmi95
    20.07.20 완료
  22. Itsyng
    코딩 잘하는법 : 중복을 제거한다
  23. 완료!
  24. Gyeong Min Lee
    0713 완료
  25. 뽀짝이
    완료
  26. rosydarling
    7/9 완료
  27. 코딩지망생
    Atom 에디터 자동 줄바꿈을 하는 방법은 뭔가요?
  28. 2020/07/08 완료
  29. 똑똑한_코딩
    20200707완료!!!!
  30. 김정원
    2020/07/05 완료
  31. 김혜린
    6월 22일 완료! 코드가 점점 간단해지는 데서 오는 짜릿함이 있네요 ㅎㅎ
  32. psyless
    20200620
  33. 김광현
    6/19오ㅓㄴ료
  34. GLORIA
    완료! 감사합니다 :)
  35. 20200616
  36. christinelee
    06122020오늘은 여기까지~
  37. christinelee
    저도 궁금했는데 감사합니다:)
    대화보기
    • 김호균
      저도 궁금했는데 감사합니다 ㅎㅎ
      대화보기
      • Cyanine
        다중 선택이 너무 궁금했는데 Ctrl + D 네요 ㅎㅎ 잘보고 있습니다.
      • 지현
        20200609
      • 20200609
      • psyless
        20200607
      • 장지윤
        완료
      • cheonsu92
        'white'뒤에 세미콜론(;) 안하셔서 그런거 아니에요 ?
        대화보기
        • 최준영
          atom 에서 저는 제 나름 편의로
          {document.querySelector('body').style.backgroundColor='black';document.querySelector('body').style.color='white' this.value='day';}
          이런 한 항목을 한줄에다 놓고 했었는데요
          이때까지는 이고잉님과 줄바꿈이 달라도 문제가 없었는데
          이번 파트에서는
          this.value='day'; 부분이 윗줄에 있으면(다른 코드들과 한줄에 있으면) 반영이 안됩니다
          다른 것 아무것도 수정한 것 없이 this.value='day'; 만 따로 줄바꿈을 해주면 반영이 되고요
          어떤 문제일까요?
        • choicode2020
          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');로 바꾸기만 하면 되용
        • choicode2020
          5/31 완료
        • HyeonHui Jeong
          5/28
        • SONSHARK
          음 한줄 한줄
          document.querySelector('body') 부분을 수정하는걸
          target 으로 설정 해두면
          var target = ~~~ 부분만 수정하면 되니까
          1억개의 줄의 document.querySelector('body') 부분을 바꾸는것보다 편하다 라는거 같습니다!!
          대화보기
          • Ddfsdfe Gdeef
            이해가 안되는 부분이 있어서요 변수로 바꾼다 해도 결국 타겟으로 전부 바꿔야 하는거 아닌가요?(1억개가 있다 치면)
            어떤 부분에서 효과적이란 건지 이해를 못했어요 ㅠㅠ JS는 거의 처음이라 ㅠㅠ ㅎㅎㅎ 아시는분 있으면
            답변 부탁드립니다
          버전 관리
          egoing
          현재 버전
          선택 버전
          graphittie 자세히 보기