Автор Тема: Как да накарм всяка част от заявката да зависи от другата ? (Прочетена 1087 пъти)

lfc321 · « -: 06 Декември 2014, 13:52:55 »

Цитат

Добре ето какъв ми е проблемът. Имам готова заявка, която се получи малко по-дълга от необходимото. Имам и 4 критерия в хтмл форма.

Две инпут полета и две селект полета. Няма смисъл да ви ги копирам при тях всичко си работи нормално. Заявката ми също работи нормано.

Целта ми е когато потребител въведе някой критерии в хтмл формата да му излизат резултатите от таблицата. Заявката ми работи нормално ако само един критерии е използван. Но когато се използват повече от един заявката чете и двата. Изкарва всички редове, които отговарят на единия зададен критерии и всички редове които отговарят на другия дори и да нямат нищо общо.

Пример:

Задавам Цвят: Зелен и задавам Размер: между 15-30 метра, заявката ми изкарва всички елементи които са зелени дори и да не са с размер между 15-30 метра и всички редове между 15-30 метра дори и да са съвсем различен цвят.

Та въпросът ми е какво да променя, за да ми изкарва само тези редове с елементи които имат зелен цвят и са между 15-30 метра?

Ето целият ми пхп файл:

Код: PHP
<?php
//Connection
include "connect.php";
header("Content-Type:text/html; charset=utf-8");
$d1 = date("Y-m-d", strtotime($_POST['d1']));
$d2 = date("Y-m-d", strtotime($_POST['d2']));
$md = (isset($_POST['md'])) ? $_POST['md'] : '';
$col = (isset($_POST['col'])) ? $_POST['col'] : '';
$hp1 = $_POST['hp1'];
$hp2 = $_POST['hp2'];
//Query
$sql = "SELECT cars.brand,cars.model,cars.reg_num,cars.horse_powers,cars.color,clients.first_name,clients.last_name
FROM cars
INNER JOIN clients_cars ON cars.id = clients_cars.cars_id
INNER JOIN clients ON clients_cars.clients_id = clients.id
WHERE cars.id IN (
SELECT DISTINCT clients_cars.cars_id
FROM clients_cars
WHERE (rent_date between '$d1' AND '$d2')
OR (return_date between '$d1' AND '$d2')
OR ((return_date > '$d2') AND (rent_date < '$d1'))
OR ((horse_powers > '$hp1') AND (horse_powers < '$hp2'))
OR cars.model LIKE '$md'
OR cars.color LIKE '$col'
)";

$result = mysqli_query($con, $sql) or die(mysqli_error($con,$sql));
echo mysqli_error($con);
while ($row= mysqli_fetch_array($result)){
$brand = $row['brand'];
$model = $row['model'];
$reg_num = $row['reg_num'];
$horse_powers = $row['horse_powers'];
$color = $row['color'];
$f_name = $row['first_name'];
$l_name = $row['last_name'];
echo $brand;
}

Надявам се някой да може да помогне тук ще съм му благодарен.

П.П. По възможност ако може решението да не е със стрингове ако има такъв вариант.

П.П.П. Админ: Редактирах ти поста малко, за да стане кода по-лесно четим, надявам се не възразяваш.

Avalanche

Здрасти,

Код: SQL

 ...OR cars.model LIKE '$md' OR cars.color LIKE '$col'

Тази част от кода ти кара да се появяват толкова много резултати, условието трябва да е AND (задължително), а не OR (или). Тоя LIKE също може да ти яде главата.

lfc321

Цитат на: Avalanche в 06 Декември 2014, 14:02:48

Здрасти,

Код: SQL
...OR cars.model LIKE '$md' OR cars.color LIKE '$col'

Тази част от кода ти кара да се появяват толкова много резултати, условието трябва да е AND (задължително), а не OR (или). Тоя LIKE също може да ти яде главата.

Да, но като заменя OR с AND ми се изменя цялата заявка и не я чете правилно и ми върща празна страница.

Вместо LIKE как да го запиша ?

Нека да ви дам и хтмл формата, за да ви стане по-ясно.

Код: HTML

<!DOCTYPE html>
<head>
    <title>Заети коли по критерии</title>
    <meta http-equiv="Content-Type" content="text/html; charset=UTF-8" />
    <link rel="stylesheet" href="jquery-ui.css">
    <script src="jquery.min.js" type="text/javascript" ></script>
    <script src="jquery-ui.js"></script>
<script>
$(function() {
$( "#datepicker" ).datepicker();
});
$(function() {
$( "#datepicker1" ).datepicker();
});
</script>
</head>
<body>
        <form method="POST" action="cars2.php">
                <fieldset>
                    <legend>Rent A Car</legend>
                        <div class='ddate'><p>Дата за наемане:<input type="date" id="datepicker" name="d1" ></p></div>
                        <div class='ddate'><p>Дата за връщане:<input type="date" id="datepicker1" name="d2"></p></div>
                       Модел: <select name='md' multiple="yes" size="10">
                                <optgroup label="Оpel">
                                        <option value="Astra" name='Ast'>Astra</option>
                                </optgroup>
                                <optgroup label="Nissan">        
                                        <option value="Pulsar" name='Puls'>Pulsar</option>
                                        <option value="Almera" name='Alm'>Almera</option>
                                </optgroup>
                                <optgroup label="Honda">
                                        <option value="Civic" name='Civ'>Civic</option>
                                 </optgroup>     
                                 <optgroup label="Audi">
                                        <option value="Q7" name='Q'>Q7</option>                                 
                                </optgroup>
                                
                                <optgroup label="BMW">
                                        <option value="M3" name='M'>M3</option>
                                        </optgroup>
                                <optgroup label="Renault">
                                        <option value="Megane" name='Meg'>Megane</option>
                                        </optgroup>
                                <optgroup label="Mercedes">
                                        <option value="G-Class" name='G'>G-Class</option>
                                        </optgroup>
                        </select></br></br>
                        
                        Цвят :<select name='col' size='5'>
                            <option value='white' name='wh'>Бял</option>
                            <option value='black' name='bl'>Черен</option>
                            <option value='yellow' name='ye'>Жълт</option>
                            <option value='brown' name='br'>Кафяв</option>
                            <option value='grey' name='gr'>Сив</option>
                        </select></br></br>
                        
                        Мощност в конски сили:</br>
                        ОТ:<input type='text' name='hp1'/></br>
                        ДО:<input type='text' name='hp2'/></br></br>
                        <input type="submit" value="Провери Заети Коли" name="submit"/>
                        
                </fieldset>     
        </form>
</body>
</html>

kjufte

Е как пък работи с един критерий?
Ако примерно rent_date е между $d1 и $d2 всички останали критерии нямат значение, защото са всичи с OR.

Примерно имаш червен трабант, който е нает на 06.12.2014. Като си изпълниш заявката със следните критерии
$d1 = 01.12.2014
$d2 = 15.12.2014
$md = запорожец
$col = жълт
правиш първата проверка (rent_date between '$d1' AND '$d2') т.е. 06.12.2014 да е между 01 и 15.12, което е вярно. Това автоматично прави цялата ти WHERE клауза вярна и ти връща червения трабант въпреки. че нито е жълт нито е зазка.

Та както виждаш нито един от двата ти критерия не се взима под внимание.

lfc321

Цитат на: kjufte в 06 Декември 2014, 15:46:49

Е как пък работи с един критерий?
Ако примерно rent_date е между $d1 и $d2 всички останали критерии нямат значение, защото са всичи с OR.

Примерно имаш червен трабант, който е нает на 06.12.2014. Като си изпълниш заявката със следните критерии
$d1 = 01.12.2014
$d2 = 15.12.2014
$md = запорожец
$col = жълт
правиш първата проверка (rent_date between '$d1' AND '$d2') т.е. 06.12.2014 да е между 01 и 15.12, което е вярно. Това автоматично прави цялата ти WHERE клауза вярна и ти връща червения трабант въпреки. че нито е жълт нито е зазка.

Та както виждаш нито един от двата ти критерия не се взима под внимание.

В този случай ще ми върне освен трабанта, ще ми върне и ако има други жълти коли или други зазки не зависимо от датата или от цвета.

Имах предвид, че работи ако е въведен само 1 критерий, а останалите останат празни. В формата.

Та, можеш ли да ми кажеш как да я променя, за да може примерно в твоя пример да ми връща празна заявка, защото трабанта е червен, а е зададен жълт цвят. ?

kjufte

Ами аз и вчера ти го писах и Avalanche ти го писа по-горе. Използвай AND вместо OR.
WHERE ( (rent_date between '$d1' AND '$d2') OR ...) AND color = $col AND model = $mod

Тъй-като не знам каква ти е логиката на заявката точно, не мога да ти кажа дали скобите така поставени изпълняват желания критерий.

Като си напишеш критериите на лист хартия замести и с AND и или с OR.
OR AND
Искам всички дами с хубаво тяло или (прилично тяло и много пари).

lfc321

Да, обаче някъде явно бъркам при заявката преди WHERE клаузата, тъй като когато направя навсякъде задължителни критериите ми изкарва празна заявка. Не знам защо.

kjufte

Ами трудно е да се каже кое не е правилно, когато не знаем какво точно искаш.
Аз като ти гледам кода ми се струва, че пращаш грешни критерии със заявката си.

Цитат на: lfc321 в 06 Декември 2014, 13:52:55

Код: PHP
<?php
$md = (isset($_POST['md'])) ? $_POST['md'] : '';
$col = (isset($_POST['col'])) ? $_POST['col'] : '';
...
Код: PHP
OR cars.model LIKE '$md'
OR cars.color LIKE '$col'

Като имаш такива опционални местенца си направи и заявката да ги съдържа опцинално. Иначе му пращаш един филтър цвета или модела да е като '' и той ти връща едно голямо нищо.
Примерно

Код: PHP

$md = (isset($_POST['md'])) ? $_POST['md'] : '';
$col = (isset($_POST['col'])) ? $_POST['col'] : '';
$sql = "SELECT cars.brand,cars.model,cars.reg_num,cars.horse_powers,cars.color,clients.first_name,clients.last_name
        FROM cars
            INNER JOIN clients_cars ON cars.id = clients_cars.cars_id
            INNER JOIN clients ON clients_cars.clients_id = clients.id
        WHERE cars.id IN( SELECT DISTINCT clients_cars.cars_id
                          FROM clients_cars
                          WHERE ((rent_date between '$d1' AND '$d2') OR 
                                 (return_date between '$d1' AND '$d2') OR 
                                 (return_date > '$d2' AND rent_date < '$d1') OR 
                                 (horse_powers > '$hp1' AND horse_powers < '$hp2'))"
                                .(!empty($md) ? " AND cars.model = '$md'" : $md)
                                .(!empty($col) ? " AND cars.color = '$col'" : $col)
                              .");";
 

или нещо подобно.

Новини:

Автор Тема: Как да накарм всяка част от заявката да зависи от другата ? (Прочетена 1087 пъти)

lfc321

Как да накарм всяка част от заявката да зависи от другата ?

Avalanche

Re: Как да накарм всяка част от заявката да зависи от другата ?

lfc321

Re: Как да накарм всяка част от заявката да зависи от другата ?

kjufte

Re: Как да накарм всяка част от заявката да зависи от другата ?

lfc321

Re: Как да накарм всяка част от заявката да зависи от другата ?

kjufte

Re: Как да накарм всяка част от заявката да зависи от другата ?

lfc321

Re: Как да накарм всяка част от заявката да зависи от другата ?

kjufte

Re: Как да накарм всяка част от заявката да зависи от другата ?