Кожному своє ) Але ідея має право бути)
Взагалі я обрав варіант не присвоювати у condition і налаштував інспекції у phpstorm

доречі як називається це правило в phpstorm для php?

поясніть будь-ласка мені нубу чому так буде погано?
працювати ж буде так само?

-----

не треба важких редакторів які хочуть гігабайти місця на диску та гігабайти оперативки ---
ставимо легесенький малесенький Notepad++
(який "вміє" багато мов програмування та підтримує будь-яке розмалювання + можна додати будь-яку мову + багато плагінів)

ставимо легесенький малесенький плагін сніппети
налаштовуємо-додаємо свої сніппети які вставляються на місце курсора по одному кліку

все геніальне - просте 

Погано буде тому що стає більше коду в першу чергу. Його важче читати так як доводиться робити реверт умови.
Якщо юзер не дорівнюж нулл тоді нічого не робимо а в іншому випадку створюємо юзера. Це ж неможливо читати.

П.с. про сніпнети знаю але генерити поганий код в такий спосіб я не хочу. Я якось покажу плагін для сніпнетів у рнрштормі який я написав)

працювати буде так же, а коду більше на 1-2 символа, я ж не помиляюся?
не можу погодитись що його читати важче, на мою думку його легше читати (я розумію що це кому як)

imho важче читати коли є заперечення заперечення чи навіть заперечення заперечення заперечення --- мені й таке щастячко зустрічалося (також заперечення нашим мозком свідомо погано сприймається чи навіть взагалі не сприймається - це взагалі окрема тема психології)

чим меньше ми створюємо конструкцій без реальної а не видуманої потреби - тим легше нам і тим хто працюватиме з кодом після нас?

ви називаєте це (1-2 зайвих символа для спрощення читання- збільшення прозорості) говнокодом?
а де ж тут говнокод?
(а як же любителі чіпляти jquery щоб один раз блок по айді смикнути чи додати два числа чи працювати з регуляркою, як же любителі чіпляти jq-ui замість десятка строк pure css ?)

Іноді все ж варто - наприклад, коли іде кілька перевірок, і одна вийшла ось такою "інвертованою", є сенс лишити пустий блок, а не інвертувати умову. Але в будь-якому разі якщо блок пустий, треба написати в ньому в коментарі, чому так.

/* preparing parameters */
if( $data !== NULL ){
  $prepared_data = prepare($data);
} else {
  $prepared_data = "";
}

if( $additional_data !== NULL ) {
  $connection = make_some_changes( $connection, $additional_data );
} 

if( $dbase !== NULL ){
  $dbase->restore_connection($connection);
} else {
  $dbase = new DBase($connection);
}

if( $user !== NULL ) {
  /* OK, user is ready */
} else {
  $user = new User();
}

$dbase->save_data_and_log( $prepared_data, $user );