Почему функция нестабильно читает файл?

Question

[Дмитрий Королев]

Функция иногда не прочитывает файл
Как это исправить?

static public void ReadParam()
        {
            BinaryReader br;
            string path = new FileInfo(Assembly.GetExecutingAssembly().Location).DirectoryName + "\\noticeprogramdata.dat";
            try
            {
                string locale = string.Empty;
                int enume = -1;
                List<ScheduleCell> cells = new List<ScheduleCell>();
                br = new BinaryReader(new FileStream(path, FileMode.OpenOrCreate));
                try
                {
                    newStart = br.ReadBoolean();
                    wD1 = br.ReadBoolean();
                    wD2 = br.ReadBoolean();
                    wD3 = br.ReadBoolean();
                    wD4 = br.ReadBoolean();
                    eD1 = br.ReadBoolean();
                    eD2 = br.ReadBoolean();
                    eD3 = br.ReadBoolean();
                    eD4 = br.ReadBoolean();
                    dayToday = br.ReadByte();
                    DateTimeOffset.TryParse(br.ReadString(), CultureInfo.InvariantCulture, DateTimeStyles.None, out dateTimeOld);

                    locale = br.ReadString();
                    enume = br.ReadInt32();
                    for (;;)
                    {
                        cells.Add(new ScheduleCell(br.ReadString(), br.ReadInt32()));
                    }
                }
                catch (EndOfStreamException exc)
                {
                    Console.WriteLine(exc.Message.ToString());
                }
                br.Close();
                schedule = new ScheduleCollection(cells, (ScheduleCollection.SortDirectionEnum)enume, locale);
            }
            catch (Exception exc)
            {
                Console.WriteLine(">>>"+exc.Message.ToString());
                if (br != null)
                    br.Close();
            }
        }

использую этот код в проекте windows forms

Comments

[Foggy Finder]

Не совсем понятно что вы имеете ввиду под "Функция иногда не прочитывает файл"

[Дмитрий Королев]

Foggy Finder, вот файл есть, а чтение файла оказывается неудачным
потому что функция не в тех условиях расположена(о разрешениях что ли) или недостаточно реализована
в результате прога думает что файла будто нет и данные программы слетают

[Foggy Finder]

Дмитрий Королев, у вас есть возможность прикрепить файл и показать функцию записи в файл?

[Роман]

Выведите в консоль/лог или посмотрите с дебагере значение path чтобы убедиться что функция пытается читать тот файл что нужно.
Так же при невозможности чтения кидается вполне конкретный Exception, по сообщению которого должно быть ясно в чём проблема.

[Дмитрий Королев]

Foggy Finder,

static public void WriteParam()
        {
            BinaryWriter bw;
            string path = new FileInfo(Assembly.GetExecutingAssembly().Location).DirectoryName + "\\noticeprogramdata.dat";
            try
            {
                bw = new BinaryWriter(new FileStream(path, FileMode.Create, FileAccess.Write));
                bw.Write(newStart);
                bw.Write(wD1);
                bw.Write(wD2);
                bw.Write(wD3);
                bw.Write(wD4);
                bw.Write(eD1);
                bw.Write(eD2);
                bw.Write(eD3);
                bw.Write(eD4);
                bw.Write(dayToday);
                bw.Write(String.Format(CultureInfo.InvariantCulture, "{0:g}", dateTimeOld));

                bw.Write(schedule.Locale);
                bw.Write((int)schedule.SortDirection);
                if (schedule.ItemsAvailable)
                {
                    foreach (ScheduleCell cell in schedule)
                    {
                        bw.Write(cell.cell1);
                        bw.Write(cell.cell2);
                    }
                }
                bw.Close();
            }
            catch (Exception exc)
            {
                System.Windows.Forms.MessageBox.Show(exc.Message);
                Console.WriteLine(">>>" + exc.Message.ToString());
                if (bw != null)
                    bw.Close();
            }
        }

[Дмитрий Королев]

Роман, тот файл
я про разрешения пытаюсь здесь узнать

[Дмитрий Королев]

Foggy Finder, я ищу литературу про разрешения и систему безопасности

Answer 1

[eRKa]

Запустите студию с правами администратора и проверьте, будет ли та же проблема. Если нет, значит дело в правах. Если будет, смотрите на размер файла, буфер, блокировку файла другими процессами, возможно делается одновременная попытка чтения из других потоков.

Comments

[Дмитрий Королев]

eRKa, дело в том что поток один в программе.
проблема возникает не когда ты ее пытаешься воспроизвести

[Дмитрий Королев]

eRKa, я ищу литературу про разрешения

[Дмитрий Королев]

eRKa, а про блокировку другими процессами поподробнее
как этого избежать?
это проги файл, какого фига что то левое будет с ним ковыряться?

[eRKa]

Дмитрий Королев, возможно, что Вы же его открыли на чтение и не закрыли. Я не знаю всего Вашего кода, поэтому мне на пальцах судить тяжело.
Я бы посоветовал создание ридера запихнуть в using

using(var fs = new FileStream(path, FileMode.OpenOrCreate))
using(var br = new BinaryReader(fs))
{
	string path = new FileInfo(...);
	try
	{ 
		...
		schedule = new ScheduleCollection(...);
	}
	catch 
	{ ... }
}

В этом случае можно быть точно уверенным, что все закроется после отработки.

[Дмитрий Королев]

eRKa, ну да, было раньше напрасно написано показать исключение в окне чтобы прервать программу при исключении исключение было про разрешения но потом я удалил эту строку
но суть в том что мне надо чтоб избежать провала
читает при запуске приложения и происходит провал не всегда правда
так раньше было написано:

...}
            catch (Exception exc)
            {
                System.Windows.Forms.MessageBox.Show(exc.Message);
                Console.WriteLine(">>>"+exc.Message.ToString());
                if (br != null)
                    br.Close();
            }

но по сравнению с провалом то что потом стало написано не так страшно
я давно с этой проблемой так то
мне в прошлый раз что то насоветовали
но я так правильно и не подошел к решению (про разрешения)
вроде в документацию отправили и я по горячим следам не раскрыл суть
недавно зашел в документацию об этом где я нашел множество непоследовательной информации как мне кажется
есть дорожка как воспринять об этом?
что трогает пользователя?

[eRKa]

Дмитрий Королев, Вы на русский гугл переводчиком переводили?)) Очень уж странные обороты речи)

Я думаю, что проблема в ручном закрытии потока br.Close(). Такая вещь должна происходить в разделе
finally, что собственно и делает using

try
	{ 	}
	catch 
	{   }
	finally
	{
		if (br != null)
            br.Close();
	}

[Дмитрий Королев]

eRKa, а я думаю в неосмысленности того как работает приложение в режиме полного или частичного доверия

Answer 2

[tupovat_bydlovat]

Понекропостчу.
Вместо ловли еоф-эксепшона и цикла фор(;;) можно сравнить position и length стрима в качестве условия while.
А по вопросу - вы там ловите один единственный конкретный эксепшон (ну или стопку конкретных эксепшонов) и только в нём закрываете ридер, при этом вызывая кучу всяких readInt32 и создавая попутно кучу других объектов реализация которых может так же хромать и возможно кидать эксепшоны.
По хорошему обернуть надо в 2 юзинга, как советовали выше. Если очень хочется ловить эксепшоны - ридер закрывать надо в finally, как тоже советовали выше, а чтобы все таки с проблемой разобраться начните с ловли и логирования остальных эксепшонов, а не только проблем с правами и кривыми путями.

Comments

[Дмитрий Королев]

tupovat_bydlovat, да, поправил на отлов других исключений
а в проблемах с правами я нуб и прошу навести меня на литературу о них