Выберите столбец mysql (который является массивом), где определен другой столбец: - и вставьте его в переменную php

Question

Выберите столбец mysql (который является массивом), где определен другой столбец: - и вставьте его в переменную php

0

У меня есть два столбца в базе данных: - copy (который является массивом) и bookid. Мой код;

$query = mysqli_query($db, "select copy from book_outward WHERE bookid like 'B1'");
while ($row = mysqli_fetch_array($query)) {
    $copyid = $row['copy'];
}

База данных выглядит так:

+----------------+
| id copy bookid |
+----------------+
| 1  1    B1     |
| 2  2,3  B1     |
| 3  4    B1     |
| 4  2    B2     |
+----------------+

но он сохраняет только последние значения, которые были введены в "B1". Я также пробовал

$copyid[] = $row['copy'];

но в этом случае я должен каждый раз менять ключи массива вручную.

Моя цель состоит в том, чтобы вставить copy into column bookid='B1' и прежде чем он должен убедиться, что в базе данных для B1 могут быть сохранены только значения UNIQUE.

HTML: -

<input type="text" name="bookid" />
<input type="text" name="copies[]" />

PHP-код для вставки: -

$book_id = $_POST['bookid']; 
$copies = implode(',',$_POST['copies']);
$result = mysqli_query($db, "insert into book_outward(bookid,copy) values ('$book_id','$copies')");

Ronit 20 сен. 2018, в 15:13

Источник

0

какой тип данных для копий в вашей БД?
suresh 20 сен. 2018, в 13:01
0

Тип данных @suresh - "VARCHAR"
Ronit 20 сен. 2018, в 13:01
0

bookid = 'B1' или bookid like '%B1%'
Jens 20 сен. 2018, в 13:02
1

Никогда не храните значения как Что означает копирование (которое является массивом) . Можете ли вы показать пример данных
Jens 20 сен. 2018, в 13:02
0

Таким образом, вы можете иметь несколько строк с bookid как B1 но каждая из них должна иметь различное значение в столбце копирования?
dWinder 20 сен. 2018, в 13:02
0

Похоже, вы храните значения в копии как CSV, что не должно быть сделано. В этом случае узнайте о нормализации
Jens 20 сен. 2018, в 13:03
0

Ваш сценарий широко открыт для атаки SQL-инъекцией Даже если вы экранируете входные данные, это небезопасно! Используйте подготовленные параметризованные операторы в API MYSQLI_ или PDO
RiggsFolly 20 сен. 2018, в 13:04
0

У меня ужасная идея , что вы хотите сделать это $copyid .= $row['copy'] . ','; что было бы очень плохой идеей. Изучите базовое руководство по проектированию баз данных в своей библиотеке и прочитайте его
RiggsFolly 20 сен. 2018, в 13:07
0

Пример базы данных @Jens предоставляется под вопросом. Любое решение?
Ronit 20 сен. 2018, в 13:23
0

Пример базы данных @RiggsFolly предоставлен под вопрос. Любое решение?
Ronit 20 сен. 2018, в 13:24
0

@Ronit Ронит, как я прокомментировал ранее, узнал о нормализации . Это очень плохой дизайн БД для хранения значений CSV
Jens 20 сен. 2018, в 13:24
0

@Ronit в вашем примере id 4 имеет copy «2», а id 2 имеет копию «2,3» - разве вы не сказали, что они уникальны?
dWinder 20 сен. 2018, в 13:25
0

@DavidWinder Да, они должны быть уникальными, но я также написал, что это только для B1. Другие bookid могут иметь ту же копию, но копия является уникальной для конкретного bookid
Ronit 20 сен. 2018, в 13:28
0

Я дал вам решение в моем предыдущем комментарии, если вы прочитали его
RiggsFolly 20 сен. 2018, в 15:32

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

Теги:

php

mysql

arrays

3 ответа

0

Решил сам, добавив только одну строку

'$copies2 = explode(",",rtrim($copies , ','));'

перед array_intersect Благодаря коду очень помогает.

Ronit 20 сен. 2018, в 13:51

0

Таким образом, мне кажется, что проблема заключается в том, что исходная схема DB данных имеет некоторые проблемы.

SQL-подобная БД не является отличным местом для размещения значения стиля "массив". У этого уже есть механизм для этого: таблица ссылок.

Это избавит вас от необходимости вручную перетасовывать ваши первичные ключи в этой таблице. Это проще проверить и проверить.

Если вы хотите сделать это с помощью php-кода, например, у вас нет доступа SQL, например, в контролируемой среде сборки, вы должны использовать "строковые ключи" для вашего $ copyid (переименование подсказки в $ copyidmap) до группироваться вместе как части данных и сохранять ключевые отношения. поэтому следующее будет уникальным: map-> {bookId} → {copyid} → {id}//Где id - это идентификатор первичной таблицы. Очевидно, проверять и выполнять ваши неопределенные назначения массива.

Zaal Tonia 20 сен. 2018, в 12:18

Ещё вопросы

какой тип данных для копий в вашей БД?
Никогда не храните значения как Что означает копирование (которое является массивом) . Можете ли вы показать пример данных
Таким образом, вы можете иметь несколько строк с bookid как B1 но каждая из них должна иметь различное значение в столбце копирования?
Похоже, вы храните значения в копии как CSV, что не должно быть сделано. В этом случае узнайте о нормализации
Ваш сценарий широко открыт для атаки SQL-инъекцией Даже если вы экранируете входные данные, это небезопасно! Используйте подготовленные параметризованные операторы в API MYSQLI_ или PDO
У меня ужасная идея , что вы хотите сделать это $copyid .= $row['copy'] . ','; что было бы очень плохой идеей. Изучите базовое руководство по проектированию баз данных в своей библиотеке и прочитайте его
Пример базы данных @Jens предоставляется под вопросом. Любое решение?
Пример базы данных @RiggsFolly предоставлен под вопрос. Любое решение?
@Ronit Ронит, как я прокомментировал ранее, узнал о нормализации . Это очень плохой дизайн БД для хранения значений CSV
@Ronit в вашем примере id 4 имеет copy «2», а id 2 имеет копию «2,3» - разве вы не сказали, что они уникальны?
@DavidWinder Да, они должны быть уникальными, но я также написал, что это только для B1. Другие bookid могут иметь ту же копию, но копия является уникальной для конкретного bookid
Я дал вам решение в моем предыдущем комментарии, если вы прочитали его

David Winder · Accepted Answer · 2018-09-20T10-42-00.000Z

Как говорится в некоторых комментариях, это не лучший способ, но это возможно.

Вы можете получить все copyid данных:

$query = mysqli_query($db, "select copy from book_outward WHERE bookid like 'B1'");
$copyid = "";
while ($row = mysqli_fetch_array($query)) {
    $copyid .= $row['copy'] . ",";
}
$copyidsFromDB = explode(",",rtrim($copyid , ','));

После этого вы можете проверить, есть ли то, что вы получили в запросе, используя array_intersect:

$copies = $_POST['copies']
// if not an array use: $copies = explode(",", $_POST['copies'])
if (count(array_intersect($copies, $copyidsFromDB) == 0) 
    // insert to DB

Благодарю. Много справки, но последний шаг "count (array_intersect)" не работает. он не проверяет дубликаты значений в обоих массивах. Будет очень полезно, если пересекающаяся часть также начнет работать.
Решил сам, добавив только одну строку: $copies2 = explode(",",rtrim($copies , ',')); before array_intersect Спасибо, код очень помогает.
@ Ронит, ты прав. Мою ошибку я имел ввиду взорвать