Знакомые с php side-effects, подскажите плз (вопрос по логике дублеловки)

Опубликовано ср, 27/08/2008 - 15:09 пользователем pkn

Вот есть (file author.inc lines 539-542) такой кусочек кода:

    if (1*$oldver >= 1*$ver) {
      unlink ($f);
      return $r . " Версия не лучше: ".bl($BookId)." $oldver:$ver<br>";
    }

Смысл операции 1*$ver если я правильно понимаю - сказать php что переменная $ver именно численная, а не строка. Вопрос: а не может ли такого быть, что (как иногда бывает в C) что результатом 1*$ver станет integer, даже если $ver содержало 1.1 ?

Комментарии

Опубликовано ср, 27/08/2008 - 15:32 пользователем jno

Может.
За все версии ПХП не скажу, но в 3 так и было.
Мне пришлось довольно навороченную процедуру ваять для корректного преобразования в число.
Что-то типа

<?php
function a2i($s) {
  $t0 = trim($s); if(!$t0) return 0;
  $t1 = "1".$t0;
  $t2 = 0 + $t1;
  return ($t2 - 10*strlen($t0));
}
?>

Опубликовано ср, 27/08/2008 - 16:08 пользователем pkn

jno написал:
Может.

Блин... :((

Опубликовано чт, 28/08/2008 - 03:22 пользователем Anarchist

Хочу напомнить, что давеча прекратили поддержку четвёртого пых-пыха.

Опубликовано ср, 27/08/2008 - 17:03 пользователем ketmar

а зачем не $ver+0? и никто ничего не теряет, и получаем число.

Опубликовано ср, 27/08/2008 - 19:26 пользователем pkn

Это не мой код.

Опубликовано ср, 27/08/2008 - 19:26 пользователем pkn

Кажется, отбой пока. Кажется, работает этот код нормально. По крайней мере при $oldver=1.0 и $ver=1.001 сравнение происходит корректно. А более экзотические варианты когда $oldver=1 или $oldver undefined меня пока не интересуют.

Опубликовано пн, 01/09/2008 - 09:51 пользователем urlin 

<?php
    if (floatval ($oldver) >= floatval ($ver)) {
      unlink ($f);
      return $r . ' Версия не лучше: ' . bl($BookId) . ' ' . $oldver . ':' . $ver . '<br>';
    }
?>

Опубликовано вт, 02/09/2008 - 10:31 пользователем pkn

Спасибо. Есть специальная floatval, значит. Правда, тесты показали, что этот код и так работает. Но все равно спасибо, конечно.
X
HNS.to is a highly insecure way of browsing Handshake domains and should only be used for demo or educational purposes. Click to see preferable resolutions methods