Вообще, в идеале одна строка кода должна делать только одно конкретное действие. В примере выше делаются два действия: a++ и +1.
То же относится к функциями. Функция должна делать только одно конкретное действие. Чтобы написать правильную функцию нужно задать вопрос: "Что сделать?".
Пример хорошей функции: возвращает максимальное значение из массива.
Пример ПЛОХОЙ функции: находит максимальное значение И выводит его. Почему такая функция плохая? Потому, что она делает два действия: 1) находит макс; 2) выводит его.
Надо сделать две функции. И действие функции отразить в названии: 1) find_max; 2) print_value.