WN - Skeptisk till ny programmerare...

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

- Klientsidans teknologier, design och grafik (https://www.wn.se/forum/forumdisplay.php?f=12)

- - Skeptisk till ny programmerare... (https://www.wn.se/forum/showthread.php?t=1052252)

Skeptisk till ny programmerare...

Tjena,

Kan någon kolla in CSSen på www.socialfrihet.se och se om allt verkligen står rätt till där? När jag kollar i style.css filen så blir jag helt förvirrad men jag kan å andra sidan ytterst lite programmering, men har aldrig sett något liknande. Tycker det ser stökigt ut och jag är orolig att programmeraren spaghettikodar sidan.

Bad honom göra om CSSen för den strulade innan och han sa att den gamla var dåligt kodad. En grej jag märkte är att han flyttade ut länken till loggan i en egen .php fil istället för att lägga den i style.css, är det normalt?

Ett utdrag från style.css:

PHP-kod:


		
			
html,body,div,span,applet,object,iframe,h1,h2,h3,h4,h5,h6,p,blockquote,pre,a,abbr,acronym,address,big,cite,code,del,dfn,em,font,img,ins,kbd,q,s,samp,small,strike,strong,sub,sup,tt,var,b,u,i,center,dl,dt,dd,ol,ul,li,fieldset,form,label,legend,caption{border:0;outline:0;font-size:100%;vertical-align:baseline;margin:0;padding:0}

body{color:#000;font-family:Arial, Helvetica, sans-serif;font-size:14px;background:#fff;line-height:28px; float:left; margin:0; padding:0;}

ul,ol{list-style:none}

img,fieldset{border:none}

.hidden{position:absolute;left:-9999px}

.clear{clear:both}

#header{width:946px;overflow:hidden;margin:0 auto;padding:0 0 0 19px}

.logo_area{border-bottom:1px solid #a9a9a9;width:946px;overflow:hidden;margin-bottom:25px; float:left;}



.logo_area .logos{float:left;width:440px;height:79px;padding:30px 0 42px}

.logo_area .logos a{width:440px;height:79px;display:block;}



.logo_search_area{float:left;width:328px;height:146px;border-right:1px solid #a9a9a9;padding:0 6px 5px 0; margin:0px 0px 0px 171px;}

.iconsearch_bg,.iconsearch_bg_lft,.iconsearch_bg_rgt{height:146px}

.iconsearch_bg{background:url(images/icon_search_bg.jpg) repeat-x}

.iconsearch_bg_lft{background:url(images/icon_search_bg_lft.jpg) no-repeat}

.iconsearch_bg_rgt{background:url(images/icon_search_bg_rgt.jpg) no-repeat right top}

.iconarea{width:308px;overflow:hidden;height:64px;padding:9px 0 0 20px}

.iconarea .box1{font-family:Arial, Helvetica, sans-serif;font-size:14px;color:#fff;width:79px;float:left;padding:15px 0 20px}

.iconarea .box2{width:229px;overflow:hidden;float:left}

.iconarea .box2 a{float:left;width:70px;display:block;margin-left:20px;background:url(images/icon_li_bg.jpg) no-repeat left top;padding:0 0 9px 14px}

.iconarea img{border:0;}

Vad säger ni? Behöver jag oroa mig?

Citat:

Ursprungligen postat av aDonis (Inlägg 20432562)

PHP-kod:


		
			
html,body,div,span,applet,object,iframe,h1,h2,h3,h4,h5,h6,p,blockquote,pre,a,abbr,acronym,address,big,cite,code,del,dfn,em,font,img,ins,kbd,q,s,samp,small,strike,strong,sub,sup,tt,var,b,u,i,center,dl,dt,dd,ol,ul,li,fieldset,form,label,legend,caption{border:0;outline:0;font-size:100%;vertical-align:baseline;margin:0;padding:0}

body{color:#000;font-family:Arial, Helvetica, sans-serif;font-size:14px;background:#fff;line-height:28px; float:left; margin:0; padding:0;}

ul,ol{list-style:none}

img,fieldset{border:none}

.hidden{position:absolute;left:-9999px}

.clear{clear:both}

#header{width:946px;overflow:hidden;margin:0 auto;padding:0 0 0 19px}

.logo_area{border-bottom:1px solid #a9a9a9;width:946px;overflow:hidden;margin-bottom:25px; float:left;}



.logo_area .logos{float:left;width:440px;height:79px;padding:30px 0 42px}

.logo_area .logos a{width:440px;height:79px;display:block;}



.logo_search_area{float:left;width:328px;height:146px;border-right:1px solid #a9a9a9;padding:0 6px 5px 0; margin:0px 0px 0px 171px;}

.iconsearch_bg,.iconsearch_bg_lft,.iconsearch_bg_rgt{height:146px}

.iconsearch_bg{background:url(images/icon_search_bg.jpg) repeat-x}

.iconsearch_bg_lft{background:url(images/icon_search_bg_lft.jpg) no-repeat}

.iconsearch_bg_rgt{background:url(images/icon_search_bg_rgt.jpg) no-repeat right top}

.iconarea{width:308px;overflow:hidden;height:64px;padding:9px 0 0 20px}

.iconarea .box1{font-family:Arial, Helvetica, sans-serif;font-size:14px;color:#fff;width:79px;float:left;padding:15px 0 20px}

.iconarea .box2{width:229px;overflow:hidden;float:left}

.iconarea .box2 a{float:left;width:70px;display:block;margin-left:20px;background:url(images/icon_li_bg.jpg) no-repeat left top;padding:0 0 9px 14px}

.iconarea img{border:0;}

Vad säger ni? Behöver jag oroa mig?

CSS är inte min grej, men jag tycker att det ser ganska vettigt ut.

Jag är lite CSS perfektionist. Tycker också spontant att det ser ok ut. Jag skriver också min CSS horisontellt. Dock kanske personen borde ha kommenterat bättre. Lagt in CSS'en i sektions block osv..

Förresten.. Du kan alltid använda en CSS processor om du vill placera om CSS koden.

http://procssor.com/

De första raderna är en så kallad CSS reset för att få alla webbläsare att utgå från samma punkt och resten (.icon*) verkar helt enkelt bara komprimerat, så ser inga konstigheter i den koden inte.

Exakt vad är det du inte sett tidigare? :)

Inga större konstigheter med koden, vilket redan flera påpekat. Normalt sett är det dock inte CSS eller dylikt man syftar på när man talar om spaghettikod utan det begreppet brukar vara förknippat med "riktig kod", i det här sammanhanget t.ex. PHP eller .NET.

Är det ganska mycket CSS kan det vara trevligt med kommenterad blockuppdelning.

Du behöver inte oroa dig, css'n ser helt ok ut den är bara skriven i kompakt form. För övrigt är det inte programmering man sysslar med i css filen.

Ser några grejer som jag reagerar på. Inget är direkt allvarligt men jag skulle fråga varför man döpt vissa klasser som man har gjort. T.ex. tycker jag att det är ganska märkligt att döpa klasser till box1, box2, nånting_bg_lft, _bg_rgt.

En stor fördel med Css är att man separerar innehåll från design men det verkar man inte ha tänkt på om man granskar klasserna. Eller så har det varit väldigt tajt med tid, det händer ju.

Är det många .box1, box2 börjar det ringa varningsklockor men det är svårt att säga nånting mer än så utan att känna till mer info.

Ser ut som att han har använt en css minifier.

Ser heller inte någonting konstigt, precis som tidigare talare har sagt.

Jag skriver också min css horisontell, på så sätt slipper jag scrolla i tusen år för att hitta en sak.

Dock kan jag hålla med om att sektionsindelning vore bra, speciellt om det är många klasser och id'ns.

Men utöver det behöver du inte oroa dig!

Tack allesammans, fortsätter med denna killen då!

Yes, gör så! :)

Sen om du nu inte gillar horisontell css kan du ju bara be honom att ändra det.

Citat:

Ursprungligen postat av dAEk (Inlägg 20432631)

Mja, box1, box2 osv kan ju vara ganska naturliga namn, beror lite på hur designen ser ut. Är detta det största problemet man har med sin "programmerare" så är det nog rätt lugnt. Sen har det ju inte sagts något om vilken prisklass vi talar om heller, det får ändå anses vara ganska så relevant i sammanhanget.

Jag skulle vilja tillägga att även om man av någon konstig anledning "gillar" horisontell CSS så är det fel. Du bör eftersträva så lättläst kod som möjligt, i alla lägen. Av den anledningen så skriver man inte CSS på det där sättet, gör man det så gör man "fel" (stora citationstecken på ordet fel).

En webbutvecklare, som ensam ska skriva all kod på frontend och backend, måste ju ha ett brett kunskapsområde. Hellre brett än djupt, men givetvis gärna båda. Men det betyder att han kanske inte skriver felfri CSS eller han kanske göra lite konstiga sql-frågor eller något annat som inte är helt klockrent ufört. Men på det stora hela gör han helt okej jobb, man måst helt enkelt väga in flera saker än en css-fil, som många redan konstaterat. Men jag hör absolut varningsklockor om jag tittar på den där filen. Men den säger inget om kvaliteten på hans arbete överlag.

Citat:

Ursprungligen postat av znap (Inlägg 20432849)

Jag skulle vilja tillägga att även om man av någon konstig anledning "gillar" horisontell CSS så är det fel. Du bör eftersträva så lättläst kod som möjligt, i alla lägen.

Jag håller inte med. Självklart är det bra att ha lättläst kod, men när man laddar upp koden till sidan så ska den vara optimerad. Att använda en CSS eller Javascript minifier är otroligt bra eftersom man kan drastiskt minska storleken på filerna.

Jag hör inga varningsklockor överhuvudtaget när jag kollar på strukturen på CSS filen. Jag ser snarare en programmerare som vill optimera laddningstiden på sin kunds sidor.

Citat:

Ursprungligen postat av tartareandesire (Inlägg 20432701)

Mja, box1, box2 osv kan ju vara ganska naturliga namn, beror lite på hur designen ser ut. Är detta det största problemet man har med sin "programmerare" så är det nog rätt lugnt.

Mja, jag har svårt att se hur box1 skulle vara ett naturligt namn men det kanske det är även om jag inte tycker det. Ett större orosmoln är de extra klasserna för att lägga in bakgrundsbilder etc. Det är inte snyggt utan känns väldigt slarvigt gjort.

Vad gäller läsbarhet håller jag med znap men det är egentligen inte ett problem iom att alla IDEs eller andra editors kan fixa formateringen via ett kortkommando eller liknande.

Citat:

Ursprungligen postat av captaindoe (Inlägg 20432852)

Jag hör inga varningsklockor överhuvudtaget när jag kollar på strukturen på CSS filen. Jag ser snarare en programmerare som vill optimera laddningstiden på sin kunds sidor.

En programmerare som vill optimera laddningstiden men som inte använder en minifier? Det borde ringa varningsklockor till både höger och vänster! ;)

Citat:

Ursprungligen postat av znap (Inlägg 20432849)

Du bör eftersträva så lättläst kod som möjligt, i alla lägen.

Ehm, nej. Som redan påpekats är den minifiering av CSS och JS som gäller i produktionsmiljö. I utvecklingsmiljö kör man självklart separerat och tydligt strukturerat, men när allt är testat finns det ingen anledning att inte ha det så kompakt som möjligt.

Men hallå grabbar? Det var då en ganska värdelös minifiering i sådana fall. Det är ganska uppenbart att den här css-filen inte är minifierad, den är bara skriven på ett något slarvigt och tämligen ostrukturerat sätt.

Jag har aldrig ifrågasatt vikten av att minifiera statiska filer i produktion. Men som sagt, det här är en mindre wordrpress-sajt, man vågar gissa att det inte finnns någon uppstyrd prod- och utveckligngsmiljö. CSS-filen förtäljer om en utvecklare som eventuellt inte är mån om läsbarheten i sin kod. Läsbarheten är A och O (tillsammans med mycket annat) när man pratar om underhåll av koden, något som bekostar företag mycket pengar. Håller man inte med om det så gör man sin arbetsgivare/uppdragsgivare en väldans otjänst...