Daten loggen mit PCANBasicExample in Visual Studio 2008

[CANorNotCAN (Gast)]

Hallo zusammen,

ich möchte CAN-Messages loggen. Als Grundgerüst verwende ich das Example 
von PCAN-Basic API in Visual Studio 2008. Die ReadMessages-Prozedur will 
ich so verändern, dass die Messwerte nicht nur gelesen, sondern auch in 
einer csv-Datei gespeichert werden:

#include <fstream>
#include <string>
#include <sstream>
#include<iostream>
using namespace std;

[...]

TPCANMsg ^CANMsg;
    TPCANTimestamp ^CANTimeStamp;
    TPCANStatus ^stsResult;

    /// added code
    String ^filename, ^head;
    ofstream logfile;
    ostringstream ssFilename;
    ///

    // Initializes Variables
    //
    CANTimeStamp = gcnew TPCANTimestamp();
    CANMsg = gcnew TPCANMsg();
    filename = fileName->Text;
    //

    ssFilename.str("");
    ssFilename.clear();
    ssFilename << &filename << "_" << &CANTimeStamp << ".csv";
    filename=ssFilename.str();
    logfile.open(filename.c_str(),ios_base::out | ios_base::app);
    
    head="Time";
    for(int i=1;i<13;i++)
    {
    head=strcat_s(head,", ");
    head=strcat_s(head,"Cell",num2str(i));
    }

    logfile << head  << endl << "Start date: " << CANTimeStamp << endl;
    /// end of added code

    // We read at least one time the queue looking for messages.
    // If a message is found, we look again trying to find more.
    // If the queue is empty or an error occurr, we get out from
    // the dowhile statement.
    //      
    do
    {
      // We execute the "Read" function of the PCANBasic                
      //
      stsResult = PCANBasic::Read(*m_PcanHandle, *CANMsg, *CANTimeStamp);

      // A message was received
      // We process the message(s)
      //
      if (*stsResult == TPCANStatus::PCAN_ERROR_OK)
        ProcessMessage(CANMsg, CANTimeStamp);

      /// added code
      if (activateLogging->Checked)
        for(int i=0;i<12;i++)    // hier ggf. anzahl der angeschlossenen zellen automatisiert einlesen und einfügen
        {
          logfile << CANMsg << ", ";
        }
        logfile << endl;
        //LogMessages(filename,CANMsg,logfile);
      /// end of added code

    } while (btnRelease->Enabled && (!Convert::ToBoolean(*stsResult & TPCANStatus::PCAN_ERROR_QRCVEMPTY)));
    
  logfile.close();

Allerdings funktioniert dieser Code nicht. Es liegt wohl (in erster 
Linie) an den ^strings (Fehlermeldung: error C2440: '=': 
'std::basic_string<_Elem,_Traits,_Ax>' kann nicht in 'System::String ^' 
konvertiert werden).

Kann mir da jemand helfen? Wenn es eine andere funktionstüchtige 
Log-Prozedur für dieses Beispiel gibt, nur her damit. Ich konnte leider 
nichts Passendes finden...

Danke im Voraus!

[auch PCAN-user (Gast)]

convertiere doch erst einmal die empfangen Daten von ihrem jeweiligen 
Format in das format string!

Die CANmsg ist eine structur die verschiedene Datentypen beinhaltet, 
also z.b. CANmsg.ID ist ein int, CANmsg.Data[0] bis CANmsg.Data[7] sind 
chars.

Das alles kann man nicht in einen string schreiben, du musst jeden Typ 
explizit und einzeln nach sting wandeln!

Habe damals auch das example als Grundgerüst für meine Software 
hergenommen, allerdings habe ich meine routinen damals in C# 
geschrieben, kann dir also leider kein Beispiel in C geben!

[CANorNotCAN (Gast)]

Danke für den Hinweis. Allerdings haben sich jetzt die Fehlermeldungen 
erhöht. Das Problem liegt wohl im Unterschied zwischen C# und C++ (und 
meinem mangelnden Wissen bzgl. der Übersetzung zwischen den Sprachen). 
Ich habe folgende Konvertierung versucht:

// string coversion of CAN messages
    for (int i=0;i<2;i++)
    {
      stringCAN[i] = gcnew String(CANMsg->Data(i));
      stringCANTimeStamp = gcnew String(CANTimeStamp);
    }
//

Da meckert der Compiler:

 error C2882: 'Data': Unzulässige Verwendung eines Namespace-Bezeichners in einem Ausdruck

Vorher hatte ich es versucht es über ToString zu konvertieren. Da waren 
die Fehler aber noch schlimmer...

Grüße

[auch PCAN-user (Gast)]

versuch mal die einzelnen datenbytes seperat zu konvertieren

..bin allerdings kein c++ spezialist

void datenschreiben (void)
{
TPCANMsg CANMsg;
string text;


   text = "";
   text  = Convert.ToString(CANMsg.DATA[0]);
   text |= Convert.ToString(CANMsg.DATA[1]);
 ...
}

[... (Gast)]

auch PCAN-user schrieb:
> ..bin allerdings kein c++ spezialist

Macht nichts, das was der TO da macht ist ja auch kein C++.

[Rufus Τ. F. (rufus)]

CANorNotCAN schrieb:
> das Problem liegt wohl im Unterschied zwischen C# und C++

Was Du da nutzt, ist kein C++, sondern eine Microsoft-Perversion 
namens "Managed C++" bzw. "C++/CLI". Sieht zwar so ähnlich aus, heißt 
auch so ähnlich, ist aber kein C++, sondern eine Mutation für den 
.Net-Unterbau.

Das hast Du bei Deinen Projekteinstellungen berücksichtigt?

[CANorNotCAN (Gast)]

@ auch PCAN-user:

an welcher Stelle (Prozedur) hast du denn geloggt?

@...:

... schrieb:
> Macht nichts, das was der TO da macht ist ja auch kein C++.

"der TO"?!

[... (Gast)]

CANorNotCAN schrieb:
> ... schrieb:
>> Macht nichts, das was der TO da macht ist ja auch kein C++.
>
> "der TO"?!

TO = Threadopener
Genau Du warst damit gemeint :)

[PCAN-user (Gast)]

Das example für C# hat die Funktion tmrRead_Tick welche eben zyklisch 
den PCAN-Dongel auf empfangene Zeichen prüft, dort wird die Funktion
"ProcessMessage(TCLightMsg MyMsg)" aufgerufen.

Hier wird bereits die Struktur TCLightMsg (Deine TPCANMsg) übergeben, 
und hier habe ich dann die CAN-Telegramme weiter verwendet und in ein 
.csv gespeichert.

        private void tmrRead_Tick(object sender, EventArgs e)
        {
            TCLightMsg MyMsg;
            CANResult Res;

            // We read at least one time the queue looking for messages.
            // If a message is found, we look again trying to find more.
            // If the queue is empty or an error occurr, we get out from
            // the dowhile statement.
            //      
            do
            {
                // We read the queue looking for messages.
   --->        Res = CANLight.Read(ActiveHardware, out MyMsg);

                // A message was received
                // We process the message(s)
                if (Res == CANResult.ERR_OK)
   --->            ProcessMessage(MyMsg);
            } while (!Convert.ToBoolean(Res & CANResult.ERR_QRCVEMPTY));
        }


....................................................


        private void ProcessMessage(TCLightMsg MyMsg)
        {
            InsertMsgEntry(MyMsg);

            if (JitterMessungenSoll != 0)
            {
                if (MyMsg.ID == NodeID)
                {
                    UpdatePosition(MyMsg);
                }
            }


            if (tmrSync.Enabled == true)
            {
                if ((MyMsg.ID == FromHex(tbAllgmCob1.Text)) ||
                   (MyMsg.ID == FromHex(tbAllgmCob2.Text)) ||
                   (MyMsg.ID == FromHex(tbAllgmCob1.Text)))
                {
                   UpdateTabAllgm(MyMsg);
                }
            }
        }
..............................................................

Viel Erfolg!

[CANorNotCAN (Gast)]

Danke PCAN-User!

Mal etwas anderes: bzgl. stringcat_s kommt die Fehlermeldung:

"error C2660: 'strcat_s': Funktion akzeptiert keine 2 Argumente"

Was soll das?! Das "muss" mit den komischen ^ bei der Initialisierung 
des Strings <head> zusammenhängen. Jetzt verstehe ich, inwiefern diese 
Sprache > eine Microsoft-Perversion ist, wobei ich vermute, dass das 
etwas CAN-spezifisches sein muss, da man im Internet nichts Gescheits 
dazu findet :(.

[CANorNotCAN (Gast)]

achso ja,
das eigentlich speichern der Datei mache ich natürlich nicht im 
tmrRead_Tick, in der Unterfunktion InsertMsgEntry befülle ich ein 
array und in die Datei schreibe ich dann wenn ich mal Zeit habe, z.B. 
wenn meine Messung beendet ist oder aber wenn ich den close-button 
anklicke!!!

Das sollte aber klar sein!?

[CANorNotCAN (Gast)]

>Was soll das?! Das "muss" mit den komischen ^ bei der Initialisierung
>des Strings <head> zusammenhängen. Jetzt verstehe ich, inwiefern diese
>Sprache > eine Microsoft-Perversion ist, wobei ich vermute, dass das
>etwas CAN-spezifisches sein muss, da man im Internet nichts Gescheits
>dazu findet :(.

Ne sorry, hab leider keine Ahnung was das soll, die komischen "Häckchen" 
sind mir gestern schon in Deinem code aufgefallen, keine Ahung was die 
bedeuten!

[PCAN-user (Gast)]

ups sorry, da habe ich den falschen user-name kopiert,
die letzten beiden postings sind von mir!

[... (Gast)]

Wobei 'strcat_s' noch ne weitere Eigenheit ist, die bei M$ auch in C++ 
(ja, richtiges C++ geht mit VisualStudio auch) auftaucht. Das normale 
'strcat' gibt es zwar noch, schmeißt aber eine Warning (es sei denn man 
stellt sie via #define wieder ab). Das 'strcat_s' will als zusätzlichen 
Parameter noch die Puffergröße haben.

[Rufus Τ. F. (rufus)]

CANorNotCAN schrieb:
> Ne sorry, hab leider keine Ahnung was das soll, die komischen "Häckchen"
> sind mir gestern schon in Deinem code aufgefallen, keine Ahung was die
> bedeuten!

Diese Häkchen (mit nur einem 'c') sind ein klares Zeichen dafür, daß es 
sich hierbei nicht um C++ handelt, sondern um die .Net-Sprache 
"C++/CLI". Sieht aus wie C++, riecht auch so ähnlich, ist aber kein C++.

Das ist ein Objektzeiger, bei dem Objekte mit gcnew statt new 
angefordert werden müssen, und deren Speicher nicht von delete, 
sondern vom garbage collector abgeräumt wird.

Wie gesagt, C++ ist das nicht.

[... (Gast)]

Rufus Τ. Firefly schrieb:
> Sieht aus wie C++
Aber nur auf den ersten Blick.
> riecht auch so ähnlich
Nö, riecht eher nach Müllkippe, 'garbage collector' halt :)

[CANorNotCAN (Gast)]

Ok, nachdem ich mich ein bissl orientiert habe (MC++-Syntax etc.) gab es 
einige Fortschritte, aber jetzt hängt es wieder. Es ist ähnlich wie bei 
Schrödingers Katze, sobald ich meinen Code wieder (teilweise) 
einkommentiere, gibt es Fehlermeldungen, obwohl (!) das Programm 
eigentlich bis zu dieser Stelle ordnungsgemäß laufen sollte. Habe mich 
an dem msdn-Beispiel zu FileStream orientiert (poste es als C-Code, auch 
wenn es das nicht ist ;) ):

private: void ReadMessages()
  { 
    TPCANMsg ^CANMsg;
    TPCANTimestamp ^CANTimeStamp;
    TPCANStatus ^stsResult;

    String ^head,^path;
    FileStream ^logfile;
    StreamWriter ^streamWriter;

    // Initializes Variables
    //
    CANTimeStamp = gcnew TPCANTimestamp();
    CANMsg = gcnew TPCANMsg();

    
    path = "C:...\\test.txt"; // In dem richtigen Programmcode ist hier der richtige Pfad angegeben. Da der aber niemanden was angeht, habe ich ihn "zensiert" ;). Hier ist aber sicherlich kein Fehler!
    head="string1,string 2,string3;"; // auch hier habe ich zensiert
    if (!File::Exists(path))
    {
      logfile = gcnew FileStream(path,FileMode::Append,FileAccess::Write,FileShare::Write);
      logfile->Close();
      streamWriter = gcnew StreamWriter(path,true,Encoding::ASCII);
      streamWriter->Write(head);
      streamWriter->Write("\n");
      streamWriter->Close();
    }
    else
      File::Open(path,FileMode::Append,FileAccess::Write,FileShare::Write);

    // We read at least one time the queue looking for messages.
    // If a message is found, we look again trying to find more.
    // If the queue is empty or an error occurr, we get out from
    // the dowhile statement.
    //      
    do
    {
      // We execute the "Read" function of the PCANBasic                
      //
      stsResult = PCANBasic::Read(*m_PcanHandle, *CANMsg, *CANTimeStamp);

      // A message was received
      // We process the message(s)
      //
      if (*stsResult == TPCANStatus::PCAN_ERROR_OK)
        ProcessMessage(CANMsg,CANTimeStamp);
      else
        if ((*stsResult == TPCANStatus::PCAN_ERROR_OK) && (activateLogging->Checked))
          ProcessMessageLog(CANMsg,CANTimeStamp,path);

    } while (btnRelease->Enabled && (!Convert::ToBoolean(*stsResult & TPCANStatus::PCAN_ERROR_QRCVEMPTY)));
  
    streamWriter->Close();
        
}

Die Funktion "ProcessMessageLog" ist eine Kopie von der bestehenden 
ProcessMessage, welche ich um die Byte-konvertierung erweitert habe und 
diese dann in die datei schreibe:

...
for (int i = 0; i<8; i++)
      {
        bMessageData[i] = theMsg->DATA[i];
      }
      msgId = theMsg->ID;
      time = itsTimeStamp->millis + 0xFFFF * itsTimeStamp->millis_overflow;

...

if (msgId==0xD7)
          {
            Number = (bMessageData[0] >> 4) & 0xF;
            Vo = ((bMessageData[0] & 0xF) << 12)^ bMessageData[1];
            streamWriter->Write(time.ToString());
            streamWriter->Write(msgId.ToString());
            streamWriter-> Write(Number.ToString());
            streamWriter-> Write(Vo.ToString());
            streamWriter->Write("\n");
          }

Typische Fehlermeldung (wird bei "streamWriter->Close();" am Ende der 
ReadMessages-Prozedur angezeigt, wobei ich vermute, dass das eher 
irrelevant ist):
"Eine nicht behandelte Ausnahme des Typs "System.NullReferenceException" 
ist in PCANBasicExample.exe aufgetreten.
Zusätzliche Informationen: Der Objektverweis wurde nicht auf eine 
Objektinstanz festgelegt."

[Rufus Τ. F. (rufus)]

In Deinem "ReadMessages wird "streamWriter" mehrfach nacheinander 
geschlossen.

Ob das so sein soll?

[CANorNotCAN (Gast)]

das problem war, dass es mehrfach nacheinander aufgerufen wurde. habe es 
bissl umgestellt (bspw. wird der filestream jetzt zu beginn bei der 
globalen objekt-initialisierung angelegt, sodass ich diesen weiter unten 
in ProcessMessages mehrfach aufrufen kann). es klappt jetzt erstmal 
hinreichend gut, sodass ich erste tests machen kann. dem realen 
härtetests wird der code wohl nicht standhalten können, da bin ich mir 
jetzt schon zimelich sicher ;).
danke für die hilfe...