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

    +165

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    $user_query .= " AND O.ID IN (";
for ($i=0;$i<count($orgs_id_array)-1;$i++){
	$user_query .= $orgs_id_array[$i];
	if($i < count($orgs_id_array) - 2)
		$user_query .= ",";
}
$user_query .= ")";

    Остается загадкой, почему нельзя было сделать:
    $query_user .= " AND O.ID IN (" . implode(",", $orgs_id_array) . ")";

    Запостил: AAS, 25 Марта 2010

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

    • ava AAS 25.03.2010 15:50 # +1.2

      Касательно последнего элемента массива $orgs_id_array - он _всегда_ пустой, неясно почему нельзя было пропускать его на этапе формирования массива (:
      Ответить

      • ava stailer 25.03.2010 16:56 # +1.4

        Пожалуй этот факт значительно забавнее самого поста =)
        Ответить

    • ava guest 25.03.2010 17:23 # +3.4

      для использования implode нужно знать про implode.
      Ответить

    • ava guest 26.03.2010 16:35 # 0

      Я тоже раньше так делал. Только мне join больше нравится, из-за своей аналогии с методом String.join() в JavaScript
      Ответить

      • ava guest 26.03.2010 19:27 # 0

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

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