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

    +89

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    function uIsCommentAllowed($id_all)
	{
		$db = 'i_all';
		$this->db->select('comment');
		$this->db->where('id_all',$id_all);
		$query = $this->db->get($db);
		if ($query->num_rows() == 0) return false;
		else 
		{
			$row = $query->row();
			if ($row->comment == 1) return true;
			if ($row->comment == 1) return false;
		}	
		return false;
	}

    Кто не видит где говно, показываю: if ($row->comment == 1) return true; if ($row->comment == 1) return false;
    Теперь главный вопрос - почему на при тестировании данная ошибка не проявляется, и все вообще работает верно?

    Запостил: increazon, 25 Апреля 2012

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

    • ava rdifb0 25.04.2012 20:11 # +1

      Потому что функция возвращает значение и дальше не выполняется, или в чем трабла ?
      Ответить

      • ava increazon 25.04.2012 20:37 # −5

        показать все, что скрытонет блядь, не угадал. $row->comment возвращает как 0 та и 1. Но все почему-то работает правильно.
        Ответить

        • ava bormand 26.04.2012 06:17 # 0

          потому что, блядь, return false в if ($row->comment == 1) return false никогда не выполняется, или условие не подходит или уже из функции вышли
          Ответить

        • ava santa_microbe 26.04.2012 08:49 # +3

          ну логично, если $row->comment == 0, то функция выйдет в 14 строке со значение false, разве нет ?
          Ответить

          • ava increazon 26.04.2012 16:17 # 0

            Да, угадал. Потому и работает правильно. Так что делать? Строку 12 отредактировать на if ($row->comment == 0) return false; или удалить вообще?
            Ответить

            • ava koodeer 26.04.2012 16:51 # +1

              > Так что делать?
              Пиздуй учить уроки!
              Здесь тебе не форум, где помогают нубам.
              Ответить

            • ava santa_microbe 27.04.2012 09:12 # +1

              ну попробуй как-то так
              if ($query->num_rows() == 0) return false;
$row = $query->row();
return $row->comment ? true : false;

              Если я правильно понял, то у тебя $row->comment может быть либо 1, либо 0
              Ответить

              • ava roman-kashitsyn 27.04.2012 09:36 # +1

                а приведение типов не сработает в этой ситуации?
                return ($query->num_rows() && $query->row()->comment);
                Ответить

                • ava santa_microbe 27.04.2012 09:54 # 0

                  Ну так даже еще лучше, я об этом не подумал. Век живи - век учись
                  Ответить

                • ava increazon 27.04.2012 21:07 # 0

                  $query->row()->comment выдаст ошибку. Нужно именно 2 строчки писать:
                  $row = $query->row();
                  $row->comment
                  Ответить

                  • ava roman-kashitsyn 27.04.2012 22:27 # 0

                    "I was really, really bad at writing parsers. I still am really bad at writing parsers." Rasmus Lerdorf
                    Ответить

                    • ava guest 27.04.2012 22:54 # 0

                      начиная с третьей версии языка лердофф к нему никакого отношения фактически не имеет
                      Ответить

    • ava FaiGovno 25.04.2012 20:25 # +1

      новый урсус?
      Ответить

    • ava LightBlack 25.04.2012 20:47 # 0

      Автор, видимо, только что с Delphi пересел на PHP. И теперь путает педали паскалевский Result и сишный return.
      Ответить

    • ava wvxvw 25.04.2012 20:50 # +3 

      if (x) return true;
if (x) return false;

      Это какое-то раздвоение личности, но у второй личности шансов нет... :(
      Ответить

    • ava Lure Of Chaos 25.04.2012 23:38 # +1

      да ладно. вот я видел что-то вроде такого (даже где-то тут было):
      fucntion get_line($file) {
  $f=fopen($file,'r');
  $l=fgets($f);
  return $l;
  fclose($f);
}
      Ответить

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