1. PHP / Говнокод #6271

    +161

    1. 1
    2. 2
    3. 3
    $Qstatus = $osC_Database->query('select max(cms_id) as cms_id from cms');
        $Qstatus->execute();
        $cms_id = $Qstatus->valueInt('cms_id') + 1;

    Код в OSCommerce использующийся для увеличения id на 1.

    Запостил: Insane18, 07 Апреля 2011

    Комментарии (17) RSS

    • ava Int 07.04.2011 17:11 # 0

      Не доверяй sql, доверяй php.
      Ответить

    • ava guest 07.04.2011 17:28 # −1

      вообще имеет смысл, если записей в cms_id нет, так как max() вернет NULL.
      Ответить

    • ava guest 07.04.2011 19:13 # 0

      Это не совсем говнокод, просто вы не поняли зачем так сделано. Я раньше тоже считал это аццким говнокодом. Первый раз увидел такое в phpBB 2.x.
      Суть в том, что на cms_id стоит autoincrement. И если вы случайно добавляете строку в конец таблицы, потом ой, удалите ее, то id все равно уже на единицу прыгнет. А вот таким способом как приведен в коде это пытаются побороть.

      ИМХО.
      Ответить

      • ava bugmenot 07.04.2011 19:23 # +1

        а "cms" в именах энтитей призвано убеждать нас что это - CMS
        Ответить

      • ava sectus 08.04.2011 01:49 # 0

        Нахрена?

        И, блин, я боюсь представить что делается после того, как удалится что-то в середине.
        Ответить

        • ava hidespb 08.04.2011 09:13 # 0

          ну, что-то вроде

          UPDATE table SET id = id -1 WHERE id > $deleted_id ORDER BY id
          Ответить

          • ava sectus 08.04.2011 09:17 # 0

            Нахрена так делать?
            Ответить

            • ava hidespb 08.04.2011 09:21 # 0

              Без понятия
              просто предположил как происходит delele в этой системе :)
              Ответить

              • ava sectus 08.04.2011 09:33 # 0

                Это риторический вопрос в пустоту. Гость скорее всего не ответит. : (
                Ответить

            • ava Lure Of Chaos 08.04.2011 10:41 # 0

              экономим айдишники, а то вдруг закончатся?
              Ответить

              • ava sectus 08.04.2011 11:19 # 0

                Хорошая вещь - экономия.
                Ответить

                • ava eth0 08.04.2011 19:37 # 0

                  Экономика должна быть экономичной, как завещал нам товарищ Горбачёв.
                  Ответить

      • ava Int 11.04.2011 10:00 # 0

        Так если есть автоинкремент, зачем +1 этот?
        Ответить

    • ava guest 08.04.2011 20:42 # 0

      Это OsCommerce 3.0 alpha5.
      Говнокод действительно служит заменой auto_increment, а таблица cms хранит статьи.
      Ответить

    Добавить комментарий