Singleton form

Og de fejl fortæller?

Error    1    Invalid expression term ')'    C:\Documents and Settings\mko\my documents\visual studio 2010\Projects\WindowsFormsApplication4\WindowsFormsApplication4\Form1.cs    22    53    WindowsFormsApplication4

Error    2    ) expected    C:\Documents and Settings\mko\my documents\visual studio 2010\Projects\WindowsFormsApplication4\WindowsFormsApplication4\Form1.cs    22    54    WindowsFormsApplication4

Det er jo en generic, dvs. du skal have typen og instancen.

SingletonFormProvider.GetInstance<Form>(this);

http://msdn.microsoft.com/en-us/library/system.windows.forms.form.aspx

Nå ja. "This" er jo ikke en type. Tak!

Jeg har rettet koden i Form1 til

private void Form1_Load(object sender, EventArgs e)
  {           
      SingletonFormProvider.GetInstance<Form2>(this).Show();
  }

Og nu kan jeg tilgå Form2 fra hele projektet uden at skulle tænke på at trække refencer. Herligt

Sender du et svar?
/Michael

Bare vær klar over det for det meste er dårlig design og frarådes. Da det senere hvis det er et stort projekt, kan blive besværligt at holde øje med hvem der egentlig tilgår den ...

Men til små projektet ... top nice.

Når du er igang ... så se om på IoC contaioners, nice pattern, hvor de fleste også tilbyder singleton.

mvh

Singleton har faaet et daarligt ry i de senere aar.

Men jeg synes altsaa at det er lidt noget teoretisk pladder.

Det er normalt muligt udfra soegning i source code eller kig paa dependencies i build at se hvor den bruges.

Hvor meget nemmere er det at trace den gennem en masse kald som argumenter i praksis?

Korrekt, test cases af en singleton er bare ikke så nem.

Men er det store projekter, og alle gør lidt som de synes, synes jeg den anden andene måde er bedre. Læg også mærke til at jeg skrev frarådes, da det bestemt også i min verden har sin ret.

Så hele tiden pros/cons for singleton. Hvis det endelig var en guld mine af kun gode ting, så ville jeg mene det er mærkeligt det ikke bliver brugt mere.

Singleton er programmørenes svar på religion :-) Enten tror man på "den" eller ej :-P

:)

Tak for input alle sammen.
Mit projekt er ikke særlig stort.
Så hvis det er helt ude  ved nær teoretisk pladder tror jeg, jeg fortsætter med den Singleton men jeg vil helt se på
IoC contaioners.
Jeg vender måske tilbage med opklarende spørgsmål.

Hvis I andre to vil have lidt at pointene sender I så et svar?

/M

nej, ikke med det input jeg kom med.

Du skal bare huske på det er singleton, og med en god hjerne, kan det gøre mange ting på en nem måde.

og ja, hvis projektet er lille, så er der ingen grund til at lave det mere besværligt.

nej tak - det var jo en lille side diskussion

Okay - tak for input

Projektet virker slet ikke som jeg havde forvente det skulle.

Jeg havde forstillet mig at kunne bruge det til kontrol af mit program ved at sende data til singletonformen fra forskellige procedure. Men data skal jo sendes for en form - det havde jeg ikke tænkt over.

Jeg kan selvfølgelig bare lave en alm. singleton og lade den oprette en for i constructoren.
Herved må jeg kunne få det ønskede.
Prøver lige....

Min alternative løsning som ser ud til at virke.
statusformen har to listbokse som jeg kan tilføje data til.


Først klassisk Singleton
using System;

namespace Singleton
{
    public sealed class SingletonLogger
    {
        static SingletonLogger instance = null;
        static readonly object padlock = new object();
        StatusrForm f;
        SingletonLogger()
        {
            f = new StatusrForm();
            f.Show();
        }
        public static SingletonLogger Instance
        {
            get
            {
                lock (padlock)
                {
                    if (instance == null)
                    {
                        instance = new SingletonLogger();
                    }
                    return instance;
                }
            }
        }

        public void logDataToForm(string str)
        {
            try
            {
                lock (this)
                {
                    f.AddToListBox1(str); 
                }
            }
            catch { }
        }
    }
}


--------------------------
formen
--------------------------
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 Singleton
{
    public partial class StatusForm : Form
    {
        public delegate void AddListBoxText(string message);
        public StatusForm()
        {
            InitializeComponent();
        }

        public ListBox ListBox
        {
            get { return listBox1; }
        }

        public void AddToListBox1(string txt)
        {
            if (listBox1.InvokeRequired)
            {
                listBox1.Invoke(new AddListBoxText(AddToListBox1), txt);

            }
            else
            {
                listBox1.Items.Add( (object)txt);
                int l = listBox1.Items.Count;
                if (l > 120)
                    for (int i = 0; i < 60; i++)
                        listBox1.Items.RemoveAt(0);
            }

        }
        public void AddToListBox2(string txt)
        {
            if (listBox2.InvokeRequired)
            {
                listBox2.Invoke(new AddListBoxText(AddToListBox2), txt);

            }
            else
            {
                listBox2.Items.Add((object)txt);
                int l = listBox1.Items.Count;
                if (l > 70)
                    for (int i = 0; i < 10; i++)
                        listBox1.Items.RemoveAt(0);
            }

        }
     
    }
}

Du bør lave din singleton om ...

public static SingletonLogger Instance
        {
            get
            {
if(instance == null)
{
                lock (padlock)
                {
                    if (instance == null)
                    {
                        instance = new SingletonLogger();
                    }

                }
}
                return instance;
            }
        }


Skulle være bedre, da du nu undgår at lave en lock, hvis 2 tråde på samme tid vil tilgå den ... og den ikke er null, så sparer du en lock som er overflødig :-)

Hmmm har prøvet men samme resultat.
Der kommer ikke noget ind i listen i statusformen og når jeg forsøger at lukke den (museklik) svarer den ikke.

Hovsa - nu virkede det lige pludselig!

Jeg lod
- singleton returner statusformen i og kaldte metoden i statusformen på referencen,
i stedet for at
-kalde en metode i singleton som kalder en metode i statusformen.

Hvorfor mon det virkede - nu ??

Kode :

Først form som jeg kalder fra:

using System;
using System.Timers;
using System.Windows.Forms;

namespace Singleton
{
    public partial class Form1 : Form
    {
       
        public Form1()
        {
            InitializeComponent();
         
        }

        private void Form1_Load(object sender, EventArgs e)
        {         
          System.Timers.Timer timer=new System.Timers.Timer(100);
          timer.Elapsed += new ElapsedEventHandler(OnTimedEvent);
          timer.Start();
         
        }
        private void OnTimedEvent(object sender,ElapsedEventArgs e)
        {
            Random r = new Random();
            string str = r.Next().ToString();
            SingletonLogger.Instance.form.AddToListBox1(str);
        }

       
       
       
    }
}
----------------------------------
singleton
----------------------------------

using System;

namespace Singleton
{
    public sealed class SingletonLogger
    {
        static SingletonLogger instance = null;
        static readonly object padlock = new object();
        public StatusForm form;
        SingletonLogger()
        {
            form = new StatusForm();
            form.Show();
        }
        public static SingletonLogger Instance
        {
            get
            {
                if (instance == null)
                {
                    lock (padlock)
                    {
                        if (instance == null)
                        {
                            instance = new SingletonLogger();
                        }

                    }
                }
                return instance;
            }
        }

   
    }
}
-----------------------------
Statusformen hvis liste opdateres
-----------------------------
using System;
using System.Windows.Forms;

namespace Singleton
{
    public partial class StatusForm : Form
    {
        public delegate void AddListBoxText(string message);
        public StatusForm()
        {
            InitializeComponent();
        }

        public ListBox ListBox
        {
            get { return listBox1; }
        }

        public void AddToListBox1(string txt)
        {
            if (listBox1.InvokeRequired)
            {
                listBox1.Invoke(new AddListBoxText(AddToListBox1), txt);

            }
            else
            {
                listBox1.Items.Add( (object)txt);
                int l = listBox1.Items.Count;
                if (l > 120)
                    for (int i = 0; i < 60; i++)
                        listBox1.Items.RemoveAt(0);

                this.Refresh();
                listBox1.Refresh();
            }

        }
       
       
     
    }
}

Hvad virkede ikke før ?

formen blev ikke opdateret.
Der kom intet ind i listboxen.