После чтение файла происходит сложение значений из файла, на выход неожидаемый результат, почуем?

Question

[георгий воронько]

class Program
    {
        static void Main(string[] args)
        {

            string name_inp_f = "input.txt";
            double c1 = 0;
            double c2 = 0;
            ;

            FileStream inp_f = new FileStream(name_inp_f, FileMode.Open);

            StreamReader reader_f = new StreamReader(inp_f);

            while (reader_f.Peek() != -1)
            {
                char c = (char)reader_f.Read();
                c1 = Convert.ToInt32(c);          
                c = (char)reader_f.Read();
                c2 = Convert.ToInt32(c);
            }

            c1 += c2;
            Console.WriteLine(c1);
            Console.ReadKey();

        }
    }

В файле есть цифра 1 и 3 (между 1 и 3 есть пробел) . ожидалось, что программа прочитает их и поместит в переменные С1 и С2. после должно произойти сложение и вывести результат на консоль. Но в итоге вместо 4 я получаю 65589. Почему? (Цель всего этого, понять как работать с фалами и их нюансы)

Comments

[GavriKos]

Отладчик в помощь, собственно.

[cicatrix]

А так вот не проще:

double c1, c2;
string[] vals = File.ReadAllText("input.txt").Split(' ');
double.TryParse(vals[0], out c1);
double.TryParse(vals[1], out c2);

Конкретно у вас проблема в том, что метод Read вашего ридера возвращает char. А файл у вас, скорее всего в кодировке UTF-8 (то бишь по 2 байта на символ). Используйте метод Parse или TryParse, чтобы сконвертировать Char в Double.
Даже в случае ASCII файла, c1 бы присвоился ASCII код символа '1' = 49, а с3 - код символа '3' = 51

[георгий воронько]

cicatrix, спасибо за решение и объяснение.

[cicatrix]

георгий воронько, И ещё совет - приучите себя сразу - всегда любые операции ввода-вывода обвязывать конструкцией try...catch, и никогда не принимайте за данность то, что вы получите из файла. Все полученные данные должны проходить валидацию на соответствие ожидаемому формату.

[георгий воронько]

cicatrix, да,это знаю.

[георгий воронько]

cicatrix, Кстати есть минус в этом, если я перенесу символ на другую сроку то ошибка мол "Индекс находился вне границ массива."

[cicatrix]

георгий воронько, Логично, потому что вы ожидаете, что в строке будет 2 элемента, разделённого пробелами, а вы программе подсовываете строку, где всего одно число.
Я о том и говорю, программный код зависит от формата входного файла.
Если хотие построчно, тогда так:

var v = File.ReadAllLines("input.txt").Select(l=> int.Parse(l)).ToArray();
System.Diagnostics.Debug.Assert(v.Length > 1);
int c1 = v[0];
int c2 = v[1];

[георгий воронько]

cicatrix,

в прошлых заданиях я просто подставлял '\n' сюда вместо ' '

string[] vals = File.ReadAllText(name_inp_f).Split('\n');

но когда у меня стало больше 2 строк то у меня работа программы прекращается на
int.TryParse(vals[3], out c4);

я предполагаю что это из за того что когда я добавляю \n то я дою ему возможность прочить только еще одну строку но не больше 2.

вот код:

Console.WriteLine("введите счет 1 команды");
            string inp_data_0 = Console.ReadLine();
            string inp_data_2 = Console.ReadLine();
            string inp_data_4 = Console.ReadLine();
            string inp_data_6 = Console.ReadLine();

            Console.WriteLine("введите счет 2 команды");
            string inp_data_1 = Console.ReadLine();
            string inp_data_3 = Console.ReadLine();
            string inp_data_5 = Console.ReadLine();
            string inp_data_7 = Console.ReadLine();

            write_0.WriteLine(inp_data_0 + " ", inp_data_2 + " ", inp_data_4 + " ", inp_data_6 + " ");
            write_0.WriteLine(inp_data_1 + " ", inp_data_3 + " ", inp_data_5 + " ", inp_data_7 + " ");

            write_0.Close();
            inp_f.Close();

            string[] vals = File.ReadAllText(name_inp_f).Split('\n');

            int.TryParse(vals[0], out c1);
            int.TryParse(vals[1], out c2);
            int.TryParse(vals[2], out c3);
            int.TryParse(vals[3], out c4);
            int.TryParse(vals[4], out c5);
            int.TryParse(vals[5], out c6);
            int.TryParse(vals[6], out c7);
            int.TryParse(vals[7], out c8);

[cicatrix]

георгий воронько, Скажите, а вы вообще отладчиком умеете пользоваться? Ну вот, например, поставить точку останова, да понаводить мышкой на нужные вам переменные? Мне кажется, многие вопросы бы у вас снялись сами собой.

write_0.WriteLine(inp_data_0 + " ", inp_data_2 + " ", inp_data_4 + " ", inp_data_6 + " ");
 write_0.WriteLine(inp_data_1 + " ", inp_data_3 + " ", inp_data_5 + " ", inp_data_7 + " ");

Записали в файл 2 строки вида
0 2 4 6
1 3 5 7

string[] vals = File.ReadAllText(name_inp_f).Split('\n');

Считали 2 (ДВЕ) строки, так как раделитель - \n
Дальше на vals[2] должно высветиться IndexOutOfRangeException так как элементов в массиве всего 2.
0 2 4 6 и
1 3 5 7

Метод WriteLine, как и предполагает название, записывает в поток ОДНУ строку, завершая её символом перевода строки и возврата каретки. Вы бы файлик открыли, да посмотрели, что туда записалось.

[георгий воронько]

cicatrix, В тот то и дело, не чего не записалось, потому, что до записи дело не доходит. Да и еще, у меня проблем не возникает когда работаю только с 2 строками в файле но когда 3 и больше то вот такая у меня фигня -_-

[cicatrix]

георгий воронько, У вас всего 2 строки в файле

[георгий воронько]

write_0.WriteLine(inp_data_0 + " ", inp_data_1 );
 write_0.WriteLine(inp_data_2+ " ", inp_data_3 )
 write_0.WriteLine(inp_data_4 + " ", inp_data_5)
 write_0.WriteLine(inp_data_6 + " ", inp_data_7)

Забыл что я поправил, а то там просто в 4 и более строк работаю.

Answer 1

[cicatrix]

StreamReader по умолчанию открывает файл в кодировке UTF-8 и считает, что в одном символе 2 байта.
Вам явно указывать кодировку файла в инициализации StreamReader.
И вообще, хорошей практикой будет считать строку (ReadLine), потом уже производить её разбор.
И, как я уже указал в комментарии, приведение Char к Int32 вернёт код символа, а не строковое значение.
Для того, чтобы превратить символ "3" в число 3 надо пользоваться методом Parse.