[C++] The most stupid C bug ever

[07520004]

Dạo blog tình cờ thấy cái này

Các bạn thử khoan đọc bài blog đầy đủ mà nhìn thử xem đoạn code này có lỗi gì?

else if (code == 200) {     // Downloading whole file
    /* Write new file (plus allow reading once we finish) */
    // FIXME Win32 native version fails because Microsoft's version of tmpfile() creates the file in C:\
   g = fname ? fopen(fname, "w+") : tmpfile();
}

Đoạn code này không chạy trong khi đoạn dưới đây chạy:

else if (code == 200) {     // Downloading whole file
    /* Write new file (plus allow reading once we finish) */
    // FIXME Win32 native version fails because Microsoft's version of tmpfile() creates the file in C:\
    //g = fname ? fopen(fname, "w+") : tmpfile();
    if(NULL != fname) {
        g = fopen(fname, "w+");
    else {
        g = tmpfile();
    }
}

Và đây là bài miêu tả đầy đủ hoàn cảnh ra đời của cái bug

http://www.elpauer.org/?p=971

[08520016]

Cái này là do sai sót của coder chứ bộ.
Cùng vì cái này mà người ta cũng khuyên thêm hạn chế dùng macro.

[07520004]

Cái này là do sai sót của coder chứ bộ.
Cùng vì cái này mà người ta cũng khuyên thêm hạn chế dùng macro.

Thì sai sót của người lập trình, nhưng cụ thể là sai sót gì?

Lỗi này có liên quan gì đến macro đâu, mà trong trường hợp này lập trình viên cũng đâu có dùng macro ?

[08520557]

Theo em thấy thì cái hàm:
g = fname ? fopen(fname, "w+") : tmpfile(); sẽ luôn gọi hàm [fopen (fname,"W+")] vì g= fname luôn có giá trị true.
Do đó, nếu fname chưa được khỏi tạo thì hàm fopen (fname,"W+") có thể bị lỗi.

[07520004]

Theo em thấy thì cái hàm:
g = fname ? fopen(fname, "w+") : tmpfile(); sẽ luôn gọi hàm [fopen (fname,"W+")] vì g= fname luôn có giá trị true.
Do đó, nếu fname chưa được khỏi tạo thì hàm fopen (fname,"W+") có thể bị lỗi.

Chúng mừng em đã sai phép ? : được tính trước phép = tính sau cùng. Thử lại lần nữa đi em.

[08520016]

èo do cái kí tự nối dòng '\' mà cuối câu comment của coder đó có C:\ .
Còn người ta khuyên hạn chế dùng macro là em nói thêm 1 kinh nghiệm thui.

[07520004]

èo do cái kí tự nối dòng '\' mà cuối câu comment của coder đó có C:\ .
Còn người ta khuyên hạn chế dùng macro là em nói thêm 1 kinh nghiệm thui.

Em tìm ra cái này trước hay sau khi đọc bài blog vậy
Đọc được cái blog đó thì cũng khá đó.

Còn macro thì cứ xài mạnh thôi, nó là thay thế code trực tiếp trước khi compile nên cho code tối ưu nhất. Khi cần viết các đoạn code có độ thực thi nhanh người ta sẽ tận dụng nó. Nếu không nên xài thì hẳn người ta bỏ từ lâu rồi =)). C vẫn còn được tiếp tục phát triển và cập nhật các chuẩn mới nên chắc nó... khó mà chứa các tính năng thừa lắm.

[08520016]

Mấy bài dạng này em đã làm qua hết rồi (khi còn làm mod C/C++ congdongcviet) ,ở các topic bên cviet: Thữ Giãn với lập trình,coder quán.....

Em tìm ra cái này trước hay sau khi đọc bài blog vậy
Đọc được cái blog đó thì cũng khá đó.

Còn macro thì cứ xài mạnh thôi, nó là thay thế code trực tiếp trước khi compile nên cho code tối ưu nhất. Khi cần viết các đoạn code có độ thực thi nhanh người ta sẽ tận dụng nó. Nếu không nên xài thì hẳn người ta bỏ từ lâu rồi =)). C vẫn còn được tiếp tục phát triển và cập nhật các chuẩn mới nên chắc nó... khó mà chứa các tính năng thừa lắm.

Người ta kêu hạn chế chứ hok kêu bỏ .
Sử dụng Macro 1 cách lạm dụng làm phá vỡ cấu trúc chương trình , quá trình bảo trì và phát triển gây khó khăn . Vì dùng macro thay hàm thường sẽ sử dụng rất nhiều dấu nối dòng '\' ,làm cho code không trong sáng.
Thường macro chỉ để dùng viết thay cho những đoạn code ngắn.

[07520004]

Mấy bài dạng này em đã làm qua hết rồi (khi còn làm mod C/C++ congdongcviet) ,ở các topic bên cviet: Thữ Giãn với lập trình,coder quán.....

Người ta kêu hạn chế chứ hok kêu bỏ .
Sử dụng Macro 1 cách lạm dụng làm phá vỡ cấu trúc chương trình , quá trình bảo trì và phát triển gây khó khăn . Vì dùng macro thay hàm thường sẽ sử dụng rất nhiều dấu nối dòng '\' ,làm cho code không trong sáng.
Thường macro chỉ để dùng viết thay cho những đoạn code ngắn.

Excellently done! Câu đố được giải sớm hơn mình nghĩ

[09520396]

tauit_dnmd là trâu về C/C++ lắm. Nổi tiếng chứ chả phải đùa đâu. Thông tin này đã được kiểm chứng.

Mấy bài dạng này em đã làm qua hết rồi (khi còn làm mod C/C++ congdongcviet) ,ở các topic bên cviet: Thữ Giãn với lập trình,coder quán.....

Người ta kêu hạn chế chứ hok kêu bỏ .
Sử dụng Macro 1 cách lạm dụng làm phá vỡ cấu trúc chương trình , quá trình bảo trì và phát triển gây khó khăn . Vì dùng macro thay hàm thường sẽ sử dụng rất nhiều dấu nối dòng '\' ,làm cho code không trong sáng.
Thường macro chỉ để dùng viết thay cho những đoạn code ngắn.