c# - “使用未分配的局部变量”是什么意思？

Question

我不断收到有关年费率、月度费用和延迟费的错误。

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

namespace Lab_5___Danny_Curro
{
    class Program
    {
        static void Main(string[] args)
        {
            string firstName;
            string lastName;
            int accNumber;
            string creditPlan;
            double balance;
            string status;
            Boolean late = false;
            double lateFee;
            double monthlyCharge;
            double annualRate;
            double netBalance;


            Console.Write("Enter First Name: ");
            firstName = Console.ReadLine();

            Console.Write("Enter Last Name: ");
            lastName = Console.ReadLine();

            Console.Write("Enter Account Number: ");
            accNumber = Convert.ToInt32(Console.ReadLine());


            Console.Write("Enter Credit Card Plan Number[Blank Will Enter Plan 0]: ");
            creditPlan = Console.ReadLine();

            Console.Write("Enter Balance: ");
            balance = Convert.ToDouble(Console.ReadLine());

            Console.Write("Is This Account Late?: ");
            status = Console.ReadLine().Trim().ToLower();

            if (creditPlan == "0")
            {
                annualRate = 0.35;  //35%
                lateFee = 0.0;
                monthlyCharge = balance * (annualRate * (1 / 12));
                return;
            }

            if (creditPlan == "1")
            {
                annualRate = 0.30;  //30%
                if (status == "y")
                {
                    late = true;
                }

                else if (status == "n")
                {
                    late = false;
                }
                if (late == true)
                {
                    lateFee = 25.00;
                }
                monthlyCharge = balance * (annualRate * (1 / 12));
                return;
            }
            if (creditPlan == "2")
            {
                annualRate = 0.20;  //20%
                if (status == "y")
                {
                    late = true;
                }

                else if (status == "n")
                {
                    late = false;
                }
                if (late == true)
                {
                    lateFee = 35.00;
                }
                if (balance > 100)
                {
                    monthlyCharge = balance * (annualRate * (1 / 12));
                }
                else
                {
                    monthlyCharge = 0;
                }
                return;
            }
            if (creditPlan == "3")
            {
                annualRate = 0.15;  //15%
                lateFee = 0.00;

                if (balance > 500)
                {
                    monthlyCharge = (balance - 500) * (annualRate * (1 / 12));
                }
                else
                {
                    monthlyCharge = 0;
                }
                return;
            }
            netBalance = balance - (lateFee + monthlyCharge);


            Console.WriteLine("Name: \t\t\t {0}  {1}", firstName, lastName);
            Console.WriteLine("Account Number: \t{0}", accNumber);
            Console.WriteLine("Credit Plane: \t\t{0}",creditPlan);
            Console.WriteLine("Account Late: \t\t{0}", late);
            Console.WriteLine("Balance: \t\t{0}", balance);
            Console.WriteLine("Late Fee: \t\t{0}", lateFee);
            Console.WriteLine("Interest Charge: \t{0}", monthlyCharge);
            Console.WriteLine("Net Balance: \t\t{0}",netBalance);
            Console.WriteLine("Annual Rate: \t\t{0}", annualRate);
            Console.ReadKey();
        }
    }
}

score 66 · Accepted Answer

编译器不够聪明，无法知道至少有一个if块将被执行。annualRate因此，无论如何都不会分配变量 like 。以下是使编译器理解的方法：

if (creditPlan == "0")
{
    // ...
}
else if (creditPlan == "1")
{
    // ...
}
else if (creditPlan == "2")
{
    // ...
}
else
{
    // ...
}

编译器知道使用 if/else 块，保证执行其中一个块，因此如果您在所有块中分配变量，它不会给编译器错误。

顺便说一句，您还可以使用switch语句而不是ifs 来使您的代码更清晰。

score 19 · Accepted Answer

将您的声明更改为：

double lateFee = 0.0;
double monthlyCharge = 0.0;
double annualRate = 0.0;

导致该错误的原因是，在您的代码中至少有一条路径，这些变量最终没有被设置为任何东西。

score 12 · Accepted Answer

因为如果没有一个 if 语句评估为 true，那么局部变量将被取消分配。在那里抛出一个 else 语句，并为这些变量分配一些值，以防 if 语句不评估为真。如果这不会使错误消失，请返回此处。

您的另一个选择是在代码开头声明变量时将变量初始化为某个默认值。

score 10 · Accepted Answer

给他们一个默认值：

double lateFee=0.0;
double monthlyCharge = 0.0;
double annualRate = 0.0;

基本上，所有可能的路径都不会初始化这些变量。

score 5 · Accepted Answer

使用关键字“默认”！！！

    string myString = default;
    double myDouble = default;

    if(!String.IsNullOrEmpty(myString))
       myDouble = 1.5;

    return myDouble;

score 4 · Accepted Answer

您的代码中有许多路径没有初始化您的变量，这就是编译器抱怨的原因。

具体来说，您没有验证用户输入creditPlan- 如果用户输入的值不是"0","1","2" or "3"，则不会执行任何指示的分支（并且creditPlan不会根据您的用户提示默认为零）。

正如其他人所提到的，编译器错误可以通过在检查分支之前对所有派生变量进行默认初始化来避免，或者确保至少执行一个分支（即，分支的互斥性，通过else陈述）。

但是，我想指出其他潜在的改进：

在您信任用户输入用于您的代码之前验证它。
将参数建模为一个整体 - 每个计划都有几个属性和计算。
为数据使用更合适的类型。egCreditPlan似乎有一个有限域，并且比 a 更适合 aenumeration或Dictionarya string。财务数据和百分比应始终建模为decimal，而不是double避免舍入问题，并且“状态”似乎是一个布尔值。
干掉重复的代码。计算，monthlyCharge = balance * annualRate * (1/12))对不止一个分支是通用的。出于维护原因，请勿复制此代码。
可能更高级，但请注意，函数现在是 C# 的一等公民，因此您可以将函数或 lambda 分配为属性、字段或参数！

例如，这是您的模型的另一种表示形式：

    // Keep all Credit Plan parameters together in a model
    public class CreditPlan
    {
        public Func<decimal, decimal, decimal> MonthlyCharge { get; set; }
        public decimal AnnualRate { get; set; }
        public Func<bool, Decimal> LateFee { get; set; }
    }

    // DRY up repeated calculations
    static private decimal StandardMonthlyCharge(decimal balance, decimal annualRate)
    { 
       return balance * annualRate / 12;
    }

    public static Dictionary<int, CreditPlan> CreditPlans = new Dictionary<int, CreditPlan>
    {
        { 0, new CreditPlan
            {
                AnnualRate = .35M, 
                LateFee = _ => 0.0M, 
                MonthlyCharge = StandardMonthlyCharge
            }
        },
        { 1, new CreditPlan
            {
                AnnualRate = .30M, 
                LateFee = late => late ? 0 : 25.0M,
                MonthlyCharge = StandardMonthlyCharge
            }
        },
        { 2, new CreditPlan
            {
                AnnualRate = .20M, 
                LateFee = late => late ? 0 : 35.0M,
                MonthlyCharge = (balance, annualRate) => balance > 100 
                    ? balance * annualRate / 12
                    : 0
            }
        },
        { 3, new CreditPlan
            {
                AnnualRate = .15M, 
                LateFee = _ => 0.0M,
                MonthlyCharge = (balance, annualRate) => balance > 500 
                    ? (balance - 500) * annualRate / 12
                    : 0
            }
        }
    };

score 3 · Accepted Answer

您的分配都嵌套在条件 if 块中，这意味着它们有可能永远不会被分配。

在类的顶部，将它们初始化为 0 或其他值

score 1 · Accepted Answer

编译器说，如果 CreditPlan 未被识别，那么每年的利率将没有值。

在创建局部变量（annualRate、monthlyCharge 和 lateFee）时，为它们分配一个默认值 (0)。

此外，如果信用计划未知，您应该显示错误。

score 0 · Accepted Answer

0

并非所有代码路径都为lateFee. 您可能希望在顶部为其设置默认值。

于 2011-04-19T01:08:56.333 回答

score 0 · Accepted Answer

您不会在 if 语句之外分配值……正如@iomaxx 指出的那样，信用可能不是 0、1、2 或 3。

尝试将单独的 if 语句更改为单个 if/else if/else if/else。或者在顶部指定默认值。

score 0 · Accepted Answer

如果您声明变量“annualRate”，例如

类程序{

**static double annualRate;**

public static void Main() {

尝试一下..

c# - “使用未分配的局部变量”是什么意思？

11 回答 11

Related

Reference