DCSIMG
Question from .Net Tapuz forum: Refactoring Code-behind to Static Methods and Properties example - Justin myJustin = new Justin( Expriences.Current );

Question from .Net Tapuz forum: Refactoring Code-behind to Static Methods and Properties example

שאלה:

 יש לי את הקוד הבא ואני בטוח שיש דרך יותר טובה ונקייה לכתוב אותו, למישהו יש רעיונות? 

 

       // טבלא ראשית
        Table MainTbl = new Table();

        // הגדרות עבור טבלא 
        MainTbl.CellPadding = 0;
        MainTbl.CellSpacing = 0;
        MainTbl.Width = Unit.Parse("456") ;

        
        TableRow MainRow = new TableRow();
        MainTbl.Rows.Add(MainRow);
        // טור קו שמאלי של טבלא ראשית
        TableCell MainLineLeftCell = new TableCell();
        MainRow.Cells.Add(MainLineLeftCell);
        MainLineLeftCell.Width = Unit.Parse("1px");
        MainLineLeftCell.Height = Unit.Parse("100%");
        MainLineLeftCell.Style.Add("background-image", "url(simages/FramePixel.gif)");

        // טור שמאלי של טבלא ראשית
        TableCell MainLeftCell = new TableCell();
        MainLeftCell.ColumnSpan = 0;
        MainLeftCell.Width = Unit.Parse("439");
        MainRow.Cells.Add(MainLeftCell);
        
        // הוספת טבלא לתוך התא השמאלי
        Table LeftTable = new Table();
        MainLeftCell.Controls.Add(LeftTable);
        // הגדרות עבור טבלא שמאלית
        LeftTable.Width = Unit.Parse("100%");
        LeftTable.CellSpacing = 0;
        LeftTable.CellPadding = 0;
        LeftTable.Height = Unit.Parse("100%");

        // שורה עליונה
        TableRow leftUpRow = new TableRow();
        LeftTable.Rows.Add(leftUpRow);

        TableCell leftUpCell = new TableCell();
        leftUpRow.Cells.Add(leftUpCell);
        leftUpCell.Style.Add("background-image", "url(simages/FramePixel.gif)");
        leftUpCell.Height = Unit.Parse("1px");
        leftUpCell.Width = Unit.Parse("100%");
        
        // שורה אמצעית
        TableRow leftMRow = new TableRow();
        LeftTable.Rows.Add(leftMRow);
        TableCell leftMCell = new TableCell();
        leftMRow.Cells.Add(leftMCell);
        
        // שורה תחתונה
        TableRow leftDRow = new TableRow();
        LeftTable.Rows.Add(leftDRow);
        TableCell leftDCell = new TableCell();
        leftDRow.Cells.Add(leftDCell);
        leftDCell.Style.Add("background-image", "url(simages/FramePixel.gif)");
        leftDCell.Height = Unit.Parse("1px");
        leftDCell.Width = Unit.Parse("100%");

        // טור ימני של טבלא ראשית
        TableCell MainRightCell = new TableCell();
        MainRightCell.ColumnSpan = 0;
        MainRightCell.Width = Unit.Parse("16");
        MainRightCell.Height = Unit.Parse("100%");
        MainRow.Cells.Add(MainRightCell);

תשובה:

בלי לשנות את הפתרון של שימוש בטבלה (במעבר ל-GridView או ל-DataList) בהחלט יש כאן מקום לשיפור הפתרון המוצא.

הפתרון הוא שימוש במתודות סטטיות ואף במאפיינים סטטיים כדי להכיל את כל ההגדרות המעניינות האלו ולרכז קוד.

נתחיל ביחד בתהליך שנקרא Refactoring. מילת המפתח כאן היא Refactoring.
הכוונה היא: |הדגש| שינוי הקוד בלי לשנות את הפונקציונליות שלו|סדגש|.
כלומר, אנחנו יודעים שהקוד עובד אבל עכשיו הגיע הזמן להפוך אותו ליותר קריא וסה"כ יותר "נכון".

נתחיל את הדוגמה בלעשות Refactoring לתאים בטבלה. נתחיל מהקוד הזה:

// טור קו שמאלי של טבלא ראשית
TableCell MainLineLeftCell = new TableCell();
MainRow.Cells.Add(MainLineLeftCell);
MainLineLeftCell.Width = Unit.Parse("1px");
MainLineLeftCell.Height = Unit.Parse("100%");

נוציא את כל קביעת המאפיינים הזו למתודה סטטית

 

// טור קו שמאלי של טבלא ראשית
MainRow.Cells.Add(TableHelper.GetNewCell()); 

public static class TableHelper
{
    public static TableCell GetNewCell()
    {
        TableCell ReturnCell = new TableCell();
        ReturnCell.Width = Unit.Parse("1px");
        ReturnCell.Height = Unit.Parse("100%");
        return ReturnCell;
    }
}

OK, אז מה עשינו כאן? לקחנו את כל המאפיינים שנקבעו ושמנו את הקביעה שלהם בתוך מתודה סטטית. המתודה הסטטית הזו גם מחזירה את התא החדש ובחוץ אנחנו דואגים להוסיף אותו לרשימת התאים.

עכשיו נטפל בקוד הזה:

// טור שמאלי של טבלא ראשית
TableCell MainLeftCell = new TableCell();
MainLeftCell.ColumnSpan = 0;
MainLeftCell.Width = Unit.Parse("439");
MainRow.Cells.Add(MainLeftCell);

אפשר לראות שההבדל בין מה ש-GetNewCell מחזיר לנו לבין מה שאנחנו צריכים זה ה-Width וה-ColumnSpan. אז נשנה את המתודה כך שעדיין GetNewCell יחזיר בדיוק אותו TableCell, אבל גם יהיה לנו Overload שמאפשר להחזיר בדיוק מה שאנחנו צריכים. 

public static class TableHelper
{
    public static TableCell GetNewCell()
    {
        return GetNewCell("1px", 1);
    }
    public static TableCell GetNewCell(string Width, int ColumnSpan)
    {
        TableCell ReturnCell = new TableCell();
        ReturnCell.ColumnSpan = ColumnSpan;
        ReturnCell.Width = Unit.Parse(Width);
        ReturnCell.Height = Unit.Parse("100%");
        ReturnCell.Style.Add("background-image", "url(simages/FramePixel.gif)");
        return ReturnCell;
    }
} 
// טור קו שמאלי של טבלא ראשית
MainRow.Cells.Add(TableHelper.GetNewCell()); 

// טור שמאלי של טבלא ראשית
MainRow.Cells.Add(TableHelper.GetNewCell("439", 0)); 




כפי שאפשר לראות, לא שנינו את העבודה של GetNewCell בלי פרמטרים, אבל קיים GetNewCell עם פרמטרים נוספים שמאפשרים לקבוע מאפיינים נוספים. נמשיך עם הקוד הבא:
TableCell leftUpCell = new TableCell();
leftUpRow.Cells.Add(leftUpCell);
leftUpCell.Style.Add("background-image", "url(simages/FramePixel.gif)");
leftUpCell.Height = Unit.Parse("1px");
leftUpCell.Width = Unit.Parse("100%");


עכשיו אנחנו נדרשים לקבוע גם גובה וגם רקע (שהוא זהה לכל התאים בעלי הרקע), אז נוסיף עוד overload של GetNewCell שמקבל את הגובה של התא וגם מקבל האם צריך להוסיף רקע.
leftUpRow.Cells.Add(TableHelper.GetNewCell("100%", "1px", 1, true)); 

public static class TableHelper
{
    public static TableCell GetNewCell()
    {
        return GetNewCell("1px", 1);
    }

    public static TableCell GetNewCell(string Width, int ColumnSpan)
    {
        return GetNewCell(Width, "100%", ColumnSpan, false);
    }

    public static TableCell GetNewCell(string Width, string Height, int ColumnSpan, bool AddBackground)
    {
        TableCell ReturnCell = new TableCell();
        ReturnCell.ColumnSpan = ColumnSpan;
        ReturnCell.Width = Unit.Parse(Width);
        ReturnCell.Height = Unit.Parse(Height);
        if (AddBackground)
            ReturnCell.Style.Add("background-image", "url(simages/FramePixel.gif)");
        return ReturnCell;
    }



נמשיך עם שני התאים האלו:
TableCell leftDCell = new TableCell();
leftDRow.Cells.Add(leftDCell);
leftDCell.Style.Add("background-image", "url(simages/FramePixel.gif)");
leftDCell.Height = Unit.Parse("1px");
leftDCell.Width = Unit.Parse("100%"); 

// טור ימני של טבלא ראשית
TableCell MainRightCell = new TableCell();
MainRightCell.ColumnSpan = 0;
MainRightCell.Width = Unit.Parse("16");
MainRightCell.Height = Unit.Parse("100%");
MainRow.Cells.Add(MainRightCell);



שהופכים ל:
leftDRow.Cells.Add(TableHelper.GetNewCell("100%", "1px", 1, true)); 

// טור ימני של טבלא ראשית
MainRow.Cells.Add(TableHelper.GetNewCell("16", "100%", 0, true));





יפה, אז עכשיו כל מקום שבו יוצרים TableCell חדש וקובעים לו מאפיינים הפך למתודה סטטית. עכשיו נשלב גם מאפיינים סטטים.
// טור קו שמאלי של טבלא ראשית
MainRow.Cells.Add(TableHelper.GetNewCell()); 

// becomes
MainRow.Cells.Add(Cells.LeftLine); 

public static class Cells
{
    public static TableCell LeftLine
    {
        get
        {
            return TableHelper.GetNewCell();
        }
    }
}


כלומר, במקום כל פעם עכשיו לקבוע מאפיינים בקוד, נשתמש במאפיינים סטטים שיתנו לנו תמיכה Strongly Typed בקוד במקום הערות מיותרות. 
public static class Cells
{
    public static TableCell LeftLine
    {
        get
        {
            return TableHelper.GetNewCell();
        }
    }

    public static TableCell LeftColumn
    {
        get
        {
            return TableHelper.GetNewCell("439", 0);
        }
    }

    public static TableCell TopRow
    {
        get
        {
            return TableHelper.GetNewCell("100%", "1px", 1, true);
        }
    }

    public static TableCell LowerRow
    {
        get
        {
            return TableHelper.GetNewCell("100%", "1px", 1, true);
        }
    }

    public static TableCell RightColumn
    {
        get
        {
            return TableHelper.GetNewCell("16", "100%", 0, true);
        }
    }
}



עכשיו הקוד שנשאר לנו נראה ככה:


 
// טבלא ראשית
Table MainTbl = new Table(); 

// הגדרות עבור טבלא 
MainTbl.CellPadding = 0;
MainTbl.CellSpacing = 0;
MainTbl.Width = Unit.Parse("456"); 

TableRow MainRow = new TableRow();
MainTbl.Rows.Add(MainRow);
MainRow.Cells.Add(Cells.LeftLine); 

// טור שמאלי של טבלא ראשית
MainRow.Cells.Add(Cells.LeftColumn); 

// הוספת טבלא לתוך התא השמאלי
Table LeftTable = new Table();
MainRow.Cells[MainRow.Cells.Count - 1].Controls.Add(LeftTable);
// הגדרות עבור טבלא שמאלית
LeftTable.Width = Unit.Parse("100%");
LeftTable.CellSpacing = 0;
LeftTable.CellPadding = 0;
LeftTable.Height = Unit.Parse("100%"); 

// שורה עליונה
TableRow leftUpRow = new TableRow();
LeftTable.Rows.Add(leftUpRow); 

leftUpRow.Cells.Add(Cells.TopRow); 

// שורה אמצעית
TableRow leftMRow = new TableRow();
LeftTable.Rows.Add(leftMRow);
leftMRow.Cells.Add(new TableCell()); 

// שורה תחתונה
TableRow leftDRow = new TableRow();
LeftTable.Rows.Add(leftDRow);
leftDRow.Cells.Add(Cells.LowerRow); 

// טור ימני של טבלא ראשית
MainRow.Cells.Add(Cells.RightColumn); 

this.Controls.Add(MainTbl);



ניקח גם את המאפיינים של טבלה ונכניס למתודות ומאפיינים: 


 
Table MainTbl = Tables.Main;
Table LeftTable = Tables.Left; 

public static class Tables
{
    public static Table Main
    {
        get
        {
            return TableHelper.GetNewTable("456");
        }
    }

    public static Table Left
    {
        get
        {
            return TableHelper.GetNewTable("100%", "100%");
        }
    }
} 




עכשיו נוסיף גם מאפיינים סטטים לשורות כך שבתוכן יתווספו תאים. 

 
// שורה תחתונה
TableRow leftDRow = new TableRow();
LeftTable.Rows.Add(leftDRow);
leftDRow.Cells.Add(Cells.LowerRow); 

//becomes
LeftTable.Rows.Add(Rows.LowerRow);

public static class Rows
{
    public static TableRow LowerRow
    {
        get
        {
            TableRow ReturnRow = new TableRow();
            ReturnRow.Cells.Add(Cells.LowerRow);
            return ReturnRow;
        }
    }
} 




בצורה דומה, נכניס את כל השורות לתוך מאפיינים, ואת כל הקשרים בין המאפיינים השונים. 
public partial class Default3 : System.Web.UI.Page
{
    protected void Page_Load(object sender, EventArgs e)
    {
        Controls.Add(Tables.Main);
    }
}

public static class TableHelper
{
    public static TableRow GetNewRow(TableCell CellInRow)
    {
        TableRow ReturnRow = new TableRow();
        ReturnRow.Cells.Add(CellInRow);
        return ReturnRow;
    }
    public static Table GetNewTable(string Width)
    {
        Table ReturnTable = new Table();
        ReturnTable.CellPadding = 0;
        ReturnTable.CellSpacing = 0;
        ReturnTable.Width = Unit.Parse(Width);
        return ReturnTable;
    }

    public static Table GetNewTable(string Height, string Width)
    {
        Table ReturnTable = GetNewTable(Width);
        ReturnTable.Height = Unit.Parse(Height);
        return ReturnTable;
    }

    public static TableCell GetNewCell()
    {
        return GetNewCell("1px", 1);
    }

    public static TableCell GetNewCell(string Width, int ColumnSpan)
    {
        return GetNewCell(Width, "100%", ColumnSpan, false);
    }

    public static TableCell GetNewCell(string Width, string Height, int ColumnSpan, bool AddBackground)
    {
        TableCell ReturnCell = new TableCell();
        ReturnCell.ColumnSpan = ColumnSpan;
        ReturnCell.Width = Unit.Parse(Width);
        ReturnCell.Height = Unit.Parse(Height);
        if (AddBackground)
            ReturnCell.Style.Add("background-image", "url(simages/FramePixel.gif)");
        return ReturnCell;
    }
}

public static class Cells
{
    public static TableCell LeftLine
    {
        get
        {
            return TableHelper.GetNewCell();
        }
    }

    public static TableCell LeftColumn
    {
        get
        {
            return TableHelper.GetNewCell("439", 0);
        }
    }

    public static TableCell TopRow
    {
        get
        {
            return TableHelper.GetNewCell("100%", "1px", 1, true);
        }
    }

    public static TableCell LowerRow
    {
        get
        {
            return TableHelper.GetNewCell("100%", "1px", 1, true);
        }
    }

    public static TableCell RightColumn
    {
        get
        {
            return TableHelper.GetNewCell("16", "100%", 0, true);
        }
    }
}




public static class Tables
{
    public static Table Main
    {
        get
        {
            Table ReturnTable = TableHelper.GetNewTable("456");
            ReturnTable.Rows.Add(Rows.Main);
            return ReturnTable;
        }
    }

    public static Table Left
    {
        get
        {
            Table ReturnTable = TableHelper.GetNewTable("100%", "100%");
            ReturnTable.Rows.Add(Rows.UpperRow);
            ReturnTable.Rows.Add(Rows.LeftRow);
            ReturnTable.Rows.Add(Rows.LowerRow);
            return ReturnTable;
        }
    }
}

public static class Rows
{
    public static TableRow LowerRow
    {
        get
        {
            return TableHelper.GetNewRow(Cells.LowerRow);
        }
    }

    public static TableRow LeftRow
    {
        get
        {
            return TableHelper.GetNewRow(new TableCell());
        }
    }

    public static TableRow UpperRow
    {
        get
        {
            return TableHelper.GetNewRow(Cells.TopRow);
        }
    }

    public static TableRow Main
    {
        get
        {
            TableRow ReturnRow = new TableRow();
            ReturnRow.Cells.Add(Cells.LeftLine);
            ReturnRow.Cells.Add(Cells.LeftColumn);
            ReturnRow.Cells[ReturnRow.Cells.Count - 1].Controls.Add(Tables.Left);
            ReturnRow.Cells.Add(Cells.RightColumn);
            return ReturnRow;
        }
    }
}


 


קישור: http://www.tapuz.co.il/tapuzforum/main/addmsg.asp?id=831&msgid=90269910&Subject=&Content=

.csharpcode, .csharpcode pre
{
	font-size: small;
	color: black;
	font-family: Consolas, "Courier New", Courier, Monospace;
	background-color: #ffffff;
Direction: ltr; 
Text-align: left;
	/*white-space: pre;*/
}
.csharpcode pre { margin: 0em; }
.csharpcode .rem { color: #008000; }
.csharpcode .kwrd { color: #0000ff; }
.csharpcode .str { color: #006080; }
.csharpcode .op { color: #0000c0; }
.csharpcode .preproc { color: #cc6633; }
.csharpcode .asp { background-color: #ffff00; }
.csharpcode .html { color: #800000; }
.csharpcode .attr { color: #ff0000; }
.csharpcode .alt 
{
	background-color: #f4f4f4;
	width: 100%;
	margin: 0em;
}
.csharpcode .lnum { color: #606060; }

            

            

                	
                Published
                Friday, December 08, 2006 11:53 PM
			    by
			    Justin-Josef Angel [MVP]          
                
                            
                
תגים:, , , , , 

            

        

    

    
    
            
                    
                    
Comments

                    

                
                    
No Comments