생활코딩

Coding Everybody

코스 전체목록

닫기

리팩토링(refactoring)

소스코드

변경사항

 

댓글

댓글 본문
작성자
비밀번호
  1. 봄숙
    리팩토링 : 중복을 모두 제거하고 변수 사용
    이 이벤트 코드 안에서는
    그 코드를 가르키도록 약속된 키워드 "this" - 중복되는 아이디값을 this로 변경가능(이 이벤트 코드의 자기자신)
    중복되는 body값을 var taget이라는 변수 지정 해서 taget은 body태그가 되는것
    body 들어갈부분에 target입력하면 var taget('body') <- 얘만 바꾸면 모두 바꿀수있음
  2. 뉴빈
    중복을 제거하기 위해 생각할 것!
  3. 엑셀함수랑 정말 비스하네요
  4. hansy
    완료!
  5. Useok
    완료
  6. moon
    항상 감사합니다.!!
  7. N0name
    night day가 모든 버튼에서 같이 바뀌게 하고 싶으면 어떻게 코드를 바꾸면 될까요?
  8. thsths
    완료
  9. 춤춤
    완료
  10. Yamto
  11. moleskindiary@gmail.com
    감사합니다!
  12. 스티븐잡숴
    완료
  13. 메메
    document.querySelector('#night_day')는 자기자신을 가리키고 있음
    →'this'키워드 사용 & id로 지칭해 줄 필요가 사라졌기 때문에 id 삭제

    변수를 지정해줌으로써 중복 제거
    var target = document.querySelector('body');
  14. 펜펜
    감사합니다!
  15. 서혜민
    완료
  16. 유화
    감사합니다!
  17. 중복된 코드를 낮추고 효율적으로 만드는 작업 : 리팩토링
    유지보수가 쉬워진다.
    <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 = '흑화';
    }
  18. Jeremy Jo
    봤습니다.
  19. 문병민
    리펙토링

    중복을 끝까지 추적해서 제거 >> 변수를 사용
  20. hr1004
    완료
  21. 오오우우얍
    다음!
  22. 샬롬
    완료
  23. 아이디
    id값은 하나에 한개밖에 안된다고 하지않았나요?
  24. 아이원
    감사합니다.
    ctr-D도 유용하네요...
  25. this, var 이용하여 간결화
  26. 김태완
    감사합니다
    대화보기
    • 리트
      CTRL+D 누를 경우 같은단어 선택이 가능하네요
    • 아빠늑대
      re는 다시 factory는 공장, 다시 보내서 개선한다는 느낌. 재가공(?)
      코딩을 하고 나면, 코드는 비효율적인 면이 생기기 마련, 동작은 그대로 두면서 효율적으로 만들고, 가독성을 높이고, 유지보수를 쉽게하고, 중복을 줄이는 작업이 바로 리펙토링
      소프트웨어가 커지고 복잡해지는데 있어서 틈틈히 리펙토링 해주는 것이 중요하다.
      (situation) 자 이 나이트 버튼을 페이지 하단에도 만들어 주고싶어서 코드를 복사해서 붙여넣으면 이렇게!
      제대로 동작하지 않는다. (id를 중복으로 사용해서 위쪽 버튼에 우선동작하게 된다.) (규약 위반)
      (그래서 영상에서는 id="night_day2"로 수정하였다.)
      (keyword : this) this라는 키워드는 자신이 포함된 tag를 가리킨다. 훨씬 코드가 간결해짐.
      document.querySelector("#night_day") --> this
      id ="night_day" 삭제
      코딩을 하는 방법을 알려드리지, 잘하는 방법은 알려드리지 않는다. 하지만 하나 알려드린다
      (tip) 중복을 끝까지 쫒아가서 없애버린다. 이는 여러기술들의 출현 동기와 관련이 많다.
      var target = document.querySelector('body') 선언
      document.querySelector('body') --> target 교체
      이제 target의 값만 바꾸어주면 모든 코드가 한번에 바뀌는 폭발적인 효과!
    • 굼벵이
      완료
    • 홍주호
      20191109 완료
    • 닞닞비
      20191108
    • 육포
      20191031
    • 은별
      완료
    • 봉가리봉봉
      감사합니다
    • 완료
    • 굼벵이
      완료
    • 박창신
      완료
    • 감자
      완료
    • 정일진
      나도 10월 1일 완료!
    • 20191001 Ted 완료!
    • 코딩만이 살길이다
      완료
    • ㅇㅇ
      코딩잘하는것 중복의제거 유지보수 편리할수록 좋은프로그래밍 췍
    • 다에
      완료
    • 쑤우
      수강완료. 감사합니다.
    • 김철호
      감사합니다
    • 정완료
      완료
    • 현우
      완료
    • 정민
      감사합니다! 유용하게 잘보고 있습니다!
    • 우왑
      9.6 끝
    • Min Sang Shin
      감사합니다.
    버전 관리
    egoing
    현재 버전
    선택 버전
    graphittie 자세히 보기