"PHP" / Говнокод #7295 Ссылка на оригинал

0

  1. 1
  2. 2
  3. 3
  4. 4
  5. 5
  6. 6
  7. 7
  8. 8
  9. 9
  10. 10
  11. 11
$varians_ids = array();
$getvaluesvariants=mysql_query("SELECT  `id` FROM `values` group by `id`") or die(mysql_error());;
while($valuesvariants = mysql_fetch_array($getvaluesvariants, MYSQL_ASSOC))
{
$varians_ids[]= $valuesvariants['id'];
}
mysql_free_result($getvaluesvariants)or die("Query failed: " . mysql_error());
for ($i = 0;  $i < count($varians_ids); $i++)
{
  $setvaluesnames = mysql_query("UPDATE `option_values` SET `option_value_ru` = (SELECT  `option_value_ru` FROM `values` WHERE `id` = '$varians_ids[$i]') WHERE  `id` = '$varians_ids[$i]'") or die(mysql_error());;
}

Наткнулся на этот шедевр. Отсутствие использования foreach кажется мелочью по сравнению с тем, как производится операция, которую можно сделать одним запросом:
$setvaluesnames = mysql_query("UPDATE `option_values`, `values`
SET
`option_values`.`option_value_ru` = `values`.`option_value_ru`
WHERE `option_values`.`id` = `values`.`id`") or die(mysql_error());

P. S.: там еще в БД пишется пробел вместо пустой строки или NULL...

Запостил: CRRaD CRRaD, (Updated )

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

      • ava manyrus # Ссылка на оригинал 0 ⇈

        Плюсы pdo по сравнению с mysqli:
        1) Более продумана в плане ооп(моё субъективное мнение)
        2) Нет привязки к одному драйверу.
        3) Нет этих "?", можно красиво сделать ":name" 🙂
        Ответить

        • ava inkanus-gray # Ссылка на оригинал 0 ⇈

          Зато mysqli побуждает выполнять s/mysql/mysqli/g, не читая документацию, после чего на форумах могут появляться новые темы «Пачиму не работает?», а здесь — новые говнокоды. Так же веселее!
          Ответить

  • ava nobody # Ссылка на оригинал 0

    в цикле запрос на апдейт, это конечно сильно ...
    Ответить

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

Помни, guest, за тобой могут следить!

    А не использовать ли нам bbcode?

    ↑
    8
    ↓