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

[2주차] 신혜빈 로또 과제 제출 #64

Open
wants to merge 15 commits into
base: main
Choose a base branch
from

Conversation

shin378378
Copy link

@shin378378 shin378378 commented Oct 2, 2024

안녕하세요 루카 리뷰어님!

정말 꼼꼼하시다고 소문으로 익히 들었는 데 만나뵙게 되어 반갑습니다 :)
잘 부탁드립니다!


신경쓴 점

  1. 비슷한 기능을 하는 메소드끼리 같은 클래스 안에 넣기
  2. 메소드 10줄 넘어가지 않게 깔끔하게 짜기
  3. 변수명 , 메소드명 이해 잘 가게 짓기

궁금한 점

List과 int[ ]가 존재하는 이유가 List는 숫자를 객체형으로 다루고 int[]는 숫자를 변수로 다룬다고 하던 데
숫자를 객체로 다루는 경우가 왜 필요한 지 궁금합니다. 숫자 하나가 객체가 되는 것이 의미가 있는 지 잘 모르겠어요 ㅜㅠ

이번 코드에서 저는 List과 int[]를 혼합해서 사용하였는 데요. 제 코드 중에 "여긴 List을 썼지만 int[]를 쓰면 더 좋겠다~" 반대로 "여긴 int[]을 썼지만 List을 쓰면 더 좋겠다~" 싶은 부분이 있는 지, 있다면 어느 부분인 지 궁금합니다.

저는 Controller에 총 3개의 필드값 두었습니다.
함수에서 저 3개의 데이터가 워낙 자주 불려서 저렇게 필드값으로 두었는 데 코드가 중복되더라도 그때그떄 데이터를 불러쓰는 게 맞는 건 지,
데이터가 많이 불릴 땐 필드값으로 두는 게 맞는 건지 궁금합니다. (Controller에 웬만하면 데이터값을 저장하지 말라는 이야기를 들어서 여쭤봅니다)


아직 테스트코드가 많이 부족한 데 틈틈히 채워넣겠습니다! ㅜㅠ

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.

1 participant