Class efficiëntie [code review]

Overzicht Reageren

Sponsored by: Vacatures door Monsterboard

Database Developer

Functieomschrijving Heb jij ongeveer 3 jaar ervaring als Database Developer met MS SQL of een vergelijkbare database? Wil jij werken voor een ambitieuze werkgever in regio Tilburg waar jij volledig de mogelijkheid krijgt jezelf te ontwikkelen? Lees dan snel verder! Hoe ziet jouw takenpakket eruit? Je gaat projecten gedurende het hele proces begeleiden. Je sluit aan bij afspraken met klanten om hun processen helder te krijgen. Vervolgens voer jij het project uit en zorgt dat dit zo goed mogelijk verloopt; Je werkt aan nieuwe softwareoplossingen die de logistieke processen verbeteren of vernieuwen; Je houdt je bezig met het ontwikkelen van

Bekijk vacature »

C# .NET Developer

Functieomschrijving Voor dit leuke softwarre bedrijf in de omgeving Vught zijn we per direct op zoek naar een C#/.NET Developer. Is development jouw passie en doe je dit graag met C#/.NET? Lees dan snel verder! Jou werkzaamheden zullen zijn: Zorgen voor de optimalisatie van de huidige software en het automatiseren van bedrijfsprocessen. Naar aanleiding van de wensen van de klant ga je, met je collega's op zoel naar passende oplossingen en je werkt dit uit tot een mooi eindproduct. Je gaat webshops, websites en webapplicaties ontwikkelen door middel van ASP.NET, C# en MVC Framework. Bedrijfsprofiel Deze opdrachtgever houdt zich bezig

Bekijk vacature »

Back-end Developer (Permanent position with the em

Bedrijfsomschrijving Dutch specialist in technical installation materials. Functieomschrijving Purpose of the position: Our client is looking for a Back-end Developer who, together with the rest of the energetic and dynamic team, is responsible for the development and management of the website. This not only concerns the development and management of the current website, but also the development of a new Headless Commerce Platform to keep the customer's website Future proof. Within the IT department, there is a real DevOps culture and the commerce team is at the forefront and tries to implement continuous improvements. Most important tasks: ï‚· Designing and

Bekijk vacature »

Full stack developer Node.js, React Remote

Functie Als fullstack JavaScript developer vind jij het uitdagend om op basis van concrete klantvragen nieuwe functionaliteiten te ontwikkelen. Bij voorkeur worden deze functionaliteiten op een bepaalde manier geprogrammeerd, zodat ze door meerdere klanten te gebruiken zijn. Je hebt dus vaak te maken met abstracte vraagstukken. Om dit te kunnen realiseren sta je nauw in contact met de product owner en/of klant. Je bent niet alleen onderdeel van het development team, maar hebt ook vaak contact met de product-owner en/of klanten om daardoor inzichten te verzamelen die leiden tot productverbeteringen. • Inzichten verzamelen bij de klant en/of product owner •

Bekijk vacature »

IT Manager team PaaS

TenneT is hard groeiende om haar ambities waar te kunnen maken. Zo nemen wij een leidende rol in het aanjagen van de energietransitie. Het werven van nieuw talent speelt daarin een cruciale rol. Wij zijn op zoek naar een gedreven Lead PaaS die hieraan wil bijdragen en misschien ben jij dat wel? Jouw bijdrage aan TenneT Je wordt de Teammanager (Lead) van een nieuw team binnen de afdeling Basic van Information Technology and Facilities (ITF) van TenneT. Het team heet Platform as a Service. Hier wordt elke dag in een goede sfeer met zijn allen hard gewerkt om vanuit IT

Bekijk vacature »

Embedded Software Developer

Functie omschrijving Ben jij een Embedded Software Developer die affiniteit heeft met de allernieuwste technieken? Voor een mooi softwarebedrijf in omgeving Gouda zijn wij op zoek naar een Embedded Software developer. Binnen deze rol houdt jij je bezig met alle werkzaamheden die nodig zijn om een functionaliteit te bouwen. Denk aan ontwerpen, architectuur, programmeren en algoritmes. Je voert test en validatie werkzaamheden uit bij de implementatie bij de klant. Ben jij enthousiast en een echte team player? Lees dan snel verder en laat wat van je horen! Bedrijfsprofiel Onze opdrachtgever bestaat uit een groot aantal creatieve en ambitieuze ontwikkelaars. Ze

Bekijk vacature »

.NET developer

Functie Als .NET ontwikkelaar ga jij aan de slag bij een van onze klanten actief in de High Tech Industrie. Onze klanten zijn voornamelijk gelokaliseerd in de omgeving van Eindhoven. Wij zijn erg selectief als het gaat om de projecten die wij accepteren en richten ons dan ook alleen op innovatieve en complexe projecten. Omdat onze klanten voornamelijk gespecialiseerd zijn in de machinebouw, werk jij ook vaak dicht tegen de machines aan. Ons team bestaat momenteel uit Embedded engineers, IOT developers en Cloud engineers. Wij werken voornamelijk aan Microsoft projecten waar er gebruik wordt gemaakt van WPF, UWP, .NET Core

Bekijk vacature »

Fullstack Software Developer

Functieomschrijving Voor een ambitieuze werkgever in regio Roosendaal zijn wij op zoek naar een Full Stack C#.NET Developer. Als software programmeur ben je verantwoordelijk voor het bouwen van webapplicaties, apps en dashboards voor de eigen IOT-oplossingen. Je werkt samen met andere developers en engineers om de sensoren in machines uit te lezen en deze data om te zetten in management informatie voor jullie klanten. Taken en verantwoordelijkheden: Verder ontwikkelen en onderhouden van webapplicaties, dashboards en apps voor de eigen IOT-oplossingen; Testen en goedkeuren van de software; Je gaat aan de slag met de volgende technologieën en frameworks: C#, JS frameworks,

Bekijk vacature »

Belastingdienst - Freelance Senior Applicatie ontw

Startdatum: 01.06.2023 Richttarief: €65,00 - €75,00 Duur van de opdracht: 6 maanden Uren per week: 36 Taal: Nederlands vereist! Gelieve in het Nederlands te solliciteren. Functieomschrijving: We verwachten van je, dat je: Brede ervaring hebt als JAVA-ontwikkelaar; Ervaring hebt met Agile/Scrum-werken en je thuis voelt in een Agile omgeving; Een aandeel levert aan het scrumproces en in de SAFe-releasetrain; Zelfstandig werkt in een scrumteam en intensief de samenwerking op zoekt met je directe collega’s en je omgeving; Ervaring meebrengt met het schattten en inplannen van taken tot en met het testen en demonstreren van de opgeleverde functionaliteit; Collega’s in je

Bekijk vacature »

Backend Developer PHP Laravel SaaS

Dit ga je doen Het ontwikkelen van nieuwe features die bijdragen aan de groei van de klanten van de organisatie; Je denkt mee over nieuwe innovaties, features en verbeteringen in de applicatiearchitectuur; Je draagt bij aan de continue ontwikkeling van jouw team doordat je elke dag streeft naar het verbeteren van jouw eigen prestaties; Je neemt actief deel aan Scrum meetings en de Backend Guild. Hier ga je werken Voor een snel groeiend bedrijf, in de regio Nieuw Vennep, zijn wij opzoek naar een ervaren Backend Developer. De organisatie is actief in de e-commercebranche en ontzorgt haar klanten middels een

Bekijk vacature »

App Developer

Samen werken aan een gezonder Nederland en toekomstbestendige zorg voor iedereen. Dat is de impact die jij kan hebben als App Developer bij VGZ. Wil jij een bijdrage leveren aan een maatschappij waarin iedereen zich thuis voelt? Bekijk dan de vacature. Uit onderzoek van Computable is VGZ verkozen tot ‘beste niet-ICT werkgever voor ICT’ers van Nederland’ Hoe ook jij het verschil maakt Als App developer werk jij aan het belangrijkste communicatiekanaal van VGZ, namelijk de App! Als App developer bij VGZ maak je onderdeel uit van een van onze App-teams. Met een goede mix van kennis en ervaring zet je

Bekijk vacature »

Grafisch vormgever

Standplaats: Maasland Aantal uren: 32 – 40 uur per week Opleidingsniveau: HBO werk- en denkniveau Ben jij een ambitieuze grafisch vormgever met een passie voor creativiteit en oog voor detail? Vind jij het daarnaast leuk om ook marketingactiviteiten op te pakken? Dan zijn wij op zoek naar jou! Bedrijfsinformatie Westacc Group BV is het zusterbedrijf van HABA en specialiseert zich in (maatwerk) oplossingen voor (elektro) techniek en verlichting in de kampeerbranche. Zij produceren en assembleren onderdelen voor caravans, campers en boten. Voor een groot aantal caravan- en campermerken leveren wij producten als zekeringkasten, invoerdozen, acculaders, schakelmateriaal en verlichting. De producten

Bekijk vacature »

Medior Java developer

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 »

.NET developer

Functie As a .NET developer you work together in a multidisciplinary development team with 1-2 Senior .NET developers, two front-end developers, Data Scientists and one UX designer. As a team you work on developing a Cloud based application and making this application more stable. Unit testing will also become very important in your new position. Together with the Senior .NET developer you will be responsible for developing the API. You work with a lot of data and occasionally there will also be data issues and some queries will have to be run. This means that you will work a lot

Bekijk vacature »

Fullstack developer

Zie jij mogelijkheden om onze tooling technisch te verbeteren en uit te bouwen? Over Jobmatix Jobmatix is een innovatieve en internationale speler op het gebied van jobmarketing. Onze jobmarketing automation tool helpt organisaties bij het aantrekken van nieuw talent door vacatures digitaal, geautomatiseerd en op een efficiënte manier te adverteren en onder de aandacht te brengen bij de doelgroep op 25+ jobboards. Volledig performance-based, waarbij organisaties betalen op basis van cost per click of cost per applicant. Maandelijks wordt onze jobmarketing automation tool al gebruikt door vele directe werkgevers, intermediairs en mediabureaus, waaronder Picnic, Rijkswaterstaat, AdverOnline, Schiphol, DPA, Teleperformance en

Bekijk vacature »

04/05/2012 09:15:02
Quote Anchor link
Beste,

Aangezien het mijn eerste keer is dat ik, uit mezelf, begin met het bouwen van classes, zou ik graag willen weten of en hoe hij efficiënter kan?

Alvast bedankt!

Class 1: Mysql connection
Class 2: password beveiligin
class 3: Email verzenden - nog niet helemaal af..
class 4: Email validatie


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
<?php

class mysql_con{
    function
mysql($mysql){
        $db_host         = "place_your_db_host";
        $db_username     = "place_your_username";  
        $db_pass         = "place_your_db_password";  
        $db_name         = "place_your_db_name";

        if (mysql_connect("$db_host","$db_username","$db_pass") == false){
             return false;
}
    
        if(mysql_select_db("$db_name") or die(mysql_error()) == false){
            return false;        
        }
        
    }
}


class pass_secur{
    function
pass($pass){
            $pepper        =    uniqid() - 1002;
            $salt         =     'abcdefghijklmnopqrstuvwxzyABCDEGHIJKLMNOPQRSTUVWXYZ1234567890';    
            $pass        =     $pass = $salt.sha1($_POST['password']).$pepper;
            
    }
}


class setmail{
    function
settingmail($setmail){
        $to      = "bob@domain_example.co.za";
        $subject = "Email from website";
        $message = $_REQUEST["body"];
        $subject = $_REQUEST["subject"];
        $email = $_REQUEST["email"];
        
        function
is_valid_email($email) {
          return preg_match('#^[a-z0-9.!\#$%&\'*+-/=?^_`{|}~]+@([0-9.]+|([^\s]+\.+[a-z]{2,6}))$#si', $email);
        }

        
        if (!is_valid_email($email)) {
          echo 'Sorry, invalid email';
          exit;
        }

        
        $headers = "From: $email";
        mail($to, $subject, $message, $headers);
        echo "Thanks for submitting.";
    }
}


class email_validate{
function
validate_email($val_mail){
        if (!eregi("^[_a-z0-9-]+(\.[_a-z0-9-]+)*@[a-z0-9-]+(\.[a-z0-9-]+)*(\.[a-z]{2,3})$", $str)) return false;
        list($usr, $domain) = split('@', $str);
        if (!@checkdnsrr($domain, 'MX')) return false;
        return true;
    }
}

?>
Gewijzigd op 04/05/2012 09:15:51 door
 
PHP hulp

PHP hulp

10/02/2025 21:35:47
 
Erwin H

Erwin H

04/05/2012 09:23:01
Quote Anchor link
Variabelen in strings zetten heeft natuurlijk geen enkele zin:
mysql_connect("$db_host","$db_username","$db_pass")
Dat moet zijn:
mysql_connect($db_host,$db_username,$db_pass)

Een die() in een class is echt not done. Je wilt nooit dat zomaar een object bepaalt dat je script stopt. Ofwel je gooit een error, ofwel je geeft een foutcode terug.

eregi (regel 54) kan je niet meer gebruiken in php, dat zal je dus moeten veranderen.

Waarom gebruik je een functie is_valid_email in de setmail class, terwijl je er een hele class vor gaat schrijven?

Toevoeging op 04/05/2012 09:25:17:

Maar misschien nog belangrijker, zoals je het nu opbouwt is het gewoon functioneel programmeren, maar dan elke functie in een class. Op deze manier OOP toepassen heeft niet zo heel veel nut, je gebruikt namelijk geen enkele meerwaarde van OOP.
 
M grootveld

M grootveld

04/05/2012 09:35:35
Quote Anchor link
Erwin H op 04/05/2012 09:23:01:
Variabelen in strings zetten heeft natuurlijk geen enkele zin:
mysql_connect("$db_host","$db_username","$db_pass")
Dat moet zijn:
mysql_connect($db_host,$db_username,$db_pass)

Een die() in een class is echt not done. Je wilt nooit dat zomaar een object bepaalt dat je script stopt. Ofwel je gooit een error, ofwel je geeft een foutcode terug.

eregi (regel 54) kan je niet meer gebruiken in php, dat zal je dus moeten veranderen.

Waarom gebruik je een functie is_valid_email in de setmail class, terwijl je er een hele class vor gaat schrijven?

Toevoeging op 04/05/2012 09:25:17:

Maar misschien nog belangrijker, zoals je het nu opbouwt is het gewoon functioneel programmeren, maar dan elke functie in een class. Op deze manier OOP toepassen heeft niet zo heel veel nut, je gebruikt namelijk geen enkele meerwaarde van OOP.


De variabelen was inderdaad een fout, die ik even over het hoofd gezien had, is aangepast.

Wat zou ik in de plaats van die() wel kunnen gebruiken? Wat is netjes in een class?

Eregi veranderd naar:
Code (php)
PHP script in nieuw venster Selecteer het PHP script
1
if (!preg_match("^[_a-z0-9-]+(\.[_a-z0-9-]+)*@[a-z0-9-]+(\.[a-z0-9-]+)*(\.[a-z]{2,3})$", $str)) return false;


Setmail is bij het verzenden van mail, bijvoorbeeld op de contact pagina. Email_validate is bijvoorbeeld voor het checken van de mail tijdens het registreren.

Je laatste alinea begrijp ik niet, zou je daar wat meer uitleg over kunnen geven?
 
Erwin H

Erwin H

04/05/2012 09:47:58
Quote Anchor link
M grootveld op 04/05/2012 09:35:35:
Wat zou ik in de plaats van die() wel kunnen gebruiken? Wat is netjes in een class?

Dat heb ik al gezegd, een error gooien of een foutcode teruggeven (kan ook gewoon false zijn).
M grootveld op 04/05/2012 09:35:35:
Je laatste alinea begrijp ik niet, zou je daar wat meer uitleg over kunnen geven?

Dan is het misschien beter om nog niet met OOP te gaan beginnen. Sorry dat ik het zeg, maar dit kan ik niet zo simpel uitleggen. Het gebruik van OOP is niet gewoon een class om een functie zetten zoals jij nu aan het doen bent. Classes hebben ieder hun eigen verantwoordelijkheid en functionaliteit. Dat gaat een stuk verder dan gewone functies. Ik vind het moeilijk om het precies uit te leggen (misschien dat iemand anders daar beter in is), maar als dit helemaal nieuw voor je is dan raad ik je aan om eerst eens wat boeken en tutorials door te gaan nemen.
 
Wouter J

Wouter J

04/05/2012 10:48:06
 



Overzicht Reageren

 
 

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.