[PHP] Echo probleem

[DeepFreeze.NL]

Momenteel heb ik deze code met hulp van google in elkaar gezet:
Dit zorgt ervoor dat nieuws vanuit een txt file wordt gelezen en dan tot regel 10 op het scherm wordt getoond. Daaronder komt dan een "Archive" link.

Wanneer er op de archive link geklikt wordt moeten alle regels uit news.txt getoond worden.
Ik heb al heel veel geprobeerd en op google gezocht, maar ik kom er niet echt meer uit?
Wie kan mij helpen?

PHP:

<?php
$fileName = file ("news.txt");
$rows = count ($fileName);
$maxRows = 10;

//Als het aantal regels meer is dan $maxRows:
if ($rows > $maxRows) { 

    for ($i = 0; $i < $maxRows; $i++) {
            echo '<br>'.$fileName[$i]; 
        }
    echo '<br><a href="'.$fileName.'">Archive</a>';

//Als het aantal regels minder is dan $maxRows:
} else {
    for ($i = 0; $i < $rows; $i++) {
            echo '<br>'.$fileName[$i];
        }
}

?>

Test voorbeeld is hier te zien: http://deepfreeze.stx.nl/shoutcast/index.php

[ Voor 99% gewijzigd door DeepFreeze.NL op 06-06-2006 18:20 ]

[Verwijderd]

a) je gebruikt $row en $rows doorelkaar

hier zit denk ik de fout

[EnnaN]

Dan kun je toch gewoon een fucntie aanroepen die de file opent, en dan iets doet van

while (filehandle){
echo filehanddle
}

dat weet google ook nog wel te vinden, lijkt me een stuk makkelijker dan niet de hele, maar een gedeelte van een file te printen

[enveekaa]

Laat maar

[ Voor 141% gewijzigd door enveekaa op 06-06-2006 16:53 ]

[DeepFreeze.NL]

Verwijderd schreef op dinsdag 06 juni 2006 @ 16:51:
a) je gebruikt $row en $rows doorelkaar

hier zit denk ik de fout

De regels worden wel gewoon getoond op het scherm met onderaan de archive link. Alleen de link van archive dat alles getoond wordt (echo $filename) werkt niet.

[JHS]

File() retourneerd de inhoud van het bestand dan ook in een array . Anders zou je er ook niet overheen kunnen loopen in je vorige functie. Je zal dus of door de count heen moeten loopen, of file_get_contents() gebruiken, welke een string teruggeeft.

[Voutloos]

DeepFreeze.NL schreef op dinsdag 06 juni 2006 @ 16:52:
[...]
De regels worden wel gewoon getoond op het scherm met onderaan de archive link. Alleen de link van archive dat alles getoond wordt (echo $filename) werkt niet.

PHP:

     echo '<br><a href="'.$filename.'">Archive</a>';

Vergelijk de kleurtjes in deze code met jouw code en de fout moge duidelijk zijn.

Overigens zou ik niet linken naar het bestand zelf, maar met GET parameters werken en zo je script aanroepen. Verder is de hele $row variabele overbodig, want deze is altijd 0 en verandert nooit. En 0 optellen/aftrekken is nogal useless.

Verder indenteer je onlogisch en zijn de 2 regels commentaar totaal onnuttig. Je gebruikt $maxRows zodat je geen magisch getal hebt midden in je script, en vervolgens zet je dat magische getal wel in je commentaar. Verder is een simpele boolean vergelijking toelichten een beetje onnodig.

[ Voor 60% gewijzigd door Voutloos op 06-06-2006 17:01 ]

[NMe]

Vind je het gek dat die link niet werkt? Je gebruikt <PHP> tags in plaats van <?php> tags, en bovendien kun je die tags binnen PHP niet eens gebruiken. Ik kan me dan ook niet echt aan de indruk onttrekken dat je geen idee hebt wat je aan het doen bent. Probeer eens wat tutorials door te nemen, want daar is iets als dit best in te vinden.

[Michali]

Nog iets. Je gebruikt zowel $fileName en $filename, en php is hoofdletter gevoelig.

[DeepFreeze.NL]

Overigens zou ik niet linken naar het bestand zelf, maar met GET parameters werken en zo je script aanroepen. Verder is de hele $row variabele overbodig, want deze is altijd 0 en verandert nooit. En 0 optellen/aftrekken is nogal useless.

Aangepast

Verder indenteer je onlogisch en zijn de 2 regels commentaar totaal onnuttig. Je gebruikt $maxRows zodat je geen magisch getal hebt midden in je script, en vervolgens zet je dat magische getal wel in je commentaar. Verder is een simpele boolean vergelijking toelichten een beetje onnodig.

Ook aangepast en die simpele boolean vergelijking is voor mij nog wel handig haha.

[Voutloos]

Overigens leest het doorgaans prettiger als je het consequent print of echo gebruikt. Bovendien kan je gewoon

PHP:

    echo $fileName[$i].'<br>';

doen.

Verder is de check op regel 14 altijd true, aangezien die conditie hetzelfde is als degene die op regel 7 al true was.

Maar goed, ik denk net als -NMe- dat het wellicht handig is om je iets meer in te lezen.

[DeepFreeze.NL]

Voutloos schreef op dinsdag 06 juni 2006 @ 17:19:
Overigens leest het doorgaans prettiger als je het consequent print of echo gebruikt. Bovendien kan je gewoon

PHP:

    echo $fileName[$i].'<br>';

doen.

Verder is de check op regel 14 altijd true, aangezien die conditie hetzelfde is als degene die op regel 7 al true was.

Maar goed, ik denk net als -NMe- dat het wellicht handig is om je iets meer in te lezen.

Ok, dankje voor je tips! Heb het ondertussen al aangepast...

[Michali]

Dit kan nog wel wat korter:

PHP:

$fileName = file("news.txt");
$rows = count ($fileName);
$maxRows = 10;
$numRowsToPrint= $rows > $maxRows ? $maxRows : $rows;
for ($i = 0; $i < $numRowsToPrint; $i++) {
    echo '<br>'.$fileName[$i]; 
}
if ($rows > $maxRows) { 
    echo '<br><a href="'.$fileName.'">Archive</a>';
}

Zulk commentaar is overigens vrij overbodig, die kun je net zo goed weglaten. Beter is om je intentie uit te leggen, niet hoe je het realiseert.

[CH4OS]

Michali schreef op dinsdag 06 juni 2006 @ 16:57:
Nog iets. Je gebruikt zowel $fileName en $filename, en php is hoofdletter gevoelig.

Hoe kom je daar nou bij? Heb dat nog nooit gemerkt eigenlijk, ook niet in eigen gemaakte functies...

[Voutloos]

In het algemeen zou die eigenschap van een programmeertaal irrelevant moeten zijn. Voor consistentie en duidelijkheid van je code zou je altijd elke variabele maar op 1 manier moeten schrijven en variabelenamen voldoende van elkaar laten verschillen. Als je je consequent aan een bepaalde codingstyle/namingstyle houd, moet dit ook als vanzelf gaan.

[Michali]

GJ-tje schreef op woensdag 07 juni 2006 @ 17:29:
[...]
Hoe kom je daar nou bij? Heb dat nog nooit gemerkt eigenlijk, ook niet in eigen gemaakte functies...

code:

<?php

error_reporting(E_ALL);
$var = "123";
print $Var;

?>

Output:

Notice: Undefined variable: Var in D:\webroot\test.php on line 5

De code in de startpost is inmiddels gewijzigd, daar werd ie wel op 2 verschillende manieren gebruikt.

[Grijze Vos]

Michali schreef op dinsdag 06 juni 2006 @ 18:31:
..
$numRowsToPrint= $rows > $maxRows ? $maxRows : $rows;
for ($i = 0; $i < $numRowsToPrint; $i++) {
..
[/php]

Waarom niet

PHP:

 for ($i = 0; $i < min($numRowsToPrint, $rows); $i++) {

?

[Michali]

Performance issue (niet dat het erg veel uitmaakt) en complexiteit van de statement. Maar als je dat fijner vindt, dan moet je dat gewoon doen.

[CH4OS]

Michali schreef op woensdag 07 juni 2006 @ 17:36:

PHP:

<?php

error_reporting(E_ALL);
$var = "123";
print $Var;

?>

Output:

[...]


De code in de startpost is inmiddels gewijzigd, daar werd ie wel op 2 verschillende manieren gebruikt.

Omdat je error_reporting op E_ALL hebt staan gaat ie idd over de zeik vanwege de hoofdletters, maar is anders niet... Met E_ALL word er stricter op de syntax gelet...

[ Voor 3% gewijzigd door CH4OS op 07-06-2006 17:53 ]

[Michali]

Andere error_reporting modes zijn alleen voor live en productie situaties, als je geen errors in de pagina wilt. Bij ontwikkelen moet je hem altijd op de zo zeikerig mogelijke mode zetten. Dit is gewoon fout. Geen onherstelbare error, vandaar een notice, maar wel gewoon fout.

In PHP5 stel ik hem altijd zo in trouwens:

PHP:

error_reporting(E_ALL | E_STRICT);

[ Voor 15% gewijzigd door Michali op 07-06-2006 17:52 ]

[Rowanov]

GJ-tje schreef op woensdag 07 juni 2006 @ 17:45:
Omdat je error_reporting op E_ALL hebt staan gaat ie idd over de zeik vanwege de hoofdletters, maar is anders niet... Met E_ALL word er stricter op de syntax gelet...

Dan is het nog wel fout

[ Voor 0% gewijzigd door Rowanov op 07-06-2006 17:56 . Reden: langzaam en zo ]

[NMe]

GJ-tje schreef op woensdag 07 juni 2006 @ 17:29:
[...]
Hoe kom je daar nou bij? Heb dat nog nooit gemerkt eigenlijk, ook niet in eigen gemaakte functies...

PHP is dan ook alleen hoofdlettergevoelig bij variabelename (en ook bij klassenamen geloof ik), en niet bij functienamen. Lekker consistent ja, maar het is nou eenmaal zo. Daarnaast is het niet krijgen van een notice niet het zelfde als het goed zijn van je code. Zoals de mensen hierboven al zeggen: je moet altijd foutloze code zien op te leveren op het meest stricte niveau van de taal, en anders ben je IMO gewoon fout bezig. PHP is al inconsistent genoeg van zichzelf als je je wèl strict aan de taal houdt; daar hoef je verder niet aan mee te helpen.

[DeepFreeze.NL]

Inmiddels heb ik, met hulp van iemand anders, het script werkende gekregen. Mocht je het willen zien: http://music.deepfreeze.nl Mocht je de source willen hebben dan hoor ik het wel.

[Javache]

Count() telt toch het aantal karakters en niet het aantal regels ?

[DeepFreeze.NL]

Javache schreef op woensdag 07 juni 2006 @ 21:25:
Count() telt toch het aantal karakters en niet het aantal regels ?

Count telt het aantal regels.

[Javache]

Ik gebruik het altijd bij array's maar php.net zegt

quote: http://be.php.net/count

int count ( mixed var [, int mode] )

count -- Count elements in an array, or properties in an object

Returns the number of elements in var, which is typically an array, since anything else will have one element.

Misschien was dit de reden dat je script niet werkte ?

EDIT : Srry laat maar :

quote: http://be.php.net/file

file -- Reads entire file into an array

array file ( string filename [, int use_include_path [, resource context]] )

Identical to file_get_contents(), except that file() returns the file in an array. Each element of the array corresponds to a line in the file, with the newline still attached. Upon failure, file() returns FALSE.

[ Voor 46% gewijzigd door Javache op 07-06-2006 21:35 ]

[FTPlus]

Count telt het aantal elementen in een array. File() maakt van elke regel een nieuw element dus daarmee krijg je het aantal regels.