30/09/2018, 18:40
Nhờ mọi người xem và nhận xét giúp code của em. Em cảm ơn
#include <stdio.h>
#include <stdlib.h>
int main()
{
char ten[28] = "";
int a;
int b = 6;
int d;
int e = -5;
int f;
int h = 36;
char c;
printf("Ten cua ban la:");
int i = 0;
do{
c = getchar();
ten[i] = c;
i++;
}
while (c != '
');
printf("Xin chao %sBat dau nao!
",ten);
printf("(9:3)x2=");
scanf("%d",&a);
while( a != b){
printf("Sai roi,Lam lai nao
(9:3)x2=");
scanf("%d",&a);
}
printf("Tot lam,tiep tuc nao
");
printf("3-5x12+52=");
scanf("%d",&d);
while( d != e){
printf("Sai roi,Lam lai nao
3-5x12+52=");
scanf("%d",&d);
}
printf("Tot lam,tiep tuc nao
");
printf("9:[2x4-(-1)]x8:2=");
scanf("%d",&f);
while(f != h){
printf("Sai roi,Lam lai nao
3-9:[2x4-(-1]x8:2=");
scanf("%d",&f);
}
printf("Xin chuc mung ban da hoan thanh 3 cau hoi");
return 0;
}
Bài liên quan
Bôi đen code và nhấn Ctrl + K để định dạng lại code đi bạn.
Mình nhìn lướt qua thì nhận xét là mình ko hiểu biến a, b, c, d, e ,f … của bạn dùng làm gì cả, nên không muốn đọc tiếp.
Bạn cố gắng dùng tên biến có ý nghĩa chứ đừng dùng a,b,c,d,e,f như thế sau này học hướng đối tượng sẽ rất là mệt.
@kyllynk,vâng em cảm ơn ạ
@nguyenchiemminhvu các biến a,b,d,e,f,h dùng để xác định đúng sai,còn biến c dùng để nhập chuỗi từ bàn phím ạ
b, e, f về cơ bản thì ko cần khai báo đâu em. Bởi vì e biết nó là gì ngay từ đầu rồi, ở phần condition của if e đặt luôn là giá trị của nó cho nhanh. Đừng tạo nhiều biến quá quản lý sẽ rất khó.
Theo mình thì bạn nên dùng hàm
gets(ten)để nhập tên. Chương trình càng ngắn gọn thì sửa lỗi dễ hơn, thực thi nhanh hơn(hầu hết).@kyllynk@ngvandung ,vâng, em cảm ơn ạ