Как проще передать все данные (поля из таблицы) в массив?

Question

[Евгений Иванов]

Вот рабочий код.
Этот метод выбирает значения полей id и field1 и возвращает массив. Если в таблице будет 100 полей, то тогда придется писать портянку вида field1...field100. Наверно, есть способ проще.
Как проще передать все данные (поля из таблицы) в массив?

static function GetList() {
$query = "SELECT  * FROM `mytable` ";
$queryResult = MySQL::MySQLQuery($query);

while ($row = $queryResult->fetch_assoc()) {
$MyListArray[] = array(
// Тут может быть 100 полей
'id'=>$row['id'],
'field1'=>$row['field1'],
// Тут однотипная "портянка"...field2...field3...
// 'field100'=>$row['field100']
);
}
return $MyListArray;
}

Comments

[ThunderCat]

Это шутка или реально тупняк?

[FanatPHP]

а в чем смысл переливать из пустого в порожнее, из массива $row делать другой, точно такой же массив?

Answer 1

[Slava Rozhnev]

Если строк не очень много забираем всё одной командой:

$query = "SELECT  * FROM `mytable` ";
$result = $mysqli->query($query);
$rows = $result->fetch_all(MYSQLI_ASSOC);

PHP online test

Иначе в цикле:

$MyListArray = [];

while ($row = $queryResult->fetch_assoc()) {
	$MyListArray[] = $row;
};

Comments

[FanatPHP]

только $MyListArray[] надо сначала инициализировать ;)

[Slava Rozhnev]

FanatPHP, как скажешь ...

[Евгений Иванов]

Slava Rozhnev, FanatPHP, странно, работает и без иницализации и вроде не жалуется (нет warning или я их просто не увидел)

$MyListArray = []; //  Это точно надо?
while ($row = $queryResult->fetch_assoc()) {
  $MyListArray[] = $row;
};

Все же рекомендуете инициализировать массив?
Без инициализации будут проблемы? (в другой версии php например )

[FanatPHP]

Евгений Иванов, дело не в версиях. а в Буратино
https://habr.com/ru/post/19463/

[Евгений Иванов]

FanatPHP, с одной стороны да, с другой нет.
Это верно, что надо инициализировать объекты/переменные. Так удобнее контролировать его содержимое (а вдруг там что то уже есть?).
С другой стороны, любой объект изначально уже инициализирован т.е. имеет какое либо значение.
Аналогично и с удалением объекта из памяти.

Во многих языках программирования мы не инициализируем объект.
В некоторых языках программирования мы не удаляем объект из памяти (есть сборщик мусора).

В общем, по моему мнению - инициализировать объект не нужно, нужно правильно использовать область видимости объекта и соблюдать парадигмы ООП (инкапсуляцию).

Вот и даже джаваскрипт к этому вроде пришел - от var к let.
Ну а C# давно уже там...

[FanatPHP]

Евгений Иванов, "С другой стороны, любой объект изначально уже инициализирован т.е. имеет какое либо значение" это я не понял совсем, а по поводу диванных рассуждений про инкапсуляцию - есть правила хорошего тона, которые позволяют сохранить немного волос на жопе. Но дело хозяйское.

[Евгений Иванов]

FanatPHP,

это я не понял совсем.

Переменная = участок памяти. Там по определению что то есть. "Мусор", например.
Но я имел ввиду неявную инициализацию.

В общем вот пример из C#.
Я тоже руководствовался "Буратино", но тимлид мне сказал, что я не прав.

public static double GetAccruals(int ServiceId, double currentReadings = 0.0, double previousReadings = 0.0)
            {
                double summ = 0; // Вот тут я не прав.
                summ = (currentReadings - previousReadings) * Convert.ToDouble(SQLiteBase.GetServiceTariff(ServiceId));
                return summ;
            }

[FanatPHP]

1. и где здесь любой объект изначально уже инициализирован? что здесь инициализировано "изначально"?

2. сам пример не в кассу, очень плохо что вы не видите разницу. здесь идет два присваивания, первое действительно лишнее. пример с массивом совсем другой, присваивание в нем только одно.