Bài tập menu, ngôn ngữ C?

[Aptechite]

Cái bài menu này em làm đã hai hôm rồi mà vẫn chưa ổn. Sau rất nhiều lần sửa của em thì các phần khác chắc là ok rồi (em nghĩ là ok rồi nhưng vẫn chưa kiểm tra kĩ) nhưng riêng cái hàm tìm số lẻ lớn nhất đến giờ em vẫn không biết là sai ở đâu, mà nó toàn in ra những giá trị "trời ơi". Đôi khi sau khi nhập xong một lượt lại còn không nhập được tiếp nữa mới chết chứ(Chắc vấn đề không chỉ nằm ở hàm tìm số lẻ lớn nhất).

#include <stdio.h>#include <conio.h>#include <stdlib.h> //1 Nhap vao cac phan tu cua mangint *nhapso(int n,int *p){int i;p=(int*)malloc(n*sizeof(int));for(i=0;i<n;i++)   {      printf("\na[%d]=",i);      scanf("%d",p+i);   } return (p);}//2 chuyen doivoid modify(int n,int *p){    int change;int i;    printf("\nBan muon thay doi phan tu thu may: ");    scanf("%d",&i);    if((i>0)&&(i<n))    {        printf("\nBan muon chuyen phan tu %d thanh so may: ",p[i]);        scanf("%d",&change);        p[i]=change;    }    else printf("\nNhap lai so can doi");}//3 Tim so le lon nhatvoid maxodd(int n,int *p){int i, max=0,j=0,*tmp;for (i=0;i<n;i++)if((p[i]%2)!=0)      {         tmp[j]=p[i];         j++;      }if((tmp[0]%2)!=0)  {  max=tmp[0];  for(j=0;j<=i;j++)      {          if(max<p[j])          max=p[j];      }    printf("\nSo le lon nhat la: %d",max);    getch();  }else    printf("\nMang khong co phan tu le");    getch();}//4 tong cua tat ca cac phan tu trong mangint sum (int n,int *p){int sum=0; int i;for(i=0;i<n;i++)    sum=sum+p[i];return sum;}//5 tong cua tat ca cac so chanint sumeven(int n,int *p){int sumeven=0,i;for(i=0;i<n;i++)  {    if((p[i]%2)==0)    sumeven =sumeven+p[i];  }return sumeven;}//6 tong cua tat ca cac so nguyen toint prime(int n){int ok=0; int i; for(i=2;i<n;i++)  if((n%i)==0)    {      ok=1;      break;    }if(ok==1) return 1;//not prime number else return 0;//prime number}int sumprim(int n,int *p){int i,sumprim=0;for(i=0;i<n;i++)    if(prime(p[i])==0) sumprim=sumprim+p[i];return sumprim;}void main(){int n, *p,sl,exit=0;   do{        clrscr();        printf("\n\n  Menu");        printf("\n\n1.Nhap vao mot mang so nguyen");        printf("\n\n2.Chuyen doi mot phan tu cua mang");        printf("\n\n3.Tim so le lon nhat");        printf("\n\n4.Tim tong cua tat ca phan tu cua mang");        printf("\n\n5.Tim tong cua cac so chan");        printf("\n\n6.Tim tong cua cac so nguyen to");        printf("\n\n7.Exit\n");        scanf("%d",&sl);        switch(sl){        case 1: printf("\nSo phan tu can nhap: ");                scanf("%d",&n);                p=nhapso(n,p);                break;        case 2: modify(n,p);                break;        case 3: maxodd(n,p);                break;        case 4: sum(n,p);                printf("\nTong cua tat ca cac phan tu cua mang la: %d",sum(n,p));                getch();                break;        case 5: sumeven(n,p);                printf("\nTong cua tat ca cac so chan la: %d",sumeven(n,p));                getch();                break;        case 6: sumprim(n,p);                printf("\nTong cua tat ca cac so nguyen to la: %d",sumprim(n,p));                getch();                break;        case 7: exit=1; break;        default: printf("\nBan hay chon lai tu 1-7");  getch();        }   }while(exit!=1);}

[Xcross87]

Cái này của bạn , mình trích ra để phân tích nè :

//3 Tim so le lon nhatvoid maxodd(int n,int *p){int i, max=0,j=0,*tmp;for (i=0;i<n;i++)if((p[i]%2)!=0)      {         tmp[j]=p[i];         j++;      }if((tmp[0]%2)!=0)  {  max=tmp[0];  for(j=0;j<=i;j++)      {          if(max<p[j])          max=p[j];      }    printf("\nSo le lon nhat la: %d",max);    getch();  }else    printf("\nMang khong co phan tu le");    getch();}

1. Bạn để ý nhé : bạn lấy các phần tử lẻ gán vào mảng *tmp : >> sau khi gán hết thì >> tmp[] chỉ có j-1 phần tử
toàn số lẻ nên câu if : if ( ( tmp[0]%2) != 0 ) === kô cần thiết
2. Vì chỉ có j phần tử nên vòng for chỉ chạy từ 0 -> j-1

Có gì sai sửa giùm nha ^o^!

[PoPoPoPo]

Có thể các nhận xét của Pete là đúng rồi. Nhưng mình có góp ý như thế này.
Thao tác thứ nhất:
for (i=0;i<n;i++)
if((p[i]%2)!=0)
{
tmp[j]=p[i];
j++;
}
là để lọc ra mảng số lẻ, nếu mảng này không có phần tử nào thì trả lại giá trị False cho hàm (Không có phần tử lẻ.)
Việc thứ hai, sau khi có mảng các số lẻ thì việc tìm phần tử max của mảng (j+1) phần tử là hoàn toàn đơn giản.
for(i=0;i<=j+1;i++)
{
if(max<tmp[i])
max=temp[i];
}
Và cuối cùng là in ra phần tử lẻ lớn nhất là max.

Cái chương trình của bạn mình thấy rườm ra quá à.

[Aptechite]

toàn số lẻ nên câu if : if ( ( tmp[0]%2) != 0 ) === kô cần thiết

Cái phần này là để phòng trường hợp mảng nhập vào toàn số chẵn. Nhưng vấn đề chính ở đây là em không hiểu tại sao cái hàm tìm số lẻ lớn nhất toàn in ra những giá trị lăng nhăng, không biết là do chương trình em viết bị sai hay là cái
Turbo C++ 3.0 của em nó bị trục trặc nhỉ?

[Xcross87]

Mình viết hàm khác để tìm số lẻ lớn nhất đây :

void maxodd(int list[], int quantity){     int i, j = 0,oddarray[100] ,max = 0;     // assign odd value     for ( i = 0 ; i < quantity ; i++ )         if ( list[i] % 2 != 0 )            oddarray[j++] = list[i];     // find max      for ( i = 0 ; i < j ; i++ )         if ( oddarray[i] > max ) max = oddarray[i];     // result     if ( max == 0 ) printf("\n Mang ko co phan tu le .. ");     else printf("\n So le lon nhat = %d " , max );}

Giá trị max = 0 là default , nếu ko có phần tử lẻ thì nó không thay đổi vì chẳng được gán cho cái gì cho nên mình mới nói :

        if ( tmp[0] %2 != 0 )

là kô cần thiết .

Khi bạn đã lọc ra được các phần tử lẻ rồi thì không cần quan tâm phần tử chẵn làm gì ^o^

[Xcross87]

Thêm nữa : trong hàm của bạn kô thấy giá trị khởi tạo cho tmp[] >> thành ra nếu như cả mảng p[] toàn số chẵn >> tmp[0] = cái số to đùng nếu xài Dev-C++ nếu xài borland chắc có warning về initializing .

Nhìn kĩ hơn vào bài của bạn mình thấy cái vô lý :
Bạn lọc ra các phấn tử lẻ vào tmp[] nhưng sau đó để lọc ra số lớn nhất bạn dùng phép so sánh

 if(max<p[j])
		  max=p[j];

>> so sánh luôn cả số chẵn trong mảng ban đầu ?_?

Mình chỉ góp ý vậy thôi . Chúc bạn sớm complete nha ^0^