0

Всем привет. Такая проблемка, с сервера загружаю объявления, в том числе и пути к изображениям. Объясню суть, я посылаю запрос серверу, он формирует список объявлений в текущей категории и пытаюсь создать массив уже в текущем массиве img_name[]. Перед добавлением строк в массив я создаю запрос в БД (если 'id_ads равен $row['id'], то добавить строку. Строки добавляются, но и добавляются те, которые ни как не подходят под моё условие. Т.е. например я вытаскиваю пути изображений для объявления с id=233, в базе которого 2 фото(пути), а в объявлении с id=234, 1 фото(путь) то в ответе от сервера я получаю на каждое объявление не нужные мне пути к фоткам. Пример:

введите сюда описание изображения

"id": "234",
        "user_id": "1",
        "img_name": [
            "1_img.jpg" --- id_ads == 234
        ]

 "id": "233",
        "user_id": "1",
        "img_name": [
            "1_img.jpg", --- id_ads == 234 !!!! - откуда?
            "2_img.jpg", --- id_ads == 233
            "3_img.jpg"  --- id_ads == 233
        ]

В общем приложу скрины ещё, объяснил как мог. Помогите пожалуйста. Вот серверная функция

function adsLoading($cat) { 
    $sql = mysql_query('SELECT * FROM `ads` WHERE `new_ads` = "0" AND `cat` = '.$cat.' ORDER BY `id` DESC');
    if(mysql_num_rows($sql) > 0) {
        while($row = mysql_fetch_array($sql)) {

            $id = $row['id'];
            $data['id'] = id;   
            $user_id = $row['user_id'];
            $data['user_id'] = $user_id;    

            $picAds = mysql_query('SELECT * FROM `upload_img_ads` WHERE `user_id` = 1 AND `temp` = 0 AND `id_ads` = '.$id.'');

            if(mysql_num_rows($picAds) > 0) {
                while($img = mysql_fetch_array($picAds)) {

                    $image = $img['img_name'];
                    $id_ads = $img['id_ads'];
                    $data['img_name'][] = $image;   
                }
            }       

            $vrl['data'][] = $data;
        }
    } else {
        $data['id'] = 0;
        $vrl['data'][] = "";    
    }

    exit(json_encode(array("data" => $vrl['data'])));
}

введите сюда описание изображения

На скриншоте видно, что в массиве с id=232, каким то чудом добавляются данные не принадлежащие данному id, не смотря на то, что в условии выборки так и обозначено.

5
  • Инициализировать нужно, молодой человек
    – vp_arth
    14 ноя 2017 в 4:59
  • 1
    $data = array(); в нужном месте решит все(нет) Ваши проблемы.
    – vp_arth
    14 ноя 2017 в 5:00
  • @vp_arth а что эта инициалищация поменяет? С ней на оборот, в массив добавляются пустые строки: "", "img_name": image.jpg
    – sergei1094
    14 ноя 2017 в 5:21
  • Вы просто должны понять что "какое-то чудо" - это Вы. Потому что Вы переиспользуете уже заполненный массив. Лишние строки остаются там с предыдущих итераций цикла.
    – vp_arth
    14 ноя 2017 в 5:23
  • Инициализация массива не изменила ничего
    – sergei1094
    14 ноя 2017 в 7:43

1 ответ 1

2

Вы просто должны понять что Вы переиспользуете уже заполненный массив.
Лишние строки остаются там с предыдущих итераций цикла.

function adsLoading($cat) { 
    $sql = mysql_query('SELECT * FROM `ads` WHERE `new_ads` = "0" AND `cat` = '.$cat.' ORDER BY `id` DESC');
    if(mysql_num_rows($sql) > 0) {
        while($row = mysql_fetch_array($sql)) {

            $id = $row['id'];

            /*
             * Инициализация!
             */
            $data = array();

            $data['id'] = id;   
            $user_id = $row['user_id'];
            $data['user_id'] = $user_id;    

            $picAds = mysql_query('SELECT * FROM `upload_img_ads` WHERE `user_id` = 1 AND `temp` = 0 AND `id_ads` = '.$id.'');

            if(mysql_num_rows($picAds) > 0) {
                while($img = mysql_fetch_array($picAds)) {

                    $image = $img['img_name'];
                    $id_ads = $img['id_ads'];
                    $data['img_name'][] = $image;   
                }
            }       

            $vrl['data'][] = $data;
        }
    } else {
        $data['id'] = 0;
        $vrl['data'][] = "";    
    }

    exit(json_encode(array("data" => $vrl['data'])));
}
5
  • Спасибо вам огромное!) Я уже понял что переиспользую его, только вот инициализировал я его после function adsLoading($cat) { .. Теперь всё работает как часики :) Спасибо ещё раз
    – sergei1094
    14 ноя 2017 в 7:51
  • Помимо всего прочего я бы порекомендовал Вам ознакомиться с красной плашкой вот тут. Ну и почитать про защиту от sql-инъекций.
    – vp_arth
    14 ноя 2017 в 7:53
  • уважаемый. Спасибо за заботу конечно) Защита будет обязательно, сейчас пока что то только каркас и архитектура.
    – sergei1094
    14 ноя 2017 в 7:55
  • Тем более, если проект ещё на этапе создания, обязательно смените API доступа к базе с mysql_ на mysqli_ или PDO
    – vp_arth
    14 ноя 2017 в 7:56
  • на mysqli_ и мне удобнее всего работать, в данном примере, это лишь "повторюсь ещё раз" - каркас :) Спасибо ещё раз вам.
    – sergei1094
    14 ноя 2017 в 7:58

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.