Dit is geen Twitter class?
Ik zou het eerder TwitLink noemen zowel als de scriptnaam als de Classnaam.
Ik zou het eerder TwitLink noemen zowel als de scriptnaam als de Classnaam.



public static function change($set_text)
{
//Standaard tekst neerzetten
$text = $set_text;
//Standaard tekst naar namen veranderen
$text = self:SMILEYYYYYYYYYYYBEGIN:set_names($text);
//Standaard tekst naar hashtags veranderen
$text = self:SMILEYYYYYYYYYYYBEGIN:set_hashtag($text);
//Tekst terugsturen
return $text;
} public static function change($string)
{
//Standaard tekst naar namen veranderen
$text = self:SMILEYYYYYYYYYYYBEGIN:set_names($string);
//Standaard tekst naar hashtags veranderen
$text = self:SMILEYYYYYYYYYYYBEGIN:set_hashtag($text);
//Tekst terugsturen
return $text;
} Aarclay schreef:In dit geval is static bezig, omdat het een eenmalige wijziging is voor een string.En waarom zou static beter zijn? Beargumentatie wordt wel op prijs gesteld.

Jenner schreef:Underscores voor functions, ok, maar voor methods kune je beter camelCase gebruikenVeel mensen gebruiken ook gewoon underscores. Past PHP ook gewoon toe. Nog een paar tips voor de volgende keer. Kun je beter deze implementatie gebruiken:Eerdere citaten tonen
Kyron schreef:Nette eerste class hoor, alleen zou ik (iig voor functies) camelCase gebruiken[COLOR= #0000BB]<?php[/COLOR] [COLOR= #FF8000]//Static function is handiger in dit geval.[/COLOR] [COLOR= #0000BB]$string[/COLOR] [COLOR= #007700]=[/COLOR] Twitter[COLOR= #007700]::[/COLOR]change[COLOR= #007700]([/COLOR][COLOR= #0000BB]$oldstring[/COLOR][COLOR= #007700]);[/COLOR] ?>
Jenner schreef:Static is handig als je het gewoon als class kunt toepassen op iets, als je OOP met classes werkt en echt een object een class wil toekennen is het beter om geen static te gebruiken. Aangezien het hier om een linkding gaat en niet om een object als een gebruiker database etc is static hier idd meer voor de hand liggend.In dit geval is static bezig, omdat het een eenmalige wijziging is voor een string.Eerdere citaten tonen
Aarclay schreef:En waarom zou static beter zijn? Beargumentatie wordt wel op prijs gesteld.
De manier die in de class wordt toegepast is omslachtig en ik denk uiteindelijk niet de meest handige manier. Omslachtig hoeft niet altijd een probleem te zijn als het extra functionaliteit bied, maar in dit geval bied het geen extra functionaliteit, dus is het beter om gewoon een static functie er van te maken
Kyron schreef:Underscores en camelCase zijn pure smaak. Er zijn zat frameworks die gebruik maken van underscores. (Te denken aan Zend Framework en FuelPHP) Het maakt qua functionaliteit niks uit, maar qua leesgemak is het soms fijner om underscores te gebruiken.Underscores voor functions, ok, maar voor methods kune je beter camelCase gebruikenEerdere citaten tonen
Jenner schreef:Veel mensen gebruiken ook gewoon underscores. Past PHP ook gewoon toe. Nog een paar tips voor de volgende keer. Kun je beter deze implementatie gebruiken:Eerdere citaten tonen
Kyron schreef:Nette eerste class hoor, alleen zou ik (iig voor functies) camelCase gebruiken[COLOR= #0000BB]<?php[/COLOR] [COLOR= #FF8000]//Static function is handiger in dit geval.[/COLOR] [COLOR= #0000BB]$string[/COLOR] [COLOR= #007700]=[/COLOR] Twitter[COLOR= #007700]::[/COLOR]change[COLOR= #007700]([/COLOR][COLOR= #0000BB]$oldstring[/COLOR][COLOR= #007700]);[/COLOR] ?>Static is handig als je het gewoon als class kunt toepassen op iets, als je OOP met classes werkt en echt een object een class wil toekennen is het beter om geen static te gebruiken. Aangezien het hier om een linkding gaat en niet om een object als een gebruiker database etc is static hier idd meer voor de hand liggend.Eerdere citaten tonen
Jenner schreef:In dit geval is static bezig, omdat het een eenmalige wijziging is voor een string.Eerdere citaten tonen
Aarclay schreef:En waarom zou static beter zijn? Beargumentatie wordt wel op prijs gesteld.
De manier die in de class wordt toegepast is omslachtig en ik denk uiteindelijk niet de meest handige manier. Omslachtig hoeft niet altijd een probleem te zijn als het extra functionaliteit bied, maar in dit geval bied het geen extra functionaliteit, dus is het beter om gewoon een static functie er van te maken
Rogier schreef:Ik kon verder niks verzinnen aan een class.Maar eerlijk gezegd vind ik het erg onnodig om hier een class voor te maken. Maak er gewoon een functie van een doe hem in je controller class ofzo.
Jenner schreef:Opzich maakt het weinig verschil maar ikzelf gebruik meestal deze naming conventions, overigens ook van Zend Framework dus ik vind het wel gek als ze zich niet aan hun eigen conventie houden haha.Underscores en camelCase zijn pure smaak. Er zijn zat frameworks die gebruik maken van underscores. (Te denken aan Zend Framework en FuelPHP) Het maakt qua functionaliteit niks uit, maar qua leesgemak is het soms fijner om underscores te gebruiken.
Kyron schreef:Oja, Zend gebruikt alleen voor classes. FuelPHP gebruikt het ook voor functie namen. Maar ik vind persoonlijk camelCase ook mooier en duidelijker, maar als je bijvoorbeeld functienamen wilt ontleden dan zijn underscores makkelijker. (Underscores worden veel gebruikt voor classes, omdat die voor de autoloader worden ontleed)Opzich maakt het weinig verschil maar ikzelf gebruik meestal deze naming conventions, overigens ook van Zend Framework dus ik vind het wel gek als ze zich niet aan hun eigen conventie houden haha.Eerdere citaten tonen
Jenner schreef:Underscores en camelCase zijn pure smaak. Er zijn zat frameworks die gebruik maken van underscores. (Te denken aan Zend Framework en FuelPHP) Het maakt qua functionaliteit niks uit, maar qua leesgemak is het soms fijner om underscores te gebruiken.
http://framework.zend.com/manual/en/coding-standard.naming-conventions.html
Maar t is natuurlijk eigen smaak alleen heb ik tot nu toe vaker camelCase gezien en ik vind dit persoonlijker ook duidelijker.