truonganhhoang/int3117-2018

NguyenThanhCong

Closed this issue · 1 comments

Đầu tiên, tên dãy số là fibonacci chứ không phải fibonanci :)

public static boolean checkSNT(int n){
if(n > 1){
for(int i = 2; i <= Math.sqrt(n); i++){
if(n % i == 0) return false;
}
return true;
}
else return false;
}
public void SearchFibonanci(int[] f, int n){
f[0] = 1;
f[1] = 1;
int i = 1; count = 1;
while(f[i] < n){
if(checkSNT( f[i] )){
System.out.print(" " + f[i]);
count++;
} i++;
f[i] = f[i-1] + f[i-2];
}
}

Trong naming convention của Java, tên hàm thì dùng lowerCamelCase, tên class thì dùng UpperCamelCase, bạn đang dùng lẫn lộn.

for (int m = 0; m <20 ; m++){
f[m] = m;
}

Đầu tiên, không nên hardcode size của mảng int là 20. Thứ hai, mình không rõ tại sao bạn phải gán giá trị cho mảng f trong khi khi đưa vào hàm SearchFibonanci thì các giá trị này lại được tính lại.

Sai tên hàm, hàm CheckFibonanci không tồn tại nhé.

tôi gửi pull request cho thầy rồi, ông close issue giúp tôi nhé