Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: code-squad/javascript-todo
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: master
Choose a base ref
...
head repository: code-squad/javascript-todo
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: dev-dongwon
Choose a head ref
Checking mergeability… Don’t worry, you can still create the pull request.
  • 5 commits
  • 17 files changed
  • 3 contributors

Commits on Oct 10, 2018

  1. lec3 - Step2.다양한 출력 지원 (#48)

    * Lecture3 - step1. 기본기능 구현 (#40)
    
    * 스켈레톤 코드 작성
    
    * 수많은 에러가 담긴 todo객체의 add함수 추가(id의 값을 제대로 가지지 못함, todo객체의 getStatusNum수정 필요
    
    * 랜덤한 id값을 못가져오는 것을 수정. checkOverlap함수를 통해서 id의 값이 같지않도록 구현
    
    * todo.add함수기능 완벽하게 구현
    
    * todo.update함수 완성
    
    * id를 입력받아 할일을 제거해주는 todo.remove함수 완성, 오류가많은 todo.printTask함수 완성
    
    * todo.printTask함수 완성, tag를 확인하는 함수 제거(printTask를 통해 각 할일들의 태그가 무엇인지 확인이 가능하므로)
    
    * todo.printTagRelate 함수 완성, todo.add함수에서 버그를 발견함
    
    * 새로운 객체 안 배열에 저장하도록 만들어 문제 해결
    
    * remove함수를 호출했을때, saveData.task배열에서 지워지는것 뿐 아니라 saveData.idArrays배열에서도 id를 지워지게 수정
    
    * 테스트코드를 주석처리함 들여쓰기를 다시 맞춤
    
    * saveData객체를 없애고 그 내용을 todo객체 안으로 집어 넣음, todo.idArrays의 값은 todo.add를 호출할때마다 추가되도록 수정.
    
    * if(saveData.idArrays.indexOf(ranNum) !== -1)-> if(this.idArrays.includes(ranNum))으로 변경
    
    * 기나긴 삼항연산자 statusNum[obj.status]++로 해결
    
    * todo.update함수에서 바뀐 상태의 상태를 반영하지 않는 버그를 찾아내서 수정
    
    * 함수마다 간단한 주석을 달아줌
    
    * 주석추가
    
    * 피드백을 위한 주석 추가
    
    * 같은 숫자를 반복하는 것을 todo.task로부터 찾아와 sync가 완벽하게 맞도록 수정
    
    * 쓸데없는 todo.idArrays에 쓰이는 배열과 메서드 제거
    
    * 변수명 변경
    
    * add나revmoe, update함수를 입력할때마다 각각 인자를 다르게 받아 다르게 출력하는 함수를 만듬
    
    * 해야할 일들 주석으로 정리
    
    * 만들어야 할 함수들 미리 작성
    
    * time obj를 saveTimeObj로 이름을 변경하고 saveTimeObj에 doing일때의 시간과 done일때의 시간을 인자로 입력받아 경과된 시간을 반환해주는 getTakeTime 함수 구현
    
    * 업데이트할 객체를 인자로 받아 id값과업데이트 될때의 시간 값을 saveTImeObj의 객체에 저장해주는 함수 구현
    
    * todo.add함수에서 새 객체를 만들때 걸린시간 이라는 항목을 추가함
    
    *  timeobj객체를 todo안으로 집어넣어 활용하기로 결정, 수정
    
    * doingTimeArray와 takenTimeArray가 굳이 필요할것 같이 않아서 제거, 대신 task배열내의 객체에 timeData추가. 이를 이용해 updateDoingTime메서드 완성
    
    * updateTakeTime메서드 구현. id가 같은 task.timeData값에 걸린 시간을 계산해서 저장해줌
    
    * showTag메서드와 그 메서드를 위한 printByTag메서드 구현. printByTag는 tag과 status를 인자로 받아 같은인자와 tag를 출력, showTag메서드는 그 값을 세번 불러서 서로다른 3가지의 status값을 모두 출력.
    
    * show메서드 구현 만약 status가 done상태일때만 시간을 출력해주도록 함
    
    * printByTag메서드의 버그 수정(상태가done인 항목을 입력해도 걸린 시간이 나오지 않았음), show메서드의 버그 수정(상태가done이면 doing항목을 출력하라고 입력했을때 done항목도 같이 출력되었었음)
    
    * 테스트 케이스 추가
    
    * showAll메서드 구현, bind메서드를 이용함
    
    * 같은태그를가짐과동시에 같은상태인것의 개수를 세는 함수 getSameTagAndStatusNum함수를 만듦
    
    * show함수를 좀더 보기좋게 수정
    
    * showTags를 위한 getSameTagArrays와 printSameTag함수를 만듬. 태그의 모든값이 담긴 배열을 만들고, 배열의 값을 통해 같은 값을 출력하는 함수를 만듬
    
    * 태그가 같은 할 일의 개수를 계산해주는 함수 getSameTagNum을 만듬
    
    * 함수의 쓰임새에 따라 정렬하여 보기편하게 수정
    
    * 테스트 케이스들을 모두 주석처리하고 id의 경우의 수를 100가지로 늘림
    
    * 사소한 버그 수정
    HTMLhead authored and crongro committed Oct 10, 2018
    Configuration menu
    Copy the full SHA
    b745355 View commit details
    Browse the repository at this point in the history

Commits on Oct 18, 2018

  1. delete. code

    crongro committed Oct 18, 2018
    Configuration menu
    Copy the full SHA
    82e620d View commit details
    Browse the repository at this point in the history

Commits on Apr 22, 2019

  1. Step4-2 mission : TUNA & SNOW (#104)

    * create todosList.json
    
    * create todos.js
    
    * create app.js for program runnung
    
    * Rix the name of function
    
    rename getCommandArrayByRegexp to getCmdList
    
    * excuteCommand 함수 수정 :  argument 개수 판별 추가
    
    인자 개수로 1개 초과  나올 경우 즉시 종료 후 프로그램 다시 실행
    
    * isValidCommand 함수 변경 : obeject argument 추가
    
    외부 변수에 접근하지 않고 객체를 인자로 받을 수 있도록 변경
    
    * idGenerator 함수 변경 : 상수가 아닌 인자로 random number 생성
    
    상수 99999 제거 후, max 값과 min 값으로 random 넘버 생성
    
    * fix the Equal operator to strcit Equal operator
    dev-dongwon authored and crongro committed Apr 22, 2019
    Configuration menu
    Copy the full SHA
    0517df6 View commit details
    Browse the repository at this point in the history

Commits on Apr 29, 2019

  1. step-5-2 : TUNA, YM (#112)

    * create todosList.json
    
    * create todos.js
    
    * create app.js for program runnung
    
    * Rix the name of function
    
    rename getCommandArrayByRegexp to getCmdList
    
    * excuteCommand 함수 수정 :  argument 개수 판별 추가
    
    인자 개수로 1개 초과  나올 경우 즉시 종료 후 프로그램 다시 실행
    
    * isValidCommand 함수 변경 : obeject argument 추가
    
    외부 변수에 접근하지 않고 객체를 인자로 받을 수 있도록 변경
    
    * idGenerator 함수 변경 : 상수가 아닌 인자로 random number 생성
    
    상수 99999 제거 후, max 값과 min 값으로 random 넘버 생성
    
    * fix the Equal operator to strcit Equal operator
    
    * create todosData.json to load the mock data
    
    * create app.js to run todos application
    
    * add the module : readLine module
    
    * add the program constructor
    
    * add the run function : to run the program
    
    * create commandParser.js file : to parse the user's input String
    
    * add the executeCmd function Object : to execute the command line
    
    * create the instruction.js file : to instruct command
    
    * add the Instruction module
    
    * implements the CommandParser method to parse the command line
    
    * create utils.js to use common
    
    * add the getArrByCondition function : get array by condition arguments
    
    * add the modules : data status, utils moudule
    
    * add the show function to show status of data
    
    * add getRandomId function
    
    * add the show function
    
    * add the adding object function
    
    * add the delete function
    
    * add the update function : update object
    
    async function to update objcet
    
    * 인자에 따라 async하게 작동할 수 있도록 메서드 추가
    
    * add the delay function : utils.js
    
    add the Promise function to delay the function
    
    * create exceptionHadling.js file
    
    * add the runtimeExceptions
    
    missingSeperatorException
    notExsitException
    sameStatusException
    commandMissingException
    
    * add the exception handling to run the application
    
    throw isValidSeperator Exception, isValidCommand
    catch and restart program
    
    * add the modules to run program
    
    * add the isValidCommand function
    
    * add the exception handling about executCmd function
    
    * add the module : ExceptionHandling module
    
    * fix datas
    
    * fix the runProgram function : missing argument
    
    * add the exception handling about add, update function
    
    * Rename constructor function : Program to RunTodoApp
    
    * Add module Objects in RunTodoApp constructor
    
    add cmdArr, commandParser, utils, instruction, customException module Obejct
    
    * Instruction 생성자에 모듈 객체 추가
    
    Instruction 생성 시 utils, customException 객체 생성 되도록 추가
    
    * show 함수 리팩토링 : 내부 분기를 everyStatus, sigleStatus function 분리
    
    * 생성자 객체를 이용하도록 prototype object 수정
    
    * CustomerException 에러 객체 리팩토링
    
    * getRandomId function에서 매직넘버 삭제, 생성자 인수 추가
    
    생성자에서 max 값과 min 값을 받을 수 있도록 수정
    
    * 생성자 함수에 모듈 객체 추가
    
    * Rename ExceptionHandling to CustomException
    
    * 함수 이름 변경 :  getradomId to getRandomID
    
    * 함수 바뀐 이름 getRandomID 반영
    dev-dongwon authored and crongro committed Apr 29, 2019
    Configuration menu
    Copy the full SHA
    430a7d3 View commit details
    Browse the repository at this point in the history

Commits on May 2, 2019

  1. Step5-3 : TUNA & YM 리뷰 내용 반영 (#136)

    * fix : resolve conflict
    
    * fix : 리뷰 내용 추가 REAMD.md
    
    * Rename : lowerCase to uppercase
    
    * Rename : cmd to command
    
    * Refactor : 배열을 변수에 할당 후 네이밍
    
    * Refactor : remove result variable
    
    * Refactor : object to destructuring
    
    * Fix : else if to else
    
    * Refactor : modified try block
    
    * feat : create getLogObj method
    
    * Refactor : class to Object
    
    * Fix : this.utils to Utils object
    
    * Fix : remove duplicated method
    
    * fix : fix the indent
    dev-dongwon authored and crongro committed May 2, 2019
    Configuration menu
    Copy the full SHA
    acb07c6 View commit details
    Browse the repository at this point in the history
Loading