Mening schrijfstijl

Overzicht

Sponsored by: Vacatures door Monsterboard

Data analist

Data analist Business Intelligence vormt binnen Red je Pakketje een belangrijke afdeling die de meest belangrijke processen binnen het hele bedrijf ondersteunt. Binnen het BI-team heb je als data analist alle data tot je beschikking en krijg je de mogelijkheid om vorm te geven aan hoe Red je Pakketje haar data gebruikt om sneller en betere beslissingen te maken. Uiteindelijk zorgen jouw oplossingen ervoor dat jouw collega’s binnen andere afdelingen beter inzicht hebben in hun processen waardoor zij data-gedreven beslissingen kunnen maken. Jouw rapportages en analyses gaan ervoor zorgen dat Red je Pakketje elke dag weer een stukje beter is.

Bekijk vacature »

Software Developer / .NET @ Deventer

2021-06-07 iSense Software Developer .NET Ben jij een Software Developer en heb je eerder ervaring opgedaan met C#? Ben je op zoek naar een nieuwe functie en lijkt het je leuk om mee te denken en te sparren met de Lead Developer? Lees snel verder en solliciteer! ISRO49470 Nieuw Organisatie Je komt te werken als Software Engineer bij een organisatie gespecialiseerd in de Automatisering voor klanten van hen in de productiebranche. Ze zijn ongeveer 100 man groot en je komt terecht in een development team van 7 man. De wens is om dit met een developer uit te breiden; iemand

Bekijk vacature »

Databasebeheerder

Informatievoorziening is een vakgebied waarin de ontwikkelingen elkaar in rap tempo opvolgen en dat staat soms in contrast met het werken voor een gemeente. Wij zien dat juist als een uitdaging. Wat ons extra motiveert is dat we hier, in tegenstelling tot in veel andere organisaties, echt voor en met mensen werken. We werken niet geïsoleerd aan een product zoals een website of applicatie, maar zoeken juist contact met andere afdelingen. De gemeentelijke organisatie en dus ook de inwoners van Haarlem en Zandvoort profiteren hiervan. Ons DBA-team is verantwoordelijk voor onze MS SQLServer, Oracle en PostgreSQL cluster-omgevingen. We hebben in

Bekijk vacature »

Senior .Net Developer (fixed & interim)

What are you going to do? All our T-Mobile Thuis systems are developed internally by our enthusiastic and self-organized scrum/DevOps teams. These are responsible for the analysis, the functional design and the realization of software solutions (applications / systems) that support the customer processes and the activation of the services in the network and on the platforms. The developments in IT are moving fast and we believe that you need hands-on expertise in implementing the new technology in order to be able to value these. We are therefore looking for experienced candidates who can and want to combine the role

Bekijk vacature »

Junior .NET developer technisch R&D team

Bedrijfsomschrijving Onze klant is een zeer technisch georiënteerde organisatie van nu in totaal 40 medewerkers waarvan 12 op de R&D development afdeling. Die afdeling bestaat naast software ook uit hardware engineers. E wordt gewerkt aan een aantal intern ontwikkelde producten die worden ingezet op bijvoorbeeld Schiphol, maar ook in rechtbanken, ziekenhuizen en andere grote ondernemingen. Functieomschrijving Als .NET developer wordt je onderdeel van een klein team van 4 .NET developers. Je gaat hier werken aan de technische producten die geleverd worden aan grote ondernemingen die onder andere de veiligheid van panden en omgevingen waarborgen, maar ook aan projecten die de

Bekijk vacature »

Functioneel Beheerder IAM en database (G!DS)

Waar het horrorverhaal IT van Stephen King, kilometers aan zeldzame boeken en functionaliteitenbeheer samenkomen. Dat is de KB in Den Haag. En het is de plek waar jij als functioneel beheerder de gegevens van miljoenen Nederlanders en honderden bibliotheken en hun aanbod verwerkt en beschermt. Maak jij het mogelijk? In deze functie maak je direct maatschappelijke impact. Je zorgt ervoor dat miljoenen Nederlanders kunnen inloggen op en gebruik kunnen maken van al onze diensten (zoals bijvoorbeeld de Online Bibliotheek en de KB catalogus). Je bent als onze functioneel beheerder IAM en database (G!DS) hierdoor actief bezig met het bieden van

Bekijk vacature »

Startersfunctie JAVA developer @ Regio Amsterdam

2021-10-02 iSense Startersfunctie JAVA developer Heb je onlangs een hbo-, wo-opleiding of een minor richting de IT afgerond en wil je graag aan de slag als Fullstack JAVA Developer? Lees dan snel verder! Want in januari 2022 start de Experis Academy weer met een leerprogramma waarin IT-experts je opleiden tot developer met als specialisatie JAVA. Dit allemaal met een salaris en een vooruitzicht op een baan bij een topwerkgever. ISNA51154 Meerdere functies beschikbaar Wat ga je doen tijdens het Fullstack JAVA ontwikkeltraject? Vanaf januari 2022 krijg jij via de Experis Academy een unieke kans om jouw IT-carrière een kickstart te

Bekijk vacature »

Back-end Developer

As a back-end Developer you are responsible for the development and operations of our lottery websites and applications. Sounds interesting? Then you might be our new back-end Developer! Functie Nederlandse Loterij is on the move and the use of IT and technology is becoming increasingly important. We have a large change agenda and are looking for enthusiastic IT colleagues who want to contribute to this. Nederlandse Loterij is in transition to become a more digital customer-oriented organization, embracing a data-driven way of working, in which we work in multidisciplinary and DevOps teams. You will work in the Technology department, the

Bekijk vacature »

Web Developer (m/w/d)

Web Developer IT and Infrastructure EMBL Heidelberg Closing date: 22 October 2021 Contract duration: 4 years (project related) Grading: 6, 7 or 8; depending on experience (monthly salary starting from 3.8k€ up to 4.8k€ after tax, plus other benefits)s Reference number: HD02055 IT Services operates and supports the IT infrastructure and services at EMBL headquarters in Heidelberg and at the laboratory’s sites in Barcelona and Rome. In collaboration with the EMBL-EBI in Cambridge our team is creating and implementing a new web infrastructure for all EMBL websites. We are looking to recruit an enthusiastic Web Developer to join the Software

Bekijk vacature »

C# .NET Developer @ Lelystad

2021-06-21 iSense C# .NET Developer Wil jij aan de slag als C# .NET software ontwikkelaar binnen een groeiende organisatie en je richten op uitdagende projecten voor verschillende klanten? Reageer dan snel! ISK49303 Nieuw Organisatie Als .NET Developer kom je te werken in de regio van Lelystad bij een organisatie die met toonaangevende klanten uit heel Nederland samen werkt. De producten en diensten van de organisatie bereiken miljoenen Nederlanders. Hierbij komt een grote hoeveelheid informatie kijken en deze moet discreet en veilig verwerkt worden. De processen die hierbij horen worden door het IT team vormgegeven. De organisatie heeft ook een adviserende

Bekijk vacature »

Senior Front-end Developer

If you are enthusiastic about front-end programming, and want to join a rapidly growing company where there is plenty of room for creativity and innovation, this is your chance.    We are currently looking for a talented Senior Front-end Developer to join our Development team in Eindhoven fulltime.  What we offer  A great opportunity to make an impact in a fast-growing logistics start-up.  The possibility to grow within the company. An informal, transparent, international and open culture.  A competitive salary and 25 vacation days per year.  Other perks: travel allowance, a pension plan, a gym membership, team building activities.  Who are you?  You are passionate for programming, you are ambitious and eager to continue developing yourself, and you are not afraid of taking full ownership

Bekijk vacature »

PHP backend ontwikkelaar met Symfony kennis gezoch

Bedrijfsomschrijving Dit bedrijf maakt voor haar klanten aansprekende en slimme webapplicaties voor haar klanten. De mensen gaan hier informeel en sympathiek met elkaar om. De eigenaar komt vanuit een andere branche en heeft inmiddels alweer een aantal jaar gelden dit bedrijf overgenomen. Ontwikkelaars krijgen hier heel veel vrijheid om zelf invulling te geven aan hun manier van werken. Inmiddels is er weer ruimte voor een extra medewerker vanwege de groei van de organisatie. Groeien is overigens geen doel van dit bedrijf. Wel het 'blij' maken van klanten en mooie technische PHP applicaties ontwikkelen. Die houding zorgt dat ze veel aanbevelingen

Bekijk vacature »

API Developer / Red Hat Fuse @ Amersfoort

2021-05-25 iSense API Developer Red Hat Fuse Heb jij als API Developer/Integratie specialist al enige ervaring met het ontwikkelen van API's en wil jij aan de slag met nieuwe oplossingen als Red Hat Fuse en 3Scale? Lijkt het jou gaaf om aan de vooravond te staan van de implementatie van een volledig nieuw applicatiehuis met tal van koppelingen? Ga jij daarnaast graag aan de slag binnen een uitdagende omgeving waarbij je de mogelijkheid krijgt jezelf verder te ontwikkelen? Lees dan snel verder! ISKO47622 Organisatie Deze organisatie is een toonaangevende speler in de vastgoedbranche en telt momenteel ruim 500 medewerkers. Met

Bekijk vacature »

Developer (PHP)

Als PHP developer werk jij mee aan de uitbreiding van onze SaaS applicatie. Je bouwt aan nieuwe functionaliteiten, verbetert ons Content Management Systeem op basis van Symfony en optimaliseert de gebruiksvriendelijkheid. Je werkt aan een mooi product dat continu uitgebreid wordt en de potentie heeft om de wereld over te nemen! Naast het meewerken aan de uitbreiding van onze SaaS applicatie, werk je ook aan specifieke modules voor onze klanten. Klanten als Scania, Fletcher Hotels, Gamma of Aditech hebben jouw hulp nodig! Wil jij meewerken om onze ambitie waar te maken? We bieden: Een salaris tussen € 2200 en €

Bekijk vacature »

Outsystems Developer @ Hoofddorp

2021-09-06 iSense Outsystems Developer Wil je werken bij een marktleider op gebied van transactieverwerking? Wil je werken aan het bouwen van een nieuw platform (greenfield) in Outsystems? Wil je jezelf blijven ontwikkelen en gebruik maken van legio opleidingsmogelijkheden? Dan is dit de ideale functie voor jou. Reageer snel! ISNO50906 Organisatie De organisatie waar je komt te werken is in Nederland marktleider op gebied van transactieverwerking. Daarnaast is de organisatie innovatief en een early adapter op gebied van Outsystems. Het willen behouden van de marktleiderspositie, maakt dat er is besloten een nieuw platform (greenfield) te bouwen waarin alle functionaliteiten optimaal met

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

17/10/2021 02:08:08
 
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.