Использование результатов одного запроса для создания другого запроса и заполнения выборки HTML

Здесь много ошибок, большинство из которых можно выяснить, проверив ваши данные. Используйте var_dump и print_r, чтобы убедиться, что переменная содержит то, что, по вашему мнению, она делает.

Во-первых, если вы назначаете псевдоним столбцу базы данных, это всего лишь один столбец. Таким образом, вы получили индексы массива от CodUso1, CodUso2 и т. д. до CodUso9, а затем categories в качестве псевдонима для CodUso10. Во-вторых, вы пытаетесь перебрать $result["categories"], которого не существует, потому что fetchAll вернет индексированный массив. И то, и другое вы могли бы реализовать, используя print_r($result); и взглянув на то, с чем вы работаете. В-третьих, вы передаете пользовательские данные непосредственно в запрос к базе данных, что является хорошим способом атаки на вашу базу данных.

Попробуйте это. Мы безопасно подготавливаем оператор, который извлекает все десять столбцов категорий, а затем перебираем их по одному за раз, выполняя второй подготовленный оператор для получения результатов. Обратите внимание, что одним из преимуществ подготовленных операторов, помимо безопасности, является снижение накладных расходов при многократном выполнении одного и того же запроса. Наконец, разделите код и презентацию настолько, насколько это возможно. Выводить HTML на экран в середине цикла некрасиво, и с ним сложнее работать. Вы должны вернуть объект JSON, а затем использовать обратный вызов Javascript для создания элементов.

<?php
$car_model = $_POST['get_car'];

// prepare the first query    
$query = "SELECT CodUso1, CodUso2, CodUso3, CodUso4, CodUso5,
        CodUso6, CodUso7, CodUso8, CodUso9, CodUso10
    FROM cars WHERE car_model = ? LIMIT 1";
$stmt = $pdo->prepare($query);
$stmt->execute([$car_model]);
// there's only one item, no loop needed
$categories = $stmt->fetch(\PDO::FETCH_ASSOC);

// here we prepare the second query
$query = "SELECT description FROM categories WHERE code = ?";
$stmt = $pdo->prepare($query);
// bind the variable to the ? parameter
$stmt->bindParam(1, $c);

$options = [];
foreach ($categories as $c) {
    if ($c == 0) {
        continue;
    }
    $stmt->execute();
    // we can fetch the single column directly
    $description = $stmt->fetchColumn();
    $options[] = ["id"=>$c, "description"=>$description];
}

header("Content-Type: application/json");
echo json_encode($options);

Затем в обратном вызове Javascript сделайте что-то вроде этого. Я предполагаю, что вы используете jQuery:

...
success: function(data, status, xhr) {
    // erase existing values
    $("select[name='category']").html("");
    // jQuery will turn this into an object automatically
    // loop through it and append an option element for each item
    data.each(function(v) {
        var sel = $("<option>").val(v.id).html(v.description);
        $("select[name='category']").append(sel);
    });
}

Причина, по которой foreach не работает, заключается в том, что $result['categories'] не является массивом. Предложение select вашего запроса просто переименовывает CodUse10 в categories.

SELECT CodUso1, 
            CodUso2, 
            CodUso3, 
            CodUso4, 
            CodUso5, 
            CodUso6, 
            CodUso7, 
            CodUso8, 
            CodUso9, 

           CodUso10 AS categories

Но реальная проблема гораздо глубже. Это восходит к вашей схеме таблицы. Каждый раз, когда у вас есть такие вещи, как field1, field2, field3, ... у вас почти наверняка есть таблица, которая не нормализована. Правильная схема будет выглядеть примерно так:

Car    Car_Category   Category 
id     car_id         id 
model  category_id    description

Если вы хотите получить список категорий определенного автомобиля, просто запросите объединенные таблицы:

select c.model, cat.id, cat.description
from car c
inner join car_category cc on c.id=cc.car_id
inner join category cat on cc.category_id=cat.id
where car.id=?

Я не совсем понимаю, пытались ли вы отобразить доступные параметры или выбранные параметры, но в любом случае будет использоваться одно и то же соединение «многие ко многим». Таким образом, вместо car_category у вас может быть таблица с именем available_options и другая с именем selected_options.

...Но я не могу изменить базу данных!

В этом случае не будет «правильного» решения; просто некоторые творческие способы обойти это. Цель должна состоять в том, чтобы свести к минимуму сумму созданного технического долга. Вероятно, самая большая часть — отделить модель (логику) от представления (html). Таким образом, если/когда схема исправлена, вам не нужно копаться в представлении и ломать что-то при изменении методов доступа; все, что вам нужно сделать, это подключить другой класс или функцию, которая получает информацию и выводит ее таким же образом.

Определите, как представление ожидает получать данные модели

Прежде чем я начну работу над моделью, мне нужно знать, как представление будет работать с ней. Для моих собственных приложений у меня есть класс, который форматирует ввод, выпадающие списки и флажки для меня, аля ruby ​​on rails:

<div class="form-group">
  <label for="category">Category</label>
  <?= FormDecorator::showAsDropDown($optionList, $defaultOption,['id'=>'category','name'=>'category','class'=>'form-control', etc...]) ?>
</div>

Чтобы использовать это, я бы предоставил ассоциативный массив, где options = array('value'=>'description'). Дело в том, что мне нужна модель для предоставления этого массива, однако она может получать информацию. Но модель не отвечает за создание какого-либо HTML-кода.

Используя тот же план, ваш результат может выглядеть так:

<div class="form-group row">
    <label class="col-sm-2 form-control-label">Category</label>
    <div class="col-sm-4">
        <select class="form-control" name="category">
            <option disabled selected hidden>Select a car first...</option>
            <?php foreach($options as $value => $description): ?>
              <option value="<?= $value ?>"><?= $description ?></option>
            <?php endforeach; ?>
        </select>
    </div>
</div>

Создайте модель для предоставления этих данных

Обычно я бы использовал объект, поскольку я использую свою собственную структуру MVC. Но для простоты я буду использовать обычную старую функцию (но я настоятельно рекомендую использовать классы, поскольку они гораздо более гибкие, а имена методов находятся в области видимости класса, а не в глобальной области видимости)

Найти описание по одному

Вы хотите прокрутить результаты 10 различных полей, все с удобным названием CodUso{$number}.

Сначала создайте функцию для получения необработанных данных.

function getCategoriesFor($car_model) {
    global $pdo; // ick, a class would avoid globals.
    $query =<<<CATEGORYQUERY
SELECT CodUso1, CodUso2, CodUso3, CodUso4, CodUso5, 
       CodUso6, CodUso7, CodUso8, CodUso9, CodUso10
FROM cars 
WHERE car_model = ?

CATEGORYQUERY;
    $stmt = $pdo->prepare($query);
    $stmt->execute($car_model);
    return stmt->fetch(\PDO::FETCH_ASSOC);
 }      

Теперь создайте функцию для получения описаний категорий.

// if this were an object, I would call it $Category->find($id)
function getCategoryDescriptionOf($code) {
    global $pdo;
    $query = "select * from categories where code=?";
    $stmt = $pdo->prepare($query);
    $stmt->execute($id);
    return $stmt->fetchAll();
}

Затем создайте функцию для перебора доступных вариантов:

function getCategoryOptionsByModel($car_model) {

    // get the row containing CodUso1 ... CodUso10
    $categoryRow = getCategoriesFor($car_model);

    // always initialize output before generating its contents
    $out = [];

    // 10 fields, iterate through them.  This is the hacky part...
    for($i=1; $i <= 10; $i++) {

        // generate the field name
        $field = 'CodUso' . $i;

        // get the code from the car_model table row 
        $code = $categoryRow[$field];

        // format the array we will use in the view
        $out[$code] = getCategoryDescriptionOf($code);
    }
    return $out;  // [code => description]
}

Вау! теперь осталось только использовать его в представлении:

<?php
$car_model = $_GET['car_model']; // or however it is assigned...

?>
<html> 
  ... snip ...

<div class="form-group row">
    <label class="col-sm-2 form-control-label">Category</label>
    <div class="col-sm-4">
        <select class="form-control" name="category">
            <option disabled selected hidden>Select a car first...</option>
            <?php foreach( getCategoryOptionsByModel($car_model) as $value => $description): ?>
              <option value="<?= $value ?>"><?= $description ?></option>
            <?php endforeach; ?>
        </select>
    </div>
</div>

Ура! Я нормализовал таблицы!

Если вы нормализуете таблицы, просто напишите новую функцию, чтобы заменить текущую функцию getCategoriesFor() (при необходимости замените имена реальных полей) и используйте ее в представлении вместо getCategoryOptionsByModel($car_model), или просто измените getCategoryOptionsByModel($car_model), чтобы она возвращала getCategoriesFor($car_model):

function getCategoriesFor($car_model) {
    global $pdo;
    $query =<<<THISISHOWITSHOULDBEDONEQUERY

select cat.code, cat.description
from cars c
inner join car_category cc on c.id=cc.car_id
inner join category cat on cc.category_id=cat.id
where cars.car_model=?

THISISHOWITSHOULDBEDONEQUERY;

    $stmt = $pdo->prepare($query)->execute($car_model);
    $out = [];
    foreach($stmt as $row) {
        $code = $row['code'];
        $description = $row['description'];
        $out[$code] = $description;
    }
    return $out;
}

Преимущества

Отделив логику от представления и убедившись, что каждая функция выполняет одну небольшую функцию, вы упростили изменение способа компиляции данных. Вам не нужно вносить какие-либо изменения в представление, кроме как указать другой источник данных. Если следующий программист после вас будет маньяком-убийцей, который знает, где вы живете, вы будете в большей безопасности!

Отказ от ответственности

Это все не в моей голове. Хотя я рекомендую PDO, я не использую ни MySQL, ни PDO, и мое использование может нуждаться в корректировке.