[jayzee]

Ursprungligen postat av P3N

Jag försökte uttrycka det försiktigt :-)

Sanningen varar alltid längst

[P3N]

Ursprungligen postat av jayzee

Om vi nu pratar om pedant så verkar du ha missat att sidan körs i en <iframe>, två (!) olika jquery versioner laddas in, sidan validerar inte för 5 öre (html4 transitional), m.m.
Orkade inte kolla på resten efter detta, men jag skulle inte anlita denna person att göra en "professionell hemsida" åt mig om man säger så.

Ooo, btw där finns 3 jquery versioner (och jag som inte ens tycker om en)

[Conny Westh]

Jag tror det skulle tillföra en del värde om någon som har koll på riktlinjerna för frontend på detaljnivå och kan hänvisa till en sån sida eller, PDF av något slag. Jag bidrog med en pdf med riklinjer ur ett användbarhetsperspektiv. Men jag jobbar inte med frontend så jag är inte tillräckligt uptodate/insatt i detaljerna för frontend. Jag utevecklar bara backend och brukar överlåta frontend till de som jobbar med det.

[jayzee]

Personligen har jag ingen PDF att referera till, har professionellt jobbat med både front- och backend i 10+ år (bla. på Opera Software där jag fortfarande har väldigt bra kontakter) så det mesta sitter i ryggmärgen

Här är dock några bra länkar:
https://developers.google.com/speed/...es/rules_intro
http://isobar-idev.github.com/code-standards/
http://taitems.github.com/Front-End-...nt-Guidelines/

[Conny Westh]

Ursprungligen postat av jayzee

Personligen har jag ingen PDF att referera till, har professionellt jobbat med både front- och backend i 10+ år (bla. på Opera Software där jag fortfarande har väldigt bra kontakter) så det mesta sitter i ryggmärgen

Här är dock några bra länkar:
https://developers.google.com/speed/...es/rules_intro
http://isobar-idev.github.com/code-standards/
http://taitems.github.com/Front-End-...nt-Guidelines/

(OBS!!! Såg att när jag har mer än ett mellanslag i rad eller tab ('\t') klipps dessa bort i mitt inlägg så källkoden jag skrivit in blir helt sjuk i inlägget, men den ser bra ut i editläget)

Jag såg på githubben att de hade en rekommendation som jag tycker är helt fel. Som jag vill kommentera.

Måsvingar ska alltid stå på egen rad när man markerar block, för att det er en tydlig linje för ögat att följa. I den ursprungliga boken om K&R C så skrevs måsvingarna på det dåliga sättet men Kernigan själv ändrade denna standard när han skrev boken om ANSI C.

Det var enmycket logisk och begriplig ändring därför att det är hart när omöjligt att följa strukturen i koden om man måste flacka med blicken till slutet på en rad för att hitta en matchande måsvinge.

På githubben så had eman motsatt rekommendation och det motsätter jag mig bestämt.

Enkelt exempel:

// Dåligt
if (foo){
bar();
}

// Bra
if (foo)
{
bar();
}

Mer komplext exempel:

// Dåligt
if (foo1){
bar1a();
if (foo2){
if (foo3){
bar3a();
}
else{
bar3b();
}
}
else{
bar2b();
}
}
else{
bar3b();
}

// Så här blir betydligt bättre...
if (foo1)
{
bar1a();
if (foo2)
{
if (foo3)
{
bar3a();
}
else
{
bar3b();
}
}
else
{
bar2b();
}
}
else
{
bar3b();
}

// Se den tydliga linjen för ögat.....
if (foo1)
{
| bar1a();
| if (foo2)
| {
| | if (foo3)
| | {
| | | bar3a();
| | }
| | else
| | {
| | | bar3b();
| | }
| }
| else
| {
| | bar2b();
| }
}
else
{
| bar3b();
}

Använd alltid måsvingar för att markera vilka block. Det är ett extremt vanligt problem att man missar att lägga till måsvingarna när man lägger till en rad i koden och då får man inte den funktionalitet man önskat. Samtidigt är det betydligt svårare att snabbt öga igenom en kod och frstå vad den gör om man använder olika "format" för att skriva sin kod, var konsekvent och använd alltid måsvingar.

// Dåligt
if (foo)
bar();

// Bra
if (foo)
{
bar();
}

Det finns ett undantag och där är om man har extremt många korta oberoende if satser (som i sig inte kan ersättas av switch/case) som i detta exempel:

// Dåligt!
if (foo1) bar1();
if (foo2) bar2();
if (foo3) bar3(); bar7(); // bar7() Kommer alltid att exekveras,
if (foo4) bar4(); // men det är lätta att missa det.
if (foo5) bar5();
if (foo6) bar6();

// Ok! men inte idealiskt.
if (foo1) bar1();
if (foo2) bar2();
if (foo3) bar3();
if (foo4) bar4();
if (foo5) bar5();
if (foo6) bar6();

Men såna här villkorssamlingar är inte så vanliga och man ska använda dem med försiktighet. För lägger man till en sats till på samma rad så kommer den alltid att exekveras och det var kanske inte vad man tänkt.

[tartareandesire]

Ursprungligen postat av ConnyWesth

Jag såg på githubben att de hade en rekommendation som jag tycker är helt fel. Som jag vill kommentera.

Måsvingar ska alltid stå på egen rad när man markerar block, för att det er en tydlig linje för ögat att följa. I den ursprungliga boken om K&R C så skrevs måsvingarna på det dåliga sättet men Kernigan själv ändrade denna standard när han skrev boken om ANSI C.

Nja, det beror ju på vilken kodstandard man väljer att följa (om man nu följer någon). Man kan välja PEAR, PSR-0 eller något helt annat.

Den där listan på webbplatser som Conny listade är inte så mycket att bry sig om. Många som arbetar med webbdesign skulle aldrig någonsin få för sig att jobba med något som ser ut som exempelvis Skatteverkets hemsida. Unika, kreativa individer utvecklar ofta en egen stil och så ska det också vara. Sedan finns det också webbdesigners som har lite mindre intuitiv design och istället designar mer rationellt. En del kan tycka det tidigare är helt onödigt ögongodis eller att det senare är tråkigt men inget av dessa alternativ, eller allting däremellan, behöver vara bättre än något annat. Var man/kvinna (det finns inget hen-ord för detta? ) har sin plats helt enkelt.

[Conny Westh]

Ursprungligen postat av tartareandesire

Nja, det beror ju på vilken kodstandard man väljer att följa (om man nu följer någon). Man kan välja PEAR, PSR-0 eller något helt annat.

Den där listan på webbplatser som Conny listade är inte så mycket att bry sig om. Många som arbetar med webbdesign skulle aldrig någonsin få för sig att jobba med något som ser ut som exempelvis Skatteverkets hemsida. Unika, kreativa individer utvecklar ofta en egen stil och så ska det också vara. Sedan finns det också webbdesigners som har lite mindre intuitiv design och istället designar mer rationellt. En del kan tycka det tidigare är helt onödigt ögongodis eller att det senare är tråkigt men inget av dessa alternativ, eller allting däremellan, behöver vara bättre än något annat. Var man/kvinna (det finns inget hen-ord för detta? ) har sin plats helt enkelt.

De webbplatser jag listade var inte de jag tycker är bäst i världen, men jag betraktar det som en miniminivå att göra något som är bättre än dessa om man erbjuder sina tjänster som webbdesigner.

Sajterna är ändå ganska bra exempel att utgå ifrån och göra något som är ännu bättre. Bland annat är färgval en viktig fråga för att öka läsbarheten på texten.

[Unik]

Ja ska tänka på allt ni skrev och försöka göra det bättre till nästa gång !

[jonssondesign]

Jag skriver alltid första måsvingen precis efter ... vad man nu kallar det ... (frågan/bestämmelsen?), och en precis i slutet av blocket på en egen rad.

Såhär gör jag:
---------------------
if (foo){
	bar();
};

eller

.wrapper {
	background:#000;
}

Har alltid gjort, och kommer alltid göra, oavsett.

Tidigare skrev jag såhär (dock bara när jag skrev css):

.wrapper {background:#000;}

Man såg mycket mer kod på ett mindre utrymme, och man slapp scrolla så mycket.

Nu skriver jag dock alltid:

.wrapper {
	background:#000;
}

Att lägga varje måsvinge på enskilda rader tar bara inte längre tid, det tar även större plats (större filer), och man får himla lite kod per "sida", så man måste scrolla himla massa.

Är man inte gammal och trött, så tror jag mitt sett fungerar lika bra som connys.

[Conny Westh]

Om du gör så här så ser det betydligt proprare ut:

Såhär gör jag:
---------------------
if (foo)
{
	bar();
};

eller

.wrapper 
{
	background:#000;
}

Det blir tydligare ändå om du har mer komplex kod:

// Dåligt 
if (foo1){
	bar1a();
	if (foo2){
		if (foo3){
			bar3a();
		}
		else{
			bar3b();
		}
	}
	else{
		bar2b();
	}
}
else{
	bar3b();
}

// Så här blir betydligt bättre... 
if (foo1)
{
	bar1a();
	if (foo2)
	{
		if (foo3)
		{
			bar3a();
		}
		else
		{
			bar3b();
		}
	}
	else
	{
		bar2b();
	}
}
else
{
	bar3b();
}