Javascript-форум (https://javascript.ru/forum/)
-   Серверные языки и технологии (https://javascript.ru/forum/server/)
-   -   помогите пожалуйста улучшить код. (https://javascript.ru/forum/server/24619-pomogite-pozhalujjsta-uluchshit-kod.html)

Duda.Ml1986@gmail.com 07.01.2012 20:36

помогите пожалуйста улучшить код.
 
Первое что вызывает вопрос, можно ли одним махом записать все поля одного массива в другой. что бы не перечеслять все, так как они не всегдя будут присутствовать в запросах (возможно будут ошибки а try нагромождать нет желания)

Насколько правильный такой подход(безопасность прошу не учитывать).
Прошу указать на недостатки.

Может более совершенный вывод?
$select = query("SELECT *  FROM  `article`  ORDER BY id DESC LIMIT 10 ","select");
	$i=0;
while(isset($select[$i]['id'])){
	
	echo $select[$i]['id']."<br>";
	$i++;
}



function query($string_query,$do){
	
		if($do="select"){
			$query = sprintf($string_query);
			$result = mysql_query($query) or die("Query failed : " . mysql_error());
			$number = 0;
		while ($line = mysql_fetch_array($result, MYSQL_ASSOC)) 
		{
			$array_select[$number]['id']=$line['id'];
			$array_select[$number]['name']=$line['name'];
			$array_select[$number]['text']=$line['text'];
			$number++;	
		}
		return $array_select;
		}

devote 07.01.2012 20:41

array_merge

devote 07.01.2012 20:44

так:
$array_select = array();
while ($line = mysql_fetch_array($result, MYSQL_ASSOC)) 
{
    $array_select = array_merge( $array_select, array( $line ) );
}

devote 07.01.2012 20:45

хотя можно и так:
$array_select = array();
while ($line = mysql_fetch_array($result, MYSQL_ASSOC)) 
{
    $array_select[] = $line;
}

Duda.Ml1986@gmail.com 07.01.2012 20:53

Спасибо Господин devote.
(Господин потому как это вежливое обращение в Чехии, если перевести на русский, а по чешски Пан)
К сожалению плюсы Вам система поставить не дает).
Сокращение кода на 6 строчек, прекрассно!


Часовой пояс GMT +3, время: 19:20.