Shoutbox werkt niet.

Door Marc de Munnik op 02-12-2008 22:53

389 views

http://www.flat3media.nl/testserver/Kitestyle/shoutbox.php

De shoutbox werkt wel, maar na plaatsing krijg ik telkens een foutmelding. Tevens wil ik een spatie tussen de berichten hoe doe ik dat?

hieronder het script:

<html>
<head>
<title>Kitestyle.nl - Shoutbox</title>
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1">
<style type="text/css">

</style>
</head>
<body leftmargin="0" topmargin="0" marginwidth="0" marginheight="0">

<div align="center">

<table width="1025" height="768" border="0" align="left" cellpadding="0" cellspacing="0" id="Tabel_01">
<tr>
<td colspan="5">
<img src="afbeeldingen/index_01.jpg" alt="" width="1024" height="133" border="0" usemap="#Map"></td>
<td>
<img src="afbeeldingen/spacer.gif" width="1" height="133" alt=""></td>
</tr>
<tr>
<td colspan="3">
<img src="afbeeldingen/index_02.jpg" width="246" height="135" alt=""></td>
<td width="663" height="575" rowspan="3" align="left" valign="top" bgcolor="#FFFFFF"><img src="afbeeldingen/deberichten.jpg" alt="de berichten" width="108" height="14"><br>
<br>
<?PHP
error_reporting(E_ALL);
$bestand = fopen('shoutboxtext.txt','a+');
$regels = file('shoutboxtext.txt');

if ( isset ( $_POST['submit'] ) ) {
if ( !empty( $_POST['name'] ) && !empty( $_POST['msg'] ) ){
$shout = '<b>' . htmlspecialchars($_POST['name']) . '</b>: ' . htmlspecialchars($_POST['msg']);
// echo $shout;
fwrite($bestand,"\n$shout");
fclose($bestand);
header('Location: shoutbox.php');
}

else {
echo 'You didn\'t fill in you\'re name and/or message';
}
}
fclose($bestand);

if ( isset ( $_GET['show'] ) ) {
if ( $_GET['show'] == 'all'){ $hoeveel = count($regels); }
else { $hoeveel = $_GET['show']; }
}
else {
$hoeveel = 10;
}

for($i=count($regels);count($regels)-$i<$hoeveel;$i--){
echo $regels[$i-1] . '<br/>';
}

?></td>
<td rowspan="4"><img src="afbeeldingen/index_04.jpg" width="115" height="635" alt=""></td>
<td>
<img src="afbeeldingen/spacer.gif" width="1" height="135" alt=""></td>
</tr>
<tr>
<td rowspan="3">
<img src="afbeeldingen/index_05.jpg" width="22" height="500" alt=""></td>
<td width="196" height="141" align="left" valign="top" bgcolor="#000000"><form action="shoutbox.php" method="post">
<span class="stijl1">Naam:
<input type="text" name="name" />
<br />
<br>
Bericht:</span>
<textarea name="msg"></textarea>
<br/>
<input type="submit" name="submit" value="Stuur bericht!">
</form> </td>
<td rowspan="3">
<img src="afbeeldingen/index_07.jpg" width="28" height="500" alt=""></td>
<td>
<img src="afbeeldingen/spacer.gif" width="1" height="141" alt=""></td>
</tr>
<tr>
<td rowspan="2">
<img src="afbeeldingen/index_08.jpg" width="196" height="359" alt=""></td>
<td>
<img src="afbeeldingen/spacer.gif" width="1" height="299" alt=""></td>
</tr>
<tr>
<td>
<img src="afbeeldingen/index_09.jpg" width="663" height="60" alt=""></td>
<td>
<img src="afbeeldingen/spacer.gif" width="1" height="60" alt=""></td>
</tr>
</table>


<map name="Map">
<area shape="rect" coords="498,90,671,130" href="shoutbox.php?show=5">
<area shape="rect" coords="682,86,855,127" href="shoutbox.php?show=all">
</map>
</div>
</body>
</html>

Niek Weevers

02-12-2008 23:04

Ik weet niet of dat het is, maar je sluit het bestand 2x. Dat lijkt me niet nodig.

Marc de Munnik

02-12-2008 23:32

dat heb ik er nu uitgehaald, alleen deze fout nog.

header('Location: shoutbox.php');

Daar geeft hij een fout op aan.

Niek Weevers

02-12-2008 23:37

Zet ob_start(); bovenin het script van shoutbox.php
Daarmee is de fout volgens mij niet weg, maar als het goed is wordt hij niet meer getoond

Marc de Munnik

02-12-2008 23:43

Het werkt bedankt,

Weet je ook wat ik aan het script moet toevoegen om er automatisch een spatie tussen de berichtjes laten komen!?

PHP Newbie

02-12-2008 23:56

Niek schreef op 02.12.2008 23:37
Zet ob_start(); bovenin het script van shoutbox.php
Daarmee is de fout volgens mij niet weg, maar als het goed is wordt hij niet meer getoond

Dat is een lapmiddel, geen oplossing.

@TS
Zorg ervoor dat je eerst al je php controles doet voordat je ook maar iets naar de browser toe stuurt.

03-12-2008 07:31

Marc de Munnik schreef op 02.12.2008 23:43
Het werkt bedankt,

Weet je ook wat ik aan het script moet toevoegen om er automatisch een spatie tussen de berichtjes laten komen!?

Dan zet je er een spatie bij?

Wouter De Schuyter

03-12-2008 08:14

stel dat het niet lukt om hem te doen werken kan je altijd deze downloade werkt goed en is volledig gratis
http://www.celerondude.com/php-shoutbox

Jelle Posthuma

03-12-2008 10:43

Kan er niet ff een sticky komen met uitleg over header('location:...')?
Want het komt nog te vaak voor dat mensen niet snappen waarom ze de error krijgen bij een header commando.

Ik krijg ook zoveel vragen over waarom ze een "header already send" error krijgen, en ik blijf constant hetzelfde uitleggen, zo erg dat ik zelfs een uitleg in een TXT document opgeslagen heb.

Dennis Mertens

03-12-2008 10:48

Helaas zijn de mensen ook de lui om zelf naar een oplossing te zoeken.

Ook wordt er heel vaak gezegd dat Ob_Start(); een slechte manier is (ben ik het mee eens), maar je vindt bijna geen alternatieven. Geen concrete voorbeelden.

PHP Newbie

03-12-2008 12:57

Dennis Mertens schreef op 03.12.2008 10:48
Helaas zijn de mensen ook de lui om zelf naar een oplossing te zoeken.

Ook wordt er heel vaak gezegd dat Ob_Start(); een slechte manier is (ben ik het mee eens), maar je vindt bijna geen alternatieven. Geen concrete voorbeelden.

Het alternatief heb ik je inmijn vorige post verteld. I.p.v. dat je eerst je design naar de browser zendt, voer je eerst al je controles uit.

Dus niet:


<html>
<head>
</head>
<body>
<?php
if($_SERVER['REQUEST_METHOD'] == 'POST'])
{
if(doeiets()
{
header('location: pagina');
}
}
?>
</body>
</html>

Maar:


<?php
if($_SERVER['REQUEST_METHOD'] == 'POST'])
{
if(doeiets()
{
header('location: pagina');
}
}
?>
<html>
<head>
</head>
<body>
</body>
</html>

Reageren

Inloggen om te reageren