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();
    }
}

我缩小了代码的大小。然而，实际上，在for ...循环中有几个if子句，我进行计算。

我要感谢能够指出任何与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;
    }

使用属性设置进行添加和检索

2 个答案: