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

    • ёж твою медь...
      Ответить
    • Ай-яй-яй. Надо было определить эти функции в терминах get_field_by_id()!
      Ответить
    • как же хочется узнать, где такое используется
      Ответить
      • система управления заявками и номерами гостиницы
        Ответить
        • адрес сайта в студию :D
          Ответить
          • Закрытая система =(

            Хотя наверное оно и к лучшему, такое показывать людям нельзя =)
            Ответить
    • Етить. А если $this->registry['sql']->getNumberRows() == 0 ?
      Ответить
    • В некоторых местах системы, есть такое:
      Внимание! Код далее - модель в 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);	
      	}
      Ответить
      • Бляяяяяяяяяя
        I have nothing more to say
        Ответить
      • если бы вы не предупредили, я бы сказал, что это вьюшка... нафига в модели хранить форматирование?
        Ответить
        • я как раз этим и занимаюсь, что выношу верстку во вьюшки, а из модели кидаю ассоциативный массив, по пути избавляясь от адовых функций типа get_user_adress($id) переписывая тонны бесполезных SQL запросов в 1.
          Ответить
          • неблагодарная работа. но полезная.
            обидно, когда кидают на поддержку высера юниора
            Ответить
    • показать все, что скрытоvanished
      Ответить

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