Skip to content

[다원님] BOJ-20546 코드 리뷰 #34

@psychology50

Description

@psychology50

기적의 매매법

image

  • 아니, 잘 참다가 이거 보고 웃참 실패한 게 자존심 상하네요. 블로그 읽다가 너무 많이 웃어서 배가 아파요. 하
  • 뭔가 리뷰를 해드리고 싶은데, 블로그 정리를 너무 잘 해놓으셨네요.
    • 코드 또한 네이밍, 함수 분리, 리팩토링 모두 깔끔하게 해놓으셔서 조언 해드릴 게 없습니다.
    • 뭐, 자다 일어나신 정보까지 다 알게 될 정도로 자세히 써주셔서 그런 거 같네요.
  • 한 가지 팁이라면, 개발할 때와는 달리 코테에선 전역 변수의 사용을 그다지 나쁘게 보지 않습니다. 필요하다면 오히려 권장하기도 해요.
    • 어차피 price는 14 크기의 고정 길이 배열이므로, 전역 변수에 할당을 해놓는다면 bnptiming의 매개변수가 한 개씩 줄어들기 때문에 함수를 읽기 위해 이해해야 할 코드가 줄겠죠.
    • 그리고 전역 변수로 하면 매개변수로 잘 넘어가고, 받는지 확인을 안 해도 되니까 간단하죠.
    • 그런데 한다고 꼭 좋다는 건 아닙니다. 오히려 전역 변수로 써서 문제가 발생하기도 하구요. 그러니 참고만 해두세요.

Metadata

Metadata

Assignees

Labels

No labels
No labels

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions