| PHP Developer Forum Hier habt ihr die Möglichkeit, eure Skriptprobleme mit anderen Anwendern zu diskutieren. Seid so fair und beantwortet auch Fragen von anderen Anwendern. Dieses Forum ist sowohl für ANFÄNGER als auch für PHP-Profis! Post your PHP questions here! |
 |

06-10-2010, 19:08
|
|
davie2000
Registrierter Benutzer
|
|
Registriert seit: Oct 2010
Beiträge: 5
|
|
Regex: eregi -> alles außer bestimmte zeichenkette
Hallo!
Ich gebe es gleich zu: dieser regex übersteigt meinen Horizont! Ich bräuchte deshalb bitte eure Hilfe bei folgendem Regex-Problem:
Ich habe zB folgenden Text:
und möchte daraus die Nummer (PID) und den Titel (gridtitle) extrahieren, also für die obigen beiden Zeilen, hätte ich gerne
1. id=6427821 und title=Sylvia
2. id=1596793 und title=Face
Leider muss ich dafür eregi verwenden; der bisherige Code (der nicht funktioniert) sieht so aus:
Code:
$regx = 'href="/productinfo.asp\?pid=([^<]*)&style=ice">(.*)<font class=gridtitle>([^<]*)</font><br>';
...
while(eregi($regx, $contents, $arrRoller)) {
array_push($retval, $arrRoller);
}
...
blöderweise ist das Ergebnis dann nur ein Treffer und zwar mit
id=6427821 (von der ersten Zeile) und
title=Face (von der letzten Zeile)
Ich müsste also statt dem (.*) (in der Mitte) irgendwie sagen "alles außer <font ...", damit mir zwei Treffer generiert werden.
Ich krieg das aber irgendwie nicht hin - bitte helft mir!
Geändert von davie2000 (06-10-2010 um 19:57 Uhr)
|

06-10-2010, 19:18
|
Kropff
  Administrator
|
|
Registriert seit: Mar 2002
Ort: Köln
Beiträge: 11.308
|
|
Vorher brichts du aber mal deinen Quellcode so um, dass man nicht kilometerweit nach rechts scrollen muss.
Danke
Peter
__________________
Nukular, das Wort ist N-u-k-u-l-a-r (Homer Simpson)
Meine Seite
|

06-10-2010, 19:42
|
|
h3ll
Registrierter Benutzer
|
|
Registriert seit: Mar 2008
Beiträge: 2.328
|
|
eregi() ist veraltet und sollte nicht mehr verwendet werden.
|

06-10-2010, 20:00
|
|
davie2000
Registrierter Benutzer
|
|
Registriert seit: Oct 2010
Beiträge: 5
|
|
@Kropff:
sorry, dachte das code-tag sorgt für den umbruch -> umbrüche manuell hinzugefügt (kommen aber im original-quellcode nicht vor!)
@h3ll:
ich weiss, dass eregi() veraltet und deprecated ist, aber es handelt sich um ein bestehendes skript, das ich erweitern/nutzen möchte und für alle bisherigen adaptionen hat eregi (immer noch) wunderbar funktioniert.
danke für eure hilfe ...
|

06-10-2010, 20:37
|
Kropff
  Administrator
|
|
Registriert seit: Mar 2002
Ort: Köln
Beiträge: 11.308
|
|
Oh Mann, der Code ist ja uralt. Mittleres Paläolithikum vermute ich mal. Also zunächst mal vergisst du eregi ganz schnell. Wenn der Code sauber ist, so geht das mit einen regulären Ausdruck:
PHP-Code:
preg_match_all('/productinfo.asp\?pid=([0-9]{1,})&style=ice">(.*)<font class=gridtitle>(.*)<\/font>/Usi',
$str, $hit, PREG_SET_ORDER);
for ($i = 0; $i < count($hit); $i++)
{
echo $hit[$i][1].'-'.$hit[$i][3].'<br>';
}
Allerdings empfehle ich dir, doch lieber auf DOM XML zurückzugreifen. Ich weiß allerdings nicht, wie das auf die fehlenden Gänsefüßchen bei den Attributen reagiert. Möglicherweise knallt es da gewaltig. Hatte in dem Zusammenhang noch nie mit so einem Schrottcode zu tun.
Peter
PS: $str steht für den Grottencode.
__________________
Nukular, das Wort ist N-u-k-u-l-a-r (Homer Simpson)
Meine Seite
Geändert von Kropff (06-10-2010 um 20:40 Uhr)
|

06-10-2010, 21:02
|
|
davie2000
Registrierter Benutzer
|
|
Registriert seit: Oct 2010
Beiträge: 5
|
|
Danke für deine Antwort, aber wie ich schon in meiner Antwort an h3ll geschrieben habe, handelt es sich um ein (viell. schon älteres *g*) coding einer bestehenden und ansonsten voll funktionierenden Datenbankapplikation.
Der "Grottencode" stammt von mir, wenn du damit den regex meinst - ich sagte ja bereits, dass regex absolut nicht meine Stärke ist.
Wenn du eregi() meinst, dann betrachte das einfach als Vorgabe "von oben"; ich muss eregi (weiter)verwenden und möchte die bestehenden Klassen auch nicht auf pregmatch umbauen, weil dann wieder alle anderen Fälle getestet und evtl. umgebaut werden müssen. Und wie gesagt für diese anderen/bisherigen Fälle funktioniert der "Grottencode" ja einwandfrei.
Der Tipp mit dem DOM-Parser nützt mir hier leider auch nichts; bei _meinen_ eigenen (fetch-)Skripten nutze ich natürlich diese sehr komfortable Möglichkeit ;-)
Wäre also sehr dankbar für einen reg. Ausdruck, der für obiges Problem auch mit eregi() funktioniert.
TIA
|

06-10-2010, 21:25
|
Kropff
  Administrator
|
|
Registriert seit: Mar 2002
Ort: Köln
Beiträge: 11.308
|
|
Ob du nun eregi oder preg_match_all nutzt, ist eigentlich lattens. Du musst nur die Schleifenverarbeitung umschreiben.
Zitat:
|
Wenn du eregi() meinst, dann betrachte das einfach als Vorgabe "von oben"
|
Dann sag denen "da oben", dass ereg Müll ist. Und bei weitem nicht so flexibel ist wie preg_match.
Zitat:
|
Wäre also sehr dankbar für einen reg. Ausdruck, der für obiges Problem auch mit eregi() funktioniert.
|
Sorry, damit habe ich zum letzten Mal vor etwa 7 Jahren gearbeitet. Außerdem denke ich, dass die Applikation mal ein Refactoring notwendig hat. Man kann nicht mit uraltem Code immer so weiter machen.
Peter
Zitat:
|
Der "Grottencode" stammt von mir, wenn du damit den regex meinst
|
Ich meinte den HTML-Code
__________________
Nukular, das Wort ist N-u-k-u-l-a-r (Homer Simpson)
Meine Seite
Geändert von Kropff (06-10-2010 um 21:32 Uhr)
|

06-10-2010, 22:31
|
|
davie2000
Registrierter Benutzer
|
|
Registriert seit: Oct 2010
Beiträge: 5
|
|
so ... jetzt habe ich mich doch breitschlagen lassen *g* und für diesen speziellen Fall auf preg_match_all() umgestellt und siehe da: es funktioniert (wie erwartet)
Dann werde ich jetzt wohl doch versuchen ganz auf preg_match_all umzusteigen und die "alten Fälle" austesten müssen.
Danke nochmals für deine Hilfe!!!
PS: der HTML-Code ist wirklich äußerst unansehlich ... wird aber von einer (sehr großen) site bis heute so ausgegeben.
PPS: mein Problem wurde gelöst - thread kann von mir aus geschlossen werden - danke
Geändert von davie2000 (06-10-2010 um 22:38 Uhr)
|

06-10-2010, 22:42
|
|
h3ll
Registrierter Benutzer
|
|
Registriert seit: Mar 2008
Beiträge: 2.328
|
|
Zitat:
Zitat von davie2000
der HTML-Code ist wirklich äußerst unansehlich ...
|
Nicht nur das. Er ist auch fehlerhaft. Pro Zeile HTML-Code komm ich locker auf über 5 Fehler.
|

06-10-2010, 22:56
|
|
davie2000
Registrierter Benutzer
|
|
Registriert seit: Oct 2010
Beiträge: 5
|
|
ja, du hast recht ... und das waren nur zwei beliebige Zeilen aus dem Dokument ... einen w3c-Validator überlebt die Seite sicherlich nicht, aber ist ja nicht meine ;-)
ich greif nur die Daten ab und das funktioniert jetzt - dank deiner Hilfe - auch für diese pfui-Seite einwandfrei
|
|
Aktive Benutzer in diesem Thema: 1 (Registrierte Benutzer: 0, Gäste: 1)
|
|
|
| Themen-Optionen |
|
|
| Thema bewerten |
|
|
Forumregeln
|
Es ist Ihnen nicht erlaubt, neue Themen zu verfassen.
Es ist Ihnen nicht erlaubt, auf Beiträge zu antworten.
Es ist Ihnen nicht erlaubt, Anhänge hochzuladen.
Es ist Ihnen nicht erlaubt, Ihre Beiträge zu bearbeiten.
HTML-Code ist aus.
|
|
|
|
PHP News
|