Netjes PHP scripten

**OmarM** · 23-12-2008, 17:01

Hallo

Ik heb ooit een beetje in PHP gescript zoals een nieuws systeem maken in PHP & MySQL. Dit is ongeveer een half jaar geleden en sinds deze vakantie had ik weer zin om te gaan PHP-en. Alleen nu wou ik het ook gelijk goed doen. Eerst weer even wat basis kennis opdoen en dan weer wat maken. Nu zijn hier waarschijnlijk mensen op het forum die ook echt als werk PHP-en hebben. Nu wou ik wat aan die mensen vragen.

Natuurlijk scripten jullie netjes etc. Maar hoe? Unsetten jullie alle vars aan het eind van een script? En doen jullie dit:

Code:

<?php
$sql = "SELECT * FROM paginas";
$res = mysql_query($sql);
?>

Of dit?

Code:

<?php
$sql = mysql_query("SELECT * FROM paginas");
?>

Gewoon uit pure intersse hoor :P.

Alvast bedankt

Omar

**Ruben K** · 23-12-2008, 17:06

Ik vind $query = ""; echt overbodig, ik doe altijd gewoon in mysql_query() de query

**Jeroen H** · 23-12-2008, 17:21

En hoe wilde je dan mysql_fetch_assoc/object/array doen?

Ik neem aan dat je dan bedoeld dat je $query = mysql_query doet en niet via de volgende regel...

**Vincent** · 23-12-2008, 17:24

Code:

$sql = "SELECT * 
    FROM paginas
    WHERE pagina = 1
        AND status = 0 
    ";
    
$query = mysql_query($sql);

Overzichtelijkheid is nooit overbodig.

Unsetten van vars an het einde van script overigens wel :p

Als je een stijl wilt leren raad ik aan:
http://pear.php.net/manual/en/standards.php

**J. Smit** · 23-12-2008, 17:36

In dit geval vind ik dit nogal onnodig:

Code:

<?php
$sql = "SELECT * FROM paginas";
$res = mysql_query($sql);
?>

Maar stel dat je een lange, ingewikkelde query hebt is het overzichtelijker om het te doen zoals Vincent al zei

**Vincent** · 23-12-2008, 17:41

Nouja, gezien mijn queries regelmatig een 100 vout aan karakters bevat is meerdere regels eigenlijk niet uit te sluiten.
Queries, joins, subqueries, etc moeten wel overzichtelijk blijven.

**Henk G.** · 23-12-2008, 19:04

Persoonlijk vind ik $sql = mysql_query("SELECT * FROM paginas"); fijner werken.
Anders krijg je erg veel regels en dat vind ik persoonlijk irritanter als dat alles in één regeltje zit.

**Martijn Dwars** · 24-12-2008, 00:12

Gewoon in 1 regel doen. Het levert nauwelijks (lees: geen) extra overzicht op en je moet wel weer een variabele aanmaken die het geheugen in moet. En waar je kan besparen zou ik het je aanraden. ;)

**Thijs Damen** · 24-12-2008, 08:24

Gewoon aan je eigen coding standards houden?

Wat is overzichtelijk. Als ik afspreek met mijn partner/klant/baas dat ik alle querys met maximaal 1 conditie (of het nu ORDER, WHERE oid. is maakt even niet uit) in 1 regel mag zetten. Dan is dat overzichtelijk.

Code:

<?php
  $res = mysql_query("SELECT * FROM tabel WHERE id = $x");
?>

Als ik afspreek alle querys, ongeacht de groote via een variable afhandel en netjes neer zet doe je het zo

Code:

<?php
  $select = "SELECT *
                FROM tabel
                WHERE id = $x
                ";
  $res = mysql_query($select);
?>

Je moet niet vergeten dat er niet voor niets regels voor gemaakt worden. Spreek met jezelf in dit geval een paar standaard regels af.
Bijvoorbeeld. waar zet ik mijn { en } neer? onder functie/if's of ernaast? Waar en waneer spring ik in, hoe zet ik een array op? Hoe benoem ik mijn variablen?

Ook geen afspraken maken is opzich een afspraak.

Om even een einde te maken. Doe wat jij wilt en wat jij het fijnst vind. Probeer voor jezelf een overzichtelijk standard te maken waarin jij je code snel terug kan vinden en je het ook kan uitleggen als iemand je script ziet. (Ik vond het er mooi uitzien is geen reden om iets te doen natuurlijk)

**H. van Lohuizen** · 24-12-2008, 08:32

PHP Code:



function SchoonMaken($variabele, $actie){
    
    switch($actie){
    
        case 1:
            
            // Opslaan
            $variabele = addslashes($variabele);
            $variabele = mysql_real_escape_string($variabele);
            $variabele = mysql_escape_string($variabele);
            break;
            
        case 0:
        
            // Weergeven
            $variabele = stripslashes($variabele);
            $variabele = htmlentities($variabele);
            break;
            
        case 2:
        
            // Weergeven content
            $variabele = stripslashes($variabele);
            break;
    }
    
    // Nieuwe variabele terug geven
    return $variabele;
}

PHP Code:



    MaakVerbinding();

    require '_config/mysql_config.php';
    
    $pagina = SchoonMaken($pagina, 1);
    $titel_pagina = SchoonMaken($_GET['titel'], 1);
    
    $q_pagina = "SELECT * FROM ".$mysql_prefix."content
                     WHERE ".$mysql_prefix."content.content_id = ".$pagina."
                     AND ".$mysql_prefix."content.content_titel LIKE '".$titel_pagina."'
                     LIMIT 0,1";
    
    $q_result = mysql_query($q_pagina) or die(mysql_error());

Zo vind ik het overzichtelijk. Maar het is vooral een persoonlijke keuze waarom je bepaalde dingen doet. Je kunt het beste kijken wat voor jou het beste is, zodat je zelf gemakkelijk kunt programmeren.

**John Timmer** · 24-12-2008, 08:59

Origineel gepost door OmarM

Hallo

Ik heb ooit een beetje in PHP gescript zoals een nieuws systeem maken in PHP & MySQL. Dit is ongeveer een half jaar geleden en sinds deze vakantie had ik weer zin om te gaan PHP-en. Alleen nu wou ik het ook gelijk goed doen. Eerst weer even wat basis kennis opdoen en dan weer wat maken. Nu zijn hier waarschijnlijk mensen op het forum die ook echt als werk PHP-en hebben. Nu wou ik wat aan die mensen vragen.

Natuurlijk scripten jullie netjes etc. Maar hoe? Unsetten jullie alle vars aan het eind van een script? En doen jullie dit:

Code:

<?php
$sql = "SELECT * FROM paginas";
$res = mysql_query($sql);
?>

Of dit?

Code:

<?php
$sql = mysql_query("SELECT * FROM paginas");
?>

Gewoon uit pure intersse hoor :P.

Alvast bedankt

Omar

Je volgt je eigen standaarden of je doet je het niet, maar je gebruikt niet 2 standaarden voor 1 oplossing.
De tweede optie is niet geschikt voor grotere queries, debuggen, etc.. dus blijft alleen de eerste over. Lijkt me logisch ;)

**Thijs Damen** · 24-12-2008, 09:08

Origineel gepost door John Timmer

Je volgt je eigen standaarden of je doet je het niet, maar je gebruikt niet 2 standaarden voor 1 oplossing.
De tweede optie is niet geschikt voor grotere queries, debuggen, etc.. dus blijft alleen de eerste over. Lijkt me logisch ;)

De eerste is wel geschikt wou je zeggen?

(net even van google gevist)

Code:

$query = "SELECT categories.id, categories.name, items.id, items.name FROM categories INNER JOIN items ON items.categoryid = categories.id AND items.id = (SELECT items2.id FROM items AS items2 WHERE items2.categoryid = category.id ORDER BY rand() limit 1)
 
$result = mysql_query($query);

**R de Valk** · 24-12-2008, 09:15

De opmaak van je code is aan je zelf vind ik. Hier zijn vele manieren voor en smaken verschillen.
Hoe je werkt met fout afhandeling en/of excepties is in mijn ogen netjes scripten. Zorg dat error reporting aan staat op het hoogste niveau en zorg dat er geen fouten in het script zitten, als er dan wel fouten optreden dan je dan deze netjes in een exceptie naar het scherm print. En variabelen buiten quotes zetten.

**John Timmer** · 24-12-2008, 09:17

Origineel gepost door Thijs Damen

De eerste is wel geschikt wou je zeggen?

(net even van google gevist)

Code:

$query = "SELECT categories.id, categories.name, items.id, items.name FROM categories INNER JOIN items ON items.categoryid = categories.id AND items.id = (SELECT items2.id FROM items AS items2 WHERE items2.categoryid = category.id ORDER BY rand() limit 1)
 
$result = mysql_query($query);

Je was te snel :)
Ik had de zin omgegooid maar de nummers laten staan :$

**Vincent** · 24-12-2008, 09:59

Origineel gepost door Martijn Dwars

Gewoon in 1 regel doen. Het levert nauwelijks (lees: geen) extra overzicht op en je moet wel weer een variabele aanmaken die het geheugen in moet. En waar je kan besparen zou ik het je aanraden. ;)

Maar hoeveel queries maak jij die zo lang zijn

Code:

SELECT * FROM tabel WHERE id = $x

(en als je het wel doet vraag ik me toch enigzins af hoe optmaal je code is als het bestaat uit veel kleine queries)

Een minimale querie die ik zou schrijven is toch wel als voorbeeld

Code:

$query = mysql_query("SELECT celnaam, celnaam, celnaam, celnaam, celnaam, celnaam FROM table WHERE table_id = " . $db->escape($var) . " AND status = 1");

Hoe vind je dat dan nog overzichtelijk, zeker voor toekomstige wijzigingen.

* gebruiken is natuurlijk geen optie als je het hebt over besparen, overigens moet je dan ook geen fetch_array gebruiken maar fetch_object aangezien de array versie 2x zoveel vars bevat.

Netjes PHP scripten