Absätze im Text mit <p></p> einfassen

[Frashier]

Ich stehe total auf dem schlauch, ich möchte in einem String/Text Absätze in <p></p> einfassen.

zb.

Lorem ipsum dolor sit amet, consectetur adipisicing elit, sed do eiusmod 
incididunt ut labore et dolore magna aliqua. Ut enim ad minim veniam.<br /><br />
Quis nostrud exercitation ullamco laboris nisi ut aliquip ex ea commodo consequat. Lorem ipsum dolor sit amet, consectetur adipisicing elit.

in

<p>Lorem ipsum dolor sit amet, consectetur adipisicing elit, sed do eiusmod 
incididunt ut labore et dolore magna aliqua. Ut enim ad minim veniam.</p>

<p>Quis nostrud exercitation ullamco laboris nisi ut aliquip ex ea commodo consequat. Lorem ipsum dolor sit amet, consectetur adipisicing elit.</p>

Kann mir jemand das preg_replace oder anderen Lösungsansatz dazu Bitte geben. Bekomm es nicht hin.

[Kropff]

String mit split aufteilen und dann für jedes Element ein <p> an den Anfang und ein </p> ans Ende.

Peter

[Frashier]

Jut habe ich:

$str = absatz(split("[\n\n]", $str));

function absatz($string) {	
 foreach($string as $val) {
  trim($val);
  if(!empty($val)) {
   echo "<p>".$val."</p>\n\n";
  }
 }
}

Habe nur das Problem das er mir immer wieder ein leeres <p></p> reinhaut und kann es mit nur nicht erklären. Was mach ich falsch?

[Kropff]

$str = absatz(split("[\n\n]", $str));
...
foreach($string as $val) 

Du solltest dich schon entscheiden. $str oder $string.

Peter

[Frashier]

$str = absatz(split("[\n\n]", $str));
...
foreach($string as $val) 

Du solltest dich schon entscheiden. $str oder $string.

Peter

Ist nicht wirklich das Problem, kommt nur weil ich das split in einer anderen Funktion habe.

[Frashier]

Ich habs, mal am Regulären Ausdruck gearbeitet (wie ich den Mist hasse)

$str = absatz(split("\r\n|\n|\r", $str));

[AmicaNoctis]

Hallo,

die Funktion trim arbeitet nicht so wie du denkst. Die ändert den String nicht, sondern gibt einen neuen zurück.

Probier's mal so:

$str = absatz(split("[\n\n]", $str));

function absatz($string) {    
 foreach($string as $val) {
  if(!trim($val)) {
   echo "<p>".$val."</p>\n\n";
  }
 }
} 

Gruß,

Amica

[fireweasel]

Jut habe ich:

...
trim($val);
  if(!empty($val)) {
   echo "<p>".$val."</p>\n\n";
  }
...

Habe nur das Problem das er mir immer wieder ein leeres <p></p> reinhaut und kann es mit nur nicht erklären. Was mach ich falsch?

Eine Zeichenkette prüft man mit !isset($string[0]) und nicht mit empty($string) auf "Leere".

Aber das ist nicht das eigentliche Problem.
"Leere" Zeilen würde ich so ausschließen:

foreach () {
  // ...
  if (preg_match('/\A\s*\z/', $val)) {
    continue;
  }
  // ...
} 

Und wenn man einmal dabei ist, könnte man das ganze Problem vielleicht auch simpler lösen:

function paragraph($string) {
  return '<p>' . preg_replace('/(?:\r\n|[\r\n]){2,}/', '</p><p>', $string) . '</p>';
} 

Für schöneren HTML-Quelltext musst du notfalls noch ein paar Zeilenumbrüche im Ersetzen-String einfügen.

[AmicaNoctis]

Eine Zeichenkette prüft man mit !isset($string[0]) und nicht mit empty($string) auf "Leere".

Wer sagt das denn außer dir?

Gruß,

Amica

[wahsaga]

Eine Zeichenkette prüft man mit !isset($string[0]) und nicht mit empty($string) auf "Leere".

Wer sagt das denn außer dir?

Hab mich auch erst gewundert - aber ich vermute, damit will fireweasel umgehen, dass auch "0" von empty als leer erkannt würde.
Quasi eine leicht "geekigere" Variante von isset($string) && $string !== ""

[AmicaNoctis]

Ah ja, danke, da find ich aber !strlen($string) trotzdem noch kürzer und intuitiver. Es könnte zwar im Allgemeinen zu ner Warnung führen, aber innerhalb von foreach (wie im Fall des Fragestellers) kann das ja nicht passieren.

[schmalle]

$str = preg_replace("%(<br />)([\s]*?)(<br />)%si", "</p><p>", $str);
$str = "<p>".$str."</p>"; 

[fireweasel]

Wer sagt das denn außer dir?

Alle die, die schlauer sind als ich (z.B. der hier). Das ist ganz sicher nicht auf meinem Mist gewachsen.

Ah ja, danke, da find ich aber !strlen($string) trotzdem noch kürzer und intuitiver.

Die Betonung sollte nicht auf der Verwendung von !isset($string[0]) liegen, sondern auf der NICHTverwendung von empty(), dessen Bezeichnung IMHO falsch gewählt ist. Sowas wie yields_to_false() würde besser passen ...

Für PHP-Verhältnisse ist !strlen($string) sicher die "intuitivste" Variante. Ich hätte lieber sowas wie $string->is_empty() oder notfalls eben is_empty($string) und als Rückgabewert TRUE für leere Strings und nichtexistierende Variablennamen. Muss man sich selber basteln.

...
Quasi eine leicht "geekigere" Variante von isset($string) && $string !== ""

Geekig? Ich verwende diese Schreibweise schon so lange, dass ich mich nicht mehr daran erinnern kann, warum ich mir sie angewöhnt hatte. Ich befürchte fast, es war aus Gründen der bösen, bösen Mikrooptimierung.
"... it's about 33% faster" sagt doch eigentlich schon alles.