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