Netjes scripten

Xjessex schreef:

Eerdere citaten tonen

Rogier schreef:

Eerdere citaten tonen

Raaoul schreef:

Alles volgens Zend (http://framework.zend.com/manual/en/coding-standard.coding-style.html) aangezien ik ook gewoon dat framework gebruik

Ik ook.

Op mijn werk moet ik alleen wel onderscheid maken in ifelse en functies.

if()
{

}

function() {

}

Hoezo dat dan?

toevoeging op 19 juni 2012, 08:45:48:
@Joey, regel 21:
Yeah

Deed ik ook vaak, alleen kreeg ik commentaar op van een heleboel mensen dat je geen shorttags moest gebruiken .

Rogier schreef:

Eerdere citaten tonen

Raaoul schreef:

Alles volgens Zend (http://framework.zend.com/manual/en/coding-standard.coding-style.html) aangezien ik ook gewoon dat framework gebruik

Ik ook.

Op mijn werk moet ik alleen wel onderscheid maken in ifelse en functies.

if()
{

}

function() {

}

Ik doe juist andersom.

if() {

} elseif() {

}

function()
{

En ik doe

if(){

}
elseif(){

}
else
{

}

en

function() {

}

OF soms zo

[COLOR= #0000BB]<?php[/COLOR]
[URL="http://www.php.net/if"]if[/URL][COLOR= #007700]([/COLOR]statement[COLOR= #007700])[/COLOR]

    [COLOR= #FF8000]// iets[/COLOR]
[URL="http://www.php.net/else"]else[/URL]

    [COLOR= #FF8000]// iets anders[/COLOR]
?>

Alleen bij redirects enz zou ik deze manier niet gebruiken.

Ik gebruik bij ifs altijd die {} accolades.
+ Onderscheid tussen functies & ifs, in spatie:

[COLOR= #0000BB]<?php[/COLOR]
  [URL="http://www.php.net/if"]if[/URL] [COLOR= #007700]([/COLOR]statement[COLOR= #007700]) {[/COLOR]
    [URL="http://www.php.net/echo"]echo[/URL] [COLOR= #DD0000]Oh my![/COLOR][COLOR= #007700];
  }[/COLOR] [URL="http://www.php.net/else"]else[/URL] [COLOR= #007700]{[/COLOR] 
    doNothing[COLOR= #007700]();
  }

  function[/COLOR] doNothing[COLOR= #007700](){[/COLOR]
    [COLOR= #FF8000]// do nothing[/COLOR]
  [COLOR= #007700]}[/COLOR]

Wel zo compact mogelijk dus, en ALTIJD inspringen, bij ALLES. Echt een aanrader voor welke taal dan ook. Dit is natuurlijk persoonlijke smaak eigenlijk, maar wat je iig altijd moet doen:

- Inspringen (Ja, ik herhaal mezelf ff, zo belangrijk is het)
- FIOP (Filter input, escape output [google maar])
- Constructors gebruiken in classes. (Het is zo kneuterig nadat je een class aanmaakt nog allerlei functies aan te roepen)
- Geen spaghettistructuur (Dus niet: teveel classes aanmaken die elkaar extenden waar het niet nodig is en van elkaar afhankelijk laten worden en.. nouja je begrijpt het wel)
- Vars en functies duidelijke namen geven (oftewel setName(), $idQuery, filterEmail(), met functies gebeurt dit vaak vanzelf maar met vars kom je tevaak $aap $noot $mies tegen (ik betrap mezelf er soms ook wel eens op))
- Switch gebruiken ipv eindeloze elseifs.
And many more...

Licht er een beetje aan Als het allemaal ifjes zijn voor controles zoals:

[COLOR= #0000BB]<?php[/COLOR]
[URL="http://www.php.net/if"]if[/URL][COLOR= #007700]([/COLOR][URL="http://www.php.net/empty"]empty[/URL][COLOR= #007700]([/COLOR]iets[COLOR= #007700])) {[/COLOR]
[COLOR= #FF8000]// error ding[/COLOR]
[COLOR= #007700]}[/COLOR] [URL="http://www.php.net/if"]if[/URL][COLOR= #007700]([/COLOR]iets[COLOR= #007700]([/COLOR]iets[COLOR= #007700])) {[/COLOR]
[COLOR= #FF8000]// error ding[/COLOR]
[COLOR= #007700]}[/COLOR]

?>

En anders

[COLOR= #0000BB]<?php[/COLOR]

[URL="http://www.php.net/if"]if[/URL][COLOR= #007700]([/COLOR]iets[COLOR= #007700]) {[/COLOR]

[COLOR= #FF8000]// iets[/COLOR]

[COLOR= #007700]}[/COLOR] [URL="http://www.php.net/else"]else[/URL] [COLOR= #007700]{[/COLOR]

[COLOR= #FF8000]// iets[/COLOR]

[COLOR= #007700]}[/COLOR]

?>

Fass schreef:

Ik gebruik bij ifs altijd die {} accolades.
+ Onderscheid tussen functies & ifs, in spatie:

[COLOR= #0000BB]<?php[/COLOR]
  [URL="http://www.php.net/if"]if[/URL] [COLOR= #007700]([/COLOR]statement[COLOR= #007700]) {[/COLOR]
    [URL="http://www.php.net/echo"]echo[/URL] [COLOR= #DD0000]Oh my![/COLOR][COLOR= #007700];
  }[/COLOR] [URL="http://www.php.net/else"]else[/URL] [COLOR= #007700]{[/COLOR] 
    doNothing[COLOR= #007700]();
  }

  function[/COLOR] doNothing[COLOR= #007700](){[/COLOR]
    [COLOR= #FF8000]// do nothing[/COLOR]
  [COLOR= #007700]}[/COLOR]

Wel zo compact mogelijk dus, en ALTIJD inspringen, bij ALLES. Echt een aanrader voor welke taal dan ook. Dit is natuurlijk persoonlijke smaak eigenlijk, maar wat je iig altijd moet doen:

- Inspringen (Ja, ik herhaal mezelf ff, zo belangrijk is het)
- FIOP (Filter input, escape output [google maar])
- Constructors gebruiken in classes. (Het is zo kneuterig nadat je een class aanmaakt nog allerlei functies aan te roepen)
- Geen spaghettistructuur (Dus niet: teveel classes aanmaken die elkaar extenden waar het niet nodig is en van elkaar afhankelijk laten worden en.. nouja je begrijpt het wel)
- Vars en functies duidelijke namen geven (oftewel setName(), $idQuery, filterEmail(), met functies gebeurt dit vaak vanzelf maar met vars kom je tevaak $aap $noot $mies tegen (ik betrap mezelf er soms ook wel eens op))
- Switch gebruiken ipv eindeloze elseifs.
And many more...

Even een aantal toevoegingen, want dit kan handig zijn.

- Gebruik geen GOTO, zorgt ook voor spaghetti code.
- Geen double quotes, maar single quotes. Maar er is één uitzondering waarbij je juist wel double quotes moet gebruiken.
- Programmeer met Engelse variable, class- en functie namen. Zowat iedere web developer kan Engels begrijpen.
- Comment lines kunnen divers zijn. Kijk maar eens naar de onderste PHP code voor instructies;

[COLOR= #0000BB]<?php[/COLOR]
[COLOR= #FF8000]// Opmerking
/* Opmerking
# Opmerking
/**
 * Documentatie
 */[/COLOR]
?>

- Zet eventueel een documentatie (uitgebreide opmerking) bij het script. Kan handig zijn als je het script gaat verkopen of releasen.
- Meerdere classes die bij dezelfde soort horen? Namespaces kunnen overzichtelijk zijn.

@Joeey: enkele quotes voor echoes met html. dubbele quotes voor SQL?

Rogier schreef:

@Joeey: enkele quotes voor echoes met html. dubbele quotes voor SQL?

Nee, altijd single quotes... Maar bij SQL is dit niet het geval.

[COLOR= #0000BB]<?php
$newsQuery[/COLOR] [COLOR= #007700]=[/COLOR] [COLOR= #DD0000]"SELECT * FROM database WHERE blabla = 0"[/COLOR][COLOR= #007700];[/COLOR]
?>

@Joeey: Je weet dat ik misschien nog wel iets meer ervaring heb aangezien je net 6 maanden programmeert? zoniet moet je verbeteren hoor. Maar wat je zegt is bullshit.

Je zegt dat er altijd maar een goeie manier is, wat natuurlijk onzin is. Het is best gebruikelijk om te echoen met dubbele quotes. Ik vind het alleen onoverzichtelijk omdat de var zijn kleurtje verliest.

Overigens moet je mijn bericht goed lezen. Je doet slim door een voorbeeld code te schrijven van een quoery. Dude O_O. Want dat is ongeveer precies wat ik net zeg. SQL met dubbele quotes. (dit staat namelijk niet in jouw bericht, en in tegenstelling tot enkele quotes bij html, is dubbele bij SQL wel verplicht)

Als je iets echot (of aan je output toevoegt) zoals JS, waarin je dus enkele quotes gebruikt, kun je ook beter dubbele gebruiken. Anders moet je het handmatig gaan zitten escapen.

Spoiler tonen

Of heredoc/nowdoc gebruiken

Rogier schreef:

@Joeey: Je weet dat ik misschien nog wel iets meer ervaring heb aangezien je net 6 maanden programmeert? zoniet moet je verbeteren hoor. Maar wat je zegt is bullshit.

Je zegt dat er altijd maar een goeie manier is, wat natuurlijk onzin is. Het is best gebruikelijk om te echoen met dubbele quotes. Ik vind het alleen onoverzichtelijk omdat de var zijn kleurtje verliest.

Overigens moet je mijn bericht goed lezen. Je doet slim door een voorbeeld code te schrijven van een quoery. Dude O_O. Want dat is ongeveer precies wat ik net zeg. SQL met dubbele quotes. (dit staat namelijk niet in jouw bericht, en in tegenstelling tot enkele quotes bij html, is dubbele bij SQL wel verplicht)

Oh, ik dacht dat je het raar vond om double quotes te gebruiken, door die vraagteken.

Er zijn ook meerdere goede manieren, maar het waren gewoon tips, niet dat dat het enige is (tevens ook nooit gezegd?).

En btw, ik heb wel langer dan 6 maanden ervaring. Geen 5 jaar zoals jij, maar ik schat zon 2 jaar.

Grappig, want ik denk nog geen 3 maanden terug vroeg je me van alles over een class. Maarja zullen we er over ophouden?

Joeey schreef:

Eerdere citaten tonen

Rogier schreef:

@Joeey: enkele quotes voor echoes met html. dubbele quotes voor SQL?

Nee, altijd single quotes... Maar bij SQL is dit niet het geval.

[COLOR= #0000BB]<?php
$newsQuery[/COLOR] [COLOR= #007700]=[/COLOR] [COLOR= #DD0000]"SELECT * FROM database WHERE blabla = 0"[/COLOR][COLOR= #007700];[/COLOR]
?>

Als het een numerieke waarde heeft hoef je niet perse quotes toe te voegen

Shahiem schreef:

Eerdere citaten tonen

Joeey schreef:

Eerdere citaten tonen

Rogier schreef:

@Joeey: enkele quotes voor echoes met html. dubbele quotes voor SQL?

Nee, altijd single quotes... Maar bij SQL is dit niet het geval.

[COLOR= #0000BB]<?php
$newsQuery[/COLOR] [COLOR= #007700]=[/COLOR] [COLOR= #DD0000]"SELECT * FROM database WHERE blabla = 0"[/COLOR][COLOR= #007700];[/COLOR]
?>

Als het een numerieke waarde heeft hoef je niet perse quotes toe te voegen

Booleans ook niet

In Java doe ik gewoonlijk dit:

public class Example  {
private naam = "Maartenf";
public Example {
if (naam == "Maartenf") {
JOptionPane.showMessageDialog("Juist!");
}
else { 
JOptionPane.showMessageDialog("Fout!");
    }
  }
}