Verstehe Fehlermeldung von Visual Studio C++ 2005 nicht

Tiger schrieb:

> -ComTerm.exe!CSerialPortChild::ThreadFunction(void * ptr=0xcdcdcdcd)

                                                           **********

Du übergibst eine ungültige Adresse.
Der Fehler liegt also beim Aufrufer.

Da fehlt noch einiges an Quelltext.
Wo ist der Start des Threads? Was wird da übergeben?

Außerdem sollte in einer Klasse, in der mindestens eine virtuelle
Methode ist, IMMER der Destruktor auch virtuell sein.
Hat damit nichts zu tun, aber fiel mir so auf.

Klaus Wachtler schrieb:
> Da fehlt noch einiges an Quelltext.
> Wo ist der Start des Threads? Was wird da übergeben?
>
> Außerdem sollte in einer Klasse, in der mindestens eine virtuelle
> Methode ist, IMMER der Destruktor auch virtuell sein.
> Hat damit nichts zu tun, aber fiel mir so auf.

Weiters:
Wenn eine Funktion in der Basisklasse virtual ist, sollte man die 
gleichnamige Funktion in der abgeleiten Klasse ebenfalls explizit 
virtual machen.

ja, wobei das nur zur Klarheit ist; virtual ist sie sowieso.

bin raus, ich kann kein rar

> Wenn ich es so mache, wie oben gezeigt
> dann erscheint folgende Fehlermeldung:
>
> Unbehandelte Ausnahme bei 0x00411b35 in ComTerm.exe: 0xC0000005:
> Zugriffsverletzung beim Schreiben an Position 0xcdcdd1cd.

...

Wenn Du das im Debugger laufen lässt, dann siehst Du auch, wie die 
Funktion mit welchen Parametern aufgerufen wurde. Wie schon von Karl 
Heinz beschrieben, ist der Pointer, der "ThreadFunction" übergeben 
wurde, nicht initialisiert.


Der von Dir gepostete Code ist übrigens unvollständig, es fehlt 
"SerialPortChild.cpp".

- auch im ZIP-Archiv.

Du brauchst doch nur am Call Stack zurückgehen und dir ansehen, wer die 
ThreadFunction aufruft und welches Argument er ihr mitgibt. Die nächste 
Frage ist dann, wo kommt dieses Argument her und warum wurde es nie 
initialisiert.

Wenn du dich mit Threads beschäftigst, solltest du solche Dinge schon 
können. Ansonsten treiben dich die schwierigeren Dinge bei Threading in 
den Wahnsinn.

ist das die SerialPortChild.cpp?
Glaube, mein Zipper ist kaputt.

Tiger schrieb:
> Und wie müsste ich den Pointer von der Methode "ThreadFunction"
> initialisieren?


Je nachdem was du eigentlich machen willst.

Im Moment wird die Threadfunction aufgerufen und ihr der Callbackpointer 
mitgegeben m_callback. Der kriegt aber nie einen Wert (ausser NULL). Und 
damit geht dann der Zugriff precv->len = 0; mächtig in die Hose.

Da dein Stack Trace einen Pointer Wert von 0xcdcdcdcd zeigt, in dem von 
dir geposteten Code m_callback aber auf 0 initialisiert wird und ich bei 
mir im Debugger den Pointer auch tatsächlich als 0x000000 sehe, gehe ich 
davon aus, dass du den Code seit deinem ersten Post verändert hast.


Mach dich mit deinem Debugger vertraut! Der ist der Helfer in der Not. 
Ohne mit ihm umgehen zu können, brauchst du nicht weitermachen.

Wo ist denn nun der Start des Threads? Dort kann man doch einen
Parameter angeben (void* oder LPVOID oder sowas), der an die
Threadfunktion durchgereicht wird. Und genau da müsstest du
die Adresse deiner Struct übergeben.

> Ich weiss nicht wo ich wie und was übergeben muss.
Müssen tust du gar nichts.
Du bist der Boss, du entscheidest, welche Daten wo in welche Funktion 
übergeben werden. Aber kein Mensch schreibt dir vor, was du tun musst.


Lass mich raten.

Code von irgendwo her kopiert. Nicht duchgeschaut und auch noch keinen 
Versuch gemacht das zu verstehen.

So funktioniert Programmieren nicht. Klar kann man Code von irgendwo her 
kopieren. Aber man muss ihn auch verstehen. Und das mindeste ist es, im 
Debugger einmal duchzusteppen.

Tiger schrieb:
> Nein dem ist nicht so. Ich hab mal vor einiger Zeit mit einem
> Echtzeitbetriebssystem zu tun gehabt.

Jetzt bin ich verwirrt.

Du schreibst einen relativ banalen 100-Zeiler und blickst dann selbst 
nicht mehr durch, warum eine Funktion einen ungültigen Pointer bekommt?

So, und wenn ich das richtig sehe, ist der vierte Parameter
(bei dir: this) das, was als void-Parameter an die
Threadfunktion weitergereicht wird.
Darin castet du das in einen Zeiger auf eine struct, was natürlich
schief geht.

Wenn ich jetzt nicht falsch liege, müsstest du als 4. Parameter die
Adresse der gewünschten struct angeben.

Was hattest du mit dem this an dieser Stelle vor?

Dazu müsste man wissen, wie deine Objekte zusammenhängen und wie
alles zusammen spielen soll. Eigentlich müsstest du das wissen;
ich habe gerade keine besondere Lust, das aus einem
undokumentierten Quelltext zu rekonstruieren.

Rein technisch geht es notfalls über eine globale Variable, die
jemand vorher setzt (sicher der schlechteste Weg), oder irgendwelche
get- oder set-Methoden, über die Thread das rechtzeitig erfährt.

Gibt es sicher, aber ist auf meinem Rechner nicht drauf.
Und ehrlich gesagt muß ich auch nicht jedes obskure Datenformat
handhaben, solange es ein paar verbreitete und frei zugängliche
Formate gibt. RAR ist meines Wissens nicht frei (zumindest das
Komprimieren).
Warum nicht ZIP?

Tiger schrieb:
>

1

> 

2

> DWORD WINAPI Thread::my_Thread(void* para)

3

> {

4

>     Thread* pthis=(Thread*)para;

5

>   int t;

6

>   if(pthis!=NULL)

7

>   {

8

>     pthis->ThreadFunction(pthis->m_callback);

9

>     t=9;

10

>     return 0;

11

>   }

12

>   return -1;

13

> }

14

> 

15

>

>
> Was ist hier bitte schön falsch daran?

Da ist gar nichts falsch daran.

pthis->m_callback wird übergeben.

Aber pthis->m_callback wird nie auf einen Wert gesetzt!
Das würde über einen Aufruf von SetCallback gemacht, aber die Funktion 
wird nie aufgerufen.

Schuss ins Blaue:

CSerialPortChild::CSerialPortChild():CSerialPort()
{
  SetThreadCallBack( &this->recv );
}

Aber eigentlich müsstest DU das wissen. Du hast es ja designed.