Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

4회차 과제 - 조민서 #9

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

JO-MINSEO
Copy link

1. 개발한 기능

가위, 바위, 보 버튼 클릭 → start 함수 : user, computer pick 정해짐 → play 함수 : 가위바위보 결과, message, recordMsg정해짐 → addResult, showReslt함수로 이동

addResult 함수 : gameRecord배열에 id와 message속성을 가지는 요소 추가 → updateRecord : gameRecord 내에 있는 요소를 돌며 리스트 영역에 message와 삭제 버튼 출력

showResult 함수 : user, computer, resultMsg에 저장된 내용을 출력 → updateScore : userScore, computerScore 출력

deleteAllResult 함수 : 전체 삭제 버튼 클릭 시 gameRecord 배열 초기화, userScore, computerScrore 0으로 초기화

deleteResult 함수 : 삭제 버튼 클릭 시 gameRecord 내의 삭제된 인덱스를 찾아 삭제하고 updateRecord해줌 + 삭제된 인덱스의 message 속성을 바탕으로 userScore와 computerScore 업데이트하고 updateScore해줌

2. 유의깊게 개발한 부분

deleteResult 함수 구현 : splice 함수와 includes 함수를 사용하여 구현

3. 개발하면서 들었던 의문 사항

없음

let userScore = 0;
let computerScore = 0;

/*기록 삭제*/
const deleteResult = (deleteId) => {
//...

const index = gameRecord.findIndex(item => item.id === deleteId)
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

filter()나 map(), forEach() 같은 메서드들을 활용하면 해당 코드들을 더 줄일 수 있을 것 같습니다 !

else if (deletemsg.includes('졌다')){
computerScore--;
}
else{
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

else 구문 내에 아무 코드도 없는데 작성하신 이유가 따로 있으신지 궁금합니다.

}

/*기록 전체 삭제*/
const deleteAllResult = () => {
//...
const deleteAllResult = (e) => {
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

매개변수 e로 이벤트를 전달받은 이유가 궁금합니다.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants