599 Regels commentaar?quote:Op maandag 13 januari 2014 13:13 schreef raptorix het volgende:
[..]
Ik heb keer een stored procedure van iets van 600 regels geschreven
Was iets met een grote huizensite zeg maar
Nee was de searchquery maar daarin ook nog eens het hele paging en parameter mechanisme, pfffff hoofdpijn van dat ding.quote:
https://github.com/Shandem/Examine ?quote:Op maandag 13 januari 2014 13:17 schreef raptorix het volgende:
[..]
Nee was de searchquery maar daarin ook nog eens het hele paging en parameter mechanisme, pfffff hoofdpijn van dat ding.
We spreken hier over 2001, toen waren dat soort technieken nog niet zo gebruikelijk, wat we wel deden was de database readonly maken, dat scheelt ongeveer 40 procent. Tegenwoordig gebruik je voor dit soort zaken gewoon een noSql server of anders wel een ORM mapper.quote:
Ah, dat plaatst het wat in de context, als je een sp van 600 regels maakt moet er toch een lampje gaan branden. The horror!quote:Op maandag 13 januari 2014 13:20 schreef raptorix het volgende:
[..]
We spreken hier over 2001, toen waren dat soort technieken nog niet zo gebruikelijk, wat we wel deden was de database readonly maken, dat scheelt ongeveer 40 procent. Tegenwoordig gebruik je voor dit soort zaken gewoon een noSql server of anders wel een ORM mapper.
1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 | <?php // fictieve $_POST array $post = array('email' => 'hoi', 'password' => 'test'); function get_variables($data) { $key = 0; do { $keys = array_keys($data); $var = (array_keys($data)[$key]); global $$var; if (!empty($data[$var])) { $$var = $data[$var]; } else { $$var = null; } $key++; } while (array_key_exists($key, $keys)); } // hier wil ik $_POST gaan gebruiken get_variables($post); // variabelen komen uit de functie echo "$email<br>$password"; ?> |
Global vind ik zelf niet zo netjes. Ik zou het via een class (service) doen.quote:Op maandag 13 januari 2014 18:59 schreef n8n het volgende:
Ben ik weer. Om in een functie specifieke $_POST input te krijgen moest ik of telkens voluit $_POST['name'] schrijven, er een variabele van maken of variabelen global aanroepen. Heb nu een functie die je overal kan aanroepen die de $_POST data omzet in losse variabelen die je lokaal kan gebruiken.
[ code verwijderd ]
het maakt van elke key in een array een variable met de naam en waarde van de key. dus $_POST[email] wordt omgezet naar $email = 'waarde van email'. Als een waarde leeg is heeft de variabele een null waarde.
Op of aanmerkingen? Kom net kijken dus heb vaak het idee dat ik metideeën kom
global is toch nodig om de variabele überhaupt te kunnen definiëren buiten de scope van de function (of class)?quote:Op maandag 13 januari 2014 19:01 schreef bondage het volgende:
[..]
Global vind ik zelf niet zo netjes. Ik zou het via een class (service) doen.
Bij een class heb je global niet nodig. Als je het object eenmaal hebt aangemaakt kun je dat meegeven aan je functies. Ook kun je op jouw manier variabelen per ongeluk overschrijven als je dezelfde namen ergens anders gebruikt.quote:Op maandag 13 januari 2014 19:02 schreef n8n het volgende:
[..]
global is toch nodig om de variabele überhaupt te kunnen definiëren buiten de scope van de function (of class)?
dus variabelen die je binnen je class (maar buiten een functie aanmaakt) kan je binnen die class in elke functie gebruiken? Ga dat even proberen. Zou wel mooi zijn met het oog op overschrijven van bestaande waarden.quote:Op maandag 13 januari 2014 19:05 schreef bondage het volgende:
[..]
Bij een class heb je global niet nodig. Als je het object eenmaal hebt aangemaakt kun je dat meegeven aan je functies. Ook kun je op jouw manier variabelen per ongeluk overschrijven als je dezelfde namen ergens anders gebruikt.
Dit is niet echt een goede oplossing. Veel te onveilig, je kunt zelf post variabelen toevoegen en die worden dan als variabele gebruikt. Dit is een paar jaar geleden een grote overstap geweest voor mensen. register_globals stond vaak aan en gaf superveel beveiligingslekken. Je hebt nu een zelfde soort functie geschreven.quote:Op maandag 13 januari 2014 18:59 schreef n8n het volgende:
Ben ik weer. Om in een functie specifieke $_POST input te krijgen moest ik of telkens voluit $_POST['name'] schrijven, er een variabele van maken of variabelen global aanroepen. Heb nu een functie die je overal kan aanroepen die de $_POST data omzet in losse variabelen die je lokaal kan gebruiken.
[ code verwijderd ]
het maakt van elke key in een array een variable met de naam en waarde van de key. dus $_POST[email] wordt omgezet naar $email = 'waarde van email'. Als een waarde leeg is heeft de variabele een null waarde.
Op of aanmerkingen? Kom net kijken dus heb vaak het idee dat ik metideeën kom
niet eens aan gedachtquote:Op maandag 13 januari 2014 19:08 schreef totalvamp het volgende:
[..]
Dit is niet echt een goede oplossing. Veel te onveilig, je kunt zelf post variabelen toevoegen en die worden dan als variabele gebruikt. Dit is een paar jaar geleden een grote overstap geweest voor mensen. register_globals stond vaak aan en gaf superveel beveiligingslekken. Je hebt nu een zelfde soort functie geschreven.
Alles wat je in een class buiten de functie zet is beschikbaar binnen de functies die in die class staan.quote:Op maandag 13 januari 2014 19:08 schreef n8n het volgende:
[..]
dus variabelen die je binnen je class (maar buiten een functie aanmaakt) kan je binnen die class in elke functie gebruiken? Ga dat even proberen. Zou wel mooi zijn met het oog op overschrijven van bestaande waarden.
ah in 1 keer snap ik $this, heb ik meteen m'n prefix te pakkenquote:Op maandag 13 januari 2014 19:13 schreef bondage het volgende:
[..]
Alles wat je in een class buiten de functie zet is beschikbaar binnen de functies die in die class staan.
Voorbeeld:
private $variable_1 = 'test';
private $variable_2 = 'test2';
public function testFunctie() {
echo $this->$variable_1 . '<br /> . $this->$variable_2';
}
$this is de referentie naar de class waar je momenteel in zit (of inherit).quote:Op maandag 13 januari 2014 19:14 schreef n8n het volgende:
[..]
ah in 1 keer snap ik $this, heb ik meteen m'n prefix te pakken
Zie editquote:Op maandag 13 januari 2014 19:14 schreef n8n het volgende:
[..]
ah in 1 keer snap ik $this, heb ik meteen m'n prefix te pakken
ze maken het wel makkelijkquote:Op maandag 13 januari 2014 19:15 schreef totalvamp het volgende:
[..]
$this is de referentie naar de class waar je momenteel in zit (of inherit).
Voor statische vars is dit self::
ja precies, nu weer uitdokteren hoe ik m'n idee alsnog werkend krijg want het is niet echt DRY als ik overal globals moet definiëren.quote:Op maandag 13 januari 2014 19:15 schreef bondage het volgende:
[..]
Zie edit![]()
$this->variable_1 (dollarteken hoort niet voor de var)
Zoiets zou je mee kunnen beginnen:quote:Op maandag 13 januari 2014 19:17 schreef n8n het volgende:
[..]
ja precies, nu weer uitdokteren hoe ik m'n idee alsnog werkend krijg want het is niet echt DRY als ik overal globals moet definiëren.
1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 | <?php class cRequest { private $request_vars; public function __construct() { $this->request_vars = $_REQUEST; } public function setRequestVar($v_key, $v_val) { $this->request_vars[$v_key] = $v_val; } public function getRequestVar($key, $ret_nonexisting = '') { if(isset($this->request_vars[$key])) { return $this->request_vars[$key]; } return $ret_nonexisting; } public function getRequestArray() { return $this->request_vars; } } ?> <?php $obj_request = new cRequest(); // print de var, als deze niet bestaat dan krijg je 'ik_was_undefined' terug. echo $obj_request->getRequestVar('bla', 'ik_was_undefined'); ?> |
1 2 3 4 5 6 7 | <?php echo testFunction($obj_request); function testFunction($request_object) { return $request_object->getRequestVar('bla', 'ik_was_undefined'); } ?> |
Dit is niet de beste oplossing, je classe is een beetje ouderwets en je kunt dan beter gebruik maken van de magic get en set methoden. Dat scheelt ook gelijk in wat je moet typen.quote:Op maandag 13 januari 2014 19:28 schreef bondage het volgende:
[..]
Zoiets zou je mee kunnen beginnen:
[ code verwijderd ]
1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 | <?php class cRequest { private $request_vars; public function __construct() { $this->request_vars = $_REQUEST; } public function __set($v_key, $v_val) { $this->request_vars[$v_key] = $v_val; } public function __get($key) { if(isset($this->request_vars[$key])) { return $this->request_vars[$key]; } return false; } public function getRequestArray() { return $this->request_vars; } } ?> <?php $obj_request = new cRequest(); // krijg een var of false echo $obj_request->bla; ?> |
Dat had ik in de eerste instantie maar dan kun je niet bepalen wat er moet worden teruggegeven indien de var niet voorkomt in de globale request array.quote:Op maandag 13 januari 2014 19:33 schreef totalvamp het volgende:
[..]
Dit is niet de beste oplossing, je classe is een beetje ouderwets en je kunt dan beter gebruik maken van de magic get en set methoden. Dat scheelt ook gelijk in wat je moet typen.
[ code verwijderd ]
Ik begreep inderdaad dat je iets specifieks terug wilde hebben. Dat is alleen niet de taak van deze class. Je geeft alleen iets terug en in je Template class of Controller zul je daar iets mee moeten doen.quote:Op maandag 13 januari 2014 19:44 schreef bondage het volgende:
[..]
Dat had ik in de eerste instantie maar dan kun je niet bepalen wat er moet worden teruggegeven indien de var niet voorkomt in de globale request array.
Stel dat je een form hebt en je wilt vooraf ingevulde waarden hebben als er geen input van de gebruiker is doorgegeven, via de functie getRequestVar in mijn class kun je dan via de tweede param aangeven wat deze waarde moet zijn. Op jouw manier lukt dat niet, of ik moet iets over het hoofd zien. Ook kun je de functie uitbreiden met een extra param waarin je bijvoorbeeld mee kunt geven of je de waarde om wilt zetten in htmlentities, dan hoef je die functie niet los aan te roepen in je template.
1 2 3 4 5 6 7 8 9 10 11 12 13 | <?php $t = new Template('test'); $r = new Request(); $t->replace('email', $r->email); class Template { protected $html; public function replace($key, $with) { if(false !== $with) $this->html = str_replace($key, $with, $this->html); // Hier kun je afhandelen wat je doet als het false is } } |
Ik ben het met je eens dat je dit soort zaken in de template zelf af hoort te handelen, echter vind ik het zelf wel handig dat de request class het in dit geval ook kan omdat je de waarden niet altijd in de template gebruikt maar misschien ook ergens anders. Je zou dit natuurlijk ook door die classes (bijv. een formulier verwerking class) laten afhandelen maar dat vond ik voor het voorbeeldje iets te diep op de stof ingaan.quote:Op maandag 13 januari 2014 19:50 schreef totalvamp het volgende:
[..]
Ik begreep inderdaad dat je iets specifieks terug wilde hebben. Dat is alleen niet de taak van deze class. Je geeft alleen iets terug en in je Template class of Controller zul je daar iets mee moeten doen.
Bijvoorbeeld:
[ code verwijderd ]
Ik moet er wel bij zeggen dat ik mijn formulieren opbouw met classes. Elk element is een class met zijn eigen validatie en manier om standaardwaarde te zetten.
Ja misschien inderdaad te geavanceerd voor een beginnerquote:Op maandag 13 januari 2014 20:10 schreef bondage het volgende:
[..]
Ik ben het met je eens dat je dit soort zaken in de template zelf af hoort te handelen, echter vind ik het zelf wel handig dat de request class het in dit geval ook kan omdat je de waarden niet altijd in de template gebruikt maar misschien ook ergens anders. Je zou dit natuurlijk ook door die classes (bijv. een formulier verwerking class) laten afhandelen maar dat vond ik voor het voorbeeldje iets te diep op de stof ingaan.
Ik bouw in de meeste tools/websites die ik heb geschreven de template op als een extension op een baseclass en set dan alle vars die nodig zijn in de template. Je kunt daar dan eventueel ook standaardwaarden zetten.
1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 | <?php $f = new System\Core\Form(); $f->addElement(new System\Core\Form\Element\Fieldset( array( 'name' => 'testFieldset', 'label' => 'Testfieldset', 'elements' => array( new System\Core\Form\Input\Text(array( 'name' => 'test', 'label' => 'Test field', 'value' => 'testvaluess', 'validator' => array( 'Length' => array('max' => 10) ))), new System\Core\Form\Input\Text(array( 'name' => 'test', 'label' => 'Testfield', 'value' => 'testvaluess', 'validator' => array( 'Length' => array('max' => 10) ))) )) )); $f->addElement(new System\Core\Form\Element\Button( array( 'name' => 'Test', 'value' => 'Test' ) )); $f->show(); ?> |
Forum Opties | |
---|---|
Forumhop: | |
Hop naar: |