bad coding practice

멤버 정의 순서

클래스 내 모든 멤버는 "이벤트", "필드", "속성", "생성자", "메서드" 순서로 정의

public partial class SA_A2000 : BaseMenu
{
    public SA_A2000()
    {
        InitializeComponent();
        // ...
    {
    
    string strOrdnum = string.Empty;   // 수주번호
    bool isSaveChk = false;            // 저장버튼 클릭시 변경
    bool isMasterChk = false;          // 기본정보 수정 여부
    bool isDetailChk = false;          // 상세정보 수정 여부
    bool isNewChk = false;    // 신규버튼 클릭시 변경
    string ordNumbering = string.Empty; //범한산업 수주번호 채번 구분
    
    protected override void OnLoad(EventArgs e)
    {
        base.OnLoad(e);
        // ...
    }
}

public partial class SA_A2000 : BaseMenu
{
    #region Fields

    private string _OrderNumber = string.Empty;   // 수주번호
    private bool _IsSaveChk = false;            // 저장버튼 클릭시 변경
    private bool _IsMasterChk = false;          // 기본정보 수정 여부
    private bool _IsDetailChk = false;          // 상세정보 수정 여부
    private bool _IsNewChk = false;    // 신규버튼 클릭시 변경
    private string _OrdNumbering = string.Empty; //범한산업 수주번호 채번 구분

    #endregion
    
    public SA_A2000()
    {
        InitializeComponent();
        // ...
    {
    
    protected override void OnLoad(EventArgs e)
    {
        base.OnLoad(e);
        // ...
    }
}

반복해서 참조하며 강제 형변환 예시

private void ButtonEditEx_Click(object sender, EventArgs e)
{
    string WorkType = string.Empty;
    if (((SimpleButtonEx)sender).Equals(btnProd))
    {
        if (((SimpleButtonEx)sender).Tag.Equals("시작"))
            WorkType = "START";
        else if (((SimpleButtonEx)sender).Tag.Equals("종료"))
        {
            if(SessionInfo.ServiceId == "20200108002") // 세원시스템
            {
                WorkType = "END_SW";
            }
            else 
                WorkType = "END";
        }
        Func_P_PR_A3000_S(WorkType);
    }
    else if (((SimpleButtonEx)sender).Equals(btnMacStop))
    {
        if (((SimpleButtonEx)sender).Tag.Equals("비가동시작"))
        {
            if (fnStopEnd())
            {
                btnMacStop.Text = "비가동종료";
                btnMacStop.Tag = "비가동종료";
            };
        }
        else if (((SimpleButtonEx)sender).Tag.Equals("비가동종료"))
        {
            if(Func_P_PR_A3000_S("StopE"))
            {
                btnMacStop.Text = "비가동시작";
                btnMacStop.Tag = "비가동시작";
            }
        }
    }



    int focusplace = gvwList.FocusedRowHandle;
    Func_P_PR_A3000_Q("PLAN");
    if (focusplace == 0 && gvwList.FocusedRowHandle == 0)
    {
        FocusedRowChanged(gvwList);
    }
}

private void ButtonEditEx_Click(object sender, EventArgs e)
{
    SimpleButtonEx simpleButton = sender as SimpleButtonEx;
    
    if (simpleButton.Equals(btnProd))
    {
        //...
    }
    else if (simpleButton.Equals(btnMacStop))
    {
        //...
    }
}

이벤트 핸들러의 반복 사용

동일한 이벤트 핸들러를 사용하며, sender 를 Equals 메서드로 판단

public partial class SY_TEST01 : BaseMenu
{
    InitializeComponent();

    grpTop.CustomButtonClick += GrpTop_CustomButtonClick;
    grpFiles.CustomButtonClick += GrpFiles_CustomButtonClick;
}

private void GrpTop_CustomButtonClick(object sender, DevExpress.XtraBars.Docking2010.BaseButtonEventArgs e)
{
    // ...
}

private void GrpFiles_CustomButtonClick(object sender, DevExpress.XtraBars.Docking2010.BaseButtonEventArgs e)
{
    // ...
}

public partial class SY_TEST01 : BaseMenu
{
    InitializeComponent();

    grpTop.CustomButtonClick += GroupControlEx_CustomButtonClick;
    grpFiles.CustomButtonClick += GroupControlEx_CustomButtonClick;
}

private void GroupControlEx_CustomButtonClick(object sender, DevExpress.XtraBars.Docking2010.BaseButtonEventArgs e)
{
    (sender as Control)?.Focus();
    string tagValue = e.Button.Properties.Tag?.ToString() ?? string.Empty;

    if (sender.Equals(grpTop))
    {
        // ...
    }
    else if (sender.Equals(grpFiles))
    {
        // ...
    }
}

문자열 보간 사용

string execPc = GetClientPCName() + "/" + GetIPAddress();

반복해서 작성한 코드의 메서드화

GridFormatRule / FormatCondition.Expression

아래 예시는 데이터를 조회할때 마다 Rule을 새로 등록하는 패턴이며, 예시 코드처럼 Expression 속성의 값이 변하지 않는 문자열일 경우 [Grid Designer] 에서 등록하는게 맞습니다.

protected override void OnLoad(EventArgs e)
{
    base.OnLoad(e);
    gvwList.FormatRules.Add(gridFormatRule1);
    // ...
    // ...
    gvwList.FormatRules.Add(gridFormatRule4);
}

public override void QueryClick()
{
    fnQRY_P_SA200440_426_Q("Q");
    AddGridRule();
    // ...
    // ...
    AddGridRule4();
}

DevExpress.XtraGrid.GridFormatRule gridFormatRule1 = new DevExpress.XtraGrid.GridFormatRule();
DevExpress.XtraEditors.FormatConditionRuleValue formatConditionRuleValue1 = new DevExpress.XtraEditors.FormatConditionRuleValue();
private void AddGridRule()
{

    gridFormatRule1.ApplyToRow = false;
    gridFormatRule1.Column = coldlvday;
    gridFormatRule1.Name = "Format0";     
    formatConditionRuleValue1.Appearance.Font = new System.Drawing.Font("돋움", 11F, System.Drawing.FontStyle.Bold);
    formatConditionRuleValue1.Appearance.ForeColor = Color.Red;
    formatConditionRuleValue1.Appearance.Options.UseFont = true;
    formatConditionRuleValue1.Condition = DevExpress.XtraEditors.FormatCondition.Expression;
    formatConditionRuleValue1.Expression = "[displayyn] > 0"; 
    gridFormatRule1.Rule = formatConditionRuleValue1;
}

// ...
// ...

DevExpress.XtraGrid.GridFormatRule gridFormatRule4 = new DevExpress.XtraGrid.GridFormatRule();
DevExpress.XtraEditors.FormatConditionRuleValue formatConditionRuleValue4 = new DevExpress.XtraEditors.FormatConditionRuleValue();
private void AddGridRule4()
{
    gridFormatRule4.ApplyToRow = false;
    gridFormatRule4.Column = colday;
    gridFormatRule4.Name = "Format0";
    formatConditionRuleValue4.Appearance.Font = new System.Drawing.Font("돋움", 11F, System.Drawing.FontStyle.Bold);
    formatConditionRuleValue4.Appearance.ForeColor = Color.Red;
    formatConditionRuleValue4.Appearance.Options.UseFont = true;    
    formatConditionRuleValue4.Condition = DevExpress.XtraEditors.FormatCondition.Expression;
    formatConditionRuleValue4.Expression = "[dday2] < 0";  
    gridFormatRule4.Rule = formatConditionRuleValue4;
}

protected override void OnLoad(EventArgs e)
{
    base.OnLoad(e);
}

public override void QueryClick()
{
    fnQRY_P_SA200440_426_Q("Q");
    
    // 등록된 Rule 초기화
    gvwList.FormatRules.Clear(); /* 중요 */
    
    // Rule 추가
    AddExpressionRuleToGridView(
        gvwList,
        coldlvday,
        "[displayyn] > 0",
        font: new Font("돋움", 11F, FontStyle.Bold),
        foreColor: Color.Red);
    // ...
    // ...
    AddExpressionRuleToGridView(
        gvwList, 
        coldlvday,
        "[dday2] < 0",
        font: new Font("돋움", 11F, FontStyle.Bold),
        foreColor: Color.Red);
}

private void AddExpressionRuleToGridView(GridView view, GridColumn gridColumn, string expression, 
    bool applyToRow = false, Font font = null, Color? backColor = null, Color? foreColor = null)
{
    var formatRule = new DevExpress.XtraGrid.GridFormatRule();
    var conditionRuleValue = new DevExpress.XtraEditors.FormatConditionRuleValue();

    conditionRuleValue.Condition = DevExpress.XtraEditors.FormatCondition.Expression;
    conditionRuleValue.Expression = expression ?? string.Empty;

    if (font != null)
        conditionRuleValue.Appearance.Font = font;
    if (backColor != null)
        conditionRuleValue.Appearance.BackColor = (Color)backColor;
    if (foreColor != null)
        conditionRuleValue.Appearance.ForeColor = (Color)foreColor;

    formatRule.Rule = conditionRuleValue;
    formatRule.ApplyToRow = applyToRow;
    formatRule.Column = gridColumn;
    
    view.FormatRules.Add(formatRule);
}

#region 의 남용

region 전처리기는 필드나 프로퍼티 각 멤버의 지역을 구분해서 묶거나 유사한 기능의 여러 메서드를 묶어 코드를 정리하는 용도로 사용한다.
아래 예시처럼 region 블럭 내 하나의 메서드만 존재할 경우엔 사용하지 않는다.
참고) https://marcduerst.com/2016/10/03/c-regions-are-evil/

#region [RESIZE]
private void MA300200_Resize(object sender, EventArgs e)
{
    grpOutkind.Width = this.Width / 5 * 3;
}
#endregion

#region [ Merge Cell ]
private void gvwBandList_CellMerge(object sender, CellMergeEventArgs e)
{
    GridView view = sender as GridView;

    string lot1 = view.GetRowCellValue(e.RowHandle1, "s_lotnum").ToString();
    string lot2 = view.GetRowCellValue(e.RowHandle2, "s_lotnum").ToString(); ;

    e.Merge = lot1.Equals(lot2);
    e.Handled = true;
}
#endregion

TabControl에서 현재 선택된 TabPage의 판단

나쁜 표현: 열거형을 정의하고 프로퍼티에 값을 할당하는 방식으로 처리한 뒤 해당 프로퍼티의 값으로 판단하는 방식
좋은 표현: XtraTabContorl.SelectedTabPage 속성의 값을 Equals 메서드를 이용해 판단 (이때 TabPage 개체의 이름이 코드만 보고도 어떤 Page 인지를 알 수 있도록 네이밍이 중요)

enum ProcessType
{
    DayWork,    //일용직일근태
    DayAgg,     //일용직집계표
    DayInd      //일용직 개인별명세서
}

ProcessType ActiveProcessType { get; set; }
        
private void TabControl_SelectedPageChanged(object sender, DevExpress.XtraTab.TabPageChangedEventArgs e)
{
    var tabControl = sender as XtraTabControl;

    //[일용직일근태]
    if (tabControl.SelectedTabPage.Equals(tpgList))
    {
        ActiveProcessType = ProcessType.DayWork;
    }
    //[일용직집계표]
    else if (tabControl.SelectedTabPage.Equals(tpgSummary))
    {
        ActiveProcessType = ProcessType.DayAgg;
    }
    //[일용직 개인별명세서]  
    else if (tabControl.SelectedTabPage.Equals(tpgPersonal))    
    {
        ActiveProcessType = ProcessType.DayInd;
    }
    
    QueryClick();
}

public override void PrintClick()
{
    switch (ActiveProcessType)
    {
        case ProcessType.DayWork:
            break;

        case ProcessType.DayAgg:
            if (textEditEx1.Text == "") return;
            spreadsheetControl2.Print();
            break;

        case ProcessType.DayInd:
            if (gvwEmpList.FocusedRowHandle < 0) return;
            if (gvwEmpList.GetFocusedRowCellValue("Menu_ID").ToString() == "")
                return;
            spreadsheetControl3.Print();
            break;

        default:
            break;
    }
}

private void TabMain_SelectedPageChanged(object sender, DevExpress.XtraTab.TabPageChangedEventArgs e)
{
    QueryClick();
}

public override void PrintClick()
{
    if (tabMain.SelectedTabPage.Equals(tpgSummary))
    {
        if (string.IsNullOrEmpty(textEditEx1.Text))
            return;

        spreadsheetControl2.Print();
    }
    else if (tabMain.SelectedTabPage.Equals(tpgPersonal))
    {
        if (gvwEmpList.FocusedRowHandle < 0 || string.IsNullOrEmpty(gvwEmpList.GetFocusedRowCellValue("Menu_ID")?.ToString()))
            return;

        spreadsheetControl3.Print();
    }
}

CTS(Common Type System) 보단 기본 자료형 키워드를 사용 할 것

Int32 minute = 0; // X
int minute = 0; // O

Double.TryParse("3.14", out double number); // X
double.TryParse("3.14", out double number); // O

String str = "A"; // X
string str = "A"; // O

빈 문자열

`""` 대신 `string.Empty` 를 사용할 것

string userId = ""; // X
string userId = string.Empty; // O

빈 문자열의 비교

if (userId != "")
{
}

if (userId != null && userId != "")
{
}

적절하지 못한 멤버 변수 선언과 조회 조건 파라미터 초기화

private DateTime _Today; 멤버 변수 정의 불필요. 필요시 GetServerDateTime() 메서드 호출로 날짜를 받아와 처리하는게 좀 더 명확
조회 조건을 초기화 하는 ResetParameters 메서드를 정리하여 OnLoad 에서 호출

public partial class PR050320 : JERPBaseForm
{
    private DateTime _Today;
    
    public PR050320()
    {
        InitializeComponent();
    }
    
    protected override void OnLoad(EventArgs e)
    {
        base.OnLoad(e);

        DeleteButton = false;
        PreviewButton = false;
        PrintButton = false;
        NewButton = false;

        _Today = GetServerDateTime();

        ymdfrdt.EditValue = GetDefault("Query", ymdfrdt, _Today.AddMonths(-1));
        ymdtodt.EditValue = GetDefault("Query", ymdtodt, _Today);
        radfinyn.EditValue = GetDefault("Query", radfinyn, "%");
        cbolocation.EditValue = GetDefault("Query", cbolocation, SessionInfo.location);
        ymdfrdt1.EditValue = GetDefault("Query", ymdfrdt1, _Today.AddMonths(-1));
        ymdtodt1.EditValue = GetDefault("Query", ymdtodt1, _Today);
        cbolocation1.EditValue = GetDefault("Query", cbolocation1, SessionInfo.location);
    }
    
    private void ResetParameters()
    {
        if (TabInOut.SelectedTabPage.Equals(tpgOut))
        {
            ymdfrdt.EditValue = GetDefault("Query", ymdfrdt, _Today.AddMonths(-1));
            ymdtodt.EditValue = GetDefault("Query", ymdtodt, _Today);

            cbolocation.EditValue = GetDefault("Query", cbolocation, SessionInfo.location);
            radfinyn.EditValue = GetDefault("Query", radfinyn, "N");
        }
        else if (TabInOut.SelectedTabPage.Equals(tpgIn))
        {

            ymdfrdt1.EditValue = GetDefault("Query", ymdfrdt1, _Today.AddMonths(-1));
            ymdtodt1.EditValue = GetDefault("Query", ymdtodt1, _Today);
            cbolocation1.EditValue = GetDefault("Query", cbolocation1, SessionInfo.location);
        }
    }
    
    private void GroupControl_CustomButtonClick(object sender, DevExpress.XtraBars.Docking2010.BaseButtonEventArgs e)
    {
        (sender as Control).Focus();
        string tagValue = e.Button?.Properties?.Tag?.ToString() ?? string.Empty;

        if (tagValue.Equals("reset"))
        {
            InitControls(sender as Control);
            ResetParameters();
        }
    }
    
    ...

public partial class PR050320 : JERPBaseForm
{
    public PR050320()
    {
        InitializeComponent();
    }

    protected override void OnLoad(EventArgs e)
    {
        base.OnLoad(e);

        DeleteButton = false;
        PreviewButton = false;
        PrintButton = false;
        NewButton = false;

        ResetParameters(tpgOut);
        ResetParameters(tpgIn);
    }
    
    private void ResetParameters(XtraTabPage tabPage)
    {
        DateTime today = GetServerDateTime();

        if (tabPage.Equals(tpgOut))
        {
            ymdfrdt.EditValue = GetDefault("Query", ymdfrdt, today.AddMonths(-1));
            ymdtodt.EditValue = GetDefault("Query", ymdtodt, today);

            cbolocation.EditValue = GetDefault("Query", cbolocation, SessionInfo.location);
            radfinyn.EditValue = GetDefault("Query", radfinyn, "N");
        }
        else if (tabPage.Equals(tpgIn))
        {
            ymdfrdt1.EditValue = GetDefault("Query", ymdfrdt1, today.AddMonths(-1));
            ymdtodt1.EditValue = GetDefault("Query", ymdtodt1, today);
            cbolocation1.EditValue = GetDefault("Query", cbolocation1, SessionInfo.location);
        }
    }
    
    private void GroupControl_CustomButtonClick(object sender, DevExpress.XtraBars.Docking2010.BaseButtonEventArgs e)
    {
        (sender as Control).Focus();
        string tagValue = e.Button?.Properties?.Tag?.ToString() ?? string.Empty;

        if (tagValue.Equals("reset"))
        {
            InitControls(sender as Control);
            ResetParameters(tabInOut.SelectedTabPage);
        }
    }
    
    ...

엑셀 파일 읽고 처리하기

단순히 파일 읽어와 GridControl에 바인딩하기

읽어온 파일의 데이터를 GridView에 표현할 때 속도 저하를 개선한 예시
코드 내 속도 저하 문제가 되었던 부분에 주석 확인
Util 클래스의 ConvertExcelToDataTable 메서드 확인

private void FileOpen()
{
    OpenFileDialog ofd = new OpenFileDialog();
    ofd.Multiselect = false;

    ofd.Filter = "Excel Files(.xlsx)|*.xlsx|Excel Files(.xls)|*.xls|Excel Files(.xlsm)|*.xlsm|Excel Files(*.xml)|*.xml|Excel Files(*.xlt)|*.xlt|Excel Files(.xltx)|*.xltx";

    if (ofd.ShowDialog() == DialogResult.OK)
    {
        try
        {
            if (ofd.FileName != string.Empty)
            {
                Workbook workBook = new Workbook();

                if (Path.GetExtension(ofd.FileName) == ".xls")
                {
                    workBook.LoadDocument(ofd.FileName, DocumentFormat.Xls);
                }
                else if (Path.GetExtension(ofd.FileName) == ".xlsx")
                {
                    workBook.LoadDocument(ofd.FileName, DocumentFormat.Xlsx);
                }
                else
                {
                    ShowMessageBox("파일의 확장자명을 확인하여 주십시오.");
                    return;
                }

                Range range = workBook.Worksheets[0].GetUsedRange();

                DataTable dt = workBook.Worksheets[0].CreateDataTable(range, false);

                int colcnt = 0;
                colcnt = 22;

                if (workBook.Worksheets[0].Columns.LastUsedIndex != colcnt)
                {
                    ShowMessageBox(GetFormMessage("BA_A0051_470_002", "업로드 양식을 확인하여 주십시오.")); 
                    return;
                }

                /*******************************************************************************************
                 *  이 부분에서 읽어 들인 엑셀 파일에 모든 Cell을 반복하며 처리하기에 
                 *  Row가 많을수록 매우 많은 시간이 소요됨
                 *******************************************************************************************/
                int rowcnt = 0;
                rowcnt = 1;
                for (int i = rowcnt; i < workBook.Worksheets[0].Rows.LastUsedIndex + 1; i++)
                {
                    DataRow dr = dt.NewRow();

                    for (int j = 0; j < workBook.Worksheets[0].Columns.LastUsedIndex + 1; j++)
                    {
                        dr[j] = workBook.Worksheets[0].Cells[i, j].DisplayText;
                    }
                    dt.Rows.Add(dr);
                }
                /******************************************************************************************/

                /*******************************************************************************************
                 *  많은 양(1천건 이상)의 DataRow를 참조하여 GridView에 행을 추가후
                 *  GridView를 참조하여 Cell에 하나씩 값을 넣어줄 경우 많은 시간이 소요됨
                 *******************************************************************************************/
                if (dt != null)
                {
                    foreach (DataRow drResult in dt.Rows)
                    {
                        GridAddNewRow(grdList2, gvwList2.RowCount);
                        int row = gvwList2.FocusedRowHandle;
                        gvwList2.SetRowCellValue(row, "ROOTITEM", drResult["Column1"].ToString());
                        gvwList2.SetRowCellValue(row, "ROOTITEMDESC", drResult["Column2"].ToString());
                        gvwList2.SetRowCellValue(row, "ROOTITEMDRAW", drResult["Column3"].ToString());
                        gvwList2.SetRowCellValue(row, "LVL", drResult["Column4"].ToString());
                        gvwList2.SetRowCellValue(row, "PARENTITEM", drResult["Column5"].ToString());
                        gvwList2.SetRowCellValue(row, "PARENTITEMDESC", drResult["Column6"].ToString());
                        gvwList2.SetRowCellValue(row, "DRAW", drResult["Column7"].ToString());
                        gvwList2.SetRowCellValue(row, "COMPONENT", drResult["Column8"].ToString());
                        gvwList2.SetRowCellValue(row, "COMPDESC1", drResult["Column9"].ToString());
                        gvwList2.SetRowCellValue(row, "DRAW2", drResult["Column10"].ToString());
                        gvwList2.SetRowCellValue(row, "QTYPERBOM", drResult["Column11"].ToString());
                        gvwList2.SetRowCellValue(row, "UM", drResult["Column12"].ToString());
                        gvwList2.SetRowCellValue(row, "STATUS", drResult["Column13"].ToString());
                        gvwList2.SetRowCellValue(row, "SFTY_STK", drResult["Column14"].ToString());
                        gvwList2.SetRowCellValue(row, "ORD_MAX", drResult["Column15"].ToString());
                        gvwList2.SetRowCellValue(row, "ORD_MIN", drResult["Column16"].ToString());
                        gvwList2.SetRowCellValue(row, "ORD_MULT", drResult["Column17"].ToString());
                        gvwList2.SetRowCellValue(row, "LOC", drResult["Column18"].ToString());
                        gvwList2.SetRowCellValue(row, "PROD_LINE", drResult["Column19"].ToString());
                        gvwList2.SetRowCellValue(row, "PUR_LEAD", drResult["Column20"].ToString());
                        gvwList2.SetRowCellValue(row, "GROUPNO", drResult["Column21"].ToString());
                        gvwList2.SetRowCellValue(row, "MFG_LEAD", drResult["Column22"].ToString());
                        gvwList2.SetRowCellValue(row, "SUPPLIER", drResult["Column23"].ToString());
                    }
                }
                /******************************************************************************************/
            }
        }

        catch (Exception ex)
        {
            ShowMessageBox(string.Format(GetFormMessage("BA_A0051_470_001", "파일을 업로드 할 수 없습니다."), ex));
                return;
        }
    }
}

// 클래스 전역 멤버
// 읽어 들인 엑셀 파일의 컬럼 순서에 맞게 선언
// (GridControl에 바로 바인딩 하기 위함)
private string[] _DataColumnNames =
{
    "ROOTITEM",
    "ROOTITEMDESC",
    "ROOTITEMDRAW",
    "LVL",
    "PARENTITEM",
    "PARENTITEMDESC",
    "DRAW",
    "COMPONENT",
    "COMPDESC1",
    "DRAW2",
    "QTYPERBOM",
    "UM",
    "STATUS",
    "SFTY_STK",
    "ORD_MAX",
    "ORD_MIN",
    "ORD_MULT",
    "LOC",
    "PROD_LINE",
    "PUR_LEAD",
    "GROUPNO",
    "MFG_LEAD",
    "SUPPLIER"
};

private void BindingDataFromExcelFile()
{
    OpenFileDialog ofd = new OpenFileDialog();
    ofd.Multiselect = false;
    ofd.Filter = "Excel Files(.xlsx)|*.xlsx|Excel Files(.xls)|*.xls|Excel Files(.xlsm)|*.xlsm|Excel Files(*.xml)|*.xml|Excel Files(*.xlt)|*.xlt|Excel Files(.xltx)|*.xltx";

    if (ofd.ShowDialog() == DialogResult.OK)
    {
        try
        {
            // 엑셀 파일의 데이터를 DataTable 형태로 반환
            DataTable excelDatas = Util.ConvertExcelToDataTable(ofd.FileName, 0);

            // 결과 DataTable의 DataColumn의 ColumnName을 변경
            // (바인딩 할 GridControl의 GridColumn.FieldName 과 동일하게)
            for (int i = 0; i < excelDatas.Columns.Count; i++)
            {
                excelDatas.Columns[i].ColumnName = _DataColumnNames[i];
            }

            // 데이터 바인딩
            SetData(grdList2, excelDatas);
        }
        catch (Exception ex)
        {
            ShowMessageBox(string.Format(GetFormMessage("BA_A0051_470_001", "파일을 업로드 할 수 없습니다."), ex));
            return;
        }
    }
}

Previousfrequently used query Nextbad coding practice (FA Team)

Last updated 4 years ago

hashtag멤버 정의 순서

hashtag반복해서 참조하며 강제 형변환 예시

hashtag이벤트 핸들러의 반복 사용

hashtag문자열 보간 사용

hashtag반복해서 작성한 코드의 메서드화

hashtagGridFormatRule / FormatCondition.Expression

hashtag#region 의 남용

hashtagTabControl에서 현재 선택된 TabPage의 판단

hashtagCTS(Common Type System) 보단 기본 자료형 키워드를 사용 할 것

hashtag빈 문자열

hashtag"" 대신 string.Empty 를 사용할 것

hashtag빈 문자열의 비교

hashtag적절하지 못한 멤버 변수 선언과 조회 조건 파라미터 초기화

hashtag엑셀 파일 읽고 처리하기

hashtag단순히 파일 읽어와 GridControl에 바인딩하기

멤버 정의 순서

반복해서 참조하며 강제 형변환 예시

이벤트 핸들러의 반복 사용

문자열 보간 사용

반복해서 작성한 코드의 메서드화

GridFormatRule / FormatCondition.Expression

#region 의 남용

TabControl에서 현재 선택된 TabPage의 판단

CTS(Common Type System) 보단 기본 자료형 키워드를 사용 할 것

빈 문자열

`""` 대신 `string.Empty` 를 사용할 것

빈 문자열의 비교

적절하지 못한 멤버 변수 선언과 조회 조건 파라미터 초기화

엑셀 파일 읽고 처리하기

단순히 파일 읽어와 GridControl에 바인딩하기