[?] strip tags programm

estartu

Schön, wenn es jetzt geht.
@Mr.B meinen Artikel habe ich schon durchkorrigiert und hoffentlich nichts übersehen.

Shade Of Mine

mein fehler. statt j=1 muss es j=0 heissen (in der for schleife). das ist noch von der argv version übriggeblieben. sorry.

estartu

Okay, ich habs korrigiert. Jetzt funktioniert es auch und die neue Version ist hochgeladen.

virtuell Realisticer

Jetzt stuertzt es wieder bei mir ab, "TODO: <Dateibeschreibung> hat ein Problem
festgestellt und muss beendet werden".

Aber vielleicht geht das wieder weg, wie mit der anderen Version ;).

mfg
v R

estartu

Hab ich mich mit den Zeigern irgendwo verzeigert? Oder auch Shade?
Ist ja ärgerlich...

Shade Of Mine

estartu_de schrieb:

Hab ich mich mit den Zeigern irgendwo verzeigert? Oder auch Shade?
Ist ja ärgerlich...

habs nachgeprüft und sehe bei mir keinen fehler... was natürlich nicht heisst, dass der code korrekt sein muss

gib mal beispiel text bei dem es abschmiert. was ist zB wenn bilder statt text in der zwischenablage sind? nur so als anregung...

reproduzierbare fehler wären klasse

estartu

Da ich mich erstmal um anderes kümmern muss, hier der Code.
Aus dem und Shades Funktion besteht es.

BOOL CStripTagsApp::InitInstance()
{
	CWinApp::InitInstance();

	if (OpenClipboard(NULL)) 
	{
		// Retrieve the Clipboard data (specifying that 
		// we want ANSI text (via the CF_TEXT value).
		HANDLE hClipboardData = GetClipboardData(CF_TEXT);
		// Call GlobalLock so that to retrieve a pointer
		// to the data associated with the handle returned
		// from GetClipboardData.
		char *pchData = (char*)GlobalLock(hClipboardData);
		char* pOut = new char[strlen(pchData)+1];
		char* pTags[] = {"[kor]","[/kor]"};

		stripBBTags(pchData, pOut, 2, pTags);

		// Unlock the global memory.
		GlobalUnlock(hClipboardData);
		// Leermachen
		EmptyClipboard();
		// Speicher reservieren (incl. \0)
		HGLOBAL hCpDaten = GlobalAlloc(GMEM_DDESHARE, strlen(pOut)+1);
		// Daten in systemweiten Speicher schieben
		pchData = (char*)GlobalLock(hCpDaten);
		strcpy(pchData, pOut);
		// Für andere zur Verfügung stellen
		GlobalUnlock(hCpDaten);
		// Daten in die Zwischenablage schieben
		SetClipboardData(CF_TEXT, hCpDaten);
		// Fertig
		CloseClipboard();
	}
	return FALSE;
}

virtuell Realisticer

Shade Of Mine schrieb:

estartu_de schrieb:

Hab ich mich mit den Zeigern irgendwo verzeigert? Oder auch Shade?
Ist ja ärgerlich...

habs nachgeprüft und sehe bei mir keinen fehler... was natürlich nicht heisst, dass der code korrekt sein muss

gib mal beispiel text bei dem es abschmiert. was ist zB wenn bilder statt text in der zwischenablage sind? nur so als anregung...

reproduzierbare fehler wären klasse

Es stuertzt ab, sobald ich es starte.

mfg
v R

[edit]
Ok, das Programm stuertzt ab, wenn keine Daten im Zwischenspeicher vorhanden sind.
[/edit]

Shade Of Mine

Fehler gefunden.
NULL Pointer fehler. ts ts ts

GlobalAlloc liefert NULL wenn es einen 'Fehler' gab. Und einen Fehler gibt es dann wenn GetClipboardData ein ungueltiges Handle liefert wenn nix in der Zwischenablage liegt.

Floglich schmiert strlen(NULL) dann ab.

btw:
2 GlobalLock
1 GlobalAlloc
und nur ein
1 GlobalUnlock

sicher dass das richtig ist...?

estartu

Ich guck mir das Montag noch mal an.
Damit wird erstmal was haben, habe ich das aus Codeproject kopiert und so lange angepasst, bis das gewünschte Ergebnis raus kam.

Hier habe ich keinen Compiler, aber wenn man weiß, dass was kopiert sein sollte, müsste es ja so auch erstmal gehen.

Mr. B

wo gibts die neuste exe zum download? link...

Mr. B

estartu

Sorry, ich habe gestern einen mittelgroßen Ringkampf mit der MSDE ausgetragen.
Geht gleich los, ich geb hier Bescheid.

Der Link ist immer der selbe, ich überschreibe die alte Version einfach.

Wow, das war ja einfach. Ich habe es jetzt mit einem Screenshot in der Zwischenablage getestet und es macht dann einfach nichts.
Oder wünscht sich jemand eine Meldung?

virtuell Realisticer

estartu_de schrieb:

Sorry, ich habe gestern einen mittelgroßen Ringkampf mit der MSDE ausgetragen.
Geht gleich los, ich geb hier Bescheid.

Der Link ist immer der selbe, ich überschreibe die alte Version einfach.

Wow, das war ja einfach. Ich habe es jetzt mit einem Screenshot in der Zwischenablage getestet und es macht dann einfach nichts.
Oder wünscht sich jemand eine Meldung?

Och ich denke, dass ist nicht noetig

mfg
v R