Использование настроек свойств для внесения дополнений и извлечения

Question

В моей программе я делаю добавление валюты из цикла for .... Он работает нормально. Но я не уверен, что было сделано правильно и в соответствии с C#.

class Program { 
    private double _amount; 
    public double amount { 
     get { 
      return _amount; 
     } 
     set { 
      _amount = value; 
     } 
    } 

    static void Main(string[] args) { 
     Program p = new Program(); 

     for (int i = 1000; i < 1300; i++) { 
      double y = 30.00; 
      double x = y + p._amount; 
      p._amount = x; 
     } 
     Console.WriteLine(p._amount.ToString()); 
     Console.ReadLine(); 
    } 
} 

Я уменьшил размер кода. По сути, однако, существует несколько if-предложений внутри цикла for ..., которые я выполняю.

Я хотел бы поблагодарить любого, кто мог бы указать на любое несоответствие принципам кодирования C#.

Answer 1

Первая вещь использовать осмысленные имена, поэтому программа может быть предоставлена ​​более значимое имя.

Модулируйте свой код (создайте отдельный класс из вашей программы) и используйте рекомендуемые методы кодирования с помощью MSDN для C#.

class Calculation 
{ 

    public double Amount { get; set; } 

    public double run(double y) 
    { 
     // No need to start at 1000. 
     for(int i = 0; i < 300; i++) 
     { 
      Amount += y; 
     } 
     return Amount; 
    } 
} 

class Program 
{ 
    static void Main(string[] args) 
    { 
     Calculation calculation = new Calculation(); 
     // pass your variable as a parameter into a class function. 
     var y = 30.0; 
     Console.WriteLine(calculation.run(y).ToString()); 
     // Console.ReadLine(); use control F5 to prevent console window from closing. 
    } 
} 

C# Coding Conventions (C# Programming Guide)

Answer 2

Я бы рекомендовал менять этот код:

public double amount 
{ 
    get 
    { 
     return _amount; 
    } 
    set 
    { 
     _amount = value; 
    } 
} 

с этим:

public double getamount() 
{ 
     return _amount; 
} 
public void setamount(int value) 
    { 
     _amount = value; 
    }