LINUX.ORG.RU

История изменений

Исправление crutch_master, (текущая версия) :

Ну поменяй 

if (!lalala) {
     return false;
}
на 
if (!lalala) return false;
Убери else там, где он не нужен 
    if (!$user = get_complete_user_data('email', $username)) {
        if (!auth_billing::create_user($username)) {
            return false;
        }

        $user = get_complete_user_data('email', $username);
    }
Вот тут не знаю, нужно ли присваивать $user второй раз, по идее присвоение должно произойти в условии. Тогда второе условие можно просто прицепить в первое (&&) или сперва присвоить $user, а потом уже плясать. Вроде в php есть еще конструкции типа $condition or return false;
В целом, не понимаю в чём проблема. Какая разница сколько там if...else. Такой код пишется один раз, а потом его никто никогда не открывает.

Исправление crutch_master, :

Ну поменяй 

if (!lalala) {
     return false;
}
на 
if (!lalala) return false;
Убери else там, где он не нужен 
    if (!$user = get_complete_user_data('email', $username)) {
        if (!auth_billing::create_user($username)) {
            return false;
        }

        $user = get_complete_user_data('email', $username);
    }
Вот тут не знаю, нужно ли присваивать $user второй раз, по идее присвоение должно произойти в условии. Тогда второе условие можно просто прицепить в первое (&&). Вроде в php есть еще конструкции типа $condition or return false;
В целом, не понимаю в чём проблема. Какая разница сколько там if...else. Такой код пишется один раз, а потом его никто никогда не открывает.

Исправление crutch_master, :

Ну поменяй 

if (!lalala) {
     return false;
}
на 
if (!lalala) return false;
Убери else там, где он не нужен 
    if (!$user = get_complete_user_data('email', $username)) {
        if (!auth_billing::create_user($username)) {
            return false;
        }

        $user = get_complete_user_data('email', $username);
    }
Вот тут не знаю, нужно ли присваивать $user второй раз, по идее присвоение должно произойти в условии. Тогда второе условие можно просто прицепить в первое (&&). Вроде в php есть еще конструкции типа $condition or return false;[br] В целом, не понимаю в чём проблема. Какая разница сколько там if...else. Такой код пишется один раз, а потом его никто никогда не открывает.

Исправление crutch_master, :

Ну поменяй 

if (!lalala) {
     return false;
}
на 
if (!lalala) return false;
Убери else там, где он не нужен 
    if (!$user = get_complete_user_data('email', $username)) {
        if (!auth_billing::create_user($username)) {
            return false;
        }

        $user = get_complete_user_data('email', $username);
    }
Вот тут не знаю, нужно ли присваивать $user второй раз, по идее присвоение должно произойти в условии. Тогда второе условие можно просто прицепить в первое (&&).
В целом, не понимаю в чём проблема. Какая разница сколько там if...else. Такой код пишется один раз, а потом его никто никогда не открывает.

Исходная версия crutch_master, :

Ну поменяй 

if (!lalala) {
     return false;
}
на 
if (!lalala) return false;
Убери else там, где он не нужен 
    if (!$user = get_complete_user_data('email', $username)) {
        if (!auth_billing::create_user($username)) {
            return false;
        }

        $user = get_complete_user_data('email', $username);
    }
Вот тут не знаю, нужно ли присваивать $user второй раз, по идее присвоение должно произойти в условии. Тогда второе условие можно просто прицепить && в первое.
В целом, не понимаю в чём проблема. Какая разница сколько там if...else. Такой код пишется один раз, а потом его никто никогда не открывает.