Mening schrijfstijl

Overzicht

Sponsored by: Vacatures door Monsterboard

Productontwikkelaar Gas & Waterstof Haarden

Ben jij iemand die graag werkt aan nieuwe innovaties en ontwikkelingen binnen gas en waterstof? Lees dan snel deze vacature! Onze opdrachtgever werd opgericht in 1754 en is daarmee een van de oudste bedrijven van Nederland. Het bedrijf is van oorsprong een ijzergieterij, maar is sinds de vondst van aardgas gespecialiseerd in de fabricage van hoogwaardige gas- en houtkachels. Naast de eerder genoemde producten zijn zij ook actief in elektrische haarden, haarden op bio-ethanol en ander soorten brandstoffen zoals waterstof. Het bedrijf is internationaal actief en levert haar kachels aan onder meer de Nederlandse, Belgische, Duitse, Franse en Engelse markt.

Bekijk vacature »

PHP ontwikkelaar

Functie Met een complex en uitgebreid e-commerce platform, een eigen PIM-systeem en eigen scan applicatie – krijg jij dagelijks te zien hoe jouw werk gebruikt wordt door miljoenen gebruikers. En we staan qua development pas in de startblokken, aangezien er nog meerdere projecten op de plank liggen te wachten! Ons huidige development team bestaat uit 8 programmeurs. Er wordt dagelijks gereflecteerd op geschreven code, Scrum taken en kennisdelen onderling is een must. Onze voertaal binnen ons team is Engels, dit omdat wij twee internationale collega’s hebben. Ons huidige “IT Landschap” bestaat voornamelijk uit allerlei losse onderdelen die individueel, maar ook

Bekijk vacature »

.NET Developer / Azure

Dit ga je doen Je dagelijkse werkzaamheden bestaan onder andere uit het: Ontwikkelen van kwalitatief sterke code met o.a. C# en .NET Core; Werken aan tal van nieuwe oplossingen binnen het Azure platform; Van scratch af aan bouwen van oplossingen, platformen, apps en andere portalen; Werken aan innovatieve (inhouse) projecten voor tal van gerenommeerde organisaties; Meedenken over architectuur vraagstukken; Leveren van technische input tijdens (pre)sales trajecten; Bouwen van koppelingen middels API's. Hier ga je werken Werken aan innovatieve en uitdagende projecten binnen een multidisciplinair team dus. Als .NET developer ga je werken in een DevOps team aan state-of-the-art oplossingen voor

Bekijk vacature »

Medior Java developer (backend)

Wat je gaat doen: Of beter nog, wat wil jij doen? Binnen DPA GEOS zijn we dan ook op zoek naar enthousiaste Java developers om ons development team te versterken. Als Java developer werk je in Agile/Scrum teams bij onze klanten en daarbij kun je eventueel ook andere ontwikkelaars begeleiden in het softwareontwikkelproces. Verder draag je positief bij aan de teamgeest binnen een projectteam en je kijkt verder dan je eigen rol. Je gaat software maken voor verschillende opdrachtgevers in jouw regio. Je bent een professional die het IT-vak serieus neemt en kwaliteit levert. Je leert snel vanwege je diepgaande

Bekijk vacature »

Medior Java developer (fullstack)

Wat je gaat doen: Of beter nog, wat wil jij doen? Binnen DPA GEOS zijn we dan ook op zoek naar enthousiaste Java developers om ons development team te versterken. Als Java developer werk je in Agile/Scrum teams bij onze klanten en daarbij kun je eventueel ook andere ontwikkelaars begeleiden in het softwareontwikkelproces. Verder draag je positief bij aan de teamgeest binnen een projectteam en je kijkt verder dan je eigen rol. Je gaat software maken voor verschillende opdrachtgevers in jouw regio. Je bent een professional die het IT-vak serieus neemt en kwaliteit levert. Je leert snel vanwege je diepgaande

Bekijk vacature »

Software Developer C# niche markt

Bedrijfsomschrijving Wil jij aan de slag met het ontwikkelen van unieke en dagelijks uitdagende software en zoek je een nieuwe baan in Rotterdam? Dan zou dit software development bedrijf wat voor jou kunnen zijn! Wij zijn voor deze klant van ons op zoek naar een sterke allround Software .NET Developer, die bedreven is in het ontwikkelen van hoogwaardige maatwerk (web)applicaties vanuit specifieke wensen van klanten. Deze organisatie is actief in een bijzonder niche van de markt: Zij ontwikkelen softwareproducten voor bijvoorbeeld opleiders, zorginstanties en uitgevers, allemaal in het kader van opleiden. Er worden standaardproducten gemaakt, maar ook wordt er gewerkt

Bekijk vacature »

Medior/Senior Python developer

Functie Jij als Senior Python developer hebt al ruime ervaring opgedaan. Bedrijven komen bij de organisatie om technische vraagstukken op te lossen. Jij als specialist bent dus de representatie van deze kwaliteit. Je zult de keuze krijgen tussen lange of korte projecten waarin je komt te werken in multidisciplinaire teams. Projecten die je gaat uitvoeren zijn zeer uitlopend. Zodoende kun je aan de ene kant kiezen voor een greenfield project en stroom je bij een ander project midden in een migratietraject in. Voor de ene klant ontwikkel je ene nieuwe portal en voor het andere project duik je veel meer

Bekijk vacature »

Front-end Developer / Angular

Dit ga je doen Als Front-end Developer zullen jouw taken voornamelijk bestaan uit: Het (door)ontwikkelen van applicaties op het SaaS platform (met o.a. TypeScript, Angular 12, CSS); Het uitvoeren van code reviews; Het meebeslissen over de inzet van bestaande en nieuwe controls en deze onderbrengen in een referentie-applicatie; Het bijdragen aan de professionalisering en verbetering van de applicaties; Het onderhouden van contact met verschillende stakeholders (ook buiten Nederland). Hier ga je werken Wat ga ik doen? Als Front-end Developer ga jij je focussen op het ontwikkelen van applicaties op het SaaS platform van een organisatie die software ontwikkelt voor de

Bekijk vacature »

Datawarehouse Specialist / ETL Ontwikkelaar

Datawarehouse Specialist / ETL Ontwikkelaar Leiden, 36 uur per week WIJ bieden een baan waar jij je ETL-ontwikkelexpertise kan inzetten binnen de lokale overheid. Je werkt aan slimme oplossingen die Leidse regiogemeentes helder inzicht geven in hun visie, voortgang en verbeterpunten. Zo maar een greep uit een dag in de functie van datawarehouse specialist: Je focust je vandaag naar hartenlust op de ontwikkeling van nieuwe ETL-stappen. Je past hier je SQL-kennis toe in de uitdaging om de door jou ontworpen queries zo snel en stabiel mogelijk te krijgen. Zo breid je het datawarehouse van verschillende regiogemeentes uit, door nieuwe managementinformatie

Bekijk vacature »

Digital Agency is looking for PHP developers!

Functie The team currently has 20 colleagues, consisting of developers (front and backend) and the operations team, which also includes management and two scrum masters. They are looking for a PHP developer who is able to work independently. You will work in one of the three scrum teams and start working on a project for the customer. The interesting thing about this is that you do have variety in terms of work, but at the same time continuously work for existing customers. This also gives you the opportunity to really go into depth and develop innovative technical solutions. In terms

Bekijk vacature »

RPA Ontwikkelaar / Modelleur

Dit ga je doen Je werkt in een team van modelleurs en verbetert de processen door RPA toe te passen in een DevOps setting; I.s.m. de business analist beoordeel je of een proces zich leent voor RPA, hierin ben je een belangrijke adviseur; Je bent verantwoordelijk voor het ontwikkelen en configureren van deze robots; Het beheren van de RPA processen; Als ervaren modelleur ondersteun je het team d.m.v. coaching en fungeer je als vraagbaak voor hen; Je haakt de juiste stakeholders aan en houdt 'de klant' aangelijnd. Hier ga je werken Onze opdrachtgever is een dynamische organisatie met internationale oriëntatie

Bekijk vacature »

Ervaren PHP developer gezocht!

Functie Het team bestaat inmiddels uit zo’n 25 collega’s met specialisten op het gebied van development, data(analyse), marketing, infrastructuur en finance. Ze hebben een supermodern pand en bieden hiernaast veel vrijheid en verantwoordelijkheid. Ze doen er alles aan om jou op te gemak te stellen. Zo kun je je eigen werkplek inrichten naar persoonlijke wensen, maar gaan ze bijvoorbeeld ook jaarlijks met elkaar wintersporten en zijn er andere leuke uitjes. Als onderdeel van één van de scrumteams ga je aan de slag, samen ben je medeverantwoordelijk voor het doorontwikkelen van hun business applicatie waar het traffic team dagelijks mee werkt.

Bekijk vacature »

Integratie Developer / Architect

Dit ga je doen Als Integratie Developer / Architect binnen deze organisatie krijg je echt de kans om impact te maken. De organisatie is groeiende maar houdt een corporate cultuur buiten de deur. Heb je een goede business case: zorg voor goede argumentatie en ga ervoor! Geen stroperig beslissingsproces dat jouw ideeën in de weg staat! Enkele van jouw taken: Je ontwerpt en ontwikkelt nieuwe integraties met behulp van interne tools (Boomi) of externe partners; Je vertaalt functionele specificaties naar technische oplossingen; Je denkt mee over strategische ontwikkelingen op het gebied van applicatie integratie; Je voert regie op leveranciers en

Bekijk vacature »

PHP Developer

As a PHP Developer at Coolblue, you ensure that our webshops work as optimal as possible. How do I become a PHP Developer at Coolblue? As a PHP Developer you work together with other development teams to make our webshop work as optimal as possible and to make our customers happy. Although you are a PHP Developer, you are not averse to a little C# or Typescript. Would you also like to become a PHP Developer at Coolblue? Read below if the job suits you. You enjoy doing this Writing pure PHP code. Working with the Customer Journey Specialist to

Bekijk vacature »

Full-Stack Web Developer

Vacancy .net Full-Stack Web Developer For our client Eclipse Are we are looking for a .net Developer. Eclipse is a young fast growing and dynamic ICT organisation based in's-Hertogenbosch. Her Mission: to offer Intelligent Multimedia marketing communication tailored to put companies in the spotlight. Sounds nice right? We also think so! Will you strenghten the team of Eclipse as .net Full-stack Web Developer? Job description At present, about 20 people are working very hard together to develop communication solutions for their Customers. These solutions focus on enriching and processing customer data from different environments through workflow-driven processes. The Eclipse culture

Bekijk vacature »
Ocirina Ocirina

Ocirina Ocirina

12/01/2011 08:00:05
Anchor link
Goede morgen, ik ben redelijk nieuw met php.
Ik heb al eerder wat vragen geplaatst hier, maar nu is mijn vraag;
Wat vinden jullie van mijn schrijfstijl? Ik hoor graag wat ik kan verbeteren.
Ook wil ik dingen graag weten wat ik beter kan doen met beveiliging of fout afhandeling.
//waarschijnlijk komen en reactie van 'het werkt niet', maar dat geeft niet.
Code (php)
PHP script in nieuw venster Selecteer het PHP script
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
32
33
<?php
if($_SERVER['REQUEST_METHOD'] == 'POST')                                                                                 //Kijken of het formulier goed is verzonden.
 {
    require_once 'connect.php';                                                                                          //Connectie db.
    if(empty($_POST['wachtwoord']))                                                                                       //kijken of het form. wel goed is ingevuld.
     {
        echo "De gegevens zijn niet correct ingevuld.";
     }

    else
     {
        if ($_POST['wachtwoord'] == $_POST['cwachtwoord'])                                                              //Controle of gegevens goed zijn ingevuld.
         {
            mysql_select_db("shop", $con);                                                                                 //$con bevind zich op de connect.php pagina.
            
            mysql_query("
            INSERT INTO klanten (unaam, upass, email)    
            VALUES ('"
. $_POST['gebruikersnaam'] . "', '". $_POST['wachtwoord']. "', '". $_POST['email']. "')");        //De invoering van de naam en pass in db.
         }
        else
         {
            echo "De ingevoerde wachtwoorden komen niet overeen!";
         }
        
        
     }
  }

 else
  {
    echo "U moet wel gegevens invullen!";
  }

 
mysql_close($con);
?>


edit: Foutje.
Gewijzigd op 12/01/2011 08:08:03 door Ocirina Ocirina
 
PHP hulp

PHP hulp

01/10/2022 20:04:43
 
Jordi Kroon

Jordi Kroon

12/01/2011 08:26:42
Anchor link
Waarom die mysql_close ?
 
Yea Rupie

Yea Rupie

12/01/2011 08:38:57
Anchor link
- Je kan nog controleren om de gebruikersnaam een minimaal aantal tekens bevat, idem voor wachtwoord.
- Je script is niet beveiligd, maak gebruik van mysql_real_escape_string()
- Gebruik je 1 database? Dan zou ik mysql_select_db("shop", $con); eruit halen en in connect.php erbij zetten.
- Mysql_close is nergens voor nodig?

mijn voorkeur geeft ook een iets andere notatie, maar dat is voor iedereen verschillend:
Code (php)
PHP script in nieuw venster Selecteer het PHP script
1
2
3
4
5
6
7
8
9
10
11
<?php
if (...) {
    
}

elseif (...) {
    
}

else {
    
}

?>


ipv

Code (php)
PHP script in nieuw venster Selecteer het PHP script
1
2
3
4
5
6
7
8
9
10
11
12
13
14
<?php
if (...)
{

}

elseif (...)
{
    
}

else
{
    
}

?>
 
- SanThe -

- SanThe -

12/01/2011 08:51:35
Anchor link
Yea Rupie op 12/01/2011 08:38:57:
mijn voorkeur geeft ook een iets andere notatie, maar dat is voor iedereen verschillend:
Code (php)
PHP script in nieuw venster Selecteer het PHP script
1
2
3
4
5
6
7
8
9
10
11
<?php
if (...) {
    
}

elseif (...) {
    
}

else {
    
}

?>


ipv

Code (php)
PHP script in nieuw venster Selecteer het PHP script
1
2
3
4
5
6
7
8
9
10
11
12
13
14
<?php
if (...)
{

}

elseif (...)
{
    
}

else
{
    
}

?>


Nee juist niet.
De tweede manier is veel duidelijker.
 
Ozzie PHP

Ozzie PHP

12/01/2011 08:52:15
Anchor link
ik doe het zo:

Code (php)
PHP script in nieuw venster Selecteer het PHP script
1
2
3
4
5
6
7
8
9
<?php
if (...) {

}
elseif (...) {

}
else {

}

?>
 
Jelmer -

Jelmer -

12/01/2011 09:04:06
Anchor link
Ik zou het in een functie doen zodat je return kan gebruiken om eruit te ontsnappen. Dat scheelt lagen van nesting met if-else statements.

Sowieso nooit wachtwoorden zomaar opslaan, altijd een afgeleide (md5 of sha1 hash bijv.). Geen uitzonderingen.

Altijd oppassen met data in je sql stoppen -> mysql_real_escape_string voor strings waarvan je niet weet wat erin zit. (Bij sha1 hoeft het bijv. niet bij omdat daar altijd iets a-zA-Z0-9 uit komt, nooit iets wat lijkt op SQL of wat je query zou kunnen breken.)

require_once 'connect.php' impliceert dat je maar op één plek je databaseverbinding kan gebruiken.

En gebruik meer functies. Functies zijn er niet voor hergebruik, maar om je code in hapbare brokken op te splitsen.
Code (php)
PHP script in nieuw venster Selecteer het PHP script
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
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
<?php

if($_SERVER['REQUEST_METHOD'] == 'POST' && do_register_user())
{

    header('Location: win.php');
    exit;
}

else
{
    // toon formulier
}


function
do_register_user()
{

    // ik check ze allemaal maar even zodat je niet tegen undefined indexes aanloopt
    if (empty($_POST['gebruikersnaam']) || empty($_POST['wachtwoord'])
        ||
empty($_POST['cwachtwoord']) || empty($_POST['email']))
    {

        echo 'De gegevens zijn niet correct ingevuld.';
        return false; // return stopt deze functie, en false geeft aan dat er
                      // iets mis was, en we dus het formulier weer terugwillen.

    }
    
    if ($_POST['wachtwoord'] != $_POST['cwachtwoord'])
    {

        echo 'De ingevoerde wachtwoorden komen niet overeen.';
        return false;
    }

    
    if (!register_user($_POST['gebruikersnaam'], $_POST['wachtwoord'], $_POST['email']))
    {

        echo 'Sorry, computer says no...';
        return false;
    }

    
    return true;
}


// Splits je code in aparte delen:
// do_register_user is om een formulier te verwerken
// register_user is om een gebruiker te registreren in je database
// get_db_connetion is om een database verbinding te pakken te krijgen.

function register_user($gebruikersnaam, $wachtwoord, $email)
{

    $con = get_db_connection();
    // sprintf om query en data wat gescheiden te houden, kan je sneller
    // je query controleren op fouten.

    $stmt = sprintf("
        INSERT INTO
            klanten (unaam, upass, email)
            VALUES ('%s', '%s', '%s')"
,
        mysql_real_escape_string($gebruikersnaam), // en al je data natuurlijk escapen.
        sha1($something_salty . $wachtwoord), // geen rauwe wachtwoorden opslaan. Nooit. Niet eens de suggestie wekken!
        mysql_real_escape_string($email));
    
    // je query gaat ook mis wanneer je bijv. een unique constraint op
    // gebruikersnaam of email hebt zitten, dus controleren of de query lukt
    // lijkt mij vrij belangrijk.

    return mysql_query($stmt, $con);
}


// met deze function kan je tenminste bij je verbinding vanuit verschillende
// contexten (functies) terwijl je nog steeds maar één keer verbinding maakt.

function get_db_connection()
{

    static $connection;
    
    if (!$connection)
    {

        require_once 'connect.php';
        mysql_select_db('shop', $con); // hoort dit niet in connect.php?

        $connection = $con;
    }

    
    return $connection;
}

?>
Gewijzigd op 12/01/2011 09:08:42 door Jelmer -
 

12/01/2011 09:06:51
Anchor link
Euh, je hebt toch al een topic lopen...?
Volgens mij is daar alles al behandeld...
 
Kris Peeters

Kris Peeters

12/01/2011 09:25:16
Anchor link
- SanThe - op 12/01/2011 08:51:35:
...
Nee juist niet.
De tweede manier is veel duidelijker.


Hier volg ik toch Yea Rupie.
Als je consequent omgaat met die indenteringsregels is er geen enkel probleem van leesbaarheid.

De sluitende accolade komt op de zelfde plaats (uiteraard lager) dan de plaats waar het commando begint, waar je de accolade opent.

bv. de i van if() komt dan op de zelfde plaats als de sluitende accolade.

Een bijkomend voordeel is dat je meer code krijgt op minder lijnen.
 
Ozzie PHP

Ozzie PHP

12/01/2011 09:31:07
Anchor link
Kris Peeters op 12/01/2011 09:25:16:
Een bijkomend voordeel is dat je meer code krijgt op minder lijnen.

Dan moet je mijn versie pakken :)

Code (php)
PHP script in nieuw venster Selecteer het PHP script
1
2
3
4
5
6
7
8
9
<?php
if (...) {

}
elseif (...) {

}
else {

}

?>


Uiteindelijk is het denk ik gewoon een kwestie van smaak / persoonlijke voorkeur. Wel grappig dat er zoveel varianten zijn. Uiteraard is die van mij het beste :P maar leuk om te zien dat iedereen z'n eigen maniertje heeft. Anyhow, het is geen kwestie van wat beter is, maar wat voor jou het prettigste werkt.
 
Kris Peeters

Kris Peeters

12/01/2011 09:38:56
Anchor link
Ten eerste, je gebruikt dit enkel voor if/elseif/else.

Maar dit is niet in overeenstemming met het principe.

Het principe is juist dat je je commando begint waar de accolade zal sluiten. Dit doe je dus niet.
Een bijkomend ding: je begint een commando niet op een volgende lijn.

Wat jij beschrijft, is enkel een systeem voor if/elseiff/else, waardoor je dus niet meer consequent omspringt met de rest.


--------

Tenzij je denkt een uitleg te kunnen geven waarin je toont dat er wel een consequent systeem zit in wat je doet.
Gewijzigd op 12/01/2011 09:41:59 door Kris Peeters
 
Ozzie PHP

Ozzie PHP

12/01/2011 09:47:36
Anchor link
Kris, wat bedoel je daar precies mee? Kun je een voorbeeld geven?

Ik doe het altijd als volgt:

If statement:

Code (php)
PHP script in nieuw venster Selecteer het PHP script
1
2
3
4
5
<?php
if (iets) {
  //doe iets
}
?>


If-else statement:

Code (php)
PHP script in nieuw venster Selecteer het PHP script
1
2
3
4
5
6
7
<?php
if (iets) {
  //doe iets
} else {
  // doe iets anders
}
?>
 
Kris Peeters

Kris Peeters

12/01/2011 09:54:49
Anchor link
Zoals ik zeg: je doet dit enkel voor if/elseif/else.
Gebeurt het op gelijk welke andere plek dat je iets zet rechts van een accolade?

Hoe past dit dan in een globaal systeem van indentering?
 
Erik Rijk

Erik Rijk

12/01/2011 10:00:53
Anchor link
Je kan wel verder gaan in je oude topic: http://www.phphulp.nl/php/forum/topic/empty-functie/75586/last/
 
 

Dit topic is gesloten.



Overzicht

 
 

Om de gebruiksvriendelijkheid van onze website en diensten te optimaliseren maken wij gebruik van cookies. Deze cookies gebruiken wij voor functionaliteiten, analytische gegevens en marketing doeleinden. U vindt meer informatie in onze privacy statement.