Chỉ giáo bài kiểm tra toàn chẵn trong mảng

#1

#include
using namespace std;
int main()
{
int k=5;
int a[5];
cout << "Nhap vao k = ";
cin >> k;

for(int i=0;i<=k;i++)
{
	if(k<0 ||k>5)
     {
	cout <<"khong hop le!";
	
	}
	else
	cout << "Nhap phan tu a[" << i << "]:";
	cin >> a[i];
}
for(int i=0;i<=k;i++)
{
	if(a[i]%2==0)
		cout << "Day so vua nhap toan chan"<<endl;
	else
		cout <<"Khong toan chan !";
	break;
}
return 0;

}
Theo các bạn có cách nào tốt hơn không… không nói phần hàm nhé!!!

#2

Kiểm tra trước khi nhập :))

#3

Đây là bài mình làm:

#include<iostream>

using namespace std;

void nhapmang(int a[100], int n){
for(int i=0;i<n;i++){
     cout<<"nhap phan tu thu a["<<i<<"]";
     cin>>a[i];
 }
}
 int ktchan(int a[100], int n){
    for(int i = 0;i<n;i++)
       if(a[i]%2!=0) return 0;
    return 1;
}

int main(){
     int a[100],n,kt;
do{
     cout<<"nhap so phan tu n=";
     cin>>n;
}while(n<=0);
     nhapmang(a,n);
     kt=ktchan(a,n);
     if(kt==0) cout<<"\nmang co so le"<<endl;
     else cout<<"\nmang toan chan"<<endl;
return 0;

}
#4

Ở đây nếu bạn dùng break thì nó sẽ dừng ngay i đầu tiên vì khi chạy xong lệnh if thì sẽ đến lênh break; rồi i mới tăng lên nên khi tới lệnh break; vòng lặp đã dừng lại. Mình nghĩ sở dĩ bạn sư dụng break vì vòng for của bạn khi chạy in ra qua nhiều dòng thông báo. Nên mình nghĩ dùng cách như trên của mình là tốt hơn.Thân ái!

#5

Duyệt mảng gặp số lẻ thì return kết luận ngay.

#6

Hàm ktchan nên return giá trị bool thay vì int :smiley:

#7

Bạn khai báo mảng tối đa 5 phần tử,nhưng khi nhập k vào cho chạy k phần tử nếu như k>5 thì sẽ bị tràn stack.vậy nên khai báo mảng int a[100] để tránh tràn bộ nhớ,hoặc dùng con trỏ cấp phát động.

#8

thanks tất cả các bạn … tất cả những ý kiến và cachs giải mình xin nhận … với phương châm :"dốt không dấu dốt … đó là câu nói các lập trình viên vẫn thường hay nói… chúc các bạn một ngày vui vẻ

#9

Không có gì bạn. thắc mắc gì cứ hỏi thôi. dạy nhau học mà :smile:

83% thành viên diễn đàn không hỏi bài tập, còn bạn thì sao?