Error in nieuwsberichten

**Kees Leurink** · 11-08-2009, 22:41

Beste SD'ers,

Vanavond was ik bezig met het updaten van mijn website. Zo wou ik een kleine upgrade invoeren in het nieuwssysteem. Niks mis mee. Uiteindelijk bleek het niet helemaal te lukken zoals ik wilde, ik kreeg een error die ik zodra even vermeld, dus drukte ik een aantal keer op CTRL + Z (ongedaan maken) waardoor ik weer de code had, die ik enkele minuten geleden ook had. Tegelijk deletete ik de kolom die ik had aangemaakt in phpMyAdmin en alles is weer het zelfde als daarvoor. Echter, de error bleef.

Op http://www.eaglesfans.nl/index2.php kunnen jullie zien dat er 5 errors staan. Voor elk nieuwsbericht 1. Ik heb al met een back-up-bestand alles geprobeert te herstellen, maar de errors blijven. Ik heb geen idee waar de fout ligt.

De code kunnen jullie in de volledigheid vinden op www.eaglesfans.nl/code.txt. De regel waar volgens de error de fout ligt heb ik dikgedrukt gemaakt.

De error is dus:

Code:

Warning:  strpos() [function.strpos]: Offset not contained in string. in ........... on line 150

Ik hoop dat iemand mij kan helpen.

Alvast bedankt!

Mvg,
Kees Leurink

**marc h** · 11-08-2009, 22:50

Kun je de code laten zien van regels 140 tm 155
dat is wat makkelijker

**Kees Leurink** · 11-08-2009, 22:54

Origineel gepost door marc h

Kun je de code laten zien van regels 140 tm 155
dat is wat makkelijker

Hmm.. het is één grote while waar hij deel uitmaakt, dus helemaal precies lukt het niet, maar hier een kleinere versie (r 148 t/m 163):

PHP Code:


while($bericht = mysql_fetch_array($nieuws))
    {
        $wanneer = strpos($bericht['nieuws']," ",310); 
        
        $reactie = mysql_query("SELECT id, naam, email, tijd, datum, bericht, nieuws_id FROM reacties WHERE nieuws_id ='".$bericht['id']."'");
        $aantal = mysql_num_rows($reactie); 
        
        echo "
        <table>
        <tr>
        <td rowspan='2'><img src='nieuws/upload/". $bericht['plaatje'] ."' width='100' height='100'></td>
        <td rowspan='3'>&nbsp;</td>
        <td colspan='2'><span style='font-size: large;'>". $bericht['titel'] ."</span>
        </td>
        <td>
        ";

...............

**Ivo van B.** · 11-08-2009, 23:02

De offset 310 bestaat niet binnen die string $bericht['nieuws'].

Verwijder $wanneer = strpos($bericht['nieuws']," ",310); en verander substr($bericht['nieuws'],0,$wanneer) in substr($bericht['nieuws'], 0, 310)

Je script is trouwens zo lek als een mandje...

**Kees Leurink** · 11-08-2009, 23:30

Origineel gepost door Ivo van B.

De offset 310 bestaat niet binnen die string $bericht['nieuws'].

Verwijder $wanneer = strpos($bericht['nieuws']," ",310); en verander substr($bericht['nieuws'],0,$wanneer) in substr($bericht['nieuws'], 0, 310)

Je script is trouwens zo lek als een mandje...

Ik heb je raad opgevolgd, maar het werkt helaas niet.

Over dat laatste ben ik het trouwens roerend met je eens. Het was een van mijn (met hulp van een ander) eerste, zoniet mijn eerste serieuze script (dus met database e.d.) Daarom heb ik nu ook zoveel problemen om alle probleempjes eruit te halen, want ik begrijp er niet altijd van wat ik destijds heb gedaan.

Maar om weer even on-topic te gaan. Heeft iemand een idee waarom het 'opeens' gebeurt dat ik een error krijg? Ik vind het vrij apart.

**Erik Bruinsma** · 11-08-2009, 23:40

Wat is eigenlijk het doel van jou strpos in dit script?

**Kees Leurink** · 11-08-2009, 23:42

Origineel gepost door Erik Bruinsma

Wat is eigenlijk het doel van jou strpos in dit script?

In de database staat een lang nieuwsbericht. Deze afkorten tot 310 tekens met daarachter de tekst LEES MEER. Als je daarop klikt krijg je het volledige nieuwsbericht met de mogelijkheid tot reageren daaronder e.d.

**Erik Bruinsma** · 11-08-2009, 23:50

Waarom staan er tekens om php?

$wanneer = strpos($bericht['nieuws']," ",310);

**Kees Leurink** · 11-08-2009, 23:51

Origineel gepost door Erik Bruinsma

Waarom staan er tekens om php?

$wanneer = strpos($bericht['nieuws']," ",310);

Was om duidelijk te maken dat het die regel was waar de fout in zat, ik dacht dat ik hem inmiddels had verwijderd.. Niet dus ;-)

**Erik Bruinsma** · 11-08-2009, 23:52

Kan het zijn dat jou berichten geen 310 tekens lang zijn?

**Kees Leurink** · 11-08-2009, 23:54

Origineel gepost door Erik Bruinsma

Kan het zijn dat jou berichten geen 310 tekens lang zijn?

Nee, ze zijn echt langer..

**Ivo van B.** · 12-08-2009, 09:03

Origineel gepost door Kees Leurink

Ik heb je raad opgevolgd, maar het werkt helaas niet.

Over dat laatste ben ik het trouwens roerend met je eens. Het was een van mijn (met hulp van een ander) eerste, zoniet mijn eerste serieuze script (dus met database e.d.) Daarom heb ik nu ook zoveel problemen om alle probleempjes eruit te halen, want ik begrijp er niet altijd van wat ik destijds heb gedaan.

Maar om weer even on-topic te gaan. Heeft iemand een idee waarom het 'opeens' gebeurt dat ik een error krijg? Ik vind het vrij apart.

Wat werkt er precies niet als je mijn raad opvolgt?

**Jacob Ras** · 12-08-2009, 09:38

Volgens mij kan werkt het op deze manie sowieso verkeerd.

PHP Code:


$wanneer = strpos($bericht['nieuws']," ",310);

Strpos is om de positie van de eerste verschijning van de needle (in dit geval de spatie: " ") op te zoeken. Als je erachter een getal zet, slaat hij dat aantal over voordat de functie de positie teruggeeft.

In dit geval zoekt de functie dus naar de 311e verschijning van de spatie (de eerste 310 worden overgeslagen) in het stukje, en ik kan me voorstellen dat die er niet is.

Je kunt toch gewoon gebruik maken van substr() ? Dus zo:

PHP Code:


$wanneer = substr($bericht['nieuws'], 0, 310);

Dan worden de eerste 310 karakters van $bericht['nieuws'] weergegeven.
Aanvullend bericht:
Ik denk trouwens dat je probeert wil dat het script na 310 karakters de eerste spatie opzoekt, en de positie van die spatie weergeeft. Vervolgens wil je dat het stukje tekst daar wordt afgekapt. Maar nu zoekt het script dus naar de 311e spatie in het stukje.

**Kees Leurink** · 12-08-2009, 09:46

Origineel gepost door Jacob Ras

Volgens mij kan werkt het op deze manie sowieso verkeerd.

PHP Code:


$wanneer = strpos($bericht['nieuws']," ",310);

Strpos is om de positie van de eerste verschijning van de needle (in dit geval de spatie: " ") op te zoeken. Als je erachter een getal zet, slaat hij dat aantal over voordat de functie de positie teruggeeft.

In dit geval zoekt de functie dus naar de 311e verschijning van de spatie (de eerste 310 worden overgeslagen) in het stukje, en ik kan me voorstellen dat die er niet is.

Je kunt toch gewoon gebruik maken van substr() ? Dus zo:

PHP Code:


$wanneer = substr($bericht['nieuws'], 0, 310);

Dan worden de eerste 310 karakters van $bericht['nieuws'] weergegeven.
Aanvullend bericht:
Ik denk trouwens dat je probeert wil dat het script na 310 karakters de eerste spatie opzoekt, en de positie van die spatie weergeeft. Vervolgens wil je dat het stukje tekst daar wordt afgekapt. Maar nu zoekt het script dus naar de 311e spatie in het stukje.

Klopt inderdaad, 't is alweer een tijdje terug dat ik dit (i.c.m. met iemand anders) heb gemaakt, dus moet goed terugdenken en begrijp niet alles (meer)

Maar om terug te komen op wat je probeerde uit te leggen. Ik heb eventjes gekeken, maar op regel 189 staat ook nog:

PHP Code:


". stripslashes(substr($bericht['nieuws'],0,$wanneer)) ."

Ik weet niet of je die ook hebt gezien?

Origineel gepost door Jacob Ras

Volgens mij kan werkt het op deze manie sowieso verkeerd.

Het heeft toch bijna een jaar op deze manier gewerkt..

**Jacob Ras** · 12-08-2009, 09:48

Ik weet niet of je die ook hebt gezien?

Die had ik inderdaad gezien. Maar het script geeft dmv de substring() de karakters 0 tot $wanneer aan. Echter zoekt $wanneer naar de 311e spatie in het stukje. Al denk ik dat ik iets over het hoofd zie... Zal nog even verder kijken in de code.

Error in nieuwsberichten