2016-09-21 96 views
0

我在縮短應用程序中的代碼時遇到了問題。我有這個示例代碼,我想讓它變小。該列表包含模型,連續劇等其他內容。我只需要做得更乾淨/可讀。 OFc,代碼遠遠大於此:簡化或縮減代碼大小

public class SmartPhones 
{ 
    public PhoneSettings() 
    { 
     Sony = true; 
     Htc = true; 
     Nokia = true; 
     Iphone = true; 

     SonyColor = Color Blue; 
     HtcColor = Color Black; 
     NokiaColor = Color Red; 
     IphoneColor = Color White; 
    } 
    public ToggleButton Sony {get; set;} 
    public ToggleButton Htc {get; set;} 
    public ToggleButton Nokia {get; set;} 
    public ToggleButton Iphone {get; set;} 

    public ColorButton SonyColor {get; set;} 
    public ColorButton HtcColor {get; set;} 
    public ColorButton NokiaColor {get; set;} 
    public ColorButton IphoneColor {get; set;} 
} 

謝謝大家的任何小幫助!

+5

該代碼沒有任何意義。不僅你有多個具有相同名稱的屬性,而且即使將它們定義爲「ToggleButton」和「ColorButton」,也爲它們分配布爾值和顏色值。請發佈一些完整的工作代碼。 – Bassie

+0

@Bassie是對的,我能想到的唯一辦法就是縮短索尼= true; Htc = true; Nokia = true; Iphone = true;'到'Sony = Htc = Nokia = Iphone = true;' –

+0

@Bassie代碼正常運行。索尼(切換開/關,ColorButton)比兩者(真/假色)的功能(get-set)。但代碼真的很大。 –

回答

0

我不太明白你想達到什麼目的,但對我來說,創建具有所有屬性的Phone類更合乎邏輯,然後創建包含具有正確屬性的Phone的所有實例的SmartPhones類。 類似這樣的東西

public class Phone 
    { 
     public string Type { get; set; } 
     public string ToggleButton { get; set; } 
     public string ColorButton { get; set; } 
    } 

    public class SmartPhones 
    { 
     private List<Phone> _list = new List<Phone>(); 

     public SmartPhones() 
     { 
      _list.Add(new Phone { Type = "Htc", ToggleButton = "aa", ColorButton = "bb"}); 
      _list.Add(new Phone { Type = "Nokia", ToggleButton = "aa", ColorButton = "bb" }); 
      _list.Add(new Phone { Type = "Sony", ToggleButton = "aa", ColorButton = "bb" }); 
     } 
    } 
+2

不要忘記初始化'_list' :) – fknx