Не корректная работа функции сравнения строк?

Question

[Artem Kirsov]

Сделал функцию сравнения двух строк с дальнейшей отметкой если строки не равны.

function checkFailureMasterSumm($masterSumm, $controlSumm){
	if(!empty($masterSumm) && !empty($controlSumm)){
		if(strcasecmp($masterSumm, $controlSumm)){
			return 'master_failure_summ';
		}
	}
}

Работает вроде нормально. Но если передать в одной строке - 0, то всё ломается.

Где я туплю?

Comments

[generate]

почему сразу не проверять strcasecmp ?
empty считает что 0 это пустая строка

Answer 1

[Slava Rozhnev]

Ваша проблема в сравнении:

if(!empty($masterSumm) && !empty($controlSumm)){

Согласно документации:

Возвращает false, если var существует и содержит непустое ненулевое значение или строку false, смотрите Преобразование в булев тип. В противном случае возвращает true.

В любом случае, если я правильно понимаю переменные $masterSumm, $controlSumm содержат числа. Если так то и сравнивать их следует как числа.

Answer 2

[Rsa97]

empty(0) === true

Comments

[Artem Kirsov]

Странная эта функция.
Переписал так

function checkFailureMasterSumm($masterSumm, $controlSumm){
	if(is_numeric($masterSumm) && is_numeric($controlSumm)){
		if(intval($masterSumm) != intval($controlSumm)){
			return 'master_failure_summ';
		}
	}
}

Вроде работает нормально.

[Vitsliputsli]

Artem Kirsov, empty реализует простой код: !isset($var) || $var == false. Т.е. во второй части используется динамическое преобразование типов, либо нужно хорошо знать все нюансы динамического преобразования в PHP, либо пользоваться более очевидными конструкциями.
Используйте явное объявление типов, приводите типы к нужным вам вручную, иначе вам придется просчитывать все возможные варианты для всех типов, а если сюда добавляется еще и динамическое преобразование, то шансы совершить ошибку очень большие.
К примеру, если сделать так:

function checkFailureMasterSumm(int $masterSumm, int $controlSumm): bool
{
    return ($masterSumm !== $controlSumm);
}

то очевидно, что приходит на вход функции, и что будет на выходе. "master_failure_summ" пусть напишет то, что занимается выводом, а эта функция просто сравнивает суммы master и control. Если под Summ понимается не вычисляемое значение или требуется значения больше чем int - то принимайте на вход строки.