Как правильно сделать if на php?

Question

[Артём Сысолятин]

Скрипт выполняет три действия: добавить информацию, удалить информацию и изменить информацию. После выполнения определённой операции происходит редирект на главную страницу с определённым GET параметром "status".
Необходимо в зависимости от это значения "status" вывести строчку, описывающую действие, а если get от "status" нет, то не выводить переменную status, либо отправить её пустой.

Мой код:

// Получение переменной статуса действия
	 if(isset($_GET['status'])) {
		$getstatus = $_GET['status'];
		if ($getstatus = "add") {
			$status = "<div class='alert alert-success alert-dismissable'><button type='button 'class='close' data-dismiss='alert' aria-hidden='true'>&times;</button>Контент успешно добавлен!</div>";
		}
		elseif ($getstatus = "del") {
			$status = "<div class='alert alert-success alert-dismissable'><button type='button 'class='close' data-dismiss='alert' aria-hidden='true'>&times;</button>Контент успешно удалён!</div>";
		}
		elseif ($getstatus = "edit") {
			$status = "<div class='alert alert-success alert-dismissable'><button type='button 'class='close' data-dismiss='alert' aria-hidden='true'>&times;</button>Контент успешно изменён!</div>";
		}
	}

Но при любом get параметре у меня выводится сообщение про успешное добавление, если в коде все elseif заменить просто на if, то выводится при любом get-запросе про успешное изменение контента.

В чём моя ошибка?

Answer 1

[Никита Маркелов]

Замени "=" на "==".
А вообще, код печален:(

Comments

[Артём Сысолятин]

Спасибо, помогло. А чем печален код?

[Никита Маркелов]

Если вы работаете на количество кода, то все окей. Иначе, зачем же одно и тоже писать?
---------------------
$action = 0;
if ($getstatus=='add') $action = "добавлен";
else if ($getstatus=='del') $action = "удален";
else if ($getstatus=='edit') $action = "изменен";
if ($action){
$status = "×Контент успешно ".$action."!";
}
else{
//обработка исключения
}
---------------------
Чтобы было красивее, можно использовать switch.
Чтобы было еще красивее, используйте framework:)

[Артём Сысолятин]

В плане php приложений только учусь, framework так и не получилось пока никакой освоить

Answer 2

[Оптимус Пьян]

Ну во-первых вроде
$getstatus = ($_GET['status']);

Во-вторых сделайте так и посмотрите что вообще у вас в $getstatus
var_dump ($getstatus)

Comments

[Никита Маркелов]

Нет, скобки ненужны, как и var_dump. Если бы проблема была в этом, условия бы вообще не выполнялись.

[Оптимус Пьян]

var_dump как раз бы показал что в $getstatus нет $_GET['status']

[Никита Маркелов]

Эм, с какой стати его там нету?

[Артём Сысолятин]

@Entii есть он там, второе = надо было поставить просто

[Никита Маркелов]

@pingvi69 Я ведь об этом и сказал.