Как правильнее будет использовать оператор if?
Зависит от содержания success/not_success.
Если внутри not_success, например, нет оператора выхода (return, throw, хоть goto), то во втором случае выполнится сначала not_success, а потом success.
Имхо, второй вариант предпочтительнее, если success - это большой кусок кода или в нем есть дополнительные проверки условий (вложенные if).