WN - feedback på min egen utvecklad e-handel

WN (https://www.wn.se/forum/index.php)

- Feedback (https://www.wn.se/forum/forumdisplay.php?f=20)

- - feedback på min egen utvecklad e-handel (https://www.wn.se/forum/showthread.php?t=1044411)

Undoredoo: anledningen till svarstiderna i kategoriträdet är detta
<a href="javascript:__doPostBack('TreeView1','tBlommo r')">
det görs med andra ord en full postback varje gång man klickar på nånting i den
att det inte märks på sidan beror på att trädmenyn ligger i en "updatepanel"
nackdelen med den lilla "uppfinningen" från .net-teamet är att ALL kod på den aktuella sidan körs likförbannat på servern.

tjenare..
yes, det var min gamla asp version som ligger kvar på .se, men som ska ersättas med .net versionen när den blir klar ;)

Citat:

Ursprungligen postat av Undoredoo (Inlägg 20374726)

Första intrycket var verkligen "wow". Ambitiöst, seriöst, stiligt.

Men jag kan inte låta bli att uppröras av att se så dålig kod år 2010. Ingen tutorial eller mänskliga lärare har ju lärt ut tabellbaserad layout på bra många år, så jag förstår inte att det fortfarande dyker upp. I synnerhet när det gäller nyproduktion. Jag kan dessvärre inte vara speciellt konstruktiv på den punkten. Förlåt.

Sen är det som sagt långa svarstider. Först tänkte jag att varukategorierna till vänster expanderas via AJAX-anrop, och att det alltså var servern som var orsaken. Men det verkar ju inte vara något annat än display-toggle via javascript. Hur kan det ta så lång tid?

Men den absolut viktigaste frågan är: har du någonting med www.minmatbutik.se att göra? Svaret på den frågan står och faller med din trovärdighet, enligt mig.

Eller också är det jag som är helt fel ute och/eller bara är på dåligt humör. Ber om ursäkt i så fall.

Citat:

Ursprungligen postat av nabil_akhlaque (Inlägg 20374750)

tjenare..
yes, det var min gamla asp version som ligger kvar på .se, men som ska ersättas med .net versionen när den blir klar ;)

Ok, jag är nöjd. :)

Men du borde fortfarande göra något åt gränssnittskoden. På riktigt.

hmm, visst ser den bra ut och så men sidan är öppen för sql injections.
Testa att logga in med ' or '1'='1 som användarnamn och lösenord då kommer man in på administratörskontot.

Designen var utan tvekan snygg, men uppbyggnaden är en av de värsta jag sett på en hemsida som till och med är gjord det här året.

Dissa tabeller över huvud taget.
Lägg gärna all text med teckensnittet "ICA" i bilder så att det är säkert att alla ser lika.
Säkra sidan från SQL-injektioner (!) - var inget problem alls att komma in på administratörkontot. Att ha ett inloggningssystem utan säkerhet är som att köra bil utan bromsar.
Måste verkligen allt vara uppbyggt med JavaScript? Sidan skulle tjäna på att istället köra med validerad HTML, både ur SEO- och användarsynpunkt.

Sen så finns det en hel del småsaker som skulle kunna optimeras - ett exempel är att lägga alla ikoner (inklusive hover) i en och samma bild och sedan visa dem som bakgrunder, och att du kör med UTF-8 istället för ISO-8859-1.

På något sätt irriterar jag mig lite på den där laddningssymbolen, men det kanske bara är jag.

lite intressant att du använt de inbyggda kontrollerna i .net till precis ALLT, oavsett om det behövs eller inte, utom just inloggningen. Den inbyggda membership-providern har ju skydd mot SQL-injections.

Fixa till din UpdatePanel4 i Default.aspx. Jag får System.Exception: No database specified.