[PHP/(My)SQL] voor dummies - Deel 52

Ik zou wel een voorbeeld template zien.
De class zelf zegt mij niet zo veel, het gebruik er van zegt meer.

En mag ik vragen waarom jij geen access control modifiers (public/protected/private) gebruikt?

quote:

Op zondag 16 december 2007 16:04 schreef colourAgga het volgende:
In ieder geval, laten we weer verder gaan met PHP want daar is dit topic voor.

Mensen nog gekeken naar die simpele template klasse? Kan die nog simpeler dan dit?

Dit vind ik een voorbeeld van bad coding practice. De parameter $bool_File geeft aan of de parameter $str_Replacement een replacement string bevat of de naam van een bestand dat je als replacement gebruikt. Waarom maak je daar niet gewoon 2 functies van, dat levert echt veel beter leesbare code op.
Dat ik een hekel heb aan hungarian prefixes had ik al eerder gezegd.

En kan het nog simpeler? Ja, Smarty gebruiken!

Wat zijn hungarian prefixes

quote:

Op zondag 16 december 2007 17:15 schreef Farenji het volgende:
Prefixes in de variable names waar je aan zou moeten kunnen zien wat voor type waarde er in de variable zit (object, boolean, string, int etc). Vrij gevaarlijk bij loosely typed languages zoals php omdat dat type nooit vaststaat.

dus in str_Needle is str de 'hungarian prefix'

quote:

Op zondag 16 december 2007 17:42 schreef Farenji het volgende:

[..]

Ja.

en dat is stom omdat str_Needle ook iets anders kan zijn dan een string?

quote:

Op zondag 16 december 2007 18:27 schreef qu63 het volgende:

[..]

en dat is stom omdat str_Needle ook iets anders kan zijn dan een string?

Klopt. In PHP kun je sowieso niet eisen dat iets van een bepaald type is. Een integer meegeven als een string gevraagd wordt (of andersom) wordt gewoon toegestaan. Daarnaast is het stom, omdat je nu heel lastig van type kunt veranderen. Als je besluit dat iets van een int naar een double gaat, dan kun je alle verwijzingen naar die variabele gaan aanpassen.

Kortom: niet doen.

Ik heb een keer een PHP boek gelezen waar ze ook classes zo een naam gaven : 'clsFooBar';

PHP 5 Super Boek was dat geloof ik.

Of "iNumber"

quote:

Op zondag 16 december 2007 13:53 schreef colourAgga het volgende:
Welke versie van safari gebruik jij dan? In 2.0.4 werkt het in ieder geval

Safari 2 is wat achterhaald ondertussen. En in 3.0.4 werkt het menu niet goed, de >> die voor de tekst komen, zorgen ervoor dat de tekst naar de volgende regel gaat. Zonder dat het vakje groter wordt overigens. Maar dat had je zelf vast ook al wel gezien.

quote:

Op zondag 16 december 2007 17:15 schreef Farenji het volgende:
Prefixes in de variable names waar je aan zou moeten kunnen zien wat voor type waarde er in de variable zit (object, boolean, string, int etc). Vrij gevaarlijk bij loosely typed languages zoals php omdat dat type nooit vaststaat.

Nooit is ook niet helmaal meer waar in php5. Je kunt bij functies wel type hinting gebruiken voor objecten en arrays. Niet voor dingen als int of string.

Weet iemand hoe ik alle 7 dagen van een bepaald weeknummer en jaartal kan verkrijgen?

Ik gebruik op het moment het volgende stukje code:



Het punt is echter dat deze natuurlijk niet werkt tijdens de jaarwisseling. Iemand? :)

Type prefixes gebruik ik meestal omdat er normaal nog een verificatie klasse overheen gaat om te checken wat voor iets een variabele is met een hoop is_string en is_int. Is ook nog iets wat is blijven hangen bij mij van andere talen etc. etc. Kunnen jullie denk ik beter overheen kijken

quote:

Op zondag 16 december 2007 18:27 schreef qu63 het volgende:

[..]

en dat is stom omdat str_Needle ook iets anders kan zijn dan een string?

Ik gebruik meestal:

En stop daar dus geen arrays in (ook al zou het kunnen).

Iemand vroeg nog om een voorbeeld template. Klik hier. Verder is dit ding nog niet met access control modifiers. Die zal ik er even inzetten.

Ik zal de hungarian prefixes er ook wel uithalen, is iets wat ik normaal gesproken voornamelijk voor mezelf gebruik omdat ik dan duidelijk maak wat ik opsla in een variabele Zelf ben ik het er niet mee eens dat het bad coding practice is. Het is pas bad coding practice als je er een gevoel van veiligheid aan gaat verbinden. Dat gaat in PHP namelijk niet op. Ik zet ze er alleen in zodat ik meteen weet wat er in een bepaalde variabele hoort te zitten.

En de hele code aanvallen als je van int naar double gaat is ook niet zo lastig met een regular expression in VIM.

Overigens heb ik dat template ding even gerecode met alle opmerkingen van hierboven meegenomen.

[ Bericht 4% gewijzigd door colourAgga op 17-12-2007 15:38:08 ]

quote:

Op maandag 17 december 2007 00:13 schreef Geqxon het volgende:
Weet iemand hoe ik alle 7 dagen van een bepaald weeknummer en jaartal kan verkrijgen?

Ik gebruik op het moment het volgende stukje code:
[ code verwijderd ]

Het punt is echter dat deze natuurlijk niet werkt tijdens de jaarwisseling. Iemand?

Deze is simpel aan te passen :

quote:

Op maandag 17 december 2007 15:27 schreef colourAgga het volgende:
Ik zal de hungarian prefixes er ook wel uithalen, is iets wat ik normaal gesproken voornamelijk voor mezelf gebruik omdat ik dan duidelijk maak wat ik opsla in een variabele Zelf ben ik het er niet mee eens dat het bad coding practice is. Het is pas bad coding practice als je er een gevoel van veiligheid aan gaat verbinden.

Mijn opmerking over bad coding practice ging neit over hungarian prefixes maar over de method replace() waarbij de betekenis van de 2e parameter sterk af hangt van de 3e parameter: de content zelf of de naam van een bestand waar de content in staat.

Waarom gebruik je eigenlijk "if( $clean == true )"? Wat is er mis met "if( $clean )"? Klopt je clean() functie wel?

De replacement is of een tekst of de tekst uit een bestand. In dat geval wordt de replacement dus het bestand. Ik snap wat je bedoelt en hij zou ook in een losse functie kunnen. Ik vind hem daar lekkerder staan.

De clean() functie doet het, niets meer en niets minder dan newlines en dubbele spaties strippen Omdat ik een $clean == true duidelijker vindt dan een ( $clean ). Persoonlijke voorkeur die bij iedereen anders is. Heel veel mensen zouden ook directory of configuration afkorten. Dingen die ik persoonlijk ook nooit doe.

quote:

Op maandag 17 december 2007 21:44 schreef colourAgga het volgende:

De clean() functie doet het, niets meer en niets minder dan newlines en dubbele spaties strippen Omdat ik een $clean == true duidelijker vindt dan een ( $clean ). Persoonlijke voorkeur die bij iedereen anders is.

Yep, we zijn allemaal anders. En da's maar goed ook. Bij vergelijken met een constante zou ik trouwens true == $clean schrijven, dan gaat php vanzelf zeuren als je een = vergeet. Het voorkomt vaag gedrag.

Vies programmeren

Net zoals ik soms als debugging ergens " if (isset($thisvarisprobarlyneverused)) " doe, wat prima ging, tot mijn collega voor de grap eens een variabele met exact deze naam had geinit. Lul

quote:

Op maandag 17 december 2007 21:44 schreef colourAgga het volgende:
De clean() functie doet het, niets meer en niets minder dan newlines en dubbele spaties strippen

Als er tussen 2 woorden eerst een regeleinde of 2 spaties staan dan staan ze daarna opeens aan elkaar vast. Lijkt me niet handig.

quote:

Omdat ik een $clean == true duidelijker vindt dan een ( $clean ). Persoonlijke voorkeur die bij iedereen anders is.

Overbodig en dus lelijk _{vind ik}

quote:

Op dinsdag 18 december 2007 10:58 schreef Farenji het volgende:

[..]

Das wel een goede tip. Ik ben zelf overigens gewend geraakt aan perl ranzigheid als "doSomething() unless $test;" of nog erger: "$test or doSomething();"

Dat kan in php ook, doSomething() or die()
Zelfde constructie, zelfde idee.

Ik gebruik eigenlijk ook wel regelmatig false == $var. Vind ik er eigenlijk stiekem zelfs mooier eruit zien maar ik had het expres niet gedaan omdat het script dan weer 'onlogisch' is voor iemand die net PHP kijkt

Maak er dan true == (false == $var) van

quote:

Op dinsdag 18 december 2007 14:48 schreef SuperRembo het volgende:
Maak er dan true == (false == $var) van

Daar zou je dan weer wel een functie voor kunnen schrijven
true_false_var() ofzo, duidelijk wat die doet toch?