.JSON file Twitter script

[Rowinio]

Op het internet vond ik het volgende script:

Demo: http://www.realiseweb.nl/examples/twitter/demo.php
Script: http://www.realiseweb.nl/examples/twitter/script.txt

Nu is het zo dat het PHP bestand wilt opslaan in een .JSON file. Hoe gaat dit in zijn werk, want heb een leeg bestand aangemaakt genaamd twitter.json en heb dit geupload naar de server (en op rechten 777 gezet). Hij geeft dan de volgende foutmelding aan:

Warning: fread() [function.fread]: Length parameter must be greater than 0 in /home/debxxxxx/domains/xxxx.xx/public_html/wp-content/themes/xxx/testertw.php on line 76

Warning: Invalid argument supplied for foreach() in /home/debxxxx/domains/xxxx.xx/public_html/wp-content/themes/xxx/testertw.php on line 93

Wat te doen om dit op te lossen?

[Anoniem: 26306]

Betere foutafhandeling inbouwen. Er wordt niet gecontroleerd of de fopen call wel gelukt is.

[CodeCaster]

Debuggen. Blijkbaar is je length-parameter 0, dus dan kun je het bestand niet lezen.

[Big4SMK]

Wat ik me voor kan stellen is dat je de file niet zelf hoeft te maken. Immers, jij hebt nu een lege file gemaakt met dus een grootte van 0. Verwijder de file eens en probeer het opnieuw

of bouw een check in dat als de lengte 0 is dat ie geen fread doet...immers zal hij toch geen tweets krijgen bij lengte 0

[Rowinio]

Big4SMK schreef op zaterdag 11 juni 2011 @ 13:29:
Wat ik me voor kan stellen is dat je de file niet zelf hoeft te maken. Immers, jij hebt nu een lege file gemaakt met dus een grootte van 0. Verwijder de file eens en probeer het opnieuw

of bouw een check in dat als de lengte 0 is dat ie geen fread doet...immers zal hij toch geen tweets krijgen bij lengte 0

Inderdaad, heb het bestand verwijderd en de volgende code komt nu tevoorschijn:

Warning: file_put_contents(/home/xxxx.xx/domains/xxxx.xx/public_html/wp-content/themes/xx/twitter.json) [function.file-put-contents]: failed to open stream: Permission denied in /home/xxxx.xx/domains/xxx.xx/public_html/wp-content/themes/xx/testertw.php on line 86

Ik zie immers wel een tweet staan!

[Voutloos]

Script houdt inderdaad geen rekening met een lege cache file, en sowieso mist elke vorm van foutafhandeling. Daarnaast is de hele stijl van de code om te janken, dus je moet het eigenlijk vooral als proof of concept gebruiken en verder uitwerken.

De fout die je nu hebt betekent dat hij geen file mag aanmaken. Dus fix de dir rechten, of maak zelf de file aan en forceer de regels die hij uitvoert zonder de cache te lezen.

[Voor 19% gewijzigd door Voutloos op 11-06-2011 13:34]

[NMe]

Rowin schreef op zaterdag 11 juni 2011 @ 13:31:
[...]

Ik zie immers wel een tweet staan!

Ja, omdat hij nu geen cachefile aan kan maken haalt 'ie het gewoon live op... Misschien moet je je eens verdiepen in wat het script doet in plaats van het blind over te nemen en te hopen dat het werkt?

[Herko_ter_Horst]

Met andere woorden: snap wat het script doet en pas aan wat voor jou niet werkt. Als je het script niet snapt, kun je beter de maker om hulp vragen (of meer PHP-kennis opdoen).

[Big4SMK]

Dat lijken me de beste tips, ik gok echter dat je van je bootstrap probleem af bent als je je twitter.json file aanmaakt, rechten op 777 zet en als content twee blokhaken [] neerzet. Dan heb je effectief een lege lijst als json opgeslagen.

[Rowinio]

Opgelost, heb de regels die naar cache verwijzen verwijderd, en het werkt! Alsnog bedankt heren (en dames?)!

[Voor 94% gewijzigd door Rowinio op 11-06-2011 13:43]

[Big4SMK]

Dat is natuurlijk ook een oplossing. Verder zou je nog wel wat meer kunnen verwijderen als je toch geen caching wilt gebruiken. De hele read functie kan weg, immers...als het bestand niet bestaat zal die nooit aangeroepen worden.

Hou er wel rekening mee dat nu voor _elke_ hit op je pagina, een hit gemaakt wordt naar twitter. Als er weinig hits zijn boeit dat niet zo veel, je cache verliep toch na een uur, maar zeker als je site groter is/wordt is dat niet de meest aangename manier.

Ik zou me als ik jou was daarom alsnog wat verdiepen in php, en zorgen dat (een opgepoetste versie van) het script met caching gaat werken.

[Voutloos]

Dat is geen oplossing. Caching van dit soort externe requests is eigenlijk gewoon verplicht, zeker gezien de uptime/api limits van Twitter. Die API limit kan je ook merken als je jouw server deelt met anderen. Als er dan iemand lekker geen caching gebruikt, is dat eigenlijk behoorlijk asociaal, omdat Twitter dan ook de calls van de anderen zal limiteren.

Hoe je het met cache aan de praat krijgt had ik al gezegd, en de methode van Big4SMK zal ook werken.

[Voor 30% gewijzigd door Voutloos op 11-06-2011 13:54]

[Rowinio]

Voutloos schreef op zaterdag 11 juni 2011 @ 13:53:
Dat is geen oplossing. Caching van dit soort externe requests is eigenlijk gewoon verplicht, zeker gezien de uptime/api limits van Twitter. Die API limit kan je ook merken als je jouw server deelt met anderen. Als er dan iemand lekker geen caching gebruikt, is dat eigenlijk behoorlijk asociaal, omdat Twitter dan ook de calls van de anderen zal limiteren.

Hoe je het met cache aan de praat krijgt had ik al gezegd, en de methode van Big4SMK zal ook werken.

De methode werkt niet, krijg ik alsnog een foutcode. Dat heb ik als eerste instantie geprobeerd.

[CodeCaster]

Dan ga je dus debuggen en foutafhandelingscode toevoegen.

[Voutloos]

Rowin schreef op zaterdag 11 juni 2011 @ 14:02:
[...]
De methode werkt niet, krijg ik alsnog een foutcode. Dat heb ik als eerste instantie geprobeerd.

Moet ik het echt voorkauwen?

Voutloos schreef op zaterdag 11 juni 2011 @ 13:33:
of maak zelf de file aan en forceer de regels die hij uitvoert zonder de cache te lezen.

Dus:
1. maak het bestaan aan, en maak het schrijfbaar.
2. zet tijdelijk een return false bovenin cache()
3. voer uit
4. haal de return false weg.

[NMe]

Rowin schreef op zaterdag 11 juni 2011 @ 14:02:
[...]


De methode werkt niet, krijg ik alsnog een foutcode. Dat heb ik als eerste instantie geprobeerd.

Tsja, als je niet wil begrijpen wat het script doet en vervolgens wil debuggen wat er fout gaat, dan houdt het op. Maar dan hoef je dit topic ook niet meer te hebben, want blijkbaar interesseert het je niet dat je hier problemen mee gaat krijgen en wil je het ook niet zelf oplossen.

Dan doen we dit topic maar dicht.