Zeilenumbruch funktioniert nicht :/

[ZarneXxX]

Hallo zusammen,
ich versuche es ganze zeit nur ich kriege es nicht hin.
Bei dem Privat Message System funktioniert kein Zeilenbruch.
Die ausgabe sieht ca. so aus:
Zeile1<br>Zeile2<br>Zeile3
Und im Quellcode sieht es so aus:
Zeile1&lt;br&gt;Zeile2&lt;br&gt;Zeile3

Das Hauptsächliche ist ja das:
$text = preg_replace('/\r\n|\r/', "<br>", html_entity_decode($_POST["nachricht"]));

Hier:
<textarea name="nachricht" id="textarea"><?php
$a = str_replace("<br />", " ", $text);
echo $a; ?></textarea>


Hier ist mein Code:

<?php
if(isset($_GET["post"])) {
if($_GET["post"]=="new") {
?>
<h1>Neue Nachricht</h1>
<div id="news" style="">
    <p>
        <form action="index_private.php" method="POST">
            <a href="index_private.php?post=new" style="font-size: 150%;color:grey;border:1px solid grey;background:#F2F2F2;padding-left:15px;padding-right:15px;">Neue Nachricht</a>
            <a href="index_private.php" style="font-size: 150%;color:grey;border:1px solid grey;background:#F2F2F2;padding-left:15px;padding-right:15px;">Back</a>
        </form>
    </p>
</div>
<?php
if(isset($_POST["back"])) {
    echo '<meta http-equiv="refresh" content="0; URL=index_private.php">';
}
if(isset($_GET["to"])) {
    $to = $_GET["to"];
} else $to = "";
if(isset($_GET["betreff"])) {
    $betreff = $_GET["betreff"];
} else $betreff = "";
?>
<div id="news">
                        <form action="index_private.php?post=new" method="POST">
                        <p id="inputtexttitel">An</p>
                        <input type="text" id="inputtext" value="<?php echo $to; ?>" name="to">
                        <p id="inputtexttitel">Betreff</p>
                        <input type="text" id="inputtext" value="<?php echo $betreff; ?>" name="betreff"><br>
                        <p id="inputtexttitel" >Nachricht</p>
                        <textarea name="nachricht" id="textarea"><?php
$a = str_replace("<br />", " ", $text);
echo $a; ?></textarea><br>
                        <input type="submit" value="Senden" name="sendengo" id="buttons">
                    </form>
</div>


                    <?php
                    if(isset($_POST["sendengo"])){
                        $to = $_POST["to"];
                        $betreff = $_POST["betreff"];
                        $text = preg_replace('/\r\n|\r/', "<br>", html_entity_decode($_POST["nachricht"]));
                        $db = new Database();
                        $db2 = new Database();
                        $db2->query("SELECT * FROM users WHERE username='".htmlentities(trim(stripslashes(mysql_real_escape_string($to))))."'");
                            foreach($db2->rows() as $row) {
                                $zu = $row["id"];
                            }
                            if($zu!="") {
                            $datum = date("d.m.Y");
                            $db->InsertInto("INSERT INTO `post` (`post_id`, `from`, `to`, `betreff`, `text`, `datum`) VALUES 
                            ('','". $username ."','".$to."','".htmlentities(trim(stripslashes(mysql_real_escape_string($betreff))))."','".htmlentities(trim(stripslashes(mysql_real_escape_string($text))))."','".$datum."')");
                            echo "<script>alert('Message Gesendet!');</script>";
                        } else echo "<script>alert('Dieser User existiert nicht!');</script>";
                    } //Button nicht gedrückt
} elseif($_GET["post"]=="ges") {
?>
<h1>Gesendete</h1>
<div id="news" style="">
    <p>
        <form action="index_private.php" method="POST">
            <a href="index_private.php?post=new" style="font-size: 150%;color:grey;border:1px solid grey;background:#F2F2F2;padding-left:15px;padding-right:15px;">Neue Nachricht</a>
        </form>
    </p>
</div>
<?php
if(isset($_POST["newmsg"])) {
    echo '<meta http-equiv="refresh" content="0; URL=index_private.php?post=new">';
}
if(isset($_POST["ges"])) {
    echo '<meta http-equiv="refresh" content="0; URL=index_private.php?post=ges">';
}
?>
<div id="news">
    <table style="font-size:140%;">
                            <tr>
                                <td style="font-size:100%;width:200px;"><h2>An</h2></td>
                                <td style="font-size:100%;width:550px;"><h2>Betreff</h2></td>
                                <td style="font-size:100%;"><h2>Datum</h2></td>
                            </tr>
                            <?php
                            $dbmsg = new Database();
                            $dbmsg->query("SELECT * FROM `post` WHERE `from`='".$username."' ORDER BY `post_id` DESC");

                            foreach($dbmsg->rows() as $row) {
                                $active = $row["active"];
                                
                                if($active==0) {
                                $blub1 = "<b>";
                                $blub2 = "</b>";
                                } else {
                                $blub1 = "";
                                $blub2 = "";
                                }
                                echo "<tr><td>".$blub1.$row["to"].$blub2."</td>
                                <td><a href='index_private.php?post=".$row["post_id"]."'>".$blub1.$row["betreff"].$blub2."</a></td>
                                <td>".$blub1.$row["datum"].$blub2."</td></tr>";
                            }
                            ?>
                        </table>
</div>
        <?php                    
} elseif($_GET["post"]!="") {


$post_id = $_GET["post"];
                    $asd = new Database();
                    $asd->query("SELECT * FROM `post` WHERE `post_id`='".$post_id."'");
                    if($asd->numRows()!=0) {
                        foreach($asd->rows() as $row) {
                            $to = $row["to"];
                            $from = $row["from"];
                            $betreff = $row["betreff"];
                            $name = $row["from"];
                            $text = $row["text"];
                            $datum = $row["datum"];
                            $active = $row["active"];
                        }
                        if($to==$username) {
                        ?>
                        <h1>Posteingang</h1>
<div id="news" style="">
    <p>
        <form action="index_private.php?post=<?php echo $post_id; ?>" method="POST">
            <a href="index_private.php?post=new&to=<?php echo $from; ?>&betreff=RE:<?php echo $betreff; ?>" style="font-size: 150%;color:grey;border:1px solid grey;background:#F2F2F2;padding-left:15px;padding-right:15px;">Antworten</a>
            <input type="submit"  style="font-size: 150%;color:grey;border:1px solid grey;background:#F2F2F2;padding-left:15px;padding-right:15px;" value="L&ouml;schen" name="delmsg">
            <a href="index_private.php" style="font-size: 150%;color:grey;border:1px solid grey;background:#F2F2F2;padding-left:15px;padding-right:15px;">Back</a>
        </form>
    </p>
</div>
<?php
if(isset($_POST["back"])) {
    echo '<meta http-equiv="refresh" content="0; URL=index_private.php">';
}
if(isset($_POST["delmsg"])) {
    $asd1 = new Database();
    $yey = $asd1->InsertInto("DELETE FROM `post` WHERE `post_id`='".$post_id."'");
    if($yey==true) {
    } echo "<script>alert('Du hast diese nachricht geloescht!');return true;</script><meta http-equiv='refresh' content='0; URL=index_private.php'>";
}
if(isset($_POST["newmsg"])) {
    echo '<meta http-equiv="refresh" content="0; URL=index_private.php?post=new&to='.$from.'&betreff=RE:'.$betreff.'">';
}
                        
                            
                            echo "<div id='news'><h2>von<i> ".$from." </i>| am <i>".$datum."</i></h2><br>";
                            echo '<form action="?post='.$post_id.'" method="POST">';
                            echo "<div style='font-family:Comic Sans MS;font-size:120%;color:grey;'>".$text."</div></div>";
                            echo "<br><br><br>";
                            //Update status
                            $dbuptodate = new Database();
                            $dbuptodate->InsertInto("UPDATE `post` SET `active`='1' WHERE `post_id`='".$post_id."'");
                            
                            
                        } elseif($from==$username) { 
                        ?>
                                                <h1>Posteingang</h1>
<div id="news" style="">
    <p>
        <form action="index_private.php?post=<?php echo $post_id; ?>" method="POST">
            <a href="index_private.php?post=new&to=<?php echo $from ?>&betreff=RE:<?php echo $betreff; ?>" style="font-size: 150%;color:grey;border:1px solid grey;background:#F2F2F2;padding-left:15px;padding-right:15px;">Antworten</a>
            <a href="index_private.php" style="font-size: 150%;color:grey;border:1px solid grey;background:#F2F2F2;padding-left:15px;padding-right:15px;">Back</a>
        </form>
    </p>
</div>
<?php
if(isset($_POST["back"])) {
    echo '<meta http-equiv="refresh" content="0; URL=index_private.php">';
}

if(isset($_POST["newmsg"])) {
    echo '<meta http-equiv="refresh" content="0; URL=index_private.php?post=new&to='.$from.'&betreff=RE:'.$betreff.'">';
}
                        
                            
                            echo "<div id='news'><h2>An<i> ".$to." </i>| am <i>".$datum."</i></h2><br>";
                            echo '<form action="?post='.$post_id.'" method="POST">';
                            echo "<div style='font-family:Comic Sans MS;font-size:120%;color:grey;'>".$text."</div></div>";
                            echo "<br><br><br>";
                            //Update status
                        
                        
                        } else echo "<script>alert('Du hast keine Rechte dies zusehen!');return true;</script><meta http-equiv='refresh' content='0; URL=index_private.php'>";
                    } else echo "<script>alert('Diese Nachricht ist nicht verfuegbar!');return true;</script><meta http-equiv='refresh' content='0; URL=index_private.php'>";    

} else echo "fehler"; 

//
} else {
?>
<h1>Posteingang</h1>
<div id="news" style="">
    <p>
        <form action="index_private.php" method="POST">
            <a href="index_private.php?post=new" style="font-size: 150%;color:grey;border:1px solid grey;background:#F2F2F2;padding-left:15px;padding-right:15px;">Neue Nachricht</a>
            <a href="index_private.php?post=ges" style="font-size: 150%;color:grey;border:1px solid grey;background:#F2F2F2;padding-left:15px;padding-right:15px;">Gesendete</a>
        </form>
    </p>
</div>
<?php
if(isset($_POST["newmsg"])) {
    echo '<meta http-equiv="refresh" content="0; URL=index_private.php?post=new">';
}
if(isset($_POST["ges"])) {
    echo '<meta http-equiv="refresh" content="0; URL=index_private.php?post=ges">';
}
?>
<div id="news">
    <table style="font-size:140%;">
                            <tr>
                                <td style="font-size:100%;width:200px;"><h2>Von</h2></td>
                                <td style="font-size:100%;width:550px;"><h2>Betreff</h2></td>
                                <td style="font-size:100%;"><h2>Datum</h2></td>
                            </tr>
                            <?php
                            $dbmsg = new Database();
                            $dbmsg->query("SELECT * FROM `post` WHERE `to`='".$username."' ORDER BY `post_id` DESC");

                            foreach($dbmsg->rows() as $row) {
                                $active = $row["active"];
                                
                                if($active==0) {
                                $blub1 = "<b>";
                                $blub2 = "</b>";
                                } else {
                                $blub1 = "";
                                $blub2 = "";
                                }
                                echo "<tr><td>".$blub1.$row["from"].$blub2."</td>
                                <td><a href='index_private.php?post=".$row["post_id"]."'>".$blub1.$row["betreff"].$blub2."</a></td>
                                <td>".$blub1.$row["datum"].$blub2."</td></tr>";
                            }
                            ?>
                        </table>
</div>
<?php
}
?>

Gruß, ZarneXxX

[Devon]

Versuch es mal mit

\n 

damit wird nämlich in PHP eine neue Zeile erzeugt.

[patlux]

Versuch mal:

$text = preg_replace('/\r\n|\r/', "<br>", html_entity_decode($_POST["nachricht"]));

ersetzen in

$text = html_entity_decode(preg_replace('/\r\n|\r/', "<br>", $_POST["nachricht"]));

Bin mir grad nich 100% sicher. Zu warum und zu müde xD

[ZarneXxX]

$text = html_entity_decode(preg_replace('/\r\n|\r/', "<br>", $_POST["nachricht"]));

Ergebnis wie vorher.

\n 

Da kommt einfach nur ein kleines N.
Sofern ich das richtig gemacht habe:
$text = preg_replace('/\r\n|\r/', " \n ", html_entity_decode($_POST["nachricht"]));

[Integer]

preg_replace('/\r\n|\r/' . " \n",

Warum denn ein Kommata das macht 0 Sinn? Und das Leerzeichen war auch unnötig hinter dem /n :-)

[DMW007]

Versuch es mal mit

\n 

damit wird nämlich in PHP eine neue Zeile erzeugt.

Und was bringt ihm das in seiner HTML-Ausgabe? HTML interessieren Escapesequenzen nicht.

Das Pattern ist unsinnig. Wenn du schon alle Plattformen abdecken willst musst du \r und \n sowohl getrennt als auch einzeln ersetzen. Linux verwendet i.d.R. nur \n, Windows \r\n und Mac OS \r. Man braucht für so Kleinigkeiten auch keine Regulären Ausdrücke, die Stringfunktionen reichen hier vollkommen aus:

$str = str_replace(array("\r\n", "\r", "\n"), "<br>", $str); 

Aber auch das ist nicht nötig, dafür gibts die PHP-Funktion nl2br($str). Die ist auch gleich plattformunabhängig und berüchsichtigt sowohl Windows, Linux als auch Mac OS.

Btw:

$post_id = $_GET["post"]; $asd = new Database(); 
$asd->query("SELECT * FROM `post` WHERE `post_id`='".$post_id."'"); 

Ich hoffe dir ist klar, dass die Abfrage so ziemlich die perfekte SQL-Injection ist.
Generell solltest du an deinem Codingstyl arbeiten, da gibts einiges Optimierungspotenzial. Den Sinn einer Datenbank-Klasse scheinst du auch nicht so richtig verstanden haben, sonst würdest du nicht 10 Millionen Instanzen davon erstellen. Variablen 'blubb', 'asdf' oder sonst wie sinnlos zu benennen ist auch nicht gerade sinnvoll, das ist gift für die Übersicht. Spätestens sobald du mehr als eine Hand voll Codezeilen so schlampig geschrieben hast und dann einen Fehler finden willst bzw den Code in 2 Monaten mal anschaust wirst du das auch selbst merken. Es hat schon seinen Grund, weshalb man Variablennamen selbst wählen kann. Das gleiche gilt für mehrere ineinander verschachtelte If-Abfragen, dafür gibts logische Operatoren.
Wenn das was größeres werden soll würde ich außerdem über ein Templatesystem und je nachdem was konkret umgesetzt werden soll ggf. auch gleich ein Framework nachdenken. Layout und Code sollte man soweit es geht trennen.

[ZarneXxX]

Ich verstehe immer noch nicht ganz was ich machen soll :S

[xOneDirectionx]

Einfach die Funktion nl2br(string); benutzen! Aber vernünftiger wäre es, wenn du den kompletten Source noch einmal neu schreiben würdest und dieses Mal nicht alles durcheinander und auch nicht so unübersichtlich programmieren würdest. Es hilft später auch dir, wenn du dein "Werk" mal erweitern willst. Vergiss die Kommentierung deines Quellcodes nicht.

[ZarneXxX]

Habs selbst raus bekommen:

$text = preg_replace('/\r\n|\r/', " \n  ", html_entity_decode($_POST["nachricht"])); 

Ersetzten durch:

$text = preg_replace('/\r\n|\r/', "<br>", html_entity_decode($_POST["nachricht"])); 

Hier wurde \n durch <br> Ersetzt!

und

('','". $username ."','".$to."','".htmlentities(trim(stripslashes(mysql_real_escape_string($betreff))))."','".htmlentities(trim(stripslashes(mysql_real_escape_string($text))))."','".$datum."')"); 

Ersetzten durch:

('','". $username ."','".$to."','".htmlentities(trim(stripslashes(mysql_real_escape_string($betreff))))."','".html_entity_decode(trim(stripslashes(mysql_real_escape_string($text))))."','".$datum."')"); 

Hier wurde .htmlentities durch .html_entity_decode ersetzt!

Danke nochmals für eure Hilfe

[DMW007]

Ich empfehle dir dringend mehr an der Logik deines Codes zu arbeiten. Es macht Sinn zuerst darüber nachzudenken was der Code eigentlich tun muss. So was wie

$text = preg_replace('/\r\n|\r/', " \n  ", html_entity_decode($_POST["nachricht"])); 

fällt dann nämlich schon mal weg, da das absolut keinen Sinn macht. Du willst Escapesequenzen für Zeilenumbrüche (\n, \r\n etc) in HTML umwandeln (<br>). Diese Zeile ersetzt \r\n und \n durch \n mit ein paar vor- und nachgestellten Leerzeichen. Spätestens jetzt sollte klar sein, dass dieser Code definitiv nicht das gewünschte tun kann. Das haben hier aber auch mit mir bereits 2 Leute gepostet.

Bei solchen Konstrukten merkt man auch, dass du scheinbar selbst nicht so genau weißt was du da tust:

html_entity_decode(trim(stripslashes(mysql_real_escape_string($text)))) 

stripslashes() ist da vollkommen unnötig, weil mysql_real_escape_string() das bereits erledigt. Mal ganz nebenbei gesagt würde ich das sowieso in die Datenbank-Klasse verlagern. Dann hast du keine 99 Funktionen die du verschachtelt ineinander aufrufst sondern ein einfaches $db->complete_clean($var) oder ähnlich. Außerdem bist du so flexibel und kannst problemlos irgendwann MySQL zb durch MySQLi austauschen, gerade bei den MySQL-Spezifischen Funktionen ist das interessant. Gerade das ist auf Dauer eh unumgänglich, da in Zukunft nur noch die MySQLi-Erweiterung von PHP supported wird.

Was ich bezüglich deines Ansatzes mit dem letzten Post sagen wollte: Es gibt wie bereits gesagt die PHP-Funktion nl2br, die das macht was du willst, und zwar plattformunabhängig. Es macht wenig Sinn das Rad neu zu erfinden. Zumal wie bereits gesagt Regex für solche Kleinigkeiten wie mit Kanonen auf Spatzen schießen ist.