Здравствуйте, d4rkme551ah, Вы писали:
D>Меня вызвали на собеседование по .NET и сказали привезти с собой проекты. Но я так волнуюсь, у меня их мало, всего 2, и немножко стесняюсь показать им свой проект. Мне нужно ваше мнение, что бы вы сказали о моих знаниях глядя на следующий код?
D>http://d4rkme551ah.hostzi.com/DBManager.cs
D>http://d4rkme551ah.hostzi.com/PlxDevice.cs
бегло посмотрел
http://d4rkme551ah.hostzi.com/DBManager.cs
могу сказать следующее
Желательно унаследовать класс MyDB от интерфейса IDisposable и соответсвенно реализовать его.
Потомучто у тебя в классе есть три переменные:
FileStream fs;
BinaryReader br;
BinaryWriter bw;
Которые используют кретические ресурсы (файл) и в твоей реализации я не нашел нигде где эти ресурсы освобождаются. Понятно что придет сборщик мусора — но когда он придет никому не известно, а в этот момент ты сделал с файлом все что тебе нужно и он может потребоваться кому нибудь другому.
То что ты не освобождаеш критические ресурсы может привести к ошибкам.
И еще комментариев вообще нет, особенно во втором файле так как он более менее сложный.