int imellem funktioner

Det kommer vel ret meget an på om, hvorfor og hvornår SetServol bliver kaldt og med hvad i forhold til din Update_Tick.

SetServo bliver kaldt hver gang der kommer en kommando ind som fortæller hvad der skal flyttes, og hvor meget. SetServo får det så behandlet, og sendt ud til Serial porten.

Timeren som tjekker den int, er sat til 1000

efter at SetServo har udført en handling, retunerer den dette

return "Channel: " + channel + nl + "Speed: " + ramp + speedTemp + nl + "Position: " + channel00 + positTemp;

Hvor nl er linieskift. Der retunerer den channel00 rigtigt nok.

Mere kode needed ...

Sikker på det er samme reference af klassen du bruger ?

http://www.google.dk/#hl=da&ei=n1mQSovzPMn2-Qb05djyDQ&sa=X&oi=spell&resnum=0&ct=result&cd=1&q=.net+value+reference+types&spell=1&fp=358d9f30dfb4c6a2

Brug:
System.Environment.NewLine til din newline ...

Hvor der bliver kaldt fra

using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Text;
using System.Windows.Forms;
using SamplePlugin;
using System.IO.Ports;
using System.IO;
using System.Net;
using ServoController;

namespace SamplePlugin
{
    public partial class SamplePluginForm : Form
    {
        ServoControl sc = new ServoControl(); //Set instance for the ServoControl class

        public SamplePluginForm(bool haveLicense, string owner)
        {
            InitializeComponent();
        }


        private void channelsUpdate_Tick(object sender, EventArgs e)
        {
            progressBar00.Value = sc.channel00;
        }
    }
}


Det som bliver kaldt

using System;
using System.Collections.Generic;
using System.Text;
using System.IO.Ports;
using System.IO;
using ParallaxPlugin;

namespace ServoController
{
    public class ServoControl
    {
        //Positions of servos stored here
        public int channel00 = 750;

        public string SetServo(int channel, int ramp, int position, string function) //Position between 280 and 1170
        {
            string nl = System.Environment.NewLine;
            if (channel == 0)
            {
                channel00 = position;
            }
            return "Channel: " + channel + nl + "Speed: " + ramp + speedTemp + nl + "Position: " + channel00 + positTemp;
        }
    }
}

Men hvor kalder du SetServo?

buzzzz mistænker dig for, at du kalder metoden på et andet objekt end det i din SamplePluginForm.

præcis som ksoren2 skriver :-)

Næsten det eneste tænkelige der sker ... og da jeg hjalp dig i går, kunne jeg forstå på dig at du var ny ... derfor jeg næsten er sikker på at du 2 steder i dit program opretter en instance af din klasser ... altså:

Servo s1 = new Servo();
Servo s2 = new Servo();

s1.SetServo(100);

s2.GetServer();

s2 indeholde stadig default værdien, men s1 indeholder 100

Men til sådan get/set operationer burde du kigger på properties i .NET

// ouT

hmm ja, sådan har jeg gjort, men jeg kalder den ene sc2 og den anden sc...

Har haft kigget på get/set og kunne ikke få det til rigtigt at gøre noget som helst... jeg må prøve igen igen

Ja, men det er netop problemet hvis du gør sådan ... så er det jo ikke samme data du retter i ...

Det er jo en helt anden instance ...

Se klassen som blueprint ... til hvordan et object skal laves.

En instance så selve klassen.

Som eksempel. Audi sælger jo mange biler lavet efter samme blueprint, men folk kører jo ikke rundt i den samme bil? :-)

// ouT

prøv at gøre variablen static....

public static int channel00 = 750;

http://www.csharp-station.com/Tutorials/Lesson10.aspx

Har fundet dette, det må være den 10-2 Accessing Class Fields With Properties

Men jeg forstår ikke hvordan det på nogen måde skulle passe ind.

Så skal jeg for hver værdi t channel/ramp/position/function oprette en private og en public? Men så er det hele jo også i den forkerte fil?
Kan ikke få det til at give nogen mening. :(

Static vil nok løse problemet som bitmatic skriver.

Men ... post nu hele dit program, vi sidder og gætter indtil vi ser _ALT_ der har med din ServoControl at gøre ...

specielt der hvor du kalder: SetServo og "new ServoControl()"

// ouT

Ændre den til static gav et brugbart resultat :)

Dog bruger jeg stadig nogle af mine egne funktioner, så må jeg finde ud af hvordan det ellers kan gøres, og så kan det komme med i næste version :)

Smider i begge et svar?

Hvis det virker med static ... så er det fordi det ikke er samme object du arbejder med i dit program, men har oprettet 2 instances af den samme klasse ... som vi har sagt fra start.

Hvis du vil have løsning på hvordan du kan få det til at virke uden static skal du enten poste din kode eller sørge for at det er den samme instance du arbejde med og ikke hele tiden opretter en ny.

Hvis du kommer ud i et multithread miljø, er det ikke smart at bruge static og tror det generelt set er bad practice at gemme ting i static variabler.

Men hvis det virker så er det jo godt. Men vil dog anbefale dig at arbejde på samme object ... for din egen skyld ... debug er noget rod mange static variabler.

// ouT

og svar

Den helt rigtige måde at gøre det på er nok noget med et Singleton pattern.

Det kunne du jo prøve at kigge på hvis du en dag vil gøre programmet helt lækkert :)

Lidt ide til pænere kode:



> Den helt rigtige måde at gøre det på er nok noget med et Singleton pattern.

Singleton er yderst yderst sjældent den "helt rigtige" måde at løse problemer på!

Dette her kode jeg ville have skrevet:

Umidbart ser en løsning med Observer Pattern ud som den rigtige løsning.

Eksempel:

Hver gang ServoControl.Channel00 ændres, bliver progress-baren opdateret. Super nemt, og enkelt.

[code]
using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Linq;
using System.Text;
using System.Windows.Forms;

namespace E
{
    class ServoControl : INotifyPropertyChanged
    {
        public event PropertyChangedEventHandler PropertyChanged;

        private int _channel00;
       
        public int Channel00
        { 
            get { return _channel00; }
            set
            {
                if(value != _channel00)
                {
                    _channel00 = value;
                    PropertyChanged(this, new PropertyChangedEventArgs("Channel00"));
                }
            }
        }
       
        public string SetServo(int channel, int ramp, int position, string function)
        {
            if(channel == 0)
            {
                Channel00 = position;
            }
           
            return "";
        }
    }   
       
    public partial class Form1 : Form
    {
        private ServoControl sc = new ServoControl();
   
        public Form1()
        {
            InitializeComponent();

            sc.PropertyChanged += new PropertyChangedEventHandler(sc_PropertyChanged);
           
            sc.Channel00 = 1;
            sc.Channel00 = 2;
            sc.Channel00 = 3;
        }

        void sc_PropertyChanged(object sender, PropertyChangedEventArgs e)
        {
            progressBar1.Value = sc.Channel00;
        }
    }
}
[/code]

Og eksperten hader kode poster *sigh*

Og lidt hjemmelæsning:

http://msdn.microsoft.com/en-us/library/system.componentmodel.inotifypropertychanged.aspx

http://en.wikipedia.org/wiki/Observer_pattern

--> "Singleton er yderst yderst sjældent den "helt rigtige" måde at løse problemer på!"

Sådan som jeg har forstået koden/hardware'en (uden egentlig at have set ret meget af den), så skal ServoControl altid pege på de samme servoer, igennem den samme serielport.

I den situation er Singleton pattern den "helt rigtige" måde at gøre det på.

Lige en lille opfølgning fra min side... Jeg har ikke glemt denne tråd, jeg kæmper lige lidt før jeg helt vil afslutte dette :)

> I den situation er Singleton pattern den "helt rigtige" måde at gøre det på.

Ikke når han ikke har flere instanser? Og hvis man bare vil skrive en række statiske metoder kan man jo bare lave en statisk klasse.

Umidbart giver en servo-controller per server jo fint mening? Men selvom man har flere, så er server-controlleren jo en "observer" som jeg ser det, derfor vil en løsning med Observer være bedre end noget roderi i singleton.