DIG Digital Corner
Alles wat je altijd al over computers, hardware, software, internet en elektronische gadgets had willen weten, maar niet op Tweakers.net durft te vragen.
Laat het een les zijn, je kan zelfs een php script er voor schrijven.quote:Op donderdag 7 november 2013 14:15 schreef pascal08 het volgende:
[..]
Maar dan zal ik toch al die duizenden URL's moeten bewerken met dat stukje base url.
Daarvoor heb je toch gewoon relatieve paden..?
In theory there is no difference between theory and practice. In practice there is.
Als het er duizenden zijn, vraag ik me toch af of er niet nog ergens wat fout gaat..quote:
[..]
Maar dan zal ik toch al die duizenden URL's moeten bewerken met dat stukje base url.
Dat vraag ik me ook altijd of als ik de reactie zo lees.quote:Op donderdag 7 november 2013 23:21 schreef Maringo het volgende:
[..]
Als het er duizenden zijn, vraag ik me toch af of er niet nog ergens wat fout gaat..
Ik ondervind nu in ieder geval geen problemen meer met een virtual host.
als het goed is alleen de base url in je template. Als de rest nu goed gaat met relatieve paden, zal dat ook goed blijven gaan.quote:
[..]
Maar dan zal ik toch al die duizenden URL's moeten bewerken met dat stukje base url.
Voordeel van ditsoort server-specifieke dingen in een config bestand zetten is dat je heel gemakkelijk kunt kopiëren van de ene naar de andere host, door gewoon al je bestanden behalve de config.php te kopiëren.
Ook de URL's van sprites, afbeeldingen, ajax-url's etc.quote:Op donderdag 7 november 2013 23:32 schreef KomtTijd... het volgende:
[..]
als het goed is alleen de base url in je template. Als de rest nu goed gaat met relatieve paden, zal dat ook goed blijven gaan.
Voordeel van ditsoort server-specifieke dingen in een config bestand zetten is dat je heel gemakkelijk kunt kopiëren van de ene naar de andere host, door gewoon al je bestanden behalve de config.php te kopiëren.
Wat je zegt over config files is iets dat steeds beter tot me door begint te dringen.
Ik ben nu Laravel aan het ontdekken en ik vind het eigenlijk best een goed framework. CodeIgniter is dood aan het gaan toch?
Ik ben bezig front-end editting te bouwen in een webshopje wat ik ooit geknutseld heb. Daarin komt nu dit prachtige stukje proza voor
Feitelijk veranderd dit een span met een nummertje in een input veld met het nummertje in de value van dat input veld:
En onblur moet hij dan het form submitten. Het form ziet er NA de DOM wijziging als volgt uit
Het form submit ook keurig, maar... alleen de id value wordt gesubmit en ac value niet.
Iemand een idee?
| 1 | <span onclick="var input = document.createElement('input'); input.setAttribute('value', this.firstChild.nodeValue); input.setAttribute('name', 'ac'); input.setAttribute('onblur', 'document.getElementById(\'front11\').submit();'); this.parentNode.replaceChild(input, this);">2015104015</span> |
Feitelijk veranderd dit een span met een nummertje in een input veld met het nummertje in de value van dat input veld:
| 1 | <input value="2014102016" name="ac" onblur="document.getElementById('front11').submit();"> |
En onblur moet hij dan het form submitten. Het form ziet er NA de DOM wijziging als volgt uit
| 1 2 3 4 | <form id="front11" action="/index.php" method="post"> <input type="hidden" name="id" value="3394" /> <input value="2014102016" name="AC" onblur="document.getElementById('front11').submit();"> </form> |
Het form submit ook keurig, maar... alleen de id value wordt gesubmit en ac value niet.
Iemand een idee?
Zo een eerste snelle blik: het input veld van ac heeft geen type.quote:Op vrijdag 8 november 2013 16:18 schreef Swetsenegger het volgende:
Ik ben bezig front-end editting te bouwen in een webshopje wat ik ooit geknutseld heb. Daarin komt nu dit prachtige stukje proza voor
[ code verwijderd ]
Feitelijk veranderd dit een span met een nummertje in een input veld met het nummertje in de value van dat input veld:
[ code verwijderd ]
En onblur moet hij dan het form submitten. Het form ziet er NA de DOM wijziging als volgt uit
[ code verwijderd ]
Het form submit ook keurig, maar... alleen de id value wordt gesubmit en ac value niet.
Iemand een idee?
type default naar text, zou geen probleem mogen zijn.
misschien wordt de onblur getriggerd vóór de change ofzo?
misschien wordt de onblur getriggerd vóór de change ofzo?
Klopt, maar ik heb hem voor de zekerheid ook een type toegevoegd met hetzelfde resultaatquote:
type default naar text, zou geen probleem mogen zijn.
Nee, onclick word de span input, dat zie ik ook als ik het element inspecteer. En pas daarna wordt de onblur getriggerd. Dus in de DOM is het element aangepast voor het form gesubmit wordt. De submit zit ook in de input, dus kan uberhaupt niet onblur getriggerd worden als de input er nog niet is denk ik?quote:misschien wordt de onblur getriggerd vóór de change ofzo?
Weet je zeker dat het formulier er na de DOM-wijziging zo uit ziet als je hier plaatst? Valt het gegenereerde input-element niet per ongeluk buiten het formulier?quote:
Ik ben bezig front-end editting te bouwen in een webshopje wat ik ooit geknutseld heb. Daarin komt nu dit prachtige stukje proza voor
[ code verwijderd ]
Feitelijk veranderd dit een span met een nummertje in een input veld met het nummertje in de value van dat input veld:
[ code verwijderd ]
En onblur moet hij dan het form submitten. Het form ziet er NA de DOM wijziging als volgt uit
[ code verwijderd ]
Het form submit ook keurig, maar... alleen de id value wordt gesubmit en ac value niet.
Iemand een idee?
Als je de inhoud van de onblur nu eens vervangt door iets als alert(this.parent), zit je dan echt in het formulier?
Schuimpje... mijn liefste. Verlaat mij nimmer weer...
Ik ben er al uit. Het form werd gestart in de ene TR en geëindigd in de ander. En dat vinden browsers blijkbaar niet leuk.quote:Op zaterdag 9 november 2013 11:39 schreef papernote het volgende:
[..]
Weet je zeker dat het formulier er na de DOM-wijziging zo uit ziet als je hier plaatst? Valt het gegenereerde input-element niet per ongeluk buiten het formulier?
Als je de inhoud van de onblur nu eens vervangt door iets als alert(this.parent), zit je dan echt in het formulier?
Dat lijkt me inderdaad geen valide HTML. Ik zou de form om je hele tabel heen gooien en/of om de tr-tags heen.quote:Op zaterdag 9 november 2013 11:50 schreef Swetsenegger het volgende:
[..]
Ik ben er al uit. Het form werd gestart in de ene TR en geëindigd in de ander. En dat vinden browsers blijkbaar niet leuk.
Schuimpje... mijn liefste. Verlaat mij nimmer weer...
Dat gaat niet aangezien er in de tabel nog wat forms staan voor het winkelmandje en wishlist.quote:Op zaterdag 9 november 2013 12:49 schreef papernote het volgende:
[..]
Dat lijkt me inderdaad geen valide HTML. Ik zou de form om je hele tabel heen gooien en/of om de tr-tags heen.
Dus ik heb nu per item blok (artikelcode + naam, omschrijving, prijs) een form. Ik submit toch per onderdeel onBlur, dus dat is sowieso geen probleem.
Deze edit mogelijkheid is natuurlijk alleen beschikbaar voor admins die zijn ingelogged. Dus deze wat messy code staat niet in de gebruikers front-end.
Kun je niet één groot formulier maken met meerdere submit-opties (met dezelfde name en een verschillende value) en dan op de server bepalen wat je wilt doen?quote:
[..]
Dat gaat niet aangezien er in de tabel nog wat forms staan voor het winkelmandje en wishlist.
Dus ik heb nu per item blok (artikelcode + naam, omschrijving, prijs) een form. Ik submit toch per onderdeel onBlur, dus dat is sowieso geen probleem.
Deze edit mogelijkheid is natuurlijk alleen beschikbaar voor admins die zijn ingelogged. Dus deze wat messy code staat niet in de gebruikers front-end.
Schuimpje... mijn liefste. Verlaat mij nimmer weer...
Heb ik ook heel even naar gekeken. Vereist wel een compleet andere opbouw en dat valt wel een beetje buiten de scope eigenlijk. Op zich werkt het nu goed. En ik heb het goed commentedquote:
[..]
Kun je niet één groot formulier maken met meerdere submit-opties (met dezelfde name en een verschillende value) en dan op de server bepalen wat je wilt doen?
quote:
[..]
Nou, best aardig van Google om de waarschuwing te geven.
Het is irritant aangezien je de manual nodig hebt
Prachtige site man, een bookmarkje waard.
Ik heb bij een verse installatie van phpbb3 de config tabel van mn oude phpbb3 in mysql gezet, maar die instellingen worden helemaal niet gebruikt..
Er staan gewoon alle standaard dingen.. als bijv:
yourdomain.com
A short text to describe your forum
en de rest is ook allemaal standaard.
Weet iemand hoe dat kan?
Er staan gewoon alle standaard dingen.. als bijv:
yourdomain.com
A short text to describe your forum
en de rest is ook allemaal standaard.
Weet iemand hoe dat kan?
En je hebt gecheckt dat in de config table van je website database jouw waarden staan (dat de default config table dus overschreven is)? En weet je ook zeker dat phpBB3 zijn configuratie-instellingen daar ook vandaan haalt?
"You know what Hell really is? It's not lakes of burning oil or chains of ice. It's being removed from God's sight."
hij staat goed in mysql ja..
Toen ik hier op mijn lokale pc die tabel gekoppeld had aan een verse install (zelfde versie als de oude was.. nou weet ik dit niet zeker maar volgens mij liet ie het toen ook niet zien totdat ik em geupdate had.. maarja nou is ie volledig up to date dus dat schiet niet op.
Toen ik hier op mijn lokale pc die tabel gekoppeld had aan een verse install (zelfde versie als de oude was.. nou weet ik dit niet zeker maar volgens mij liet ie het toen ook niet zien totdat ik em geupdate had.. maarja nou is ie volledig up to date dus dat schiet niet op.
--
[ Bericht 100% gewijzigd door Chandler op 13-11-2013 08:42:58 ]
[ Bericht 100% gewijzigd door Chandler op 13-11-2013 08:42:58 ]
The people who lost my respect will never get a capital letter for their name again.
Like trump...
Like trump...
Heet de database ook hetzelfde?quote:
hij staat goed in mysql ja..
Toen ik hier op mijn lokale pc die tabel gekoppeld had aan een verse install (zelfde versie als de oude was.. nou weet ik dit niet zeker maar volgens mij liet ie het toen ook niet zien totdat ik em geupdate had.. maarja nou is ie volledig up to date dus dat schiet niet op.
Ik weet niks van Hindoes. Wel van Samoerai en andere dingen.
Onleesbaar.quote:Op woensdag 13 november 2013 19:42 schreef Chandler het volgende:
Wat vinden jullie van me scrippie?
http://pastebin.com/g2xTLLtP
Baggerquote:
Wat vinden jullie van me scrippie?
http://pastebin.com/g2xTLLtP
Lekker vatbaar voor SQL Injectionquote:
Wat vinden jullie van me scrippie?
http://pastebin.com/g2xTLLtP
Op dinsdag 9 augustus 2011 23:01 schreef SuperrrTuxxx het volgende:
Ik hou zoveel van jou, ik doe alles voor je! O+
Ik hou zoveel van jou, ik doe alles voor je! O+
en wat de twee heren boven mij zeiden
Op dinsdag 9 augustus 2011 23:01 schreef SuperrrTuxxx het volgende:
Ik hou zoveel van jou, ik doe alles voor je! O+
Ik hou zoveel van jou, ik doe alles voor je! O+
Niet uitvoerbaar. Als ik de code copy-paste vanaf pastebin, krijg ik de melding "This archive is broken, get a good version". Dat blijkt dan over table structure, table data en file data te gaan.quote:
Wat vinden jullie van me scrippie?
http://pastebin.com/g2xTLLtP
dafuq is this?quote:
Wat vinden jullie van me scrippie?
http://pastebin.com/g2xTLLtP
Je moeder is een hamster
What did I just read ?quote:
Wat vinden jullie van me scrippie?
http://pastebin.com/g2xTLLtP
@Light: Klopt, dat de data onleesbaar is, moet zo zijn.
@Pakspul; bedankt voor je zeer nuttige reactie van 1 woord..
Juicyhil: Klopt, maar het script heeft ook niet als doel beschikbaar te zijn voor de 'hele' wereld... maar opbouwende tips kan ik altijd wat mee
@Light: Het klopt inderdaad dat het om bestands en mysql data te gaan, en daaromheen zit een installer verwerkt, het rare is alleen dat het bestand bij mij deze melding niet geeft en bij jullie wel...
@Crutch: Versleutelde 'data' met een installer er aan vast geknoopt.
@GI: Read above
Het is vreemd dat de data op jullie systemen als 'broken' wordt weergegeven terwijl hier de code uitgevoerd kan worden, snap niet geheel waar het probleem zit maar daar moet ik dus nog mee spelen..
@Pakspul; bedankt voor je zeer nuttige reactie van 1 woord..
Juicyhil: Klopt, maar het script heeft ook niet als doel beschikbaar te zijn voor de 'hele' wereld... maar opbouwende tips kan ik altijd wat mee
@Light: Het klopt inderdaad dat het om bestands en mysql data te gaan, en daaromheen zit een installer verwerkt, het rare is alleen dat het bestand bij mij deze melding niet geeft en bij jullie wel...
@Crutch: Versleutelde 'data' met een installer er aan vast geknoopt.
@GI: Read above
Het is vreemd dat de data op jullie systemen als 'broken' wordt weergegeven terwijl hier de code uitgevoerd kan worden, snap niet geheel waar het probleem zit maar daar moet ik dus nog mee spelen..
The people who lost my respect will never get a capital letter for their name again.
Like trump...
Like trump...
Alvast een tip dan:
Magic constants horen te worden geschreven met hoofdletters, dus __DIR__ en __FILE__.
Magic constants horen te worden geschreven met hoofdletters, dus __DIR__ en __FILE__.
Thanks, kleine vraag; jullie testen vast allemaal in een linux omgeving?
het probleem zit hem in PHP_EOL die in windows omgeving anders is dan in de linux omgeving....
[ Bericht 44% gewijzigd door Chandler op 14-11-2013 08:46:32 ]
het probleem zit hem in PHP_EOL die in windows omgeving anders is dan in de linux omgeving....
[ Bericht 44% gewijzigd door Chandler op 14-11-2013 08:46:32 ]
The people who lost my respect will never get a capital letter for their name again.
Like trump...
Like trump...
Op Debian werkt het niet.quote:
Thanks, kleine vraag; jullie testen vast allemaal in een linux omgeving?
het probleem zit hem in PHP_EOL die in windows omgeving anders is dan in de linux omgeving....
Op donderdag 2 juli 2009 22:41 schreef RTB het volgende:
als ik elk rap"liedje" een kans moest geven was ik aan het eind van dit millennium nog bezig met het tempo waarin die kotshoop uitgebraakt wordt.
👾
als ik elk rap"liedje" een kans moest geven was ik aan het eind van dit millennium nog bezig met het tempo waarin die kotshoop uitgebraakt wordt.
👾
Daar ging ik al vanuit, alles anders dan Windows servers zal deze archive als broken betitelen... komt door PHP_EOL in PHP die onder windows anders is dan linux .....quote:
The people who lost my respect will never get a capital letter for their name again.
Like trump...
Like trump...
PHP_EOL gedraagt zich op ieder platform hetzelfde, namelijk: geef de line-ending van dit platform.
Als je specifiek een windows line-ending nodig hebt, slaat het dus nergens op om PHP_EOL te gebruiken.
Als je specifiek een windows line-ending nodig hebt, slaat het dus nergens op om PHP_EOL te gebruiken.
KomtTijd; Klopt! daar ben ik dan nu ook achter gekomen, dus ga ik daar mee aan de slag!
The people who lost my respect will never get a capital letter for their name again.
Like trump...
Like trump...
Ok, reactie in één woord: MVCquote:
@Light: Klopt, dat de data onleesbaar is, moet zo zijn.
@Pakspul; bedankt voor je zeer nuttige reactie van 1 woord..
Juicyhil: Klopt, maar het script heeft ook niet als doel beschikbaar te zijn voor de 'hele' wereld... maar opbouwende tips kan ik altijd wat mee
@Light: Het klopt inderdaad dat het om bestands en mysql data te gaan, en daaromheen zit een installer verwerkt, het rare is alleen dat het bestand bij mij deze melding niet geeft en bij jullie wel...
@Crutch: Versleutelde 'data' met een installer er aan vast geknoopt.
@GI: Read above
Het is vreemd dat de data op jullie systemen als 'broken' wordt weergegeven terwijl hier de code uitgevoerd kan worden, snap niet geheel waar het probleem zit maar daar moet ik dus nog mee spelen..
Now you have two problems? Als in: hoe is MVC een oplossing voor de problemen van Chandler? :-)quote:
Tegenwoordig moet je Dr. Ir. zijn om een beetje correct Nederlands te kunnen neerpleuren.
Abusing semicolons since 1987.
Abusing semicolons since 1987.
Je mag toch op zijn minst wel je templates en je code gescheiden houden, niets problematisch aan hoor.quote:
[..]
Now you have two problems? Als in: hoe is MVC een oplossing voor de problemen van Chandler? :-)
Ik moest de cache legen in phpbb en daarna deed ie het..quote:
[..]
Heet de database ook hetzelfde?
Maar nu is mijn vraag waarom phpbb ./ voor de links zet voor smilies, ranks etc.
dus als mn smilie path is ingesteld als images/smilies (gewoon standaard dus)
dan maakt ie er ./images/smilies van waardoor de afbeelding niet word weergegeven.
??
Dat is opzich handig, aangezien je dan geen "harde" paden hoeft te gebruiken...quote:
[..]
Ik moest de cache legen in phpbb en daarna deed ie het..
Maar nu is mijn vraag waarom phpbb ./ voor de links zet voor smilies, ranks etc.
dus als mn smilie path is ingesteld als images/smilies (gewoon standaard dus)
dan maakt ie er ./images/smilies van waardoor de afbeelding niet word weergegeven.
??
-- ViPeRII --
Ja? slaat in mijn ogen compleet de plank misquote:
[..]
Now you have two problems? Als in: hoe is MVC een oplossing voor de problemen van Chandler? :-)
want het enige wat het script moet doen is steeds het zelfde maar dan voor andere 'data' pakketten. Maar goed, MVC is gewoon een geil woord dat je overal voor moet gebruiken... voor countertjes, voor simpele contact formulieren, voor alles eigenlijk 
Niet als het doel 1 bestand is, zonder includes! of wat voor externe bestanden dan ookquote:
Je mag toch op zijn minst wel je templates en je code gescheiden houden, niets problematisch aan hoor.
zelfs de afbeeldingen staan in CSS in base64 
ennuh op zich heb ik mijn html toch grotendeels gescheiden van php, alleen zit het wel verwerkt in het php bestand als 1 lap tekst...Maar goed, heb het probleem reeds 'denk ik' opgelost, nog even op een paar externe servers testen plus weer de nodige veranderingen (verbeteringen)...
Oh en kan natuurlijk altijd de nodige suggesties gebruiken, het kan altijd mooier, gemakkelijker of logischer
[ Bericht 4% gewijzigd door Chandler op 14-11-2013 20:25:41 ]
The people who lost my respect will never get a capital letter for their name again.
Like trump...
Like trump...
MVC is zo 2012. MVP, MVVM, noem het maar op!
Tegenwoordig moet je Dr. Ir. zijn om een beetje correct Nederlands te kunnen neerpleuren.
Abusing semicolons since 1987.
Abusing semicolons since 1987.
LOL wat is dan 2013? gewoon weer ouderwets?
Maaruh een update: http://pastebin.com/VtwXQHur voor diegene die het leuk vind om dingen te testen en opbouwend kritiek wil geven
Oh pass is password123
gewoon weer ouderwets? Maaruh een update: http://pastebin.com/VtwXQHur voor diegene die het leuk vind om dingen te testen en opbouwend kritiek wil geven
Oh pass is password123
The people who lost my respect will never get a capital letter for their name again.
Like trump...
Like trump...
T werkt, op je quit knop na. deze brengt alles terug naar de 1e stap van je script.quote:Op zaterdag 16 november 2013 11:39 schreef Chandler het volgende:
LOL wat is dan 2013?
Maaruh een update: http://pastebin.com/VtwXQHur voor diegene die het leuk vind om dingen te testen en opbouwend kritiek wil geven
Oh pass is password123
Vervolgens zou het mooi zijn als je je script ook op andere "gebruikelijke" database systemen zou kunnen installeren, welke makkelijk te scripten zijn via PDO; Dus: mssql, postgres en sqlite, om de gebruiker wat meer opties te geven.
Daarnaast wil ik je ten zeerste aanraden het script netjes te maken, oftewel alles netjes in OOP schrijven in plaats van de procedural code die je nu geschreven hebt. Dat lijkt nu onzin, maar naar mate dat je script groeit, heb je daar altijd wat aan.
Ook dien je je unsets te testen om te kijken of ze wel dan niet geset zijn, je weet maar nooit wat er in je script gebeurd...Dat ben je bijvoorbeeld vergeten bij stap 7;
Ook heb ik nog de vraag waarom je de set_time_limit gebruikt..
Voor het mooie zou ik ook het aantal stappen wat proberen te verminderen.
-- ViPeRII --
Je gebruikt twee keer "case 7:". Dat werkt niet, iedere case moet uniek zijn.quote:
Maaruh een update: http://pastebin.com/VtwXQHur voor diegene die het leuk vind om dingen te testen en opbouwend kritiek wil geven
En $_SESSION unsetten is ook niet netjes. Da's een PHP superglobal, die moet je niet willen unsetten. Je kunt er wel een andere waarde aan geven, bijvoorbeeld null.
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?
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?
The people who lost my respect will never get a capital letter for their name again.
Like trump...
Like trump...
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:
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?
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.
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:
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.
verder nog grote php fouten gezien? nee toch?
The people who lost my respect will never get a capital letter for their name again.
Like trump...
Like trump...
