Как переписать код?

Question

[Капоот]

Как переписать без goto этот код?

<?php

if (!is_dir("messages")) {
mkdir("messages");
};
a:
goto a;
};
postMessages();
goto a;
};
?>

Answer 1

[nokimaro]

<?php
// Guestbook, Гостевая книга

if (!is_dir("messages")) {
mkdir("messages");
};

function postMessages() {
foreach (glob('messages/*.txt') as $message) {
$message = file($message);
echo str_replace("\n", "", $message[0]) . "<br/>";
echo str_replace("\n", "", "<a href='mailto:$message[1]'>$message[1]</a>") . "<br/>";
for ($i = 2; $i < sizeof($message); $i++) {
echo htmlspecialchars($message[$i]) . "<br/>";
};
echo "<br/>";
};
};

$messages = scandir("messages");

if (!$_POST) {
postMessages();
//a:
print_form();
} else {
if (filter_var($_POST['email'], FILTER_VALIDATE_EMAIL) !== false && $_POST['author'] && $_POST['email'] && $_POST['message']) {
$author = str_replace("\r\n", "", $_POST['author']);
$email = str_replace("\r\n", "", $_POST['email']);
$msg = $_POST['message'];
$time = (int)(microtime(true) * 1000000);
file_put_contents("messages/$time.txt", "$author
$email
$msg");
} else {
echo 'Error while adding your message.';
//goto a;
print_form();
};
postMessages();
//goto a;
print_form();
};

function print_form()
{
    echo
    '<form action="index.php" method="post">
    <p><input type="text" name="author" required /></p>
    <p><input type="text" name="email" required /></p>
    <p><textarea name="message" required></textarea></p>
    <p><input type="submit" /></p>
    </form>';
}
?>

Comments

[FanatPHP]

по-моему один вызов тут лишний. Но в этом бардаке хрен разберешься

[nokimaro]

hackerman 234, замените эту строчку
if(isset($_POST['submit']))

на вот это
if ($_SERVER['REQUEST_METHOD'] == 'POST')

[FanatPHP]

nokimaro, тебе как, вообще не стыдно за этот код? :)

[nokimaro]

FanatPHP, это не мой код. Автор стёр свой вчерашний вопрос, где изначально был этот код за его авторством.
Я лишь внёс пару правок в его код, чтобы решить вопрос и при этом сохранить "авторский стиль".

[nokimaro]

FanatPHP, автор решает какое-то тестовое задание.
Там было длинное ТЗ вчера в впоросе, типа гостевой книги на файлах, с чётчикм указанием используйте microtime, выведите такой-то текст ошибки, используйте htmlspecialchars и тд и тп.

[FanatPHP]

nokimaro, возможно это не он сам удалил, а модераторы.
Я на этот вопрос тоже сейчас жалобу отправлю

[FanatPHP]

hackerman 234, ну так прочти сообщение об ошибке, там английским по белому написано, что надо сделать.
Или так и будешь выклянчивать каждую строчку?

Answer 2

[FanatPHP]

Код, конечно, адовый.
Переписать как-то так

if ($_POST) {
    if (filter_var($_POST['email'], FILTER_VALIDATE_EMAIL) && $_POST['author'] && $_POST['message']) {
        $author = str_replace("\r\n", "", $_POST['author']);
        $email = str_replace("\r\n", "", $_POST['email']);
        $msg = $_POST['message'];
        $time = (int)(microtime(true) * 1000000);
        file_put_contents("messages/$time.txt", "$author\n$email\n$msg");
        header("Location:".$_SERVER['REQUEST_URI']);
    } else {
        echo 'Error while adding your message.';
    }
}
postMessages();
?>
<form action="index.php" method="post">
<p><input type="text" name="author" required /></p>
<p><input type="text" name="email" required /></p>
<p><textarea name="message" required></textarea></p>
<p><input type="submit" /></p>
</form>

Если коротко, то если все дороги ведут к :а, то просто поместить код внизу, без условий, чтобы он всегда выполнялся.