Nhờ mọi người xem và nhận xét giúp code

#1 
#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 != '\n');
    printf("Xin chao %sBat dau nao!\n",ten);
    printf("(9:3)x2=");
    scanf("%d",&a);
    while( a != b){
        printf("Sai roi,Lam lai nao\n(9:3)x2=");
        scanf("%d",&a);
    }
    printf("Tot lam,tiep tuc nao\n");

        printf("3-5x12+52=");
        scanf("%d",&d);
        while( d != e){
            printf("Sai roi,Lam lai nao\n3-5x12+52=");
            scanf("%d",&d);
        }
        printf("Tot lam,tiep tuc nao\n");
        printf("9:[2x4-(-1)]x8:2=");
        scanf("%d",&f);
        while(f != h){
            printf("Sai roi,Lam lai nao\n3-9:[2x4-(-1]x8:2=");
            scanf("%d",&f);
        }
        printf("Xin chuc mung ban da hoan thanh 3 cau hoi");

    return 0;
}
#2

Bôi đen code và nhấn Ctrl + K để định dạng lại code đi bạn.

3 Likes
#3

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.

3 Likes
#4

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.

1 Like
#5

@kyllynk,vâng em cảm ơn ạ

#6

@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 ạ

#7

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ó. :smiley:

#8

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).

#9

@kyllynk@ngvandung ,vâng, em cảm ơn ạ

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