на первый взглфд вроде все верно, единственное лучше держать структуру однотипных данных