Ошибка выполнения в экземпляре StringBuilder - PullRequest
Новая
       12

Ошибка выполнения в экземпляре StringBuilder

0 голосов
/ 02 марта 2010

Пожалуйста, помогите мне понять, что не так с этим кодом. (Я пытаюсь построить строку, извлекая ее части за строкой из текстового файла).

Я получаю ошибку во время выполнения "В случае ссылки на объект, не установленной для объекта" в строке strbuild.Append(str);

        StreamReader reader = new StreamReader("buf.txt", System.Text.Encoding.ASCII);
        StringBuilder strbuild = new StringBuilder();
        strbuild = null;

        while (reader.Peek() >= 0)
        {
            string str = null;
            str = reader.ReadLine().ToString();

            string segment = str.Substring(0, 1);

            if (segment == "A")
            {
                strbuild.Append(str); //here  i get an error
            }
            else if (segment == "B")
            {
                strbuild.Append("BET");
            }

        }
        printstr = strbuild.ToString();
        reader.Close();

        MessageBox.Show(printstr);

Ответы [ 4 ]

10 голосов
/ 02 марта 2010

Посмотрите на эти строки: 

StringBuilder strbuild = new StringBuilder();
strbuild = null;

Что вы ожидаете, когда произойдет, когда вы тогда позвоните strbuild.Append(...)? Почему вы устанавливаете strbuild на ноль вообще?

Вы, кажется, любите инициализацию двухстрочной переменной - вот еще один пример: 

string str = null;
str = reader.ReadLine().ToString();

Это было бы легче прочитать, как просто: 

string str = reader.ReadLine();

(ReadLine уже возвращает строку, поэтому вам не нужно вызывать ToString() для результата.)

Тем не менее, я делаю , чтобы вы использовали оператор using для StreamReader - в противном случае, когда возникнет исключение, вы оставите читателя открытым.

Одна приятная вещь в TextReader.ReadLine() заключается в том, что она возвращает ноль, когда вы закончите. Вам не нужно заглядывать и , затем читать.

Наконец, если вы тестируете только один символ, вам не нужна подстрока - просто используйте индексатор строк, чтобы получить символ. Итак, вы можете иметь: 

StringBuilder builder = new StringBuilder();

// Consider using File.OpenText
using (StreamReader reader = new StreamReader("buf.txt", Encoding.ASCII))
{
    string line;
    // Normally side-effect + test is ugly, but this is a common and
    // neat idiom
    while ((line = reader.ReadLine()) != null)
    {
        // TODO: What do you want to happen for empty lines?
        char segment = str[0];
        if (segment == 'A')
        {
            builder.Append(line);
        }
        else if (segment == 'B')
        {
            builder.Append("BET");
        }
    }
}
MessageBox.Show(builder.ToString());
6 голосов
/ 02 марта 2010

Вы задаете для строителя строк значение null после инициализации.

Изменение 

StringBuilder strbuild = new StringBuilder(); 
strbuild = null;

до 

StringBuilder strbuild = new StringBuilder();

опуская линию 

strbuild = null;
2 голосов
/ 02 марта 2010

Измените 

  StringBuilder strbuild = new StringBuilder();
  strbuild = null;

на 

  StringBuilder strbuild = null;
  strbuild = new StringBuilder();

или, чтобы предотвратить этот тип ошибки: 

  StringBuilder strbuild = new StringBuilder();
1 голос
/ 02 марта 2010

В вашем примере много ошибок, вот первая исправленная версия: 

StringBuilder strbuild = new StringBuilder();

// Put resource into using statements, for deterministic cleanup
using (TextReader reader = new StreamReader("buf.txt", System.Text.Encoding.ASCII))
{
    string line;

    //Maybe looks a little ugly the first time, but is commonly used to
    //process all the lines of a file (.ReadToEnd() can cause memory problems
    //on really big files)
    while ((line = reader.ReadLine()) != null)
    {
        //Instead of if, else if, else if, etc just take a switch
        //statement. Makes it much easier to read.
        switch (line[0])
        {
            //If you need case insensitivity put both versions of the character
            //before your first line of code
            case 'A':
            case 'a':
                strbuild.Append(line);
                break;
            //Otherwise just use the lower or upper case version you like
            case 'B':
                strbuild.Append("BET");
                break;
        }
    }
}

//Put the result of the StringBuilder directly into the Show() function
MessageBox.Show(strbuild.ToString());
...