Citat:
Hej!Här är några synpunkter från mig:
* Använd inte "class" i ett klassnamn.
* Kommentera mer
* Läs på om private/protected/public
Kolla här till exempel:
Kod:
public class ClassMultipleIncome {
protected List<int> MultipleIncomeList = new List<int>();
protected int listSumma;
//Add to list
protected void AddtoList(int x){
MultipleIncomeList.Add(x);
}
protected int ShowSum(){
listSumma = MultipleIncomeList.Sum();
return listSumma;
}
// Loop out the items.
protected void ShowAllItemsInList(){
foreach (int value in this.MultipleIncomeList ){
Console.Write(" + {0}",value);
}
}
}
Klassen är public samtidigt som alla variabler och funktioner är protected, det är helt random. Mitt råd är att hålla dig till public/private tills du får en djupare förståelse för hur det fungerar. I det här fallet skulle du göra såhär:
Kod:
public class ClassMultipleIncome {
private List<int> multipleIncomeList;
private int listSumma;
// Construction
public ClassMultipleIncome(){
multipleIncomeList = new List<int>();
}
//Add to list
public void AddtoList(int x){
multipleIncomeList.Add(x);
}
// Metoden returnerar summan, den displayar den inte. Därför namnbyte
public int GetSum(){
return multipleIncomeList.Sum();
}
// Loop out the items.
public void ShowAllItemsInList(){
foreach (int value in this.MultipleIncomeList ){
Console.Write(" + {0}",value);
}
}
}
Det finns massa annat att säga, men vi kan börja där
