WEB2 - JavaScript

리팩토링(refactoring)

소스코드

변경사항

 

댓글

댓글 본문
  1. goodluck
    this, target
  2. Dallae
    Cut off duplication-!
  3. 코딩하는렌즈쟁이
    2020-05-21
    키워드 this
    변수를 활용한 중복 제거
  4. 이서연
    완료
  5. 호야
    완료
  6. hhyun
    완료
  7. green diseny
    5월3일 finish
  8. corrcorea
    4/27 완 료
  9. 솔솔
    4/26 완료
  10. 민스티티아
    완료!
  11. 하나해요
    완료!
  12. Mongsil
    2020/4/2 완료!
  13. 야루
    완료
  14. 2020/04/02
    리팩토링 : 코드의 유지보수나 간결함, 중복의 제거를 위해 하는 행위

    자기 자신을 칭하는 this와 변수 선언을 이용한 중복의 제거로 코드를 리팩토링함

    잘 배우고 갑니다!
  15. 이름
    실습해보니 *.style.backgroundColor나 *.style.color 같은 부분은 변수로 만들지 못하나 보네요

    감사합니다~
  16. 완료
  17. Yohanesty
    완료
  18. 뿡뿡팡야
    완료
  19. OneJae EE
    20.03.22 일 / 37세 비전공자 / 완료
  20. 혹등고래
    완료
  21. 완료!
  22. -refactoring : 중복된 소스를 제거하여 문장을 간결하게 만듦
    -this : 자기자신의 태그를 가리킴

    var target = document.querySelector('body');
    if(this.value === "night') {
    target.style.backgroundColor = 'black';
    ...}
  23. 완료
  24. 짱아찌
    완료
  25. Always
    잘 보고 갑니다! 감사합니다!
  26. 리팩토링 : 중복된 코드를 줄임으로써 유지보수가 편리하도록 개선하는 작업.

    var target = document.querySelector('body');
    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';
    }
    ">
  27. 숩숩이
    완료
  28. daniellee123456
    완료
  29. 코딩하쟈
    완료
  30. 뚠뚠이
    완료
  31. eddylee123456
    완료
  32. 규몽
    완료
  33. 감사합니다.완료
  34. 봄숙
    리팩토링 : 중복을 모두 제거하고 변수 사용
    이 이벤트 코드 안에서는
    그 코드를 가르키도록 약속된 키워드 "this" - 중복되는 아이디값을 this로 변경가능(이 이벤트 코드의 자기자신)
    중복되는 body값을 var taget이라는 변수 지정 해서 taget은 body태그가 되는것
    body 들어갈부분에 target입력하면 var taget('body') <- 얘만 바꾸면 모두 바꿀수있음
  35. 뉴빈
    중복을 제거하기 위해 생각할 것!
  36. 엑셀함수랑 정말 비스하네요
  37. 완료!
  38. 완료
  39. 항상 감사합니다.!!
  40. N0name
    night day가 모든 버튼에서 같이 바뀌게 하고 싶으면 어떻게 코드를 바꾸면 될까요?
  41. thsths
    완료
  42. 춤춤
    완료
  43. moleskindiary@gmail.com
    감사합니다!
  44. 스티븐잡숴
    완료
  45. 메메
    document.querySelector('#night_day')는 자기자신을 가리키고 있음
    →'this'키워드 사용 & id로 지칭해 줄 필요가 사라졌기 때문에 id 삭제

    변수를 지정해줌으로써 중복 제거
    var target = document.querySelector('body');
  46. 펜펜
    감사합니다!
  47. 서혜민
    완료
  48. 유화
    감사합니다!
  49. 중복된 코드를 낮추고 효율적으로 만드는 작업 : 리팩토링
    유지보수가 쉬워진다.
    <input id="night_day" type="button" value="흑화" onclick="
    var target = document.querySelector('body');
    if(this.value === '흑화'){
    target.style.backgroundColor = 'black';
    target.style.color = 'white';
    this.value = '사람';
    } else {
    target.style.backgroundColor = 'white';
    target.style.color = 'black';
    this.value = '흑화';
    }
버전 관리
egoing
현재 버전
선택 버전
graphittie 자세히 보기