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

0

  1. 1
  2. 2
  3. 3
  4. 4
  5. 5
  6. 6
  7. 7
  8. 8
  9. 9
  10. 10
  11. 11
  12. 12
  13. 13
  14. 14
  15. 15
  16. 16
  17. 17
  18. 18
  19. 19
function SentMessage($Text,$Title) {

	$r=mysql_query("SELECT `id`,`login`,`email` FROM `".$this->TableUsers."` WHERE `access` = 1 and `check_email`=1");

	while( $res = mysql_fetch_array($r) ){               
		mysql_query("
			INSERT INTO `".$this->MessTable."` ( 
				`From` , `To` , `Date` , `Title` , `Text` 
			) VALUES (
				'".$this->MainUser['id']."', '".$res[id]."', NOW(), '".$Title."', '".$Text."'
			)
		");

		$body=BuildBody("mail.message", $res);

		$headers=' /*... заголовок  ...*/ ';
		mail ( $res[email], " /*... отправитель ..*/ ", $body, $headers);
	}
}

вот такое чудо враждебной техники откопалось при колупании одной cms'ки...
(при том что выборка из $this->TableUsers с такими условиями получалась порядка ~500 т. строк )

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

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

  • ava Lure Of Chaos # Ссылка на оригинал 0

    запрос в цикле к запросу - развидьте меня это видеть....... при этом BuildBody - глобальная функция, ойй.
    Ответить

  • ava Анонимус # Ссылка на оригинал 0

    люди так пафосно юзают объекты, и при этом тут же в куче mail, сборка письма и обращение к базе.
    И ведь наверное думают что они пишут ОО код. Карго-культ блин)

    а уж инсерты в цикле это круть
    Ответить

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

      ну ООП это же круто. И пох что он не совсем ООП и не везде..... ООП тут у нас как декорации, да
      Ответить

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

    function Kernel_panic(){
  $r = mysql_query("SELECT * FROM table");
  while($res = mysql_fetch_array($r) ){
    mysql_query("INSERT INTO table VALUES('...','...','...')");
  }
}
    Ответить

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

      И где Здесь паник. ? мускул отработает не даже не вспатеет. Вот 61 join вызовет локальный коласпс. 🙂
      Ответить

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

        если вкл persistent connection, то даже при большой табле мускул даже не возмутится кол-вом работы.

        даа, 61 жойн, особенно если много данных...
        Ответить

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

      Об этой функции известно всему коллективу. Новоприбывших, кто когда-нибудь собственноручно добирается до неё встречают фразой: "Ни в коем случае не запускай!", - но никто не решается это удалить почему-то... Так и висит...
      Ответить

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

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

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

    ↑
    8
    ↓