Как подправить код php?

Question

[smoky-jo]

здравствуйте, не могу сообразить, как правильно подправить код, чтобы функция clearCreditsTransAction если вот это условие не выполнено при обновлении базы, не запускалась

`email`='$email_get'"

function clearCreditsTransAction($id) {

	global $db;
	$email_i=users::getEmail($id);
	$clear_credits = $db->fetchRow("update ".TABLE_USERS." set `no_credits` = 0 where `id`='$id' and `email` ='$email_i'");
	}

и второй

function trasactionCredits($id='') {

		$host = 'localhost';
		$database = 'strue';
		$user = 'root';
		$password = '';
		$link = mysqli_connect($host, $user, $password, $database) or die("Ошибка: ".mysqli_error($link));
	$email_get=users::getEmail($id);

global $db;
$no_credits_user_el = $db->fetchRow("select `no_credits` from ".TABLE_USERS." where `id`='$id'");
$no_credits_rub = $no_credits_user_el*65;
$transaction_to_account=$link->query("update `class_users` set `no_credits` = `no_credits`+$no_credits_rub where `email`='$email_get'");
$email_i=users::clearCreditsTransAction($id);
	}

я понимаю что сам запускаю эту функцию после обновления базы, но мне нужно ее запустить только в том случае, если email = $email_get, в остальных случаях эта функция не нужна, не могу сообразить как сделать, подскажите пожалуйста

Answer 1

[bedolazhka]

примерно так:

set no_credits = if(email = $email_i, 0, no_credits) where id = $id

если email у $id соответствует $email_i, то пишем 0, иначе оставляем без изменения

Comments

[smoky-jo]

bedolazhka спасибо большое, попробовал так

$clear_credits = $db->fetchRow("update ".TABLE_USERS." set `no_credits` = if(email = $email_i, 0, no_credits) where `id`='$id' and `email` ='$email_i'");

но теперь не обнуляется значение даже если email соответствует..

[bedolazhka]

smoky-jo, убрать из запроса: and `email` ='$email_i' - эта проверка уже есть в условии if(), проверить синтаксис по кавычкам и проверить нужно ли запрос апдейта присваивать переменной $clear_credits внутри функции, где нет проверки подсчитанных строк.

[smoky-jo]

bedolazhka,

$clear_credits = $db->fetchRow("update ".TABLE_USERS." set `no_credits` = if(`email` = '$email_i',0, `no_credits`) where `id`='$id_i'");

сделал вот так, проверка срабатывает, но, если вызвать первую функцию

trasactionCredits при этом даже если проверка срабатывает,

то значение no_credits обновляется на 0..

[bedolazhka]

smoky-jo, а если не срабатывает проверка?

[smoky-jo]

bedolazhka, то получается как раз то что мне нужно было, одна строка обнуляется, во второй базе прибавляется

[bedolazhka]

smoky-jo, ну у Вас в вопросе условие если не выполнено, то не нужно обновлять, я Вам как бы под него дал пример

делайте, как Вам нужно

[smoky-jo]

bedolazhka, так в том то и дело, не понимаю почему он обновляет на 0, если условие не выполнено, откуда он его берет..

[bedolazhka]

smoky-jo, по умолчанию может задано значение 0, поэтому 0 не берётся откуда-то, но остаётся при условии несовпадения, то есть и так и так 0 из-за значения по дефолту

[smoky-jo]

bedolazhka, все, я понял в чем проблема была, допилил, вместо 0 подставил текущее значение этого поля просто

$get_balanse_cred=users::getUserCreditsForMenu($id);
	$clear_credits = $db->fetchRow("update ".TABLE_USERS." set `no_credits` = if(`email` = '$email_i',$get_balanse_cred, `no_credits`) where `id`='$id_i'");

спасибо огромное за помощь))

[bedolazhka]

smoky-jo,

`no_credits` = if(`email` = '$email_i',$get_balanse_cred, `no_credits`)

текущее значение поля `no_credits` и будет `no_credits` иначе БД будет производить запись в БД переменной, которую Вы лишним запросом определили, Вы не понимаете суть if() а данном ответе