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.
Re: php e-SHOP
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 (-;
Re: php e-SHOP
...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.
Warning: count(): Parameter must be an array or an object that implements Countable in /home/pasokime/domains/mysql.lt/public_html/forumas/include/parser.php on line 820
4 2007-10-24 02:05:14 (edited by DY 2007-10-24 02:08:59)
Re: php e-SHOP
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.
Mintys - blogas apie gyvenimą, politiką, ekonomiką, kultūrą, mokslą ir kitką.
Adventure Lietuva - kompiuteriniai žaidimai, kurie yra meno forma.
Pasaulio šalių himnai - įrašai, žodžiai ir jų vertimai, informacija.
Adventure Lietuva - kompiuteriniai žaidimai, kurie yra meno forma.
Pasaulio šalių himnai - įrašai, žodžiai ir jų vertimai, informacija.
Re: php e-SHOP
manjana wrote:
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'ų".
Re: php e-SHOP
DY wrote:
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.