PDA

Bekijk Volledige Versie : Editscript met fouten



Bertez[z]
08/07/03, 21:50
Ik heb inmiddels een scriptje van iemand gekregen, maar ik ben bang dat ie
vol fouten zit.
Krijg parseerrors...
Het gaat dus om een nieuwsscriptje.
Lap tekst... Dat je kan veranderen.

Wie kan me even helpen dit te debuggen?
Alvast Hartelijk Dank!


----Pagina.php-----
<?php include ("ding.txt"); ?>
<a href="geefop.php">Klik hier om het nieuws te veranderen.</a>

----ding.txt------
(leeg) (chmod 664)

----geefop.php------
<?php
if (!empty($_POST)) {
$bericht = nl2br($bericht);
$bestand = "ding.txt";
$inhoud_bestand = file($bestand);
$schrijf_bestand = fopen($bestand,"w");
fwrite ($schrijf_bestand, "_POST['bericht']"
fwrite ($schrijf_bestand, join('', $inhoud_bestand));
fclose ($schrijf_bestand);
echo "<script language=\"javascript\">setTimeout(\"window.location.href =
'Index.php'\", 1000);</script>";
}
else {
?>
<form action="<?php echo ($_SERVER['PHP_SELF']) ?>" method="post">
<textarea name="iets" value="hier je ding" onClick="if (this.value=='')
this.value=''" name="bericht">
</textarea>
<input type="submit">
<?php
}
?>



--
Greetz,
Bertez[z]
-
Mail:
Bertweb @ Home . NederLand

Rick
08/07/03, 22:05
"Bertez[z]" <bertweb@@home.nl> schreef in bericht
news:bef77q$qik$1@news4.tilbu1.nb.home.nl...
> Ik heb inmiddels een scriptje van iemand gekregen, maar ik ben bang dat ie
> vol fouten zit.
> Krijg parseerrors...
> Het gaat dus om een nieuwsscriptje.
> Lap tekst... Dat je kan veranderen.
>
> Wie kan me even helpen dit te debuggen?
> Alvast Hartelijk Dank!
Welke errors krijg je en op welke regels? Wat heb je al geprobeerd?
--
Slechte webdesigners? Geef ze hier aan:
http://webdesignert.vectoris.nl/

Bertez[z]
08/07/03, 22:20
"Rick" <spammans@chello.nl> schreef in bericht
news:bef7qt$17d3$1@nl-news.euro.net
> "Bertez[z]" <bertweb@@home.nl> schreef in bericht
> news:bef77q$qik$1@news4.tilbu1.nb.home.nl...
>> Ik heb inmiddels een scriptje van iemand gekregen, maar ik ben bang
>> dat ie vol fouten zit.
>> Krijg parseerrors...
>> Het gaat dus om een nieuwsscriptje.
>> Lap tekst... Dat je kan veranderen.
>>
>> Wie kan me even helpen dit te debuggen?
>> Alvast Hartelijk Dank!
> Welke errors krijg je en op welke regels? Wat heb je al geprobeerd?


Regel 7 van geefop.php heb ik het einde even een ) neergezet...
http://bertweb.dev-c.nl/Miniwijk35/news/
Maar ik kan nog steeds niets openen of wijzigen...


--
Greetz,
Bertez[z]
-
Mail:
Bertweb @ Home . NederLand

Rick
08/07/03, 22:20
"Bertez[z]" <bertweb@@home.nl> schreef in bericht
news:bef85h$6vf$1@news4.tilbu1.nb.home.nl...
> "Rick" <spammans@chello.nl> schreef in bericht
> news:bef7qt$17d3$1@nl-news.euro.net
> > "Bertez[z]" <bertweb@@home.nl> schreef in bericht
> > news:bef77q$qik$1@news4.tilbu1.nb.home.nl...
> >> Ik heb inmiddels een scriptje van iemand gekregen, maar ik ben bang
> >> dat ie vol fouten zit.
> >> Krijg parseerrors...
> >> Het gaat dus om een nieuwsscriptje.
> >> Lap tekst... Dat je kan veranderen.
> >>
> >> Wie kan me even helpen dit te debuggen?
> >> Alvast Hartelijk Dank!
> > Welke errors krijg je en op welke regels? Wat heb je al geprobeerd?
>
>
> Regel 7 van geefop.php heb ik het einde even een ) neergezet...
> http://bertweb.dev-c.nl/Miniwijk35/news/
> Maar ik kan nog steeds niets openen of wijzigen...
>
>
wil je info erbij schrijven of in plaats van?

Bertez[z]
08/07/03, 22:35
"Rick" <spammans@chello.nl> schreef in bericht
news:bef8mt$17vk$1@nl-news.euro.net
> "Bertez[z]" <bertweb@@home.nl> schreef in bericht
> news:bef85h$6vf$1@news4.tilbu1.nb.home.nl...
>> "Rick" <spammans@chello.nl> schreef in bericht
>> news:bef7qt$17d3$1@nl-news.euro.net
>>> "Bertez[z]" <bertweb@@home.nl> schreef in bericht
>>> news:bef77q$qik$1@news4.tilbu1.nb.home.nl...
>>>> Ik heb inmiddels een scriptje van iemand gekregen, maar ik ben bang
>>>> dat ie vol fouten zit.
>>>> Krijg parseerrors...
>>>> Het gaat dus om een nieuwsscriptje.
>>>> Lap tekst... Dat je kan veranderen.
>>>>
>>>> Wie kan me even helpen dit te debuggen?
>>>> Alvast Hartelijk Dank!
>>> Welke errors krijg je en op welke regels? Wat heb je al geprobeerd?
>>
>>
>> Regel 7 van geefop.php heb ik het einde even een ) neergezet...
>> http://bertweb.dev-c.nl/Miniwijk35/news/
>> Maar ik kan nog steeds niets openen of wijzigen...
>>
>>
> wil je info erbij schrijven of in plaats van?

In plaats van... Nou, hij moet de inhoud gewoon weergeven in een tekstbox,
zodat het bestaande bewerkt kan worden.
Of klopt er gewoon niets van dit script...


--
Greetz,
Bertez[z]
-
Mail:
Bertweb @ Home . NederLand

Daniel Tryba
08/07/03, 22:35
Bertez[z] <bertweb@@home.nl> wrote:
>> Welke errors krijg je en op welke regels? Wat heb je al geprobeerd?
>
>
> Regel 7 van geefop.php heb ik het einde even een ) neergezet...
> http://bertweb.dev-c.nl/Miniwijk35/news/
> Maar ik kan nog steeds niets openen of wijzigen...

Het script werkt precies zoals te verwachten volgens de source (na
correctie van de parse error).

punten van aandacht:
-0,0 error handling
-0,0 filelocking
-variabelen komen uit de lucht vallen: register_globals probleem (FAQ)
-useless use of javascript award nominated
-textarea heeft geen attribuut value
-de onclick is zinloos
-nl2br() wordt IMHO op het verkeerde moment gebruikt (niet bij het
toevoegen maar bij het weergeven).
-form is gaar.

Maar het is je vast nog niet opgevallen dat er elke keer precies het
zelfde aan de textfile wordt toegevoegd ("['bericht']")?

Het lijkt me zinnig om de persoon van wie je dit script gekregen hebt,
eens te vragen of hij het kan fixen.

--

Daniel Tryba

Maarten Wierda
08/07/03, 23:05
Bertez[z] wrote:

[...]

> <textarea name="iets" value="hier je ding" onClick="if (this.value=='')
> this.value=''" name="bericht">

Hier bedoel je vast:

onClick="if (this.value == this.defaultValue) this.value = '';"

Voor inhoudelijk commentaar op je PHP-code, zie de post van Daniel.

--
Cheers, Maarten Wierda

"Music is first, lyrics are secondary."

John Bokma
08/07/03, 23:20
Bertez[z] wrote:

> Ik heb inmiddels een scriptje van iemand gekregen, maar ik ben bang dat ie
> vol fouten zit.
> Krijg parseerrors...
> Het gaat dus om een nieuwsscriptje.
> Lap tekst... Dat je kan veranderen.
>
> Wie kan me even helpen dit te debuggen?
> Alvast Hartelijk Dank!
>
>
> ----Pagina.php-----
> <?php include ("ding.txt"); ?>
> <a href="geefop.php">Klik hier om het nieuws te veranderen.</a>
>
> ----ding.txt------
> (leeg) (chmod 664)
>
> ----geefop.php------
> <?php
> if (!empty($_POST)) {
> $bericht = nl2br($bericht);
> $bestand = "ding.txt";
> $inhoud_bestand = file($bestand);
> $schrijf_bestand = fopen($bestand,"w");
> fwrite ($schrijf_bestand, "_POST['bericht']"

We locken tegenwoordig ook al niet meer. En je kan ik weet niet wat
erinzetten, *ook* (en bovenal) HTML... Leuk, waar komt het te staan?

John

--
email: mail(at)johnbokma.com (or reply) home: http://johnbokma.com/
website design tips: http://johnbokma.com/websitedesign/ (preliminary)