Topic: php e-SHOP
Sweiki,
Parasiau PHP e-shop moduliuka, manau kad jums tai gali buti idomu imeciau pas save i saita. Tai kam tai pades, atrasykite forume, man idomi kritika.
LINKAS
http://www.ipsistemos.org/index.php?id= … c7e7ab8cd1
Aciu.
PHP ir MySQL programavimas, SQL užklausos, duomenų bazės, PHP scriptai, pagalba, diskusijos, pamokos ir straipsniai.
You are not logged in. Please login or register.
PHP ir MySQL → PHP → php e-SHOP
Sweiki,
Parasiau PHP e-shop moduliuka, manau kad jums tai gali buti idomu imeciau pas save i saita. Tai kam tai pades, atrasykite forume, man idomi kritika.
LINKAS
http://www.ipsistemos.org/index.php?id= … c7e7ab8cd1
Aciu.
Na, pakritikuoti tai yra ką (-;
Aišku, negalima čia kritikuoti visko, nes nesimato niekur "krepšelio" minimalios specifikacijos/reklavimų. Todėl neaišku kiek sprendimas atitinka užduoties reikalavimus, todėl bandysiu remtis internete priimta praktika "kaip veikia prekių krepšeliai".
Iš pradžių iš karto kyla klausimas, kodel metodas delItem() sumažina prekės kiekį krepšelyje vienetu, ir tik jeigu yra viena prekė - ji ištrinama. Realiai prekių krepšeliuose, kalbant apie prekės kiekį, reikalingi metodai:
a) gauti prekės kiekį;
b) keisti prekės kiekį;
Kadangi gauti prekės kiekį galima ir per objektas->cart masyvą tai bendru atveju rekomenduoju kurti atskirą metoda tik tuo atveju, jeigu prisireiks naudoti papildomas funkcijas prekės kiekio gavimui (pvz.: if(isset($obj->cart[$id])) ), kurias ir galima sudėti į konkretų metodą.
Tuo tarpu metodas "keisti prekės kiekį" tikrai pravestų, nes:
- galima lengvai padidinti ar sumažinti prekės kiekį iki bet kokio norimo kiekio (taip pat ir didinti/mažinti vienetu);
- galima ištrinti prekę (padavus nulinį kiekį);
O dabartinis variantas tikrai apsunkintu prekės ištrinimą, jeigu tarkim aš krepšelyje turiu 100 prekės vienetų, ir pan.
Toliau, kas čia per maniera taip išvedinėti duomenis:
echo "Jusu krepselis tuscias<";
echo 'br>';
Kodėl negalima viską sudėti į vieną eilutę?
Kodėl bereikalingai naudoji intval() funkciją? Jeigu jau ją naudojai pridėdamas prekę ir takrim keistamas jos kiekį krepšelyje, tai tikriausiai to pakanka būti garantuotam, kad krepšelyje prekės kiekis tikrai yra INT tipo. Kitaip sakant reikia tik tuose vietose, kur tas kiekis keičiamas užtikrinti, kad patektu teisinga reikšmė ir spausdinant neeikvoti resursų.
Toliau manau, kad tokios klasės/objektai neturi išvedinėti jokio HTML kodo, ir aplamai nenaudoti echo ir kitų spausdinimo funkcijų/metodų. Duomenų išvedimą rekomenduoju palikti kitiems programos, kuri naudos šią klasę, elementams, nes dabar gaunasi taip, kad šitos klasės nelabai išeina panaudoti jos nemodifikavus.
Taip pat klausimas, kam konstruktoriuje sukuriamas krepšelio masyvas (cart), jeigu galima klasės antraštėje tai atlikti? Kintamojo sukūrimas konstruktoriuje pačią klasę daro sunkiau skaitomą (t.y. kažkas kitas skaitydamas klasę ne iš karto matys, kad joje yra kintamasis cart).
Pradžiai tiek, yra ir kitų niuansų, bet kadangi čia tik kaip suprantu pasižaidimui tai palinkėsiu sėkmės programuojant (-;
...xm..
siaip as rasiau si Clase kaip principa "kaip galima padaryti " - ja gi galima tobulinti kaip tiktai tau patinka.
Dabar pradesiu atsakyneti.
1-Del prekes kiekio keitimo sutinku cia tu teisus, bet konkreciu atveju rodant kaip PVZ nejaugi to reikia?
2-Del maneros
echo "Jusu krepselis tuscias<";
echo 'br>'; - padariau tai todel kad pas mane saitas traktuoja <br> kaip TAG - taip pat ji ir atvaizduoja surades.
3-Del intval() irgi teisus... trinsiu lauk,nezinau kodel ikisau :).
Aciu tau uz pastebejimus Geros dienos.
Dar jei rašoma lietuviškai (kalbu apie teksto išvedimą), tai reikėtų naudoti ir lietuviškas raides. šiaip bendrai kalbant apie jūsų puslapį ( http://www.ipsistemos.org/ ) tai galėtų jis ir profesionaliau atrodyti - kažkodėl lietuviškame tinklapyje įterpiami angliški žodžiai ("Read more", "Search", "Go") - čia gal problema su turinio valdymo sistema ir negalima sulietuvinti tų žodžių, bet šiaip reikėtų, juolab kai viena jūsų veiklos rūšių kaip suprantu yra prekyba turinio valdymo sistemomis. Dar ir kitur lietuviškos raidės tai naudojamos, tai ne (pvz. "Atsiusti kodą galite paspaudę čIA"), rašoma su klaidomis (pvz. "profeseonalumas"). šiaip gal pastebėjimas ne visai susijęs su tema, bet manau, kad blogas tinklapio vaizdas gadina įvaizdį kompanijos, kuri būtent ir užsiima tinklapių kūrimu.
echo "Jusu krepselis tuscias<";
echo 'br>'; - padariau tai todel kad pas mane saitas traktuoja <br> kaip TAG - taip pat ji ir atvaizduoja surades.
O kaip manai, kaip pvz. šiame forume sėkmingai atvaizduojama eilutė:
echo "Jusu krepselis tuscias<br>";
(-; Tiesgiog yra bent keli būdai, kaip išvesti norimą tekstą neatliekant tokių "work-around'ų".
Dar jei rašoma lietuviškai (kalbu apie teksto išvedimą), tai reikėtų naudoti ir lietuviškas raides. šiaip bendrai kalbant apie jūsų puslapį ( http://www.ipsistemos.org/ ) tai galėtų jis ir profesionaliau atrodyti - kažkodėl lietuviškame tinklapyje įterpiami angliški žodžiai ("Read more", "Search", "Go") - čia gal problema su turinio valdymo sistema ir negalima sulietuvinti tų žodžių, bet šiaip reikėtų, juolab kai viena jūsų veiklos rūšių kaip suprantu yra prekyba turinio valdymo sistemomis. Dar ir kitur lietuviškos raidės tai naudojamos, tai ne (pvz. "Atsiusti kodą galite paspaudę čIA"), rašoma su klaidomis (pvz. "profeseonalumas"). šiaip gal pastebėjimas ne visai susijęs su tema, bet manau, kad blogas tinklapio vaizdas gadina įvaizdį kompanijos, kuri būtent ir užsiima tinklapių kūrimu.
Del lietuvybes. Gerbiamas DY i tavo citata atsakysiu taip...
Rasiau koda ne i cenzuros departamenta o i programuotoju foruma. Speju kad rasant koda pirminybe teikiama
ne komentaru Sulietuvinimu,surusinimu ir.t.t...
O del profeseonalumo - kiekvienas turi teise rinktis, patinka imk ir nauduokis - nepatinka niekas nevercia nauduotis.
Aciu tau uz komentarus.
PHP ir MySQL → PHP → php e-SHOP
Powered by PunBB, supported by Informer Technologies, Inc.