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

    +179

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    $sql=mysql_query("SELECT id FROM table");
$id=mysql_num_rows($sql)+1;

$qry=mysql_query("INSERT INTO table(id, .....)  VALUES ('$id','.....')")
    or die (mysql_error());

    Осуществление auto_increment-а!
    Угадайте, что будет если кто то удалить одну строку из таблицу?

    Запостил: gotha, 14 Июля 2010

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

    • ava Vasiliy 14.07.2010 10:47 # 0

      тупа как то ну хоть select max(id) использовал
      Ответить

      • ava kal1sha 14.07.2010 10:49 # +5

        а лучше в таблице указать auto_increment и не делать порнографию
        Ответить

        • ava Vasiliy 14.07.2010 10:53 # −3

          не всегда. иногда нужны записи пронумерованные с от 1 до N инкрементом такого не добьешся он хранит своё максимальное значение
          Ответить

          • ava kal1sha 14.07.2010 10:58 # +2

            в этой задаче он делал auto_increment это чётко видно
            Ответить

            • ava murz 17.07.2010 21:14 # 0

              Если удалить последнюю запись, а потом снова начать добавлять, то уже не будет работать как auto_increment.
              Ответить

          • ava Lure Of Chaos 14.07.2010 11:04 # −1

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

          • ava Uchkuma 14.07.2010 11:26 # +6

            id он на то и id, что должен быть уникальным для каждой конкретной записи. Если запись удалили, то больше не должно быть записи с таким id. А если нужны по порядку пронумерованные записи, то нужно вводить еще одно поле с нумерацией. Например number. И уж там нумеровать как твоей душе угодно.
            Ответить

            • ava Vasiliy 14.07.2010 11:31 # −2

              вот реальная ситуация нужно выгружать в таблицу инфу о рейтингах топ 100 за день для последующей обработки. зачем там уникальное поле. есть дата есть позиция в топе. все зачем нужны лишние поля
              Ответить

              • ava kal1sha 14.07.2010 11:40 # −2

                если есть дата можно применить такую магию:

                id=секунды - const, где const=секунда в которой запустился проект
                Ответить

              • ava Uchkuma 14.07.2010 12:31 # +2

                Мне тебя жаль, если ты не знаешь, зачем нужны уникальные поля.
                Например, что ты будешь делать, если надо будет принудительно поменять местами 2 позиции в топе?

                Узкоспецифических примеров, конечно, можно много придумать.
                Ответить

              • ava turdman 14.07.2010 12:46 # 0

                Это натуральный ключ, получаемый из входных данных, и быть он может чем угодно.

                В говне - синтетический. И наложение условий 'от 1 до N' на такой ключ само по себе говно
                Ответить

    • ava kal1sha 14.07.2010 10:47 # 0

      будет больно.
      Ответить

    • ava bugmenot 14.07.2010 12:32 # +5

      Подполковник Очевидный поясняет: mySQLные обезьяны имеют весьма смутное представление о том, что такое первичный ключ; зато в них палками вдолбили, что первая колонка должна называться id
      Ответить

      • ava Анонимус 14.07.2010 14:14 # +7

        Серди MySQLных есть и грамотные люди. Но мало.
        А вот пересечение множества PHPшников и множества MySQLщиков точно является подмножеством обезъян.
        Ответить

    • ava inj 15.07.2010 21:51 # +2

      За такое надо убивать ещё в детстве. Только-только из пизды вынул и сразу за две ноги на его об пол.
      Ответить

      • ava Lure Of Chaos 15.07.2010 22:14 # +1

        а заодно и папку укуренного и мамку проститутку
        Ответить

      • ava WebKill_HochetVBan 15.07.2010 22:39 # 0

        Неужто в жизни ниразу ошибок не делали? Пускай живёт. Нужно давать шанс исправиться.
        Ответить

        • ava inj 15.07.2010 23:25 # +1

          Тут как бы эмоции.

          Но если серьёзно, то
          а) зачем вытаскивать все записи чтобы узнать их количество
          б) как можно по количеству назначать ИД?
          ц) КО как бы намекает, что даже select max(id) тут плох, потому что к БД может обращаться более одного пользователя одновременно.
          д) используются непараметризованые запросы => д.1) возможны проблемы с безопасностью, д.2) проблемы с производительностью БД
          Чел не шарит в программировании (см пп.а, б), чел не шарит в программировании БД (пп. ц, д).
          Говнокод фееричен и я решил высказаться в таком же духе. Тут некоторые яйца рвут налево и направо, а я вот так поступаю.
          Ответить

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