Структура if

[zumm]

Доброго всем! Вот возник у меня вопрос, как лучше организовывать структуру проверок. Вот так:

result = 'complete';

if(Проверка 1)
{
  result = 'Ошибка 1';
}

if(Проверка 2)
{
  result = 'Ошибка 2';
}

if(result == 'complete')
{
  //какие то действия
}

return result;

или вот так

if(Проверка 1)
{
  if(Проверка 2)
  {
      //какие то действия
  }
  else
  {
    result = 'Ошибка 2';
  }
}
else
{
  result = 'Ошибка 1';
}

return result;

понятно что во втором случае происходит меньше проверок, да и если для какой то проверки нужны какие то еще действия, например запрос, то они будут произведены только если никаких посторонних ошибок не было.

но в первом случае код имеет меньшую вложенность и более удобен к восприятию.

[Stilet]

Третий (лучший на мой взгляд) вариант:

result = 'complete';

if(Проверка 1)        result = 'Ошибка 1';
else if(Проверка 2)  result = 'Ошибка 2';
else  //какие то действия


return result;

[zumm]

Stilet, а чем он отличается от второго?

[Stilet]

Читабельнее. По крайней мере для меня
О! И я койче увидел:

if(! Проверка 1)        result = 'Ошибка 1';
else if(! Проверка 2)  result = 'Ошибка 2';
else  //какие то действия

А то получается с точностью да наоборот

[Arigato]

Я бы сделал так:

result = 'complete';
switch ( true )
{
  case <Проверка 1>: result = 'Ошибка 1'; break;
  case <Проверка 2>: result = 'Ошибка 2'; break;
  default: // какие-то действия;
};
return result;

[zumm]

Stilet, вот часть функции, над которой я работаю в данный момент, она хороша для примера:

		
function initLogin($userEmail, $userPassword)
{			
	if($userEmail <> null and $userPassword <> null)
	{
		$userPassword = md5(md5($userPassword).md5(md5($userPassword).md5($userEmail)));
		$temp = mysql_fetch_array(mysql_query("SELECT * FROM `users` WHERE 
				`email` = '".$userEmail."' AND `pass` = '".$userPassword."'"));
		
		if(count($temp) <> 1)
		{
			if($temp['ban'] == 0)
			{
				if($user['status'] <> 0)
				{
				
				}
				else
				{
					$result = 'Аккаунт не активирован.';
				}
			}
			else
			{
				$result = 'Аккаунт заблокирован.';
			}
		}
		else
		{
			$result = 'Данные введены не верно.';
		}
	}
	else
	{
		$result = 'Заполните все поля.';
	}
}

разве так не более читабельно, в отличии от того, если бы заполнение "result" было бы там же, где и промежуточные действия, то бишь сверху?

[Stilet]

Мне вложенка по глазам бьет.
Честно, я не люблю так писать. При анализе приходится глазами водить вверх-вниз.
Но это касается только стиля написания. Если тебя это устраивает то почему бы нет

[flance]

zumm

Первый вариант не равен второму.

Во-первых, во 2 варианте результатом будет "ошибка х", если проверки НЕ пройдены, а в 1 варианте наоборот, если проверки пройдены.
Вы скажите, что в первом случае подразумевается

if(!Проверка 1)

но в некоторых случаях это влияет на читаемость, а ведь тема именно об этом.

Во-вторых, во 2 варианте если уже Ошибка 1, то проверок больше не будет и "какие-то действия" выполняться не будут. Это оптимальнее по скорости. В 1 варианте не так.

В-третьих, в 1 варианте Ошибка 2 перекрывает Ошибку 1, если она есть. Спрашивается - зачем мы тогда о ней узнавали? Это является ошибкой программиста.

[Arigato]

Сообщение от zumm

Stilet, вот часть функции, над которой я работаю в данный момент, она хороша для примера:

Тут, как минимум, можно сократить объем кода:

function initLogin($userEmail, $userPassword)
{			
	if($userEmail <> null and $userPassword <> null)
	{
		$userPassword = md5(md5($userPassword).md5(md5($userPassword).md5($userEmail)));
		$temp = mysql_fetch_array(mysql_query("SELECT * FROM `users` WHERE 
				`email` = '".$userEmail."' AND `pass` = '".$userPassword."'"));
		if(count($temp) <> 1)
		{
			if($temp['ban'] == 1) $result = 'Аккаунт заблокирован.';
                          elseif($user['status'] == 0) $result = 'Аккаунт не активирован.';
		} else $result = 'Данные введены не верно.';
	} else $result = 'Заполните все поля.';
}

Кстати, меня смущает еще это условие: (count($temp) <> 1)

[Вадим Мошев]

Мне по душе больше первый (он читабельнее, но пусть работает чуть медленне), хотя я тоже считаю, что первый код и второй - не одно и то же, хотя бы потому, что результат первого кода в случае выполнения условия <Проверка 2> будет таким:

result = 'Ошибка 2'

, в то время, как у второго кода будет результат такой

//какие то действия

. Может автор допустил какую-то ошибку, тогда можно взять свои слова обратно.

Хотя могу предложить ещё вот примерно такое

if (Result != 'Complete')
	{
	if (Проверка 1) Return 'Ошибка 1'
		
		else Return 'Ошибка 2'
	}

// Какие-то действия