Changeset 21582 for branches/version-2_12-dev/data/class/helper/SC_Helper_HandleError.php

Timestamp:

2012/03/04 04:20:00 (14 years ago)

Author:

Seasoft

Message:

#1633 (エラーハンドリングの改善)
#1676 (ログファイルを分離する)
#1677 (デバッグログの出力設定と画面へのエラー出力設定とを分離する)
#1678 (SQL の実行ログを提供する)
#1613 (typo修正・ソース整形・ソースコメントの改善)

File:

• branches/version-2_12-dev/data/class/helper/SC_Helper_HandleError.php (modified) (7 diffs)

branches/version-2_12-dev/data/class/helper/SC_Helper_HandleError.php

@@ -25 +25 @@
  * エラーハンドリングのクラス
  *
- * @package Page
+ * 依存するクラスに構文エラーがあると、捕捉できない。よって、依存は最小に留めること。
+ * 現状 GC_Utils_Ex(GC_Utils) に依存しているため、その中で構文エラーは捕捉できない。
+ * @package Helper
  * @version $Id$
  */
@@ -87 +89 @@
         }
 
+        $error_type_name = GC_Utils_Ex::getErrorTypeName($errno);
+
         $now = date('Y/m/d H:i:s');
         // 本来 realpath() で正規化したいところだが、NULL を返すケースがあるため避けている (#1618)
@@ -92 +96 @@
         switch ($errno) {
             case E_USER_ERROR:
-                $message = $now . " [$errfile:$errline] FATAL Error($errno) $errstr from ". $_SERVER['REMOTE_ADDR'] . "\n";
-                error_log($message, 3, $log_file_path);
-
-                self::displaySystemError($errstr);
+                $message = "Fatal error($error_type_name): $errstr on [$errfile($errline)]";
+                GC_Utils_Ex::gfPrintLog($message, ERROR_LOG_REALFILE, true);
+
+                SC_Helper_HandleError_Ex::displaySystemError($message);
                 exit(1);
                 break;
@@ -103 +107 @@
             case E_CORE_WARNING:
             case E_COMPILE_WARNING:
-                $message = $now . " [$errfile:$errline] WARNING($errno) $errstr from ". $_SERVER['REMOTE_ADDR'] . "\n";
-                error_log($message, 3, $log_file_path);
+                $message = "Warning($error_type_name): $errstr on [$errfile($errline)]";
+                GC_Utils_Ex::gfPrintLog($message, ERROR_LOG_REALFILE);
                 return true;
 
@@ -126 +130 @@
      */
     static function &_fatal_error_handler(&$buffer) {
-        if (preg_match('/<b>(Fatal) error<\/b>: +(.+) in <b>(.+)<\/b> on line <b>(\d+)<\/b><br \/>/i', $buffer, $matches)) {
+        if (preg_match('/<b>(Fatal error)<\/b>: +(.+) in <b>(.+)<\/b> on line <b>(\d+)<\/b><br \/>/i', $buffer, $matches)) {
             $now = date('Y/m/d H:i:s');
-            // 本来 realpath() で正規化したいところだが、NULL を返すケースがあるため避けている (#1618)
+            // realpath() で正規化したいが、NULL を返すケースがあるため避けている (#1618)
             $log_file_path = DATA_REALDIR . 'logs/site.log';
-            $message = $now . " [$matches[3]:$matches[4]] FATAL Error: $matches[2] from ". $_SERVER['REMOTE_ADDR'] . "\n";
-            error_log($message, 3, $log_file_path);
+            $message = "$matches[1]: $matches[2] on [$matches[3]($matches[4])]";
+            GC_Utils_Ex::gfPrintLog($message, ERROR_LOG_REALFILE, true);
             if (DEBUG_MODE !== true) {
                 $url = HTTP_URL . 'error.php';
-                if (defined('ADMIN_FUNCTION') && ADMIN_FUNCTION) {
+                if (defined('ADMIN_FUNCTION') && ADMIN_FUNCTION === true) {
                     $url .= '?admin';
                 }
@@ -174 +178 @@
         }
 
-        $errstr = "[{$arrError[file]}:{$arrError[line]}] FATAL Error({$arrError[type]}) {$arrError[message]}";
-
-        // ログの書き出し
-        $now = date('Y/m/d H:i:s');
+        $error_type_name = GC_Utils_Ex::getErrorTypeName($arrError['type']);
+        $errstr = "Fatal error($error_type_name): {$arrError[message]} on [{$arrError[file]}({$arrError[line]})]";
+
         // 本来 realpath() で正規化したいところだが、NULL を返すケースがあるため避けている (#1618)
         $log_file_path = DATA_REALDIR . 'logs/site.log';
-        error_log($now . " $errstr from ". $_SERVER['REMOTE_ADDR'] . "\n", 3, $log_file_path);
+        GC_Utils_Ex::gfPrintLog($errstr, ERROR_LOG_REALFILE, true);
 
         // エラー画面を表示する
-        self::displaySystemError($errstr);
+        SC_Helper_HandleError_Ex::displaySystemError($errstr);
     }
 
@@ -215 +218 @@
         $objPage->init();
         if (isset($errstr)) {
-            $objPage->arrDebugMsg[]
-                = "▼▼▼ エラーメッセージ ▼▼▼\n"
-                . $errstr . "\n"
-                . "▲▲▲ エラーメッセージ ▲▲▲\n"
-            ;
+            $objPage->addDebugMsg($errstr);
         }
         $objPage->process();