Видимо, так оно и выглядит. Непонятно, зачем такую тяжёлую функцию вызывать в while, почему там while, а не for (количество писем ведь не меняется), и почему так извращённо увеличивается i. Но это уже называется "стилем" - каждый волен сам выбирать путь решения.
Юбилейный DevConfX пройдет 21-22 июня в Москве. Как всегда — Вы решаете, кто попадет в программу секции Backend — голосуйте за интересные доклады