Skip to content

과제 제출 [ABC-LAB_FE_김희환]#6

Open
BlankCodeStack wants to merge 1 commit intocodestates-seb:mainfrom
BlankCodeStack:김희환

Hidden character warning

The head ref may contain hidden characters: "\uae40\ud76c\ud658"
Open

과제 제출 [ABC-LAB_FE_김희환]#6
BlankCodeStack wants to merge 1 commit intocodestates-seb:mainfrom
BlankCodeStack:김희환

Conversation

@BlankCodeStack
Copy link
Copy Markdown

과제제출

@BlankCodeStack BlankCodeStack changed the title 김희환 과제 제출 [김희환] Jan 25, 2023
@BlankCodeStack BlankCodeStack changed the title 과제 제출 [김희환] 과제 제출 [ABC-LAB_FE_김희환] Jan 25, 2023
@0xmiles
Copy link
Copy Markdown

0xmiles commented Jan 26, 2023

문자열을 이용한 다양한 메서드를 사용해서 구현해보셨군요!
연습할 때는 이렇게 다양한 방법을 사용하는 것도 좋습니다.
output같은 경우에는 따로 사용되지 않는 배열이기에 지워주셔도 좋을 것 같습니다.

첫 if문의 경우에도 큰 역할이 없기 때문에 불필요한 코드인 것 같습니다.

변수를 선언할 때에는 할당을 할 때에 선언을 하는 것이 좋습니다.
let a의 경우 return 하기 전에 let a = count.split(','),map(parseFloat);으로 선언하거나,
바로 return count.split(','),map(parseFloat);을 해보시는 것도 좋을 것 같네요 :)

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