Bạn code như thế này thì đúng là chết thật. Mình sẽ đưa ra một vài nhận xét và hướng dẫn chỗ bạn phải xem lại.
1. Tên hàm bạn đặt không đúng. Tên hàm phải bắt đầu là một động từ và ý nghĩa phải rõ ràng. Hai hàm PhanTuCuoi và NguyenTo của bạn chẳng hề mang ý nghĩa gì cả.
2. Hàm NguyenTo (giả sử đổi tên thành TinhTongCacSoNguyenTo) thì bạn nên tách ra làm 2 hàm. Một là KiemTraSoNguyenTo và TinhTongCacSoNguyenTo để tận dụng tính tái sử dụng sau này.
3.Lỗi nặng nhất (gây crash chương trình) đó là khai báo biến mà không khởi tạo :(.
Code:
int PhantuCuoi(int a[],int n)
{
int flag;
for(int i=n-1;i<=0;i--)
if(a[i]%2==0 && a[i]>10) //gap phan tu thoa
{
flag=a[i]; // crash
break;
}
return flag; // crash
}
Code:
int NguyenTo(int a[],int n)
{
int tong;
for(int i=0;i<n;i++)
{
for(int j=2;j<=a[i];j++)
{
if(a[i]%j==0)
{
break;
}
else
tong=tong+a[i]; // crash !!!
}
}
return tong; // crash !!!
}
Hi vọng bạn hiểu ý mình muốn nói và tự tìm ra phong cách coding cho mình.
Thân.