From 410dc2d04dbecf019129cd8cd65a3be2c62b4d0c Mon Sep 17 00:00:00 2001
From: ulysseskao <ulysseskao@gmail.com>
Date: Tue, 03 May 2016 18:36:41 +0800
Subject: [PATCH] refator code

---
 CCSTrace/CCS/EventAI/JudgeCase.cs |   39 +++++++++++++++++----------------------
 1 files changed, 17 insertions(+), 22 deletions(-)

diff --git a/CCSTrace/CCS/EventAI/JudgeCase.cs b/CCSTrace/CCS/EventAI/JudgeCase.cs
index 447c7d3..c7d92ca 100644
--- a/CCSTrace/CCS/EventAI/JudgeCase.cs
+++ b/CCSTrace/CCS/EventAI/JudgeCase.cs
@@ -22,7 +22,7 @@
         private int _inferFsc; // 推測出可能受損的設備種類
         private int _inferUfid; // 推測出可能受損的設備編號
         private int _parentCaseId = 0;
-        private Equipment _mEquip;
+        private Equipment _equip;
 
         private readonly OracleConnection _connectionTpc;
         private readonly OracleTransaction _transaction;
@@ -30,13 +30,13 @@
 
         // int Range = 360; //比事故案件還早多久時間內的案件才作合併 單位:分
 
-        public JudgeCase(int mFdrId, int mFsc, int mUfid, int mCaseId, string mAcceptDate, OracleConnection connection, OracleTransaction trx, string traceConnection)
+        public JudgeCase(int fdrId, int fsc, int ufid, int caseId, string acceptDate, OracleConnection connection, OracleTransaction trx, string traceConnection)
         {
-            _damageFdrId = mFdrId;
-            _damageFsc = mFsc;
-            _damageUfid = mUfid;
-            _newCaseId = mCaseId;
-            _acceptDate = mAcceptDate;
+            _damageFdrId = fdrId;
+            _damageFsc = fsc;
+            _damageUfid = ufid;
+            _newCaseId = caseId;
+            _acceptDate = acceptDate;
 
             _connectionTpc = connection;
             _transaction = trx;
@@ -82,18 +82,15 @@
             }
             catch (Exception e)
             {
-                Logger.Error(e.Message);
-                Console.WriteLine(e.StackTrace);
+                Logger.Error(e, e.Message);
                 Logger.Error("尋找母案件時發生錯誤.");
-                throw e;
+                throw ;
             }
             finally
             {
-                if (command != null)
-                    command.Dispose();
+                command?.Dispose();
 
-                if (reader != null)
-                    reader.Close();
+                reader?.Close();
             }
 
             // 若事故案件為該饋線上的第一件事故案件時,無須再作追蹤比較
@@ -129,7 +126,7 @@
             catch (Exception ex)
             {
                 Logger.Error(ex.Message);
-                throw ex;
+                throw;
             }
             Logger.Info("不同變壓器,開始追蹤(New Version)...");
 
@@ -185,7 +182,7 @@
                                     _parentCaseId = Convert.ToInt32(((ArrayList)_mCase[i])[0].ToString());
                                     _inferFsc = ((Equipment)_newCase[j]).Fsc;
                                     _inferUfid = ((Equipment)_newCase[j]).Ufid;
-                                    _mEquip = (Equipment)((ArrayList)_mCase[i])[5];
+                                    _equip = (Equipment)((ArrayList)_mCase[i])[5];
                                     Logger.Info("與案件" + ((ArrayList)_mCase[i])[7] + "合併,故障點Fsc: " + _inferFsc + " Ufid: " + _inferUfid);
                                     return;
                                 }
@@ -217,7 +214,7 @@
                                     _parentCaseId = Convert.ToInt32(((ArrayList)_mCase[i])[0].ToString());
                                     _inferFsc = damageEquipment.Fsc;
                                     _inferUfid = damageEquipment.Ufid;
-                                    _mEquip = (Equipment)((ArrayList)_mCase[i])[5];
+                                    _equip = (Equipment)((ArrayList)_mCase[i])[5];
                                     Logger.Info("與案件" + ((ArrayList)_mCase[i])[7] + "合併,故障點Fsc: " + _inferFsc + " Ufid: " + _inferUfid);
                                     return;
                                 }
@@ -250,7 +247,7 @@
 
         public Equipment GetOldrEquipment()
         {
-            return _mEquip;
+            return _equip;
         }
 
         public int GetParentCaseId()
@@ -287,8 +284,7 @@
             }
             catch (Exception e)
             {
-                Logger.Error("Problems occur: " + e.Message);
-                Console.WriteLine(e.StackTrace);
+                Logger.Error(e, "Problems occur: " + e.Message);
             }
             finally
             {
@@ -322,8 +318,7 @@
             catch (Exception e)
             {
                 // CCS.GlobalVariable.ErrorLog.setErrorLog("Error in JudgeCase(RealDamageEquipment) :" + e.getMessage());
-                Logger.Error("Problems occur: " + e.Message);
-                Console.WriteLine(e.StackTrace);
+                Logger.Error(e, "Problems occur: " + e.Message);
             }
             finally
             {

--
Gitblit v0.0.0-SNAPSHOT