serialport i en class (visual c++)

eyeR_sp skal vel aedres fra lokal variabel til et field i klassen?

hvordan gør man det?

ref class B
{
    private:
        A^ a;
    public:
        // metoder
};

Nu kan a bruges i alle metoder.

jeg tror nok det næsten er magen til det jeg har.. jeg har bare delt det op i 2 filer... en header og en cpp fil...

Header:
ref class eyeR
{
public:
    static void connect(void);
    void disconnect(void);
    void write_servo_new_def(int, int, int, int, int);

};



cpp-fil:
#include "stdafx.h"
#include "classes.h"

using namespace System::IO::Ports;

void eyeR::connect(void)
{
    SerialPort^ eyeR_sp = gcnew SerialPort();
    if(eyeR_sp->IsOpen == true) { eyeR_sp->Close(); } // tjekker om porten allerede er åben og lukker den hvis det er tilfældet
    eyeR_sp->PortName = "COM6";
    eyeR_sp->BaudRate = 57600;
    eyeR_sp->DataBits = 8;
    eyeR_sp->Open();
}
void eyeR::disconnect()
{
}
void eyeR::write_servo_new_def(int servo_number, int min_raw_value, int max_raw_value, int index_num, int initial_index )
{
}


burde det ikke lave alle funktionerne public?

for det hjælper ihvertfald ikke

Header:
ref class eyeR
{
private:
    SerialPort^ eyeR_sp;
public:
    static void connect(void);
    void disconnect(void);
    void write_servo_new_def(int, int, int, int, int);

};



cpp-fil:
#include "stdafx.h"
#include "classes.h"

using namespace System::IO::Ports;

void eyeR::connect(void)
{
    eyeR_sp = gcnew SerialPort();
    if(eyeR_sp->IsOpen == true) { eyeR_sp->Close(); } // tjekker om porten allerede er åben og lukker den hvis det er tilfældet
    eyeR_sp->PortName = "COM6";
    eyeR_sp->BaudRate = 57600;
    eyeR_sp->DataBits = 8;
    eyeR_sp->Open();
}
void eyeR::disconnect()
{
}
void eyeR::write_servo_new_def(int servo_number, int min_raw_value, int max_raw_value, int index_num, int initial_index )
{
}

det giver en masse fejl
hvis jeg så tilføjer namespace't i headerfilen:
using namespace System::IO::Ports;

så kommer der bare nogle andre fejl:
1>.\classes.cpp(8) : error C2597: illegal reference to non-static member 'eyeR::eyeR_sp'
1>.\classes.cpp(9) : error C2227: left of '->IsOpen' must point to class/struct/union/generic type
1>.\classes.cpp(9) : error C2227: left of '->Close' must point to class/struct/union/generic type
1>.\classes.cpp(10) : error C2227: left of '->PortName' must point to class/struct/union/generic type
1>.\classes.cpp(11) : error C2227: left of '->BaudRate' must point to class/struct/union/generic type
1>.\classes.cpp(12) : error C2227: left of '->DataBits' must point to class/struct/union/generic type
1>.\classes.cpp(13) : error C2227: left of '->Open' must point to class/struct/union/generic type

Kan du poste begge filerne komplette?

Jeg tror at det er en ganske lille ting der skal rettes.

det er de komplette filer... jeg inkluder bare header filen i min windows form header og bruger den der

Det virker med de rette using og hvis connect laves ikke-static.

#using <mscorlib.dll>

using namespace System::IO::Ports;

ref class eyeR
{
private:
    SerialPort^ eyeR_sp;
public:
    void connect(void);
    void disconnect(void);
    void write_servo_new_def(int, int, int, int, int);

};

#using <mscorlib.dll>
#using <system.dll>

using namespace System::IO::Ports;

#include "foobar.h"

using namespace System::IO::Ports;

void eyeR::connect(void)
{
    eyeR_sp = gcnew SerialPort();
    if(eyeR_sp->IsOpen == true) { eyeR_sp->Close(); } // tjekker om porten allerede er åben og lukker den hvis det er tilfældet
    eyeR_sp->PortName = "COM6";
    eyeR_sp->BaudRate = 57600;
    eyeR_sp->DataBits = 8;
    eyeR_sp->Open();
}

void eyeR::disconnect()
{
}

void eyeR::write_servo_new_def(int servo_number, int min_raw_value, int max_raw_value, int index_num, int initial_index )
{
}

så giver den fejl i min form1.h fil istedet...

den siger:
1>c:\users\bjarne\documents\visual studio 2008\projects\eyer windows\eyer windows\Form1.h(390) : error C2352: 'eyeR::connect' : illegal call of non-static member function
1>        c:\users\bjarne\documents\visual studio 2008\projects\eyer windows\eyer windows\classes.h(10) : see declaration of 'eyeR::connect'

og det er lige meget om jeg har using af de 2 dll filer med, det giver samme fejl med og uden de usings

i form1.h står der:
private: System::Void connectToolStripMenuItem_Click(System::Object^  sender, System::EventArgs^  e) {
            eyeR::connect(); //connect til eyeR

        }

Problemet må være her:

eyeR::connect(); //connect til eyeR

eyeR er en class men skulle være en instance.

hjælper ikke.

hvis jeg tilføjer:
eyeR hej();
hej.connect();

skriver den bare:
1>c:\users\bjarne\documents\visual studio 2008\projects\eyer windows\eyer windows\Form1.h(392) : error C2228: left of '.connect' must have class/struct/union

Den kode jeg postede i #9 compiler fint.

At det muligvis giver fejl i noget andet kode som du ikke har vist eller forklaret om er svaert at forhindre.

Ja. Med den aendring kraver det en instans af klassen.

Du skal sikre dig at Form1.h inkluderer .h filen der definerer klassen.

Og proev saa med:

eyeR^ hej = gccnew eyeR();
hej->connect();

jeg har en include i toppen af form1.h så det skulle være i orden
men det melder stadig fejl hvis jeg bruger gcnew... så sker der bare dette:

1>c:\users\bjarne\documents\visual studio 2008\projects\eyer windows\eyer windows\Form1.h(17) : error C2072: 'eyeRwindows::hej' : initialization of a function
1>c:\users\bjarne\documents\visual studio 2008\projects\eyer windows\eyer windows\Form1.h(17) : error C2440: 'initializing' : cannot convert from 'eyeR ^' to 'eyeR ^(void)'

Hvordan ser den relevante kode i Form1.h ud nu?

der er faktisk ikke den store kode i filen

det er bare en form hvor der er lavet nogle knapper og en progress bar men der er ikke tilføjet nogen funktioner til dem endnu...

har så en knap (toolstrip) der hedder connect som skal kalde connect kommandoen

eyeR^ hej = gccnew eyeR(); og den her linje har jeg smidt helt oppe for oven hvor alle de andre using namespace xx er...

eyeR^ hej = gccnew eyeR();

skal være i koden hvor du skal bruge et eyeR objekt. Ikke oppe mellem using.

hvis jeg smider dem samme sted giver den bare 3 fejl istedet:

1>c:\users\bjarne\documents\visual studio 2008\projects\eyer windows\eyer windows\Form1.h(98) : error C2072: 'hej' : initialization of a function
1>c:\users\bjarne\documents\visual studio 2008\projects\eyer windows\eyer windows\Form1.h(98) : error C2205: 'hej' : cannot initialize extern variables with block scope
1>c:\users\bjarne\documents\visual studio 2008\projects\eyer windows\eyer windows\Form1.h(98) : error C2440: 'initializing' : cannot convert from 'eyeR ^' to 'eyeR ^(void)'

btw det skal vel være gcnew og ikke gccnew?

nu fik jeg det til at virke...

det var mig der havde kopieret linjen og glemt at slette den for oven...

men vil det sige at jeg skal lave en ny instance af class'en hver gang jeg skal bruge den et nyt sted?

Ja. gccnew var en skrivefejl.

Du skal lave en ny instans af klassen hver gang du skal bruge en.

Måske opretter du en ny i hver Form1 metoder der skal bruge den.

Måske opretter du eb i Form1 constructor og bruger den samme i alle metoder.

Det afhænger af hvilken funktionalitet/logik du har brug for.

Hej
  Jeg tror du skal læne dig tilbage og overveje hvad det er din funktion skal gør.
  Hvis der kun skal være en instance af classen så kan du bruge den samme instance over det hele hvis den er defineret det rigtige sted, hvis der er noget der skal gælde for alle instances af classen kan du lave noget static function.
  Sandsynligvis skal du bare oprette en instance mm. du bruger forskellige methoder med forskellige parameter, hvilket din origanale post ikke antyder.

har forsøgt at oprette den i contructoren for form1 men så kan den ikke findes der hvor jeg skal bruge den

jeg har kun brug for en instance af classen men den skal bare bruges mange steder i form1

Har du lavet det som et felt i klassen som bliver initaliseret i constructor?

er ikke sikker på jeg forstår hvad du mener men har prøvet på ldit forskellige måder... lige nu står det sådan her:
    public:
        Form1(void)
        {
            InitializeComponent();
            //
            //TODO: Add the constructor code here
            eyeR^ hej = gcnew eyeR();
            //
           
        }

det giver ingen fejl men kan heller ikke bruge den... de andre ting jeg har prøvet meldt alt sammen fejl

private:
        eyeR^ hej;
public:
        Form1(void)
        {
            InitializeComponent();
            //
            //TODO: Add the constructor code here
            hej = gcnew eyeR();
            //
         
        }

hmm ja så virker det... hvorfor er det at den skal stå i private for at det virker?

Det afgoerende er at det kommer ud som felt i klassen fremfor at vaere lokal variabel.

At den er i private fremfor public er kun en detalje - jeg valgte det fordi det er god skik og brug.

okay tak for hjælpen
smider du et svar så får du point..

svar