php, mysql, вставка данных с помощью цикла

Question

php, mysql, вставка данных с помощью цикла

0

Я получаю данные API, которые я должен хранить в базе данных. Я должен использовать цикл, потому что я не могу знать, сколько предметов клиент поставит в корзину. Для получения дополнительной информации, пожалуйста, посмотрите мой пример. Зависит от цикла. Мне удается сохранить только первый или последний элемент... Любая помощь будет оценена.

<?php

$data = array
  (
  array("Item1"),
  array("Item2"),
  array("Item3"),
  array("Item4"),

  );

$count = count($data);    

$servername = "localhost";
$username = "root";
$password = "";
$dbname = "test";

// Create connection
$conn = new mysqli($servername, $username, $password, $dbname);


$x = 0;
while($x <= $count-1) {
    $mytest = $data[$x][0];
echo $mytest;
$sql = "INSERT INTO test (mytest)
VALUES ('$mytest')";


  $x++;
}

if ($conn->query($sql) === TRUE) {
    echo "New record created successfully";
} else {
    echo "Error: " . $sql . "<br>" . $conn->error;
}

$conn->close();

user9155057 04 март 2018, в 22:16

Источник

1

ВНИМАНИЕ : При использовании mysqli вы должны использовать параметризованные запросы и bind_param для добавления пользовательских данных в ваш запрос. НЕ используйте интерполяцию или конкатенацию строк для достижения этой цели, поскольку вы создали серьезную ошибку внедрения SQL . НИКОГДА не помещайте $_POST , $_GET или любые пользовательские данные непосредственно в запрос, это может быть очень вредно, если кто-то попытается использовать вашу ошибку.
tadman 04 март 2018, в 20:36
0

Примечание. Попробуйте избавиться от привычки объявлять операторы SQL в одноразовых переменных, которые используются только один раз. Намного проще следовать коду, в котором запрос передается непосредственно в функцию, и больше нет шансов испортить и отправить $sql3 вместо визуально подобного $sql8 .
tadman 04 март 2018, в 20:36
0

Постарайтесь избавиться от привычки загромождать ваш код ненужными вещами, такими как === true . Многие функции предназначены для возврата значений, которые оцениваются как логически истинные или ложные, поэтому это избыточно.
tadman 04 март 2018, в 20:36
0

Вы никогда не говорили нам, что не работает так, как вы этого хотите.
Funk Forty Niner 04 март 2018, в 20:36
0

вы запускаете запрос вне цикла, будет добавлен только последний элемент
user6763587 04 март 2018, в 20:39
0

@Funk Forty Niner Теперь я храню только последний элемент (Item4). И мне нужно хранить их все.
user9155057 04 март 2018, в 20:47
0

Чтобы перебрать массив, используйте foreach , а не while .
Barmar 04 март 2018, в 20:51
1

И почему вы пишете $x <= $count-1 вместо более идиоматического $x < $count ?
Barmar 04 март 2018, в 20:52
0

Конструкция foreach обеспечивает простой способ перебора массивов. foreach работает только с массивами и объектами и выдает ошибку, когда вы пытаетесь использовать его для переменной с другим типом данных или неинициализированной переменной. Существует два синтаксиса: оператор foreach (array_expression as $ value) оператор foreach (array_expression as $ key => $ value)
aravind3 05 март 2018, в 03:55

Показать ещё 7 комментариев

Теги:

php

mysql

1 ответ

Ещё вопросы

ВНИМАНИЕ : При использовании mysqli вы должны использовать параметризованные запросы и bind_param для добавления пользовательских данных в ваш запрос. НЕ используйте интерполяцию или конкатенацию строк для достижения этой цели, поскольку вы создали серьезную ошибку внедрения SQL . НИКОГДА не помещайте $_POST , $_GET или любые пользовательские данные непосредственно в запрос, это может быть очень вредно, если кто-то попытается использовать вашу ошибку.
Примечание. Попробуйте избавиться от привычки объявлять операторы SQL в одноразовых переменных, которые используются только один раз. Намного проще следовать коду, в котором запрос передается непосредственно в функцию, и больше нет шансов испортить и отправить $sql3 вместо визуально подобного $sql8 .
Постарайтесь избавиться от привычки загромождать ваш код ненужными вещами, такими как === true . Многие функции предназначены для возврата значений, которые оцениваются как логически истинные или ложные, поэтому это избыточно.
Вы никогда не говорили нам, что не работает так, как вы этого хотите.
вы запускаете запрос вне цикла, будет добавлен только последний элемент
@Funk Forty Niner Теперь я храню только последний элемент (Item4). И мне нужно хранить их все.
Чтобы перебрать массив, используйте foreach , а не while .
И почему вы пишете $x <= $count-1 вместо более идиоматического $x < $count ?
Конструкция foreach обеспечивает простой способ перебора массивов. foreach работает только с массивами и объектами и выдает ошибку, когда вы пытаетесь использовать его для переменной с другим типом данных или неинициализированной переменной. Существует два синтаксиса: оператор foreach (array_expression as $ value) оператор foreach (array_expression as $ key => $ value)

aravind3 · Answer 1 · 2018-03-05T01-00-00.000Z

Цикл foreach всегда лучше, чем при повторении массива.

Конструкция foreach обеспечивает простой способ перебора массивов. foreach работает только с массивами и объектами и выдает ошибку при попытке использовать ее для переменной с другим типом данных или неинициализированной переменной. Существует два синтаксиса:

foreach (array_expression as $value)
    statement

foreach (array_expression as $key => $value)
    statement

Решение вашей проблемы:

foreach($data as $val) {
    echo $val;
    $sql = "INSERT INTO test (mytest) VALUES ('$mytest')"; 
    if ($conn->query($sql) === TRUE) {
        echo "New record created successfully";
    } else {
        echo "Error: " . $sql . "<br>" . $conn->error;
    }
}