[RESOLVED] Need Lots Of Help! PHP GUESTBOOK

[define_php]

Hi there... I am new here so if you would give me some tips around here but the point is...

My script doesn't work and keeps bringing up the error "Parse error: syntax error, unexpected T_ECHO in /index.php on line 58". Here is the code for you to examine (PLEASE DON'T LEECH):

	<?php
	if (isset($_POST["name"])) {
		if (isset($_POST["name"])) {
			$nameofperson = strip_tags($_POST["name"]);
			$postofperson = strip_tags($_POST["text"]);
			$myFile = "comments.db";
			$fh = fopen($myFile, 'a') or die("Robbit Shout is down, try again later.");
			$stringData = "$nameofperson : $postofperson\n";
			fwrite($fh, $stringData);
			fclose($fh);
			echo file_get_contents("comments.db");
		}
	} elseif (
		echo file_get_contents("comments.db");
	}
?>

Please help? It's the line with the file_get_contents bit in it.

[bradgrafelman]

You have a parenthesis instead of a curly brace here:

} elseif ( 

Also, why do you have duplicate if() statements:

    if (isset($_POST["name"])) {
        if (isset($_POST["name"])) { 

?

[define_php]

Good point on both of those, going to fix it now. See what comes up.

[define_php]

Sorry, but the same error comes up.

"Parse error: syntax error, unexpected T_ECHO in /index.php on line 58"

	<?php
	if (isset($_POST["name"])) {
		if (isset($_POST["text"])) {
			$nameofperson = strip_tags($_POST["name"]);
			$postofperson = strip_tags($_POST["text"]);
			$myFile = "comments.db";
			$fh = fopen($myFile, 'a') or die("Robbit Shout is down, try again later.");
			$stringData = "$nameofperson : $postofperson\n";
			fwrite($fh, $stringData);
			fclose($fh);
			echo file_get_contents("comments.db");
		}
	} elseif ( 
		echo file_get_contents("comments.db");
	}
?>

[laserlight]

By the way, instead of this:

$myFile = "comments.db";
$fh = fopen($myFile, 'a') or die("Robbit Shout is down, try again later.");
$stringData = "$nameofperson : $postofperson\n";
fwrite($fh, $stringData);
fclose($fh);

You could write:

if (file_put_contents("comments.db", "$nameofperson : $postofperson\n", FILE_APPEND) === false) {
    die("Robbit Shout is down, try again later.");
}

EDIT:

define_php wrote:

Sorry, but the same error comes up.

"Parse error: syntax error, unexpected T_ECHO in /index.php on line 58"

You still did not change the opening parenthesis ( to an opening brace {.

[define_php]

Sorry, nothing changed except for the line number of the error.

"Parse error: syntax error, unexpected T_ECHO in /www/sitegoz.com/p/a/r/partypad/htdocs/index.php on line 56"

	<?php
	if (isset($_POST["name"])) {
		if (isset($_POST["text"])) {
			$nameofperson = strip_tags($_POST["name"]);
			$postofperson = strip_tags($_POST["text"]);
			if (file_put_contents("comments.db", "$nameofperson : $postofperson\n", FILE_APPEND) === false) {
			    die("Robbit Shout is down, try again later.");
			} 
			echo file_get_contents("comments.db");
		}
	} elseif ( 
		echo file_get_contents("comments.db");
	}
?>

[laserlight]

Try again:

<?php
if (isset($_POST["name"])) {
    if (isset($_POST["text"])) {
        $nameofperson = strip_tags($_POST["name"]);
        $postofperson = strip_tags($_POST["text"]);
        if (file_put_contents("comments.db", "$nameofperson : $postofperson\n", FILE_APPEND) === false) {
            die("Robbit Shout is down, try again later.");
        }
        echo file_get_contents("comments.db");
    }
} elseif {
    echo file_get_contents("comments.db");
}
?>

[define_php]

Ok, now I get a different error.

"Parse error: syntax error, unexpected '{', expecting '(' in /index.php on line 55"

<?php
if (isset($_POST["name"])) {
    if (isset($_POST["text"])) {
        $nameofperson = strip_tags($_POST["name"]);
        $postofperson = strip_tags($_POST["text"]);
        if (file_put_contents("comments.db", "$nameofperson : $postofperson\n", FILE_APPEND) === false) {
            die("Robbit Shout is down, try again later.");
        }
        echo file_get_contents("comments.db");
    }
} elseif {
    echo file_get_contents("comments.db");
}
?> 

[laserlight]

Sigh, I had to really rewrite it:

<?php
if (isset($_POST["name"], $_POST["text"])) {
    $contents = strip_tags($_POST["name"]) . ' : '
        . strip_tags($_POST["text"]) . "\n";
    if (file_put_contents("comments.db", $contents, FILE_APPEND) === false) {
        die("Robbit Shout is down, try again later.");
    }
    echo $contents;
} else {
    echo file_get_contents("comments.db");
}
?>

My reasoning is that you probably want to print the contents of the file even if the name is provided but the text is not. Then, there is no point reading from the file right after you have written to it.

EDIT:
Oh wait, that is wrong, since you are appending. As such, it should be:

<?php
if (isset($_POST["name"], $_POST["text"])) {
    $contents = strip_tags($_POST["name"]) . ' : '
        . strip_tags($_POST["text"]) . "\n";
    if (file_put_contents("comments.db", $contents, FILE_APPEND) === false) {
        die("Robbit Shout is down, try again later.");
    }
}

echo file_get_contents("comments.db");

?>

[define_php]

Thanks, but I am using this for my school website, and I want to be able to strip swear words. I need someone else to help me because I was never good with replace(). Help?

<?php
if (isset($_POST["name"], $_POST["text"])) {
    $contents = strip_tags($_POST["name"]) . ' : '
        . strip_tags($_POST["text"]) . "<br>";
    if (file_put_contents("comments.txt", $contents, FILE_APPEND) === false) {
        die("Robbit Shout is down, try again later.");
    }
    echo $contents;
} else {
	echo '<font size=\"2\">';
    echo file_get_contents("comments.txt");
}
?> 

EDIT: Please? Someone?

[laserlight]

Well, if you are done with this part, mark the thread as resolved and then start another thread about filtering out swear words.