Здравствуйте, MScanner, Вы писали:

MS>могу сказать следующее
MS>Желательно унаследовать класс MyDB от интерфейса IDisposable и соответсвенно реализовать его.
MS>Потомучто у тебя в классе есть три переменные:
MS>FileStream fs;
MS>BinaryReader br;
MS>BinaryWriter bw;

MS>Которые используют кретические ресурсы (файл) и в твоей реализации я не нашел нигде где эти ресурсы освобождаются. Понятно что придет сборщик мусора — но когда он придет никому не известно, а в этот момент ты сделал с файлом все что тебе нужно и он может потребоваться кому нибудь другому.
MS>То что ты не освобождаеш критические ресурсы может привести к ошибкам.

Тут создается всего один статический экземпляр MyDB на всю программу, поэтому я не задумался об освобождении ресурсов.
А насчет комментариев, да, все время откладываю написание комментов, а потом и вовсе забиваю. Собственно тут это не так критично, потому что делаю проект только я. Но я стараюсь исправится.