Je hoeft er m.i. niet per se OOP van te maken, maar het helpt wel (ik ben er wel een voorstander van). Maar je zou wel aparte functies kunnen maken, dat maakt je script makkelijker testbaar. Je kan dan elke functie testen op de werking. Dat lukt niet in je huidige model.quote:Op zondag 17 november 2013 13:30 schreef Chandler het volgende:
Thanks voor jullie inhoudelijke reacties, de installer heeft een datablock en een script onderdeel, het script onderdeel ga ik niet oop maken, zou kunnen maar zie daar zelf het nut niet van in. Een keuze in database systemen ben ik zeker voor en gaat ook zeker verwerkt worden in het script en qua unset moet ik inderdaad nog wel het een en ander aanpassen... oh 2x case 7 had ik blijkbaar over het hoofd gezien... ooeps.. zelfde als set_time_limit, die heb ik gebruikt om te testen, iets wat bij kleine 'installers' echt niet nodig is maar bij installers van 30mb misschien wel, daar moet ik het allemaal nog op testen.
Paar vragen:
- Waarom stappen verminderen? zoveel stapjes zijn het niet? kijk eens naar programma's die je onder windows installeert? die hebben vaak ook vele stappen..
- Waarom OOP? zoveel code zit er niet in zeg maar... op zich zou't wel gemakkelijk zijn als ik er een ander ontwerp omheen wil gooien!
- Andere tips?
Oh de quit button is domweg bedoeld om de installer te resetten... je begint weer bij het begin want echt 'afsluiten' van het script zoals een programma gaat niet werken... tenzij ik window.close() ga gebruiken
thanks voor inhoudelijk feedback, maar hoe voelde de installer verder aan?
1 2 3 4 5 6 7 8 9 | $array = array ( 'ding' => .. , 'dat' => .. , 'zus' => .. , 'zo' => .. ); # zou ik herschrijven naar: $array = array( ding => .., dat => .., zus => .., zo => .., ); |
Mijn OOP is extreem brak, maak daarin vaak fouten en leer daar pas van als iemand er wat over zegt (mijn manier van leren). Ik zal eens kijken wat ik allemaal in functies kan stoppen wat er nu nog niet inzit...quote:Op zondag 17 november 2013 13:36 schreef slacker_nl het volgende:
Je hoeft er m.i. niet per se OOP van te maken, maar het helpt wel (ik ben er wel een voorstander van). Maar je zou wel aparte functies kunnen maken, dat maakt je script makkelijker testbaar. Je kan dan elke functie testen op de werking. Dat lukt niet in je huidige model.
Klopt, daarin heb je totaal gelijk alleen hoeft een gemiddelde gebruiker van dit script niets met de data, zelfde als jij niets moet met de inhoudt van een installer anders dan de installer zijn werk laten doen... dus de data hoeft er niet mooi uit te zien (data block) maar qua scripten werkt het zeker beter...quote:Verder zou ik wel iets aan je code conventies gaan doen:
Dat leest en edit prettiger.
Het gaat me erom dat als je later in dat script kijkt dat het makkelijk edit en leest. Dat de gebruiker er niet in hoeft te lezen.. maar andere coders wel en die wil je wel helpen met schone, zelf-gedocumenteerde code.quote:Op zondag 17 november 2013 14:19 schreef Chandler het volgende:
[..]
Mijn OOP is extreem brak, maak daarin vaak fouten en leer daar pas van als iemand er wat over zegt (mijn manier van leren). Ik zal eens kijken wat ik allemaal in functies kan stoppen wat er nu nog niet inzit...
[..]
Klopt, daarin heb je totaal gelijk alleen hoeft een gemiddelde gebruiker van dit script niets met de data, zelfde als jij niets moet met de inhoudt van een installer anders dan de installer zijn werk laten doen... dus de data hoeft er niet mooi uit te zien (data block) maar qua scripten werkt het zeker beter...
verder nog grote php fouten gezien? nee toch?
1 2 3 4 | $installer = new InstallerClass( # named arguments ); $installer->run(); |
Je moet altijd je documentatie goed op orde hebben. Maar dat staat los van de manier van code. Al helpt het ook dat je code zichzelf uitlegt. Maar je functies moeten gewoon gedocumenteerd zijn.quote:Op zondag 17 november 2013 14:49 schreef Chandler het volgende:
Ook met OOP moet je je inlezen in code want ->run() zegt ook compleet niets, want wat gebeurd er in al die functies? dus denk dat documentatie van het script zelf stukken belangrijker is.. Al zou ik het wel leuk vinden om er een OOP versie van maken maken dat ik zo de schil (layout/vormgeving) on the fly kan aanpassen!
Op m'n werk is het moeten. Want anders zegt de testsuite: je hebt de pod niet op orde en dan falen de tests.quote:Op zondag 17 november 2013 16:19 schreef Chandler het volgende:
moeten is onzin, handig true; zal ik ook zeker gaan doen, maar blijkbaar vinden jullie het idee toch best leuk is een vervolg op een heel oud projectje (php2hex )
Testbaarheid.quote:Op zondag 17 november 2013 18:10 schreef Chandler het volgende:
Waarom in aparte files? leg dat eens uit?
Hoe heb je die code getest?quote:Op maandag 18 november 2013 01:09 schreef Chandler het volgende:
Zo werk ik inderdaad ook al, alle code is gewoon al getest (toen nog niet op meerdere platformen maar goed) met de losse modules die ik toen werkend met elkaar heb verbonden in het huidige script. Ik zal vannacht nog eens naar wat puntjes van het script kijken (heb toch nachtdienst, dus alle tijd)
Nog goede suggesties voor een test-suite?quote:Op zondag 17 november 2013 16:21 schreef slacker_nl het volgende:
[..]
Op m'n werk is het moeten. Want anders zegt de testsuite: je hebt de pod niet op orde en dan falen de tests.
Overigens zeg ik niks over je idee, want ik zou je script en je installer in aparte files frotten, maar dat is mijn mening.
Onder PHP, ik ken er geen. Onder Perl heb je prove.quote:Op maandag 18 november 2013 13:16 schreef HalveZware het volgende:
[..]
Nog goede suggesties voor een test-suite?
Waarom in hemelsnaam een NoSQL database voor een loginsysteemquote:Op maandag 18 november 2013 13:52 schreef RetRy32 het volgende:
Sjooooooooooow ik was aan het kloten met dat MongoDB gebeuren, ik heb dus een Auth template voor mijzelf gemaakt, is wel een beetje rommelig en comments heb ik in het Engels gezet btw...
Voor de liefhebbers...
https://anonfiles.com/file/9e49f9c2059cdd78233e284435781ffa
En dat MongoLab remote hosting is gewoon gratis tot 500MB
[ afbeelding ]
Waarom niet?quote:Op maandag 18 november 2013 14:35 schreef KomtTijd... het volgende:
Waarom in hemelsnaam een NoSQL database voor een loginsysteem
Het is een template, en er staat ook nog een comment erover erbij manquote:Op maandag 18 november 2013 14:35 schreef KomtTijd... het volgende:
[..]
...en zie ik daar MD5 password hashes? , , !!!!!!!!!
Normaliter wil je behoorlijk joinen met je usersquote:
Daar heb ik niet naar gekeken. Maar MD5 gebruiken voor passwordhashes zou gewoon bij wet verboden moeten zijn.quote:[..]
Het is een template, en er staat ook nog een comment erover erbij man
Want met no SQL databases zou je niet kunnen joinen? Ik bel gelijk even Twitter en Facebook dat hun architectuur niet klopt!quote:Op maandag 18 november 2013 15:16 schreef KomtTijd... het volgende:
[..]
Normaliter wil je behoorlijk joinen met je users
Het kan, maar er is geen reden om het jezelf extra moeilijk te maken.quote:Op maandag 18 november 2013 15:22 schreef raptorix het volgende:
[..]
Want met no SQL databases zou je niet kunnen joinen? Ik bel gelijk even Twitter en Facebook dat hun architectuur niet klopt!
Forum Opties | |
---|---|
Forumhop: | |
Hop naar: |