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

    +175

    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
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    23. 23
    24. 24
    25. 25
    26. 26
    27. 27
    28. 28
    29. 29
    30. 30
    31. 31
    32. 32
    33. 33
    34. 34
    35. 35
    36. 36
    37. 37
    38. 38
    39. 39
    40. 40
    41. 41
    42. 42
    43. 43
    44. 44
    45. 45
    46. 46
    47. 47
    48. 48
    49. 49
    50. 50
    51. 51
    52. 52
    53. 53
    54. 54
    55. 55
    56. 56
    57. 57
    58. 58
    59. 59
    60. 60
    61. 61
    62. 62
    63. 63
    64. 64
    65. 65
    66. 66
    67. 67
    68. 68
    69. 69
    70. 70
    71. 71
    72. 72
    73. 73
    74. 74
    75. 75
    76. 76
    77. 77
    78. 78
    79. 79
    80. 80
    81. 81
    82. 82
    83. 83
    84. 84
    85. 85
    86. 86
    87. 87
    88. 88
    89. 89
    90. 90
    function get_user_name($id) {
		$query = "
			SELECT name
			FROM users 
			WHERE id = '".$id."'
		";
		$this->registry['sql']->query($query);
		if ($this->registry['sql']->getNumberRows()>0) {
			$result = $this->registry['sql']->getFetchObject();
			return $result[0]->name;
		}
	}
	
	function get_user_login($id) {
		$query = "
			SELECT login
			FROM users 
			WHERE id = '".$id."'
		";
		$this->registry['sql']->query($query);
		if ($this->registry['sql']->getNumberRows()>0) {
			$result = $this->registry['sql']->getFetchObject();
			return $result[0]->login;
		}
	}
	
	function get_user_password($id) {
		$query = "
			SELECT password
			FROM users 
			WHERE id = '".$id."'
		";
		$this->registry['sql']->query($query);
		if ($this->registry['sql']->getNumberRows()>0) {
			$result = $this->registry['sql']->getFetchObject();
			return $result[0]->password;
		}
	}
	
	function get_user_role($id) {
		$query = "
			SELECT role
			FROM users 
			WHERE id = '".$id."'
		";
		$this->registry['sql']->query($query);
		if ($this->registry['sql']->getNumberRows()>0) {
			$result = $this->registry['sql']->getFetchObject();
			return $result[0]->role;
		}
	}
	
	function get_user_adress($id) {
		$query = "
			SELECT adress
			FROM users 
			WHERE id = '".$id."'
		";
		$this->registry['sql']->query($query);
		if ($this->registry['sql']->getNumberRows()>0) {
			$result = $this->registry['sql']->getFetchObject();
			return $result[0]->adress;
		}
	}
	
	function get_user_valuta($id) {
		$query = "
			SELECT valuta
			FROM users 
			WHERE id = '".$id."'
		";
		$this->registry['sql']->query($query);
		if ($this->registry['sql']->getNumberRows()>0) {
			$result = $this->registry['sql']->getFetchObject();
			return $result[0]->valuta;
		}
	}
	
	function get_room_type($id){
		$query = "
			SELECT type
			FROM rooms 
			WHERE id = '".$id."'
		";
		$this->registry['sql']->query($query);
		if ($this->registry['sql']->getNumberRows()>0) {
			$result = $this->registry['sql']->getFetchObject();
			return $result[0]->type;
		}	
	}

    в догонку к Говнокод #7684

    Запостил: vkontakte, 29 Августа 2011

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

    • ava Lure Of Chaos 29.08.2011 11:11 # +2

      ёж твою медь...
      Ответить

    • ava roman-kashitsyn 29.08.2011 11:16 # +3

      Ай-яй-яй. Надо было определить эти функции в терминах get_field_by_id()!
      Ответить

    • ava striker 29.08.2011 12:05 # 0

      как же хочется узнать, где такое используется
      Ответить

      • ava vkontakte 29.08.2011 12:26 # +1

        система управления заявками и номерами гостиницы
        Ответить

        • ava Lure Of Chaos 29.08.2011 12:40 # 0

          адрес сайта в студию :D
          Ответить

          • ava vkontakte 29.08.2011 12:45 # 0

            Закрытая система =(

            Хотя наверное оно и к лучшему, такое показывать людям нельзя =)
            Ответить

    • ava ling 29.08.2011 12:30 # 0

      Етить. А если $this->registry['sql']->getNumberRows() == 0 ?
      Ответить

    • ava vkontakte 29.08.2011 12:33 # +1

      В некоторых местах системы, есть такое:
      Внимание! Код далее - модель в MVC
      function show_colonization_list(){
		global $status_arr;
		global $class_arr;
		
		$filter = "";
		$html = "";
		
		if(isset($_POST['filter_application'])) {
			$filter .= "WHERE parent>0";
			if(!empty($_POST['start_date'])) {
				$start_date_str = strtotime($_POST['start_date']);
				$start_date = date('Y-m-d', $start_date_str);
				$filter .= " AND start ='".$start_date."'";
			}
			if(!empty($_POST['end_date'])) {
				$end_date_str = strtotime($_POST['end_date']);
				$end_date = date('Y-m-d', $end_date_str);
				$filter .= " AND end ='".$end_date."'";
			}
		} 
		
		$query = "
				SELECT *
				FROM application 
				".$filter."
				ORDER BY date_action DESC
			";
		$this->registry['sql']->query($query);
		if ($this->registry['sql']->getNumberRows()>0) {
			$html .= "<table class=\"ap_list\">
						<tr>
						<th>№ заявки</th>
						<th>Наименование турфирмы</th>
						<th>Дата заезда</th>
						<th>Дата выезда</th>
						<th>Тип номера</th>
						<th>Корпус</th>
						<th>Этаж</th>
						<th>Номер</th>
						</tr>";
			foreach ($this->registry['sql']->getFetchObject() as $oRow) {
				if($oRow->bed > 0) $bed = '<img src="'.site_url.'/_styles_/pics/bed_icon.gif" class="bed_icon">'; 
				else $bed = '';
				$html .= '<tr>
							<td>'.$bed.$oRow->id.'</td>
							<td>'.$this->get_user_name($oRow->parent).'</td>
							<td>'.date('d-m-Y', strtotime($oRow->start)).'</td>
							<td>'.date('d-m-Y', strtotime($oRow->end)).'</td>
							<td>'.@$class_arr[$this->get_room_type($oRow->room)].'</td>
							<td>'.$this->get_room_corpus($oRow->room).'</td>
							<td>'.$this->get_room_floor($oRow->room).'</td>
							<td>'.$this->get_room_number($oRow->room).'</td>
							</tr>';	
			}
			$html .= "</table>";
		}
		$this->registry['template']->set('col_list', $html);	
	}
      Ответить

      • ava roman-kashitsyn 29.08.2011 12:36 # 0

        Бляяяяяяяяяя
        I have nothing more to say
        Ответить

      • ava Lure Of Chaos 29.08.2011 12:41 # +2

        если бы вы не предупредили, я бы сказал, что это вьюшка... нафига в модели хранить форматирование?
        Ответить

        • ava vkontakte 29.08.2011 12:44 # 0

          я как раз этим и занимаюсь, что выношу верстку во вьюшки, а из модели кидаю ассоциативный массив, по пути избавляясь от адовых функций типа get_user_adress($id) переписывая тонны бесполезных SQL запросов в 1.
          Ответить

          • ava Lure Of Chaos 29.08.2011 12:46 # +2

            неблагодарная работа. но полезная.
            обидно, когда кидают на поддержку высера юниора
            Ответить

    • ava guest8 09.04.2019 11:24 # −999

      показать все, что скрытоvanished
      Ответить

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