Tính tổng các số nguyên tố

[coixuong]

#include <conio.h>#include <stdio.h>//#define MAX 100 void NhapMang(int a[],int &n){    printf("Nhap so phan tu :");    scanf("%d",&n);    for(int i=0;i<n;i++)    {        printf("a[%d]=",i);                scanf("%d",&a[i]);    }} void XuatMang(int a[],int n){    printf("Noi Dung Mang: ");    for (int i=0;i<n;i++)            printf("%d\t",a[i]);}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];                        break;        }        return flag;                }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];                }    }        return tong;} void main(){     int a[10],n,pt;    NhapMang(a,n);    XuatMang(a,n);    //PhantuCuoi(a,n);        printf("%d",PhantuCuoi(a,n));    printf("Kaka %d",NguyenTo(a,n));        getch();}

Mọi người xem qua đoạn code ,thử không biết mình viết sai chỗ nào.Khi xuất mảng thì in ra,nhưng khi đến số nguyên tố thì không in ra,mà ăn ra địa chỉ.
cảm ơn

[nhc1987]

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 :(.

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
}

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.