Løkke

double ialt = 0;
for(i=0; i<16; i++)
{
ialt =+ Convert.ToDouble(tbPris+i.Text);
}

Proev noget a la:

            decimal ialt = 0;
            foreach(Control c in Controls)
            {
                if(c is TextBox)
                {
                    if(c.Name.IndexOf("tbPris") == 0)
                    {
                        ialt += decimal.Parse(((TextBox)c).Text);
                    }
                }
            }

Error    2    The name 'tbPris' does not exist in the current context    C:\revision\WindowsApplication1\WindowsApplication1\Form2.cs    54    42    WindowsApplication1

Jeg tror at den ikke godtager +i inde i et navn?

Nej, var lige for hurtig der.

Det skal også vær som arne angiver. Dog nok lige me3d en try-catch om Parse-delen.

decimal ialt = 0;
            foreach (Control c in Controls)
            {
                if (c is TextBox)
                {
                    if (c.Name.IndexOf("tbPris") == 0)
                    {
                        try
                        {
                            ialt += decimal.Parse(((TextBox)c).Text);
                        }
                        catch { }
                    }
                }
            }

            MessageBox.Show("Sum: " + ialt);

Hej arne. Den kunne i compaile nu. Men den ligger dem desværre ikke sammen: Du får lige koden :)

using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Text;
using System.Windows.Forms;

namespace WindowsApplication1
{
    public partial class Form2 : Form
    {
        double ialt = 0;

        public Form2()
        {
            InitializeComponent();
            this.Text = "Revision v 1.0 beta - Faktura";

            timer1 = new Timer();
            timer1.Interval = 100;
            timer1.Start();
            timer1.Tick += new EventHandler(Timer_Tick);
        }

        private void Form2_Load(object sender, EventArgs e)
        {

        }



        public void Timer_Tick(object sender, EventArgs eArgs)
        {

            if (sender == timer1)
            {
                //
                //Beregner ialt feltet.
                //
                decimal dtbIalt = belobIAlt();
                string stbIalt = Convert.ToString(dtbIalt);
                tbIalt.Text = stbIalt;
            }
            Invalidate();
        }

        public decimal belobIAlt()
        {

            decimal ialt = 0;
            foreach (Control c in Controls)
            {
                if (c is TextBox)
                {
                    if (c.Name.IndexOf("tbPris") == 0)
                    {
                        ialt += decimal.Parse(((TextBox)c).Text);
                    }
                }
            }
            return ialt;
           
        }
 
    }

   
}

Hvad sker der ? exception ? 0.00 ?

Nielle: Alt virker, men den skriver at summen er = 0 :(

Arne v: Der sker bare ingen ting. Med nielle message.show viser den i en messagebox
sum: 0

Kunn e det tænkes at dine textboxe ikke hedder "tbPris..." ?

En anden mulighed er at du har lagt dem inde i et panel?

Check Name property paa dine TextBox'e, maaske hedder de textBox7 etc..

Ja, de ligger inde i et panel

Hvis de goer saa aendre dem til at hedde det de skal.

Ahhh panel - et øjeblik :^)

Skal jeg smide dem ud af det? (det vil jeg helst ikke). Mit panel/gruopbox hedder:

groupBox2

Ikke nødvendig:

Sådan:

        public decimal belobIAlt(Control c1)
        {
            decimal ialt = 0;
            foreach (Control c2 in c1.Controls)
            {
                if (c2 is TextBox)
                {
                    if (c2.Name.IndexOf("tbPris") == 0)
                    {
                        try
                        {
                            ialt += decimal.Parse(((TextBox)c2).Text);
                        }
                        catch { }
                    }
                }
                else if (c2 is Panel)
                {
                    ialt += belobIAlt(c2);
                }
            }

            return ialt;
        }

og sådan:

        public void Timer_Tick(object sender, EventArgs eArgs)
        {

            if (sender == timer1)
            {
                //
                //Beregner ialt feltet.
                //
                decimal dtbIalt = belobIAlt(this);
                string stbIalt = Convert.ToString(dtbIalt);
                tbIalt.Text = stbIalt;
            }
            Invalidate();
        }

Nielle: Viser stadig 0. Men det er en groupBox alle boxene ligger i. Og der er 16 forskellige. Giver lige koden som den er nu:

using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Text;
using System.Windows.Forms;

namespace WindowsApplication1
{
    public partial class Form2 : Form
    {
        double ialt = 0;

        public Form2()
        {
            InitializeComponent();
            this.Text = "Revision v 1.0 beta - Faktura";

            timer1 = new Timer();
            timer1.Interval = 100;
            timer1.Start();
            timer1.Tick += new EventHandler(Timer_Tick);
        }

        private void Form2_Load(object sender, EventArgs e)
        {

        }



        public void Timer_Tick(object sender, EventArgs eArgs)
        {

            if (sender == timer1)
            {
                //
                //Beregner ialt feltet.
                //
                decimal dtbIalt = belobIAlt(this);
                string stbIalt = Convert.ToString(dtbIalt);
                tbIalt.Text = stbIalt;
            }
            Invalidate();
        }

        public decimal belobIAlt(Control c1)
        {
            decimal ialt = 0;
            foreach (Control c2 in c1.Controls)
            {
                if (c2 is TextBox)
                {
                    if (c2.Name.IndexOf("tbPris") == 0)
                    {
                        try
                        {
                            ialt += decimal.Parse(((TextBox)c2).Text);
                        }
                        catch { }
                    }
                }
                else if (c2 is GroupBox)
                {
                    ialt += belobIAlt(c2);
                }
            }

            return ialt;
        }
    }

   
}

Løsningen er da heldigvis let at udvide:

        public decimal belobIAlt(Control c1)
        {
            decimal ialt = 0;
            foreach (Control c2 in c1.Controls)
            {
                if (c2 is TextBox)
                {
                    if (c2.Name.IndexOf("tbPris") == 0)
                    {
                        try
                        {
                            ialt += decimal.Parse(((TextBox)c2).Text);
                        }
                        catch { }
                    }
                }
                else if (c2 is Panel)
                {
                    ialt += belobIAlt(c2);
                }
                else if (c2 is GroupBox)
                {
                    ialt += belobIAlt(c2);
                }
            }

            return ialt;
        }

Hvordan ser form koden ud ?

prøv den her:)

private decimal recursiveMethod(Control controls)
        {
            decimal amount = 0;
            foreach (Control control in controls.Controls)
            {
                amount += recursiveMethod(control);
                TextBox textbox = control as TextBox;
                if(textbox == null) continue;
                try
                {
                    amount += decimal.Parse(textbox.Text.Trim());
                }
                catch { }
            }
            return amount;
        }


og


public void Timer_Tick(object sender, EventArgs eArgs)
        {

            if (sender == timer1)
            {
                //
                //Beregner ialt feltet.
                //
                decimal dtbIalt = recursiveMethod(this);
                string stbIalt = Convert.ToString(dtbIalt);
                tbIalt.Text = stbIalt;
            }
            Invalidate();
        }

JAAAAAA, Tager lige en runde rundt om blokken ;D DET VIRKER!!! TAKKKK!!! jeg er i himlen :D

Send et svar så du kan få nogle point!

Jeg tror ikke, at du har benyttet min metode, men den burde virke uanset om dine textboxe placeret i paneler eller andra kontroller.

Nielle, din løsning er rigtig! Kalp din virker også delvist, Den bliver ved med at ligge sammen hvert sekund

Jep, den er faktisk lidt mere elegant, men den skal nu lige udvides med et ekstra tjek på navnet af textboxene da summen ellers ville blive forkert.

okay martin, men det er noget helt andet;)
selve logikken i min er nok mere sikker, men it's your call.. du kan frit benytte koden;o)

nielle -> korrekt:) jeg kom til at fokusere på, at gøre den generisk, men det er en mindre detalje:P

private decimal recursiveMethod(Control controls)
        {
            decimal amount = 0;
            foreach (Control control in controls.Controls)
            {
                amount += recursiveMethod(control);

                TextBox textbox = control as TextBox;
                if (textbox == null ||
                    !textbox.Name.StartsWith("tbPris")) continue;
                try
                {
                    amount += decimal.Parse(textbox.Text.Trim());
                }
                catch { }
            }
            return amount;
        }

Jeg syntes begge er geniale ;) Jeg siger RIGTIGT mange tak! Og jeg kan KALP din kan jeg bruge et andet sted i mit program, til et problem jeg heller ikke lige kunne løse, så det virker nu! :D

Virkelig godt! 100000 TAK til jer alle ! :D

Er det ikke bare en 3-vejs deler denne her?

Det er okay:)
Jeg kan godt lide, at pusle med rekursiv kode:P

Du kunne nu have ventet til de andre to havde lagt et svar :^)

http://www.eksperten.dk/spm/800612

Der er nu noget magisk ved rekursion ;^)

        private decimal CalcSum(Control control)
        {
            if (control is TextBox)
            {
                if (control.Name.StartsWith("tbPris"))
                {
                    try { return decimal.Parse(control.Text.Trim()); }
                    catch { }
                }
                return 0;
            }
            else
            {
                decimal sum = 0;
                foreach (Control childControl in control.Controls)
                {
                    sum += CalcSum(childControl);
                }
                return sum;
            }
        }

Note: det var ikke et tilfaelde at jeg valgte decimal fremfor double. Det strengt forbudt
at bruge float eller double til beloeb.

I hvert fald hvis man er fra den financielle sektor - de kan ikke lide afrundingdfejl!

Forhåbentligt er der også andre brancher hvor man ikke er glade for det.

Gu' ve' hvordan de mon så har det med tomme "catch-clauses" i parsningen :x

(Til trådstarter: I stedet for at ligge og rode med en timer der ligger og kører hver 1/10 sekund, kunne man prøve at overveje en eventbaseret løsning i stedet)

/B :)

Jeg ville nok ikke lave den catch (jeg lavede ikke den catch).

At et ikke udfyldt felt tælles som 0 er vel helt fint.

Men hvis folk skriver teksten "fjorten" i feltet er det nok tvivlsomt om de forventer
at det bliver talt som 0 uden advarsler.

Man kunne putte noget i den catch.

Men jeg ville føle det mere fristende at have noget "kun tal" kode andet steds
og så lade en exception her ryge langt op og blive betragtet som en kode bøf
i validerings koden.

tjah.. jeg lavede try catch af ren dovenskab...
problemet i fokus var, at udregningen slet ikke kom med.
Exception håndtering er et helt andet kapitel.

men i hvertfald så kan man fra .NET 2.0 og op benytte sig af decimal.TryParse metoden da den ikke kaster en exception.

Problem stillingen er stadig den samme.