abonnement Unibet Coolblue
pi_133340467
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?
Just say hi!
  zondag 17 november 2013 @ 13:36:56 #152
187069 slacker_nl
Sicko pur sang
pi_133340624
quote:
0s.gif 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?
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.

Verder zou ik wel iets aan je code conventies gaan doen:

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   => ..,
);

Dat leest en edit prettiger.
In theory there is no difference between theory and practice. In practice there is.
pi_133341618
quote:
0s.gif 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.
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:
Verder zou ik wel iets aan je code conventies gaan doen:

Dat leest en edit prettiger.
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? :D
Just say hi!
  zondag 17 november 2013 @ 14:29:34 #154
187069 slacker_nl
Sicko pur sang
pi_133341873
quote:
0s.gif 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? :D
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.

Je kan toch heel makkelijk een class maken met functies en dan

1
2
3
4
$installer = new InstallerClass(
# named arguments
); 
$installer->run(); 
In theory there is no difference between theory and practice. In practice there is.
pi_133342363
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! *)
Just say hi!
  zondag 17 november 2013 @ 15:02:56 #156
187069 slacker_nl
Sicko pur sang
pi_133342845
quote:
14s.gif 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! *)
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.
In theory there is no difference between theory and practice. In practice there is.
pi_133345051
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 ;))
Just say hi!
  zondag 17 november 2013 @ 16:21:37 #158
187069 slacker_nl
Sicko pur sang
pi_133345141
quote:
0s.gif 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 ;))
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.
In theory there is no difference between theory and practice. In practice there is.
pi_133348505
Waarom in aparte files? leg dat eens uit?
Just say hi!
  zondag 17 november 2013 @ 18:43:06 #160
187069 slacker_nl
Sicko pur sang
pi_133349368
quote:
5s.gif Op zondag 17 november 2013 18:10 schreef Chandler het volgende:
Waarom in aparte files? leg dat eens uit?
Testbaarheid.
In theory there is no difference between theory and practice. In practice there is.
pi_133349465
Ik snap je nog steeds niet, want ook met een 'iets' groter bestand is testen allemaal geen probleem hoor :? of doel je op het feit dat ik deze modules dan apart kan testen? zo ja; dat doe (deed) ik al voordat ik ze in de installer verwerkte...
Just say hi!
  zondag 17 november 2013 @ 19:06:27 #162
187069 slacker_nl
Sicko pur sang
pi_133349990
Do one thing and do it well. En ik doel inderdaad dat je 1 module per keer test. Waarom zou je allemaal modules/klasses laden als je er maar eentje wilt hebben?
In theory there is no difference between theory and practice. In practice there is.
pi_133366050
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)
Just say hi!
pi_133367961
quote:
14s.gif 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)
Hoe heb je die code getest?
pi_133369465
Gewoon 1 scriptje met daarin de class/functie en daaronder het test gedeelte?
Just say hi!
pi_133373517
quote:
0s.gif 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.
Nog goede suggesties voor een test-suite?
  Moderator / Redactie Sport / Devops maandag 18 november 2013 @ 13:20:55 #167
176766 crew  zoem
zoemt
pi_133373605
Wat betreft php: PHPUnit icm een CI-omgeving.
  maandag 18 november 2013 @ 13:21:05 #168
187069 slacker_nl
Sicko pur sang
pi_133373614
quote:
0s.gif Op maandag 18 november 2013 13:16 schreef HalveZware het volgende:

[..]

Nog goede suggesties voor een test-suite?
Onder PHP, ik ken er geen. Onder Perl heb je prove.
In theory there is no difference between theory and practice. In practice there is.
pi_133374459
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 _O_


Overigens heb ik het lokaal werkend gehad met alleen Ampps te installeren en daarvandaan MongoDB starten.
Er is wel echt belachelijk weinig documentatie over MongoDB met de PHP driver te vinden zeg...
En wat er te vinden valt is verouderd of niet noob-friendly...

[ Bericht 10% gewijzigd door #ANONIEM op 18-11-2013 14:54:46 ]
pi_133375739
quote:
0s.gif 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 _O_
[ afbeelding ]
Waarom in hemelsnaam een NoSQL database voor een loginsysteem :?
...en zie ik daar MD5 password hashes? ;(, ;(, ;(!!!!!!!!!
pi_133376203
quote:
14s.gif Op maandag 18 november 2013 14:35 schreef KomtTijd... het volgende:
Waarom in hemelsnaam een NoSQL database voor een loginsysteem
Waarom niet?
quote:
14s.gif Op maandag 18 november 2013 14:35 schreef KomtTijd... het volgende:

[..]
...en zie ik daar MD5 password hashes? ;(, ;(, ;(!!!!!!!!!
Het is een template, en er staat ook nog een comment erover erbij man :')

[ Bericht 3% gewijzigd door #ANONIEM op 18-11-2013 14:50:40 ]
  maandag 18 november 2013 @ 14:52:53 #172
25889 Sitethief
Fulltime Flapdrol
pi_133376345
Ik heb zelf goede ervaringen met CouchDB :).
Stroek: Sitethief, die is heel groot en sterk :Y.
Faat: *zucht* zoals gewoonlijk hoor Sitethief weer in de bocht >:)
pi_133377000
quote:
0s.gif Op maandag 18 november 2013 14:48 schreef RetRy32 het volgende:

[..]

Waarom niet?
Normaliter wil je behoorlijk joinen met je users
quote:
[..]

Het is een template, en er staat ook nog een comment erover erbij man :')
Daar heb ik niet naar gekeken. Maar MD5 gebruiken voor passwordhashes zou gewoon bij wet verboden moeten zijn.
pi_133377158
quote:
14s.gif Op maandag 18 november 2013 15:16 schreef KomtTijd... het volgende:

[..]

Normaliter wil je behoorlijk joinen met je users

Want met no SQL databases zou je niet kunnen joinen? Ik bel gelijk even Twitter en Facebook dat hun architectuur niet klopt!
🕰️₿🕰️₿🕰️₿🕰️₿🕰️₿🕰️ TikTok next Block
pi_133377235
quote:
0s.gif 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!
Het kan, maar er is geen reden om het jezelf extra moeilijk te maken.
abonnement Unibet Coolblue
Forum Opties
Forumhop:
Hop naar:
(afkorting, bv 'KLB')