Betalen via de telefoon (Targetpay)

- Regel 37: Variabelen horen buiten quotes
- Vanaf regel 59: Hoezo gebruik je geen switch() ?
- Waarom niet direct hier: https://www.targetpay.com/signup aanmelden?
- Controleer met if($_SERVER['REQUEST_METHOD']=="POST") {
of er ge-POST is.

@Aar

waarom hij niet een link heeft direct naar het signup gedeelte (jou link dus), zie je wel op deze pagina. Denk alleen niet dat het veel voordeel voor hem zal hebben :P

Ik zou die reacties met een switch doen...

cURL ipv file_get_contents, is veel sneller.

Goed, waar is de prullenbak?
Wat moet dit voorstellen?
Je doet wel style en div troep, als je dat doet, doe dan ook de rest van een pagina d'r bij.
Elke regel een echo, wat een tik werk, en je kunt het net zo goed in één echo gooien.
En misschien eerst nog even verdiepen in API's?

Waarom moeten wij aanmelden met een reference naar jouw? Die link slaat nergens op net zoals dit script.

Oh, en als je boehoe (klagen dat ik niet vriendelijk ben) gaat doen, dat zal mij een worst wezen. Als je een script hier plaatst dan moet het enig kwaliteit hebben. Dit script heeft dat niet. Ook ben je misschien nog een beginner, dan moet je ook geen boehoe doen dat je afgekraakt wordt. In de categorie beginner worden scripts geplaatst voor beginners en niet door beginners.
En als je wilt laten zien wat je kan, plaats dan een topic in het forum, ga niet de scriptlib. vervuilen met nog meer rotzooi.

@ Aar:
Bedankt voor je opmerkingen! Ik hoop ze nu verbeterd te hebben!

@ Geert:
Klopt. Wanneer je mensen via jouw layoutcode laat aanmelden kun je per transactie die de account van de ander maakt €0,01 verdienen. Vandaar! Ookal levert het voor mij weinig op zo, dat klopt!

@ Karl:
Zover ik weet blijft dit phpHULP!! En is er dus niets mis met 'slechte' scripts. Ik vind het gebruiken van echo voor iedere regel minder chaotisch in mijn scripts, en wanneer het jou niet aanstaat is het voor jou net zo makkelijk als knippen en plakken om ze eruit te halen. Verder staat de style er in voor o.a. de errors. En ook die kun jij er toch zelf uitslopen als je zelf alles zo goed weet ;-). Verder wil ik niet ingaan op je kritiek. Nog 1 keer dan: Kopieren en plakken en dan bij 2 bijna de zelfde scripts posten is wel logisch?

Bedankt in ieder geval, iedereen die me wel willen helpen!

Het argument past niet bij de stelling. Het klopt dat dit phphulp is. De bedoeling is dus dat je op het forum vragen kunt stellen over hoe je iets moet doen o.i.d. Daar is de hulp. De hulp die via scripts wordt geboden is kant en klare scripts. Daarom moeten die scripts wel kwalitatief goed zijn. Dit gedeelte is er niet voor jou om hulp te krijgen, maar om hulp te geven.
Als een script niet goed is, vind ik dat mensen dat moeten weten. Dit script is weliswaar niet hartstikke gevaarlijk (sql injections o.i.d.), toch kan het beter.
En i.p.v. dat je je script gaat verdedigen dat het wel goed is, kan je het ook gewoon beter maken. Dan vervalt mijn kritiek toch?

Mij lijkt het juist dat het daardoor veel chaotischer wordt, er is veel minder overzicht omdat elke regel hetzelfde begint. Het is eigenlijk ook niet de bedoeling dat er op elke regel een echo is. Voor de parser is het ook beter als je het gewoon in eentje stopt.

Doe er dan tenminste ook de rest van de html bij. Zoveel is dat niet.

Waarom moeten dit twee verschillende scripts zijn? Lijkt me juist dat het veel mooier is als het in een script is.

Kijk karl, dit is een beter reactie. Je reactie in het sms-topic vind ik gewoon echt te ver gaan.

@Karl
Ik heb het script nu aangepast. En hoop dat het nu wel een voldoende scoort. ik zal het andere ook nog aanpassen.
Ik heb trouwens expres gekozen om de scripts te scheiden aangezien de kosten voor betalingsverkeer via sms vaak duurder zijn dan via telefoon. En ja, dat kan je ook allemaal scheiden en dat het aan of uit gezet kan worden, maar ik heb voor deze manier gekozen.
Overigens heb ik ook al een intern bericht gehad van iemand die al een hele tijd op zoek was naar zo'n script en er nu dus heel blij mee was hem eindelijk te hebben gevonden. En dat jij het goed genoeg voor de prullenbak vind wil niet zeggen dat iedereen dat vind.

"En is er dus niets mis met 'slechte' scripts."

Eeh, jawel..

Door slechte scripts leer je beginners alleen maar foute technieken.

Waarom heet dit 'Pincode per telefoon' en niet gewoon 'Betalen via de telefoon'? Het gaat toch om een betaalmogelijkheid?

Ik had een vraagje, Dit werkt prima als je bijvoorbeeld een betaling van 1,30 wilt doen, Hoe doe je het als je product nou bijv 3,90 is? (3x bellen) hoe check je dat?

Neem aan dat de code drie keer moet worden ingevoerd. En dat TargetPay zelf wel het signaal geeft dat er voldoende keer gebeld is.

ik zou niet weten hoe dat zit aangezien er enkel staat vermeld over 1x bellen op de website en in het script.

ik zou niet weten hoe dat zit aangezien er enkel staat vermeld over 1x bellen op de website en in het script.