Ты сначала проверяешь строку на соответствие регулярке, а потом парсишь Integer.parseInt. Можно сразу строку распарсить в инт (только нужно в try-catch) и проверить на то, что оно больше нуля и сразу вернуть из функции это число, чтобы не выполнять лишние операции

Выбор коллекции не кажется обоснованным. Каждый новый продукт по сути это новое значение в мапе, а мапа занимает больше памяти, чем обычный список. Поэтому кажется, что простой ArrayList подойдёт лучше для хранения какого-нибудь класса Product с двумя полями - названием и ценой