Là lỗi đàn áp thực hành xấu?

Trong một câu hỏi SO tôi đã hỏi ở đây về một số mã mà tôi không chắc chắn, có người đã trả lời "BTW, mã khủng khiếp ở đó: nó sử dụng biểu tượng triệt tiêu lỗi (@) rất nhiều."

Có một lý do tại sao điều này là thực hành xấu? Với những thứ như:

$db=@new mysqli($db_info) or die('Database error');

, nó cho phép tôi chỉ hiển thị một thông báo lỗi tùy chỉnh. Nếu không có lỗi, thì nó vẫn hiển thị thông báo PHP điển hình của:

Cảnh báo : mysqli :: mysqli (): php_network_getaddresses: getaddrinfo không thành công: Không có máy chủ nào như vậy được biết đến. trong một số \ file \ path trên dòng 6

cũng như 'Lỗi cơ sở dữ liệu'.

Là lỗi triệt tiêu luôn luôn xấu, và nếu vậy, những gì cụ thể về những điều trên là xấu?

Cập nhật: mã thực tế mà tôi đang sử dụng là:

or error('Datatabase error', 'An error occurred with the database' . (($debug_mode) ? '<br />MySQL reported: <b>' . $db->error . '</b><br />Error occurred on line <b>' . __LINE__ . '</b> of <b>' . __FILE__ . '</b>' : ''))

trong đó loại bỏ tất cả đầu ra trước đó và hiển thị một thông báo lỗi. Vì vậy, thực tế là thông báo lỗi không bao gồm chi tiết về những gì đã xảy ra cụ thể (mà mọi người dường như đang đề xuất là lý do tại sao việc khắc phục lỗi là xấu) là không liên quan.

Tôi nghĩ rằng bạn đang làm điều đúng đắn bằng cách khắc phục lỗi, bởi vì bạn đang thực hiện xử lý lỗi của riêng mình.

Có thể dễ dàng hơn để xem xét tương đương trong, giả sử, Java. Loại bỏ lỗi sử dụng @tương tự như nuốt một ngoại lệ. Mã sau đây, tương tự như mã của bạn, là hợp lý:

try {
    db = new MySQLi(dbInfo);
} catch (SQLException connectionFailure) {
    die("Database error");
}

(Chà, trong Java, bạn sẽ không muốn công cụ servlet chết, nhưng bạn có ý tưởng.)

Tuy nhiên, điều này không được chấp nhận:

try {
    db = new MySQLi(dbInfo);
} catch (Exception e) {
}

Hầu hết việc loại bỏ lỗi PHP được thực hiện một cách bất cẩn, tương tự như ví dụ sau, do đó phản ứng giật đầu gối đối với mã của bạn.

Lỗi ức chế là xấu, bởi vì nó không chỉ ẩn thông tin mà bạn đã biết ( một cái gì đó đã đi sai). Nó cũng có thể ẩn thông tin quan trọng để gỡ lỗi mà bạn không biết ( cái gì , ở đâu và tại sao ).

Trong ví dụ cụ thể này, ăn trưa cơ sở dữ liệu lỗi Lỗi không phải là một thông báo lỗi rất tốt. Đã xảy ra lỗi với DB. Không giác ngộ lắm. Các “ Cảnh báo: mysqli :: mysqli (): php_network_getaddresses: getaddrinfo thất bại: Không có máy chủ như vậy được biết đến. trong một số \ file \ path trên dòng 6, thực sự là một thông báo lỗi tốt hơn. Nó cung cấp cho bạn một vị trí và lý do cho vấn đề. Bạn có thể nhảy ngay vào và bắt đầu gỡ lỗi, vì lỗi đã được định vị.

Tất nhiên, các nhà thiết kế thư viện đôi khi có xu hướng đưa ra nhiều cảnh báo không liên quan. Tôi không biết PHP đủ rõ để biết liệu nó có cách nào để lọc các cảnh báo mà bạn không quan tâm hay không. Tôi biết rằng việc đọc qua một ngăn xếp hàng trăm dòng không phải là rất sáng tỏ. Nhưng phản ứng chính xác với quá nhiều thông tin không phải là giảm lượng thông tin về 0 , mà là lọc ra những phần không liên quan ở một số giai đoạn. Các@ nhà điều hành không có granularity này (đó là phương châm là tất cả hoặc không có gì ), và do đó không phải là một công cụ phù hợp để quản lý lỗi hiệu quả.

Có một số trường hợp bạn biết rằng một lỗi nào đó sẽ xuất hiện và việc khắc phục sự cố thực tế sẽ tốn kém hơn so với việc tắt tiếng tin nhắn (và bị ảnh hưởng từ đó). Đây có thể là trường hợp trong một one-off kịch bản, nhưng gắn bó ngón tay của bạn vào tai của bạn và sẽ “ la la la ” là không một phản ứng chuyên nghiệp để lỗi (tiềm năng).

Việc khắc phục lỗi là xấu vì nó che giấu vấn đề, điều mà nhiều lần chúng ta thậm chí không nhận ra và nó có thể dẫn đến hành vi bất ngờ có thể chứng minh rất tốn kém trong một số ứng dụng quan trọng, ví dụ như trong các ứng dụng được sử dụng trong lĩnh vực tài chính, y tế và quốc phòng.

Nó cũng không phải là một ý tưởng tốt để xử lý các ngoại lệ trong mã và hiển thị các thông báo lỗi thực tế cho người dùng vì nó có thể dẫn đến các vấn đề bảo mật vì thông báo lỗi thường tiết lộ rất nhiều về mã của bạn, điều này có thể giúp người dùng và tin tặc độc hại thao túng hệ thống.

Vì vậy, cách tốt nhất là xử lý các lỗi ở các cấp độ khác nhau, ví dụ ở cấp cao nhất, bạn có thể xử lý lỗi bằng cách chỉ ghi lại lỗi thực tế và hiển thị một thông báo đơn giản và thân thiện với người dùng.

Có, toán tử triệt tiêu lỗi nói chung là một ý tưởng tồi.

Bạn nên quản lý báo cáo lỗi trong cấu hình ( php.ini). Vì vậy, bạn có thể chọn một cài đặt khác nhau cho từng môi trường (ví dụ: để lại các cảnh báo trong quá trình phát triển và ẩn chúng trong sản xuất).

Tình huống duy nhất khi sử dụng @có thể có ý nghĩa là khi bạn phát triển thư viện cho các nhà phát triển khác. Nếu bạn tự nguyện chọn làm điều gì đó tạo ra cảnh báo và không muốn làm phiền người dùng thư viện của bạn bằng một cảnh báo không phụ thuộc vào họ, @có thể là một giải pháp.

Tôi không thể nghĩ ra cách sử dụng nào khác.