@ngoisaochip wrote:

include "stdafx.h"

include "conio.h"

include "iostream"

include "stdio.h"

struct SinhVien { char strMSSV[9]; char strHoTen[30]; char strNamSinh[5]; float fDiemTrungBinh; }; struct SinhVienNode { SinhVien info; SinhVienNode pNext; }; struct List { SinhVienNode pHead; SinhVienNode pTail; }; void XuatDanhSach(SinhVienNode p); void init(List & L); SinhVienNode getNode(char MSSV[9], char HoTen[30], char NamSinh[5], float DiemTrungBinh); void addFirst(List & l, SinhVienNode p); void addTail(List & l, SinhVienNode p); SinhVienNode insertHead (List & l, char MSSV[9], char HoTen[30], char NamSinh[5], float DiemTrungBinh); SinhVienNode insertTail (List & l, char MSSV[9], char HoTen[30], char NamSinh[5], float DiemTrungBinh); void addAfter(List & l, SinhVienNode q, SinhVienNode p); void main() { List lop18TH01; int a, b; init(lop18TH01); do { printf("Nhap so luong sinh vien muon them: "); scanf_s("%d", &a); } while (a < 1); printf("\n"); SinhVienNode p; char strMSSV[9]; char strHoTen[30]; char strNamSinh[5]; float fDiemTrungBinh; for (int i=1; i<=a; i++) { printf("Sinh vien thu %d:", i); printf("Nhap vao MSSV: "); scanf_s("%s", &p->info.strMSSV); printf("Nhap vao ho va ten: "); scanf_s("%s", &p->info.strHoTen); printf("Nhap vao nam sinh: "); scanf_s("%s", &p->info.strNamSinh); printf("Nhap vao diem trung binh: "); scanf_s("%f", &p->info.fDiemTrungBinh); p = getNode(strMSSV, strHoTen, strNamSinh, fDiemTrungBinh); addTail(lop18TH01, p); } /

_getch(); system("cls"); do { printf("1 \t\n"); printf("2 \t\n"); printf("3 \t\n"); printf("4 \t\n"); printf("5 \t\n"); printf("0 \t\n"); printf("Chon: "); scanf_s("%d", &b); system("cls"); switch (b) { case 1: break; case 2: _getch(); break; case 3: _getch(); break; case 4: _getch(); break; case 5: _getch(); break; }; system("cls"); } while (a != 0);

/ _getch(); } void XuatDanhSach(SinhVienNode p) { printf("%s - %s - %s - %.1f\n", p->info.strMSSV, p->info.strHoTen, p->info.strNamSinh, p->info.fDiemTrungBinh); } void init(List & l) { l.pHead = l.pTail = NULL; } SinhVienNode getNode(char MSSV[9], char HoTen[30], char NamSinh[5], float DiemTrungBinh) { SinhVienNode p = new SinhVienNode; if (p == NULL) return NULL; strcpy_s (p->info.strMSSV, MSSV); strcpy_s (p->info.strHoTen, HoTen); strcpy_s (p->info.strNamSinh, NamSinh); p->info.fDiemTrungBinh = DiemTrungBinh; p->pNext = NULL; return p; } void addFirst(List & l, SinhVienNode p) { if (l.pHead == NULL) l.pHead = l.pTail = p; else{ p->pNext = l.pHead; l.pHead = p; } } void addTail(List & l, SinhVienNode p) { if (l.pHead == NULL) l.pHead = l.pTail = p; else { l.pTail->pNext = p; l.pTail = p; } } SinhVienNode insertHead(List & l, char MSSV[9], char HoTen[30], char NamSinh[5], float DiemTrungBinh) { SinhVienNode p = getNode (MSSV, HoTen, NamSinh, DiemTrungBinh); if (p != NULL) { addFirst (l, p); return p; } return NULL; } SinhVienNode insertTail (List & l, char MSSV[9], char HoTen[30], char NamSinh[5], float DiemTrungBinh) { SinhVienNode p = getNode (MSSV, HoTen, NamSinh, DiemTrungBinh); if (p != NULL) { addTail (l, p); return p; } return p; } void addAfter(List & l, SinhVienNode q, SinhVienNode *p)

{ if (q != NULL) { p->pNext = q->pNext; q->pNext = p; } if (q = l.pTail) l.pTail = p; addFirst(l, p); }

Posts: 4

Participants: 3

Read full topic

Dễ dàng để nhận ra đoạn code trên dùng để tính tổng các số từ 0 tới n. Tuy nhiên khi bạn chạy ở nhiều máy khác nhau hay chạy ở trên CodeLearn với số lần đủ lớn, bạn sẽ thấy kết quả sẽ ko phải luôn luôn giống nhau!

Lỗi xảy ra chính ở việc bạn không khởi tạo giá trị cho biến số a. Theo đó, mỗi một lần chạy, biến a sẽ được hệ điều hành gán cho 1 giá trị ngẫu nhiên. Và nếu giá trị ngẫu nhiên đó không bằng 0, thì chương trình này chắc chắn là chạy sai rồi, đúng không?

Có rất nhiều bạn nói với tôi rằng, khi khởi tạo 1 biến số thì biến đó sẽ nhận 1 giá trị mặc định. Xin thừa rằng, điều ấy phụ thuộc khá nhiều vào trình biên dịch hoặc cờ biên dịch (compiler flag). Có những trình biên dịch tự động cấp phát và khởi tạo biến số nguyên bằng 0, có những trình biên dịch tự động cấp phát và khởi tạo cả một mảng số nguyên gồm những số 0, nhưng có những trình biên dịch mặc kệ, việc khởi tạo hoàn toàn nằm trong tay bạn.

Đặc biệt, trong 1 số đoạn mã liên quan tới mảng hay ma trận, nếu sử dụng mảng hay ma trận mà không khởi tạo giá trị cho mảng kiểu như bool array[100] hay int array[100][100] thì tỉ lệ gặp lỗi của bạn sẽ cao hơn bình thường.

Tại sao C/C++ thường bị lỗi này?

Với Java hay C#, nếu bạn gõ thử đoạn code tương tự, bạn sẽ bị gặp báo lỗi: variable … might not have been initialized (Java) hay Use of unassigned local variable ‘…’ (C#) và buộc phải sửa lỗi này.

Với C hay C++, nếu bạn sử dụng IDE với các tham số mặc định, thì thường lỗi trên sẽ bị trình biên dịch bỏ qua, ko nhìn thấy. Có một số trình biên dịch nếu cấu hình cờ biên dịch chặt hơn một chút, bạn sẽ nhìn thấy warning. Tuy nhiên đa phần coder chúng ta đều mặc kệ warning, và đó là lí do chương trình của bạn chạy không đúng.

Tại sao lại có chuyện lúc thì không warning, lúc thì warning và lúc thì báo lỗi, bạn hãy đọc và hiểu thêm về cờ biên dịch ở phía bên dưới nhé

Cờ biên dịch:

Cờ biên dịch được hiểu là những tuỳ chọn, tham số khi trình biên dịch biên dịch code của các bạn

Xét đoạn mã sau:

`

include <iostream> using namespace std; int main() { int a; cout << a << endl;}`

Bây giờ hãy thử biên dịch đoạn code trên với 2 câu lệnh khác nhau như hình dưới đây:

Sự khác nhau khi biên dịch trong trường hợp có và không có cờ "-Wall"

Như bạn thấy rồi đấy, trong trường hợp có cờ "-Wall", trình biên dịch sẽ bật cảnh báo về việc không khởi tạo biến. Thông thường, trên máy bạn hay IDE của bạn sẽ không bật cờ này, đó là lí do bạn ko phát hiện ra được lỗi không khởi tạo biến của mình.

Tiếp theo hãy xem sự kết hợp của hai cờ "-Wall" và "-Werror" khi biên dịch đoạn mã trên. Bạn sẽ thấy bây giờ trình biên dịch báo lỗi chứ không còn là cảnh báo nữa.

Chi tiết các cờ biên dịch bạn có thể tham khảo ở link sau: //caiorss.github.io/C-Cpp-Notes/compiler-flags-options.html

Kết luận

Giờ thì các bạn có thể hiểu sơ sơ về lỗi không khởi tạo biến và tác hại của nó rồi chứ.

Lỗi này không chỉ gặp với C++, mà có thể 1 số ngôn ngữ khác cũng có thể gặp (với 1 số ngôn ngữ mà mình chưa biết). Do đó, hãy luôn rèn cho mình thói quen khởi tạo 1 giá trị mặc định khi bạn khai báo một biến hay một đối tượng nhé.