gdg-korea/gdg_codelab_1403

2014년 3월 GDG Codelab용 코드 저장소입니다.

gdg_codelab_1403's Issues

실행 코드들을 별도 메소드로 분리시키는게 좋을것 같습니다.

실행코드들이 모두 main() 메소드에만 집중되어 있네요.
예외코드 나 실행 코드를 별도 메소드로 분리하는게 좋을 것 같습니다.

LICENSE 파일을 추가하는게 좋을것 같습니다.

LICENSE 가 있으면 좀 더 편안한 마음으로 코드를 사용할 수 있지 않을까요?

인자로 음수값을 넣어도 정상적으로 수행되도록 만들어주세요.

https://en.wikipedia.org/wiki/Generalizations_of_Fibonacci_numbers#Extension_to_negative_integers
(Extension to negative integers항목 참고)
JaehyunAhn님이 댓글에서 언급하셨던 음수 확장 부분을 구현하면 좋을 것 같습니다.

Refactoring

Fibonacci 와 FibonacciTest 정도로 클래스 분리하는 것이 어떨까요?

자바맹인데요 ㅠ.ㅠ 어떻게 실행해야 하나요?

자바소스는 어떻게 실행해야 하나요?

주석이 깨집니다

빌드 시 생성되는 파일들은 버전 관리에서 제외해 주세요.

.class 삭제해야되지 않을까요?

인자로 47이상을 주면 음수값이 나왔다 말았다 합니다.

지나가던 어떤 분이 실행하는 방법을 알려주셔서, 이렇게 저렇게 실행해봤는데요.

실행할 때 값을 47보다 크게 주면 출력물에서 음수가 나옵니다.

뭔가 이상한데요. 해결해주세요.

실행가이드가 필요.

인자가 없이 실행 했을 때, 에러만 발생하므로 실행에 대한 --help 와 같은 가이드 필요.

Exception in thread "main" java.lang.ArrayIndexOutOfBoundsException: 0
at Fibonacci.main(Fibonacci.java:4)

결과 출력을 간결하게 하는 건 어떨까요?

F1 = 0
F2 = 1
F3 = 1
F4 = 2
...
이런 방식은 어떨까요? 지금 방식은 수열을 확인하기 약간 난해하게 보입니다

변수명이 이상하네요.

입력 받은 값이 계산 종료 마지막 값이기 때문에

start 보다는 end, until 등의 변수명으로 바꾸는게 readability를 높여줄것 같습니다.

readme 에서 입력범위를 알려주면 좋을것같아요

입력범위를 알려주면 좋을것같아요

Please follow me.

I want you guys effort Please follow me https://github.com/ragingwind

인자가 없을 땐 에러 메세지를 보기 좋게 보여주면 좋겠어요.

인자가 없을 땐 어떤 인자가 필요하다 보여주면 좋겠어요.

For loop 보다 Recursion 이 좋을까요? For loop 이 O(n) 인데, recursion 은 계산이 안되네요... -_-...

함수를 main에서 분리해서 public 함수로 만들어주세요.

재사용 가능하게 로직을 main에서 분리해주세요.

인자가 1일 때 에러가 발생합니다.

https://github.com/GDG-Korea/gdg_codelab_1403/blob/master/Fibonacci.java#L7

이 부분에서 start = 1일때 문제가 발생됩니다. 수정해주세요^^

code indent 가 Tab 랑 Space 가 혼재되어 있습니다 ㅠ

code 를 보니까 intent 가 tab 이랑 space 가 혼재되어 있습니다.
한가지로 통일하는게 좋을것 같은데요.. 어떤게 나을까요?

gradle build 시스템을 추가하고 싶습니다.

요즘 gradle 도 많이 사용하고 있으니 gradle build script 도 있으면 좋을것 같습니다.

피보나치수열이 0번째 항부터 출력됬으면 좋겠어요!

fib2부터 출력되는 것이 가독성을 해치는듯합니다!

fib0 : 0
fib1 : 1 요것부터 위에서 나오면 좋을것같아요.

Star 갯수가 너무 적은 것 같습니다.

Fork는 34개인데 여러분 Star버튼 눌러주세요!

Fibonacci 수열 리턴 값을 변경했으면 합니다.

Fibonacci 수열 리스트를 원하는 다른 함수가 필요할 경우, 지금의 기능으로는 그 값을 전달할 수 없습니다.
list 나 array로 결과값을 반환하고, 출력은 return값을 받아서 처리하면 재활용성에 좋을 것 같습니다.

인자의 검증 필요

숫자 이외의 값이나 인자가 없으면 익셉션이 납니다. 에러 메시지 필요~

Readme를 영문으로 작성해주세요.

이 프로젝트가 Global하게 진행될 수 있게 영문화가 필요하다고 생각됩니다.

.gitignore 에 intellij 옵션이 없음.

intellij 용 설정파일을 제외하는 게 필요함.

Test Case가 필요할 것 같습니다.

능력자 분들 테스트 자동화 시스템 넣어주세요~

코딩 룰이 필요합니다.

Readme.md에 같이 기술하면 좋겠습니다.

주석이 영어이면 좋을 것 같습니다.

일부 환경에서 EUC_KR인코딩 때문에 깨지는데

개별 세팅보다는 영어주석이 좋을 것 같습니다..

.class 파일은 repo에 올라오지 않으면 좋겠습니다.

.class파일은 repo에 없으면 좋겠는데.

어떻게 하는거죠? 아시는 분이 있으실텐데 말이죠 ;-)

Execution time을 보여주세요.

속도가 얼마나 걸렸는지 알고싶어요.

README.md 파일 수정

https://github.com/GDG-Korea/gdg_codelab_1403#coding-conventions
Indentation 항목 중 Space 4개가 아니라 5개로 변경해주세요.

Can't compile cause encoding

$ javac -encoding utf8 ./Fibonacci.java
./Fibonacci.java:1: illegal character: \65279
?import java.math.BigInteger;
^
1 error

Here is my log. I'm using OS X 10.9.
Is there any mistake about compiling source?

Network is extreamly slow.

from here, the network is extreamly slow.
It is totally impossible to write code !!!

메뉴얼을 만들어 주세요~

메뉴얼이 보이지 않네요.
우리들의 편한 작업을 위한 메뉴얼을 만들어 주세요~^^