Проблема при вычитании?

Question

[viktorross]

здравствуйте, не могу разобраться в чем проблема, не работает вычитание меньшего из большего числа. Например скрипт считает 1,500.00 меньше чем 900.00. Но допустим 999,00 меньше чем 900.00 не считает и нормально высчитывает все.

Вот кусок кода который должен

require_once "include/include.php";
require_once "classes/coupons.php";
require_once "classes/priorities.php";
require_once "classes/validator.php";
require_once "classes/users.php";
require_once "classes/payment_processors.php";
else { 

			$processor = escape($_POST['processor']);

			$found=0;
			$processors = new payment_processors();
			$payment_processors = $processors->getActivePaymentProcessors();
			foreach ($payment_processors as $p) if($p['processor_code'] == $processor) { 
				$found=1;
			}

			if($processor=="credits") {

				$needed_credits = $amount/$credits_settings['unit'];
				$needed_credits = format_price($needed_credits, '', '', '.');
				$current_credits = credits::creditsForUser($crt_usr);

				if($needed_credits>$current_credits) { 

					$ret['error']=$lng['credits']['not_enough_credits']; 
					$ret['response'] = 0; 
					
					global $appearance_settings;
					if(strtolower($appearance_settings['charset'])!="utf-8") $ret = utf8_encode_all($ret);

					echo json_encode($ret);
					exit(0);

				}
			}

			if(!$found && ($processor != "credits" || !$allowed)) { 

				$ret['error'] = $lng['buy_package']['error']['invalid_processor']; 
				$ret['response'] = 0;
				
				global $appearance_settings;
				if(strtolower($appearance_settings['charset'])!="utf-8") $ret = utf8_encode_all($ret);

				echo json_encode($ret);
				exit(0);

			}

Выводится в шаблон так через smarty 

<code>
<span class="price">{$current_credits}</span>
</code>

Куда копать , в чем может быть проблема?

Answer 1

[Stalker_RED]

Проблема в том, что вы пытаетесь выполнять арифметические действия и сравнивать СТРОКИ, а не числа.

https://www.php.net/manual/ru/function.floatval.php

Comments

[viktorross]

Stalker_RED получается мне надо переписывать весь код заново? этот подправить не получится?

[Stalker_RED]

viktorross, я же ссылку дал.

[viktorross]

Stalker_RED да я не могу понять что сделать, я только недавно начал php изучать, сижу второй день пытаюсь исправить ничего не получается

[Stalker_RED]

viktorross, ну вот смотри, сравнение строк происходит посимвольно.

Сравним строки "ABC" и "ACB". Первые символы одинаковые. На второй позиции "B" и "C". Сравниваем их коды. "C" больше чем "B", значит вторая строка больше.

Если сравнить строки "900" и "99", то первые символы одинаковы, на второй позиции девятка и ноль. Так как строка "9" больше чем "0", то "99" больше чем "900".

Чтобы такой херней не случалось, нужно сравнивать не строки, а числа. Открой уже ссылку и попробуй прочесть.

[viktorross]

Stalker_RED уже понятнее, спасибо большое
Я прочитал про floatval
но я все равно не пойму как это оформить..
ошибка у меня как я понял в этой строке

$current_credits = credits::creditsForUser($crt_usr);

Число credits сравнивается с creditsForUser так вот я не пойму как заменить это все на сравнение чисел как Вы написали, логику я понял , как это вообще работает , а саму реализацию в моем случае не пойму

[Stalker_RED]

$x = '99.01'; // это строка '99.01'
$y = floatval($x); // это число 99.01

хз зачем я это пишу, в примерах же есть.

[viktorross]

Stalker_RED, я думал все гораздо сложнее, смотрите как я делал после чего ошибка 500 появлялась

if($processor=="credits") {

				$needed_credits = $amount/$credits_settings['unit'];
				$needed_credits = format_price($needed_credits, '', '', '.');
                                $current_credits = floatval($credits_formatted);
				$current_credits = credits($credits_formatted)::creditsForUser($crt_usr,$credits_formatted);

так без ошибки,но ничего не меняется

$current_credits = floatval($current_credits);
			    $current_credits = credits::creditsForUser($crt_usr);

[Stalker_RED]

viktorross,

$current_credits = floatval($current_credits); // переменную current_credits вы переводите во float, отлично.
// правда непонятно откуда взялось ее значение, но допустим оно там есть.

$current_credits = credits::creditsForUser($crt_usr); // вызываете какой-то credits::creditsForUser() который
// перезаписывает эту $current_credits. То есть работа проделанная на предыдущей строке пропала.

так без ошибки,но ничего не меняется

[viktorross]

Stalker_RED, отсюда берется creditsForUser может тут что-то изменить?

public static function creditsForUser($user_id)
    {
        if (!$user_id) {
            return 0;
        }
        global $db;
        $credits = $db->fetchRow("select `no_credits` from " . TABLE_USERS . " where `id`='" . $user_id . "'");
        if (!$credits) {
            return 0;
        }
        return number_format($credits, 2);
    }
    public static function userHasCredits($user_id, $price)
    {
        if (!$user_id) {
            return 0;
        }
        global $db;
        global $credits_settings;
        $no_credits = ceil($price / $credits_settings["unit"]);
        $has_credits = $db->fetchRow("select `no_credits` from " . TABLE_USERS . " where `id`='" . $user_id . "'");
        if ($has_credits < $no_credits) {
            return 0;
        }
        return $has_credits;
    }

[Stalker_RED]

viktorross, number_format() возвращает строку.

[viktorross]

Stalker_RED, все равно не работает..

return number_format(floatval$credits, 2);

точно также считает число 1,000,00.00 меньше чем 100.00

[Stalker_RED]

viktorross, забей