Question

可能重复：
Why do I have to assign a value to an int in C# when defaults to 0?

我刚刚开始学习C＃，编写一个名为Journal的人工应用程序。在解析日志文件的函数中，我已经声明了变量DateTime currentEntryDate。在我到达定义新条目的行之前，它不会获得值。 second 时间到达输入行，该变量将用于为前一个条目创建类JournalEntry的实例。

问题是使用变量的代码不会编译：

使用未分配的本地变量＆＃39; currentEntryDate＆＃39;

这对我没有意义。为了让编译器满意，我真的必须给我的变量一个浪费的初始值吗？当然，我误解了某些内容，或者我的代码中出现了错误。

Pastebin上的代码：Journal.cs。我已经强调了相关的内容。

代码：

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

namespace Journal
{
    class Journal
    {
        public List<JournalEntry> Entries;

        private static readonly string EntryLineRegex =
            @"-- Entry: (?<title>.*) \((?<year>\d{4})-(?<month>\d{2})" +
            @"-(?<day>\d{2})\)";

        public static Journal FromFile(string filePath)
        {
            Journal returnValue = new Journal();

            StreamReader fileReader = new StreamReader(filePath);

            // Prepare variables for parsing the journal file.
            bool hitFirstEntry = false;
            DateTime currentEntryDate;
            string currentEntryTitle;
            StringBuilder currentEntryText = new StringBuilder();

            // Prepare a regular expression for the entry lines.
            Regex entryLineRegex = new Regex(EntryLineRegex);

            while (!fileReader.EndOfStream)
            {
                string line = fileReader.ReadLine();

                if (line.StartsWith("--"))
                {
                    // Is this the first entry encountered? If so, don't try to
                    // process the previous entry.
                    if (!hitFirstEntry)
                    {
                        hitFirstEntry = true;
                    }
                    else
                    {
                        // Create a JournalEntry with the current entry, then
                        // reset for the next entry.
                        returnValue.Entries.Add(
                            new JournalEntry(
                                currentEntryText.ToString(), currentEntryDate
                            )
                        );

                        currentEntryDate = new DateTime();
                        currentEntryText.Clear();
                    }

                    // Extract the new entry title and date from this line and
                    // save them.
                    Match entryMatch = entryLineRegex.Match(line);
                    GroupCollection matches = entryMatch.Groups;

                    currentEntryDate = new DateTime(
                        Convert.ToInt16(matches["year"].Value),
                        Convert.ToInt16(matches["month"].Value),
                        Convert.ToInt16(matches["day"].Value)
                    );

                    currentEntryTitle = matches["title"].Value;
                }
                else
                {
                    currentEntryText.Append(line);
                }
            }

            return returnValue;
        }
    }

    class JournalEntry
    {
        public string Text;
        public DateTime EntryDate;

        public JournalEntry(string text, DateTime entryDate)
        {
            this.Text = text;
            this.EntryDate = entryDate;
        }
    }
}

Answer 1

我认为这里的问题是编译器不够聪明，无法掌握读取输入的方式，并且存在一个执行路径，变量不会被初始化，即它是否经过{{1首先，在else之前。为避免这种情况，您可能需要在定义时进行初始化。

Answer 2

像这样重构你的循环怎么样？这将确保currentEntryDate在您使用之前具有值：

string line = fileReader.ReadLine();
while (line != null)
{
    // Extract the new entry title and date from this line and
    // save them.
    Match entryMatch = entryLineRegex.Match(line);
    GroupCollection matches = entryMatch.Groups;

    currentEntryDate = new DateTime(
        Convert.ToInt16(matches["year"].Value),
        Convert.ToInt16(matches["month"].Value),
        Convert.ToInt16(matches["day"].Value)
    );

    currentEntryTitle = matches["title"].Value;

    while ((line = fileReader.ReadLine()) != null && !line.StartsWith("--"))
    {
        currentEntryText.Append(line);
    }

    // Create a JournalEntry with the current entry, then
    // reset for the next entry.
    returnValue.Entries.Add(
        new JournalEntry(
            currentEntryText.ToString(), currentEntryDate
        )
    );

    currentEntryText.Clear();
}

Answer 3

在这种情况下，编译器没有意识到hitFirstEntry和currentEntryDate之间的“依赖关系”。

即使您可以“证明”每当hitFirstEntry更改为true时，currentEntryDate将在不久之后分配，currentEntryDate也不会第一次读直到（最早）在循环的下一次迭代中，编译器复杂。也许你可以重新编写代码。

修改：这是您代码的“最小”版本：

        bool isFirstTime = true;
        DateTime localVarToBeAssigned;

        while (true)
        {
            if (isFirstTime)
            {
                isFirstTime = false;
            }
            else
            {
                // this reads the variable
                Console.WriteLine(localVarToBeAssigned);
            }

            // this assigns the variable
            localVarToBeAssigned = DateTime.Now;
        }

Answer 4

如果你真的，真的不想实例化它：

DateTime? currentEntryDate = null

问号使DateTime可以为空，通常不是。

Answer 5

您声明局部变量这些变量没有默认值并从堆栈中获取内存你应该初始局部变量使用它们使用此代码的更改行：

currentEntryDate = new DateTime（）;

你的代码太高了，

我真的必须为我的所有变量赋予初始值吗？

5 个答案: