소스 코드 오염없이 로깅을 소개합니다

Question

이 질문은 당분간 머리가 고개를 들고 있습니다 ... 로깅을 유용하게 사용하려면 코드에 모든 것이 있어야하지만 코드를 읽기가 어렵게 만듭니다. 다음 코드와 같습니다 :

public IDictionary<decimal, Status> GetStatus(decimal[] keys) 
{ 
    _logger.Debug("ENTERED GetStatus"); 

    IDictionary<decimal, Status> statuses = new Dictionary<decimal, Status>(); 
    string inClause = null; 

    inClause = FormatInClause(keys, inClause); 
    _logger.DebugFormat(" inClause: '{0}' ", inClause); 
    if (string.IsNullOrEmpty(inClause)) 
    { 
     _logger.Error("Key collection is null or empty."); 
     throw new Exception("Key collection is null or empty."); 
    } 

    if (!IsOpen) 
     Connection.Open(); 

    using (IDbCommand cmd = Connection.CreateCommand()) 
    { 
     cmd.CommandText = " select id, date, status " + 
      " from ORDERS where id in (" + inClause + ") "; 

     inClause = null; 

     using (IDataReader reader = cmd.ExecuteReader()) 
     { 
      int i = 0; 
      while (reader.Read()) 
      { 
       object[] values = new object[reader.FieldCount]; 

       reader.GetValues(values); 
       DebugHelper.LogValues(_logger, " reader.Read() #" + i + " reader.GetValues(values): ", values); 

       statuses[(decimal)values[0]] = new Status(
        (decimal)values[0], 
        ValueOrDefult<string>(values[1]), 
        ValueOrDefult<string>(values[2]), 
        (decimal)values[3], 
        ValueOrDefult<DateTime>(values[4])); 
       _logger.DebugFormat(" reader.Read() #{0} created new Status() ", i); 

       values = null; 
       i++; 
      } 
     } 
    } 

    _logger.Debug("EXITED GetStatus"); 
    return statuses; 
} 

소스 코드의 가독성을 떨어 뜨리지 않기위한 전략이 있습니까?

Answer 1

Aspect oriented programming은 로깅과 같이 cross-cutting concerns을 도와야합니다. postsharp하지만 더 전통적인 방법에 의지하지 않으면 로그 된 내용에 대해 아주 세부적으로 제어 할 수 없습니다.

Answer 2

소스 코드가 나에게 잘 어울립니다. 실제로 로그 메시지를보고 각 메시지 2 개가 무엇인지 파악할 수 있기 때문에 사실 가장 좋습니다.

은 비록 한 가지 _ 그것은 참으로 의, 날 귀찮게한다_logger에.

일부 로깅 API는 같은 단축 API를 제시하는 경향이 :

방법의 위 또는 당신의 방법

l.d("debug") 
l.c("critical") 
...etc 

모두 좋은 이럴입니다.

편집

당신은 아직도 그것에 대해 뭔가를하고 싶은 경우, 단지 #regions에 로깅 라인을 감아을 축소합니다.

Answer 3

몇 가지 규칙을 따를 수 있습니다.

1) 실제로 "상대하고있는"오류 만 기록하십시오.

2) 모든 메소드의 시작과 종료시 디버깅 문을 사용할 필요가 없도록 AOP를 사용하여 메소드를 래핑하십시오. 또한 AOP 호출에 메소드의 들어오는 및 나가는 매개 변수/응답을 기록하게 할 수 있습니다. 코드가 너무 너무 때문에 로깅 이럴 측면 위버 같은 PostSharp

Answer 4

봐는 복잡하다. 솔리드 원칙을 읽어야합니다.

예를 들어 독자 코드를 별도의 메소드로 이동하십시오.

Answer 5

에서