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

오오오오오 #64

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

오오오오오 #64

wants to merge 4 commits into from

Conversation

jsmssu
Copy link

@jsmssu jsmssu commented Mar 15, 2014

오옹

jsmssu added 4 commits March 15, 2014 16:02
오오오오오오오오오오??
띄어쓰기
print 입력범위
@jsmssu
Copy link
Author

jsmssu commented Mar 15, 2014

leave a comment

int start = Integer.parseInt(args);
double fib1 = 0;
double fib2 = 1;
if(start>=2 && start<47) {

Choose a reason for hiding this comment

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

47로 제한한 이유가 있나요? 47로 쓰는 것보다는 const variable로 작성하는 것이 더 좋을 것 같습니다.

Choose a reason for hiding this comment

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

47로 제한이 있다면, readme 파일에 제약사항에 대해서 기술을 같이 추가했어야 할 것 같습니다.

@jsmssu jsmssu closed this Mar 15, 2014
@jsmssu jsmssu reopened this Mar 15, 2014
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