Drobny problem z wyrażeniem regularnym Opcje

[gac3k]

Witam, to mój pierwszy post z prośbą o pomoc, banalny bezsensowny i zaraz ktoś mnie zjedzie że takich idiotycznych pytań się nie zadaje ale mam lekkiego kaca i ciężko u mnie z myśleniem więc potrzebuje pomocy (IMG:style_emoticons/default/wink.gif)

Tak więc mam oto, taką metodę w modelu w kohanie, choć to akurat nieistotne.

[PHP] pobierz, plaintext 
<?php defined('SYSPATH') or die('No direct script access');
 
class Model_Post extends ORM
{
	public $_belongs_to = array(
		'advert'	=> array(),
	);
 
	public function replace_songs()
	{
		$pattern = '[\[piosenka tytul="(.*)" wykonawca="(.*)"\]]is';
		$replacer = '<a href="\1">\1</a> - <span>\2</span>';
		return preg_replace($pattern,$replacer,$this->content);
	}
}
 
?>
[PHP] pobierz, plaintext 

rzecz w tym że metoda ta ma na celu wyciągnięcie jak gdyby z czegoś w stylu BBCODE tytułu i autora piosenki. I generalnie rzecz biorąc skrypt działa poprawnie jeżeli mam tylko jeden taki kod. Czyli np. z fragmentu

To może być [piosenka tytul="Jakiś tytuł" wykonawca="Wykonawca"], a przynajmniej tak sądzę.

elegancko wyciągnie wyrażenie i zapisze mi prawidłową wartość, problem pojawia się gdy mam dwa takie kody lub więcej, np.

To może być [piosenka tytul="Jakiś tytuł" wykonawca="Wykonawca"], a przynajmniej tak sądzę, ale może to też [piosenka tytul="Inny tytuł" wykonawca="Inny Wykonawca"].

to parsuje się tylko część pierwszego kodu, a cała reszta się psuje.

Wiecie może jak powinno wyglądać prawidłowe wyrażenie, lub może błąd jest w innym miejscu, będę wdzięczny za każdą odpowiedź (IMG:style_emoticons/default/biggrin.gif)

Ten post edytował gac3k 22.05.2011, 20:03:30

[BaN]

* jest operatorem zachłannym, zatem musi być

[PHP] pobierz, plaintext 
$pattern = '[\[piosenka tytul="([^"]*)" wykonawca="([^"]*)"\]]is';
[PHP] pobierz, plaintext 

albo

[PHP] pobierz, plaintext 
$pattern = '[\[piosenka tytul="(.*?)" wykonawca="(.*?)"\]]is';
[PHP] pobierz, plaintext 

[gac3k]

Wielkie dzięki, działa jak trzeba. Wiedziałem że to jakaś pierdółka (IMG:style_emoticons/default/wink.gif)