dwa problemy ze sklepem Opcje

[john_doe]

Witam,
tworzę sobie zupełnie dla treningu koszyk zakupów. Mam już dodawanie i edycję ilości. Pojawiły się dwa problemy. Jeden estetyczny drugi nie.
Mam sobie plik addToCart.php w którym wyświetlam koszyk. Ma on postać takiego linku

http://127.0.0.1/myShop/addToCart.php?id=1&productId=3

gdzie id to id kategorii a produktId wiadomo. plik ten ma postać

[PHP] pobierz, plaintext 
<?php
session_start();
 
include('connect.php');
include('functions.php');
 
print '<h1>Zawartosc koszyka</h2>';
 
if(isSet($_SESSION['koszyk']))
{
    $checkExistance = findValue($_GET['productId'], $_SESSION['koszyk']); 
}
if( $checkExistance == 1 )
{
    print 'TOWAR JEST JUZ w KOSZU <br />';
}elseif(isSet($_GET['productId'])) {
 
        $_SESSION['koszyk'][$_SESSION['id']] = array('id' => $_GET['productId'], 'ile' => 1);
        
        $_SESSION['id']++;    
}
 
 
print '<a href="shop.php">Enter to my shop centre</a><br /><br />';
print '<a href="dump.php">Dump</a><br /><br />';
 
 
 
 
if ( count($_SESSION['koszyk'] ) > 0)
{
 
        print '<table border="1">
                    <tr>
                        <th>ID</th>
                        <th>NAME</th>
                        <th>PRICE</th>
                        <th>VOLUME</th>
                        <th>ILE</th>
                        <th>edit</th>
                    </tr>';
        
        $iter = -1;
        
        foreach($_SESSION['koszyk'] as $product)
        {    
            $iter++;
            
            $id = $product['id'];    
            
            $sql = "SELECT product_id, product_name, product_price, product_volume 
                            FROM products 
                    WHERE product_id = " . $id;
            
            $result = mysql_query($sql);
            
            while ($row = mysql_fetch_array($result))
            {
                print '<tr>';
                print '<td>' . $row[0] . '</td>';
                print '<td>' . $row[1] . '</td>';
                print '<td>' . $row[2] . '</td>';
                print '<td>' . $row[3] . '</td>';
                print '<td>' . $product['ile'] . '</td>';
                print '<td><a href="java script:showEditForm('editForm.php?id=' . $iter . '')">edit</a></td>';
            }
            
        }
        print '</table>';
} else print 'koszyk pusty';
 
 
 //print var_dump($_SESSION);
?>
 
<script type="text/javascript">
 
function showEditForm(address)
{
    okno = window.open(address, "okno", "toolbar=no, location=no, status=no, menubar=no, width=210, height=150, scrollbars=no, resizable=no, left=450, top=200");
}
 
</script>
[PHP] pobierz, plaintext 

teraz muszę wkleić jeszcze pliki dwa służące edycji (krótkie są)

editForm.php

[PHP] pobierz, plaintext 
session_start();
 
print_r($_SESSION['koszyk']);
 
print '<br>';
 
print 'ile: ' . $_SESSION['koszyk'][$_GET['id']]['ile'];
 
print '<br>';
 
?>
 
<form action="edit.php?id=<?php print $_GET['id'] ?>" method="POST">
    
    <select name="increaseVolume">
        <option value="0">0</option>
        <option value="2">2</option>
        <option value="3">3</option>
        <option value="4">4</option>
        <option value="4">5</option>
        <option value="4">6</option>
    </select>
    
    <input type="submit" value="Zmien" />
</form>
[PHP] pobierz, plaintext 

oraz sama akcja edit.php

[PHP] pobierz, plaintext 
<?php
session_start();
 
//if( $_POST['increaseVolume'] == 0 )
//{
    //unset session
//    unset( $_SESSION['koszyk'][$_GET['id']] );
//}else {
 
            $_SESSION['koszyk'][$_GET['id']]['ile'] = $_POST['increaseVolume'];
  //    }
 
 
print '<script>opener.location.reload(true); window.close();</script> ';
?>
[PHP] pobierz, plaintext 

o i teraz tak. Jeden estetyczny bląd bo gdy zmienie liczbe sztuk to oczywiście zmienia ale wypisuje, że towar jest już w koszyku. I dobrze bo tak jest zakodowane i nie wiem jak to poprawić. Jest coś co namierza skąd pochodzą zmiany? wtedy by sie zrobiło że gdy z pliku edit to coś tam coś tam.
Drugi problem to usuwanie z koszyka. Linie w edit są zakomentowane bo to usuwanie jakoś nie kosi. Gdy ilość updejtowana = 0 wtedy unset session.
Nie wiem czy to, że źle działa nie jest spowodowane wyświetlaniem koszyka? nawet wtedy źle updejtuje ilość gdy <> 0. Np. jak dodam jeden towar do koszyka i chce go usunąć to nie usuwa. gdy dodam drugi to moge usunąć który kolwiek, jednak ten co zostanie nie mogę nadać mu innej ilosci. Zwariuje.
sorry za długiego posta.

pozdro

[dr_bonzo]

Wez uzyj jakiegos prostego Frameworka, np Kohana, tam bedziesz mial wszystkie akcje podzielone na metody, tak ze edycja nie bedzie ci sie mieszac z dodawaniem. I prosciej bedzie zrozumiec/zakodzic co w danej chwili sie uruchomi.
Do tego odzielisz troche SQL, od PHP i od HTMLa

[john_doe]

wiem i masz napewno rację, jednak robie to zupelnie na surowo i po raz 1szy i chciałbym aby najpierw mój pomysł zadziałał a potem wezmę się za frameworki.
jakieś sugestie co do poprawki mojego zamąconego kotła?

wydaje mi się, że źle używam funkcji unset(); bo po jej wykonaniu zostaje taki array

Array (
[1] => Array ( [id] => 2 [ile] => 1 ) [0] => Array ( [ile] => 4 ) )
ile: 4

czyli tam gdzie wyboldowałem nie usuwa się wszystko zostaje ile. Tablica winna przyjąć postać
Array (
[1] => Array ( [id] => 2 [ile] => 1 )

Ten post edytował john_doe 29.06.2009, 00:15:56

[dr_bonzo]

jakieś sugestie co do poprawki mojego zamąconego kotła?

No chyba zartujesz ze bede probowal to spagetti przeanalizowac, wlasnie temu sugeruje ci FW bo zmusi cie do lekkiego porzadku w kodzie./

[john_doe]

bez przesady 60 linijek kodu to nie jakieś spaghetti

Ten post edytował john_doe 29.06.2009, 08:06:52

[deirathe]

Tragedia kolejny osc się rodzi skąd ludzie takie nawyki biorą
To może jak w koszu masz zero to daj unset całego przedmiotu a nie zmieniasz ilość na 0, bo przedmiot Ci zostaje i podejrzewam że wyświetla że jest przez tą funkcję findValue czy jakoś tak.
A w tablicy masz indeks 'ile', który pokazuje że jest 0 sztuk.
Ablo 2 sprawa zmień funkcję findValue, żeby sprawdzała ilość sztuk danego przedmiotu w koszyku i jak jest 0 to żeby zwracała false.

Ten post edytował deirathe 29.06.2009, 08:38:03

[john_doe]

no właśnie jak daje

[PHP] pobierz, plaintext 
<?php
unset( $_SESSION['koszyk'][$_GET['id']] );
?>
[PHP] pobierz, plaintext 

to usuwam tylko id produktu z koszyka, pozostaje mi ilość sztuk

Array (
[1] => Array ( [id] => 2 [ile] => 1 ) [0] => Array ( [ile] => 4 ) )

i mam kłopot by usunąć wszystko.

P.S. Każdy się czegoś uczy. Robię to cudo po raz 1wszy. Kombinuje po swojemu i dlatego tak to wygląda. Najpierw chcę osiągnąć za wszelką cenę funkcjonalność. Potem to porządkować, przerabiać itp. Nie chce się przekomarzać ale nawyków nie mam skąd nabierać bo nie pracuję jako programista php w jakimś zespole choć byłoby fajnie. Programuje codziennie po 10 h w Transact SQLu.

[deirathe]

Wiesz czasem po osiągnięciu funkcjonalności właśnie tym sposobem który tu przedstawiasz, będziesz musiał wszystko pisać od nowa- niestety taki fakt. Co do programowania jeżeli chcesz się nauczyć jak pisać ciekawie, a masz jakieś podstawy to polecam Ci książkę
http://webmade.org/recenzje/wzorce-projekt...e-obiektowe.php

Co do problemu- masz źle zbudowaną w takim razie tablicę. Przebuduj ją na
koszyk ->idproduktu->(ilosc,nazwa,i inne)
i wtedy jak zrobisz unset koszyk[id_produktu] to masz pewnosc ze wywali wszystko

I trochę o bezpieczeństwie poczytaj

Ten post edytował deirathe 29.06.2009, 10:38:56

[john_doe]

spoko wiem, że nie jest to bezpieczne ale nie o to chodzi. Co do funkcjonalności to na bank będę musiał napisać na nowo ale już z bogatszym portfelem doświadczenia.
Tablica może nie jest źle zbudowana tylko coś mi nie wychodzi by to zunsetować
Przecież ma postać

Array (
[1] => Array ( idProduktu, ileTegoProduktu )
)

[phpion]

Mając 1 produkt w koszyku musisz wykonać 1 zapytanie aby wyświetlić zawartość koszyka. Mając 10 produktów zapytań tych masz 10, przy 20 produktach 20. Niezależnie od ilości produktów możesz pobrać produkty 1 zapytaniem. Jak? Zamiast w pętli wykonywać:

[SQL] pobierz, plaintext 
... WHERE product_id=1
... WHERE product_id=2
... WHERE product_id=3
[SQL] pobierz, plaintext 

lepiej użyć do tego IN:

[SQL] pobierz, plaintext 
... WHERE product_id IN (1, 2, 3)
[SQL] pobierz, plaintext 

czyli zrobić implode(', ', $tablica_id_produktow).

[john_doe]

racja, dzięki. podziałam po pracy. Dobre spostrzeżenie <gitara>