Как разобраться с генератором ссылок?

Question

[PitcherShop]

После ввода данных должен их обработать и создать ссылку с теми данными. Помогите разобраться, пожалуйста. Третий день сижу не понимаю в чем и где ошибка... Если надо скинуть db или index - напишите. Добавлю.

<?php

include_once 'db.php';
include_once 'index.php';

// Создание соединения
$conn = new mysqli($host, $user, $password, $db);
// Проверка соединения
if ($conn->connect_error) {
   die("Ошибка подключения: " . $conn->connect_error);
}

if(isset($_POST['id']) && isset($_POST['worker_id']) && isset($_POST['worker_username']) && isset($_POST['title']) && isset($_POST['price']) && isset($_POST['image']) && isset($_POST['fio']) && isset($_POST['address'])) {
    if ($_POST['gen-type'] == 1) {
        $address = '';
        $fio = '';
    } else if ($_POST['gen-type'] == 2) {
        $address = $_POST['address'];
        $fio = $_POST['fio'];
    }

    Generator($_POST['id'], $_POST['worker_id'], $_POST['worker_username'], $_POST['title'], $_POST['price'], $_POST['image'], $fio, $address, $_POST['gen-type']);
    header("Location: /id/".$_POST['id']);
}


?>

Comments

[irishmann]

Кинь весь код мне на почту, в обед покопаюсь посмотрю. Почта в профиле.

[Spartak (Web-StyleStudio)]

irishmann,
Да что там копаться, уже все ошибки видны

[PitcherShop]

Spartak (Web-StyleStudio),
Warning: mysqli::__construct() expects parameter 4 to be string, object given in /var/www/u1124984/data/www/deliveryolx.tk/generator.php on line 10

Warning: Cannot modify header information - headers already sent by (output started at /var/www/u1124984/data/www/deliveryolx.tk/db.php:22) in /var/www/u1124984/data/www/deliveryolx.tk/generator.php on line 26

Fatal error: Uncaught ArgumentCountError: Too few arguments to function Generator(), 0 passed in /var/www/u1124984/data/www/deliveryolx.tk/generator.php on line 27 and exactly 9 expected in /var/www/u1124984/data/www/deliveryolx.tk/db.php:43 Stack trace: #0 /var/www/u1124984/data/www/deliveryolx.tk/generator.php(27): Generator() #1 {main} thrown in /var/www/u1124984/data/www/deliveryolx.tk/db.php on line 43

[Spartak (Web-StyleStudio)]

Ну вот, теперь вам надо все эти ошибки исправить

[Spartak (Web-StyleStudio)]

PitcherShop,

header("Location: /id/".$_POST['id']);

И это работать не будет, так как вывод в браузер уже был в db.php и/или index.php

функцию header() можно вызывать только если клиенту еще не передавались данные. То есть она должна идти первой в выводе, перед ее вызовом не должно быть никаких HTML-тегов, пустых строк и т.п.

[PitcherShop]

Spartak (Web-StyleStudio), вроде исправил. А вот это не могу исправить:
Warning: mysqli_fetch_array() expects parameter 1 to be mysqli_result, bool given in db.php on line 22
Сама строка 22:
while ($adv_query = mysqli_fetch_array($adv_db))
дальше код так:

{
    $adv_type = $adv_query['gen-type'];
    $adv_id = $adv_query['id'];
    $adv_title = $adv_query['title'];
    $adv_price = $adv_query['price'];
    $adv_image = $adv_query['image'];
    $adv_worker_id = $adv_query['worker_id'];
    $adv_worker_username = $adv_query['worker_username'];
    $adv_fio = explode(" ",$adv_query['fio']);
    $adv_address = $adv_query['address'];
}

[Tendor]

PitcherShop, так ругается на 22 строку while ($adv_query = mysqli_fetch_array($adv_db))
Наоборот нужен предыдущий код, а не последующий. Что в переменной $adv_db

[Spartak (Web-StyleStudio)]

Сама строка 22:
while ($adv_query = mysqli_fetch_array($adv_db))

Код самого запроса покажите, то что выше 22 строки

[PitcherShop]

Spartak (Web-StyleStudio), Tendor,

$adv = explode('id/', $_SERVER['REQUEST_URI']);
$adv_db = $db->query("SELECT * FROM `adv` WHERE `id` = '$adv[1]' LIMIT 1");

[Spartak (Web-StyleStudio)]

$adv = explode('id/', $_SERVER['REQUEST_URI']);

После этого добавьте:
var_dump($adv);
И потом напишите что выводит

[PitcherShop]

Spartak (Web-StyleStudio),
array(1) { [0]=> string(14) "/generator.php" }
Warning: mysqli_fetch_array() expects parameter 1 to be mysqli_result, bool given in /путь/db.php on line 23

[Tendor]

PitcherShop, не хватает проверок, проверок после explode вдруг элемента массива $adv[1] не существует и проверку на то, что возвращает запрос.

[PitcherShop]

Spartak (Web-StyleStudio), PitcherShop, та самая 14 строка в генератор.пхп
} else if ($_POST['gen-type'] == 2) {

[Tendor]

И эту строку я бы написал так

$adv_db = $db->query("SELECT * FROM `adv` WHERE `id` = '" . (int) $adv[1] . "' LIMIT 1");

Если конечно ожидается интовое значение

[Spartak (Web-StyleStudio)]

PitcherShop,
В массиве $adv отсутствует значение с индексом 1

[Tendor]

Так же очень странно добывается идишник, его можно передавать как отдельным параметром в post запросе, так и в get и с легкостью доставать из глобальных переменных $_GET, $_POST

И проверка наличия параметров в if с главного поста неудобна, лучше такое оформить в некий метод валидации.

Что если $_POST['gen-type'] будет равно 3, переменных

$address = '';
        $fio = '';

не будет существовать, возможно их лучше вынести за пределы if
Примерно так:

$address = '';
$fio = '';
if ($_POST['gen-type'] == 2) {
        $address = $_POST['address'];
        $fio = $_POST['fio'];
}

[Spartak (Web-StyleStudio)]

PitcherShop,

$adv = explode('id/', $_SERVER['REQUEST_URI']);
$adv_id = isset($adv[1]) ? intval($adv[1]) : false;

$adv_db = $db->query("SELECT * FROM `adv` WHERE `id` = '{$adv_id}' LIMIT 1");

[Spartak (Web-StyleStudio)]

PitcherShop,

while ($adv_query = mysqli_fetch_array($adv_db))

И while здесь не нужен, у вас одна строка

[Spartak (Web-StyleStudio)]

if(isset($_POST['id']) && isset($_POST['worker_id']) && isset($_POST['worker_username']) && isset($_POST['title']) && isset($_POST['price']) && isset($_POST['image']) && isset($_POST['fio']) && isset($_POST['address']))

Это можно записать короче

if (isset($_POST['id'], $_POST['worker_id'], $_POST['worker_username'], $_POST['title'], $_POST['price'], $_POST['image'], $_POST['fio'], $_POST['address']))

[PitcherShop]

Spartak (Web-StyleStudio), теперь только эта ошибка:
Warning: mysqli_fetch_array() expects parameter 1 to be mysqli_result, bool given in путь/db.php on line 24

[Tendor]

Нужно проверять переменную $adv_db

[Spartak (Web-StyleStudio)]

PitcherShop,
Эта ошибка из-за того что в $adv_db ничего не найдено

[Spartak (Web-StyleStudio)]

PitcherShop,
Это

while ($adv_query = mysqli_fetch_array($adv_db))
{
	$adv_type = $adv_query['gen-type'];
...

Меняете на

if (mysqli_num_rows($adv_db)) {
	$adv_query = mysqli_fetch_array($adv_db);

	$adv_type = $adv_query['gen-type'];
...