깨끗한 기능 코드 또는 많은 코드 덩어리

Question

이 질문에서 저는 기본 관행에 대해 물어볼 것입니다. 오늘 나는이 코드로 데이터베이스 테이블 값을 업데이트해야하는 상황에 직면했다.

public void updateUsrProfileName(string usrId, string name) 
    { 
     query = "update [db_user].[dbo].[usr_profiles] set [Name]=@name where [usrid]=@usrid "; 
     try 
     { 
      com = new SqlCommand(query,con); 
      com.Parameters.AddWithValue("@name",name); 
      com.Parameters.AddWithValue("@usrid",usrId); 
      con.Open(); 
      com.ExecuteNonQuery(); 
      con.Close(); 
     } 
     catch (Exception e) 
     { 
      con.Close(); 
      throw e; 
     } 
    } 

나는 테이블의 모든 컬럼에 대한 기능 이상 반복해야합니다, 그래서 내가 코드를

public void Commonfunction(SqlCommand com, string var) 
    { 
     try 
     { 
      com.Parameters.AddWithValue("@usrid", var); 
      con.Open(); 
      com.ExecuteNonQuery(); 
      con.Close(); 
     } 
     catch (Exception e) 
     { 
      con.Close(); 
      throw e; 
     } 
    } 

을 줄이고이

 public void updateUsrProfileName(string usrId, string name) 
    { 
     query = "update [db_user].[dbo].[usr_profiles] set [Name]=@name where [usrid]=@usrid "; 
     try 
     { 
      com = new SqlCommand(query,con); 
      com.Parameters.AddWithValue("@name",name); 
      Commonfunction(SqlCommand com, string name); 
     } 
      catch (Exception e) 
      { 
       con.Close(); 
       throw e; 
      } 
     } 

같은 기능 이상 전화를 하나 개의 공통 기능을 마련 장점 : - 중복 코드가없는 깨끗한 코드.

단점 차이가 매우 적은 적은 코드 길이 : - 함수의 어떤으로 호출은 매우 적은 코드가 어디 있는지지도 기능은이 상황에서 등

호출, 성능이 인수와 일치해야함에 따라 감소 증가 이중화는 공통 함수로 이동하는 것이 좋으며 어떤 조건에서는 함수를 나누는 것에 대해 생각해야한다.

Answer 1

항상 코드를 반복하는 것이 아니라 공유 방법으로 "이동"하십시오. 드물게 프로파일 링이 그 영역에서 문제가 있음을 보여줄 때까지

Answer 2

일반 논리를 별도의 방법으로 입력하는 것이 좋습니다. 과도한 try/catch 블록을 제거하여 코드를 단순화하고 finally 절을 추가했습니다.

public void Commonfunction(SqlCommand com, string var) 
{ 
    try 
    { 
     com.Parameters.AddWithValue("@usrid", var); 
     con.Open(); 
     com.ExecuteNonQuery(); 
    } 
    finally 
    { 
     if (con != null) 
     { 
      con.Dispose(); 
     } 
    } 
} 

public void updateUsrProfileName(string usrId, string name) 
{ 
    query = "update [db_user].[dbo].[usr_profiles] set [Name]=@name where [usrid]=@usrid "; 

    com = new SqlCommand(query, con); 
    com.Parameters.AddWithValue("@name", name); 
    Commonfunction(com, name); 
}