c# - 无法使用将覆盖 C# 中的默认值的新类构造函数值

Question

我编写了一些代码来接收五个社会安全号码、五个总收入条目，并为每个条目计算税金。该程序将使用美元限额、低税率和高税率的默认值来计算欠税。但它也会让用户选择是否要使用他们自己的美元限额、低税率和高税率来计算所选纳税人的欠税额。

问题:每当我输入自己的美元限额、低税率和高税率来计算欠税时，它仍然使用默认值(限额 = 30000，低税率 = .15，高税率 = .28)。这就是错误计算的欠税值的原因。

问题。可能是因为我的“public static void GetRates(int income)”没有返回类型(它是 void)？在我调用“Taxpayer.GetRates(taxPayer[x].grossIncome);”之后，我是否应该将一个值返回给 main？方法？

部分代码

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;

namespace assignment
{
    public class Rates
    {
        public int incomeLimit; // income limit
        public double lowTaxRate; // low tax rate
        public double highTaxRate; // high tax rate

        public int IncomeLimit // read only property
        {
            get
            {
                return incomeLimit;
            }
        }
        public double LowTaxRate // read only property
        {
            get
            {
                return lowTaxRate;
            }
        }
        public double HighTaxRate // read only property
        {
            get
            {
                return highTaxRate;
            }
        }

        public Rates()
        {
            DoRates();
        }

        public void DoRates() // class constructor that assigns default values 
        {
            // set default values for limit, lowRate, and highRate
            incomeLimit = 30000;
            lowTaxRate = .15;
            highTaxRate = .28;
        }

        public void DoRates(int limit, double lowRate, double highRate) // class constructor that takes three parameters
        {
            incomeLimit = limit; 
            lowTaxRate = lowRate;
            highTaxRate = highRate;
        }
        // CalculateTax method that takes an income parameter and computes the tax
        public int CalculateTax(int income)
        {
            int taxOwed = 0;

            if (income < incomeLimit)
            {
                taxOwed = Convert.ToInt32(income * lowTaxRate);

            }
            if (income >= incomeLimit)
            {
                taxOwed = Convert.ToInt32(income * highTaxRate);

            }
            return taxOwed;
        }  
    }
    public class Taxpayer : IComparable
    {
        string socialSecurityNum;
        int grossIncome;
        int taxOwed;

        // Use get and set accessors.
        public string SocialSecurityNum
        {
            get
            {
                return socialSecurityNum;
            }
            set
            {
                socialSecurityNum = value;
            }
        }
        // Use get and set accessors.
        public int GrossIncome
        {
            get
            {
                return grossIncome;
            }
            set
            {
                grossIncome = value;
            }
        }
        //  Use read-only accessor
        public int TaxOwed
        {
            get
            {
                return taxOwed;
            }
        }
        // objects are comparable to each other based on tax owed.
        int IComparable.CompareTo(Object o)
        {
            int returnVal;
            Taxpayer temp = (Taxpayer)o;
            if (this.taxOwed > temp.TaxOwed)
                returnVal = 1;
            else
                if (this.taxOwed < temp.TaxOwed)
                    returnVal = -1;
                else
                    returnVal = 0;
            return returnVal;
        }

        public static void GetRates(int income)
        {

            int incomeLimit;
            double lowRate;
            double highRate;

            char input;
            Rates rates = new Rates();

            Console.Write("Do you want default values (enter D) or enter your own (enter O)? ");
            input = Convert.ToChar(Console.ReadLine());


            switch (char.ToUpper(input)) // start switch
            {        
                case 'D': // if the input latter is d or a D
                    rates.DoRates();
                    break;
                case 'O': // if the input latter is o or an O
                    Console.Write("Enter the dollar limit ");
                    incomeLimit = Convert.ToInt32(Console.ReadLine());

                    Console.Write("Enter the low rate ");
                    lowRate = Convert.ToDouble(Console.ReadLine());

                    Console.Write("Enter the high rate ");
                    highRate = Convert.ToDouble(Console.ReadLine());

                    rates.DoRates(incomeLimit, lowRate, highRate);
                    rates.CalculateTax(income);
                    break;
                default: Console.WriteLine("You entered and incorrect option"); // display this messages if the input was something other than D or O
                    break;
            }


        }

        public static void Main()
        {
            // instantiate an array of five (5) Taxpayer objects.
            Taxpayer[] taxPayer = new Taxpayer[5];
            Rates taxRates = new Rates();

            // Implement a for-loop that will prompt the user 
            // to enter the Social Security Number and gross income.
            for (int x = 0; x < taxPayer.Length; ++x)
            {

                taxPayer[x] = new Taxpayer();
                Console.Write("Enter Social Security Number for taxpayer {0} ", x + 1);
                taxPayer[x].socialSecurityNum = Convert.ToString(Console.ReadLine());


                Console.Write("Enter gross income for taxpayer {0} ", x + 1);
                taxPayer[x].grossIncome = Convert.ToInt32(Console.ReadLine());

                Taxpayer.GetRates(taxPayer[x].grossIncome);


                taxPayer[x].taxOwed = taxRates.CalculateTax(taxPayer[x].grossIncome);

            }

谢谢大家的帮助。我想我在编写这段代码时确实有点忘乎所以了。在将代码与我的问题隔离开来之后，我终于弄明白了。这是我所做的，以防有人想看。

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;

namespace assignment
{
    public class Rates
    {
        public int incomeLimit; // income limit
        public double lowTaxRate; // low tax rate
        public double highTaxRate; // high tax rate

        public int IncomeLimit { get { return incomeLimit; } }// read only property
        public double LowTaxRate { get { return lowTaxRate; } } // read only property
        public double HighTaxRate { get { return highTaxRate; } }// read only property

        public Rates()
        {
            incomeLimit = 30000;
            lowTaxRate = .15;
            highTaxRate = .28;
        }

        public Rates(int incomeLim, double lowRate, double highRate)
        {   
            incomeLimit = incomeLim;
            lowTaxRate = lowRate;
            highTaxRate = highRate;

        }

        // CalculateTax method that takes an income parameter and computes the tax
        public int CalculateTax(int income)
        {
            int taxOwed = 0;

            if (income < incomeLimit)
            {
                taxOwed = Convert.ToInt32(income * lowTaxRate);
            }
            if (income >= incomeLimit)
            {
                taxOwed = Convert.ToInt32(income * highTaxRate);
            }
            return taxOwed;
        }
    }
    public class Taxpayer
    {
        string socialSecurityNum = null;
        int grossIncome = 0;
        int taxOwed = 0;

        // Use get and set accessors.
        public string SocialSecurityNum { get {return socialSecurityNum;} set {socialSecurityNum = value;} }
        // Use get and set accessors.
        public int GrossIncome { get { return grossIncome; } set { grossIncome = value; } }
        //  Use read-only accessor
        public int TaxOwed { get { return taxOwed; } }

        public void GetRates(int income)
        {
            int incomeLimit = 0;
            double lowRate = 0;
            double highRate = 0;
            char input;

            Console.Write("Do you want default values (enter D) or enter your own (enter O)? ");
            input = Convert.ToChar(Console.ReadLine());


            switch (char.ToUpper(input)) // start switch
            {
                case 'D': // if the input latter is d or a D
                    Rates rates = new Rates();
                    taxOwed = rates.CalculateTax(income);
                    break;
                case 'O': // if the input latter is o or an O
                    Console.Write("Enter the dollar limit ");
                    incomeLimit = Convert.ToInt32(Console.ReadLine());
                    Console.Write("Enter the low rate ");
                    lowRate = Convert.ToDouble(Console.ReadLine());
                    Console.Write("Enter the high rate ");
                    highRate = Convert.ToDouble(Console.ReadLine());
                    Rates myrates = new Rates(incomeLimit, lowRate, highRate);
                    taxOwed = myrates.CalculateTax(income);
                    break;
                default: Console.WriteLine("You entered and incorrect option"); // display this messages if the input was something other than D or O
                    break;
            }

        }
        public static void Main()
        {
            Taxpayer[] taxPayer = new Taxpayer[5];
            Rates taxRates = new Rates();

            Taxpayer myTaxpayer = new Taxpayer();
            // Implement a for-loop that will prompt the user 
            // to enter the Social Security Number and gross income.
            for (int x = 0; x < taxPayer.Length; ++x)
            {
                taxPayer[x] = new Taxpayer();
                Console.Write("Enter Social Security Number for taxpayer {0} ", x + 1);
                taxPayer[x].socialSecurityNum = Convert.ToString(Console.ReadLine());
                Console.Write("Enter gross income for taxpayer {0} ", x + 1);
                taxPayer[x].grossIncome = Convert.ToInt32(Console.ReadLine());
                myTaxpayer.GetRates(taxPayer[x].grossIncome);

                taxPayer[x].taxOwed = myTaxpayer.taxOwed;
            }
            // Implement a for-loop that will display each object 
            // as formatted taxpayer SSN, income and calculated tax.
            for (int y = 0; y < taxPayer.Length; ++y)
            {
                Console.WriteLine("Taxpayer # {0} SSN: {1} income {2:C} Tax is {3:C}", y + 1, taxPayer[y].socialSecurityNum,
                    taxPayer[y].grossIncome, taxPayer[y].taxOwed);
            }
        }
    }
}

Answer 1

我会建议您更改代码，使用更干净的代码并使用更好的 OOP 技术。

您应该检查 Taxpayer 类，特别是名为 GetRates() 的方法。

在该方法中，您创建了一个 Rates 类型的对象，现在，如果您检查类 Rates 的构造函数

public Rates()
    {
        DoRates();
    }

它调用方法 Dorates() 但没有参数，所以，它总是会调用这些 DoRates 方法

public void DoRates() // class constructor that assigns default values 
    {
        // set default values for limit, lowRate, and highRate
        incomeLimit = 30000;
        lowTaxRate = .15;
        highTaxRate = .28;
    }