for это плохо, потому что в нем можно ошибиться, а View у нас тестами не покрыта.

Рекомендуется использовать метод setModel у JComboBox, он позволяет вставлять *списки* целиком вместо того чтобы писать циклы по обработке отдельных записей.