Итог-сумма в форме

[Наталья Баева]

Подскажите что исправить чтобы заработало

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 WindowsFormsApplication1
{
    public partial class Form1 : Form
    {
        public Form1()
        {
            InitializeComponent();
        }

        private void btnOrder_Click(object sender, EventArgs e)
        {
            DateTime time = DateTime.Now;
            textBox1.Text = time.ToString();

            string stPayment = "";
            //1)oplata
            if (radioButton1.Checked)
            { stPayment = "Visa-Card"; }
            else if (radioButton2.Checked)
            { stPayment = "Master-Card"; }
            else stPayment = "При получении";
            textBox1.Text = time + "\r\nСпособ оплаты - " + stPayment;
            //2)dostavka
            string stShipping = "";
            for (int i = 0; i < groupBox2.Controls.Count; i++)
            {
                RadioButton rd = groupBox2.Controls[i] as RadioButton;
                if (rd.Checked)
                    stShipping = rd.Text;
            }
            textBox1.Text += "\r\nСпособ доставки - " + stShipping;

            string stGoods = "";


            for (int j = 0; j < groupBox3.Controls.Count; j++)
            {
                CheckBox cd = groupBox3.Controls[j] as CheckBox;
                if (cd.Checked)
                    stGoods += cd.Text + "\r\n";
                
            }
            textBox1.Text += "\r\n\r\nТовары: \r\n" + stGoods;

            int itog = 0;
            int cena = 0;
            int a = 0;
            for (int j = 0; j < groupBox3.Controls.Count; j++)
            {
                CheckBox cd = groupBox3.Controls[j] as CheckBox;
                NumericUpDown nv = groupBox3.Controls[j] as NumericUpDown;
                if (checkBox1.Checked)
                {
                    cena = (int)cd.Tag;
                    int n = (int)numericUpDown1.Value;
                    label1 = (n.ToString()) * (cena.ToString());
                }
                else
                    label1 = (a.ToString());

                if (checkBox8.Checked)
                {
                    cena = (int)cd.Tag;
                    int n = (int)numericUpDown1.Value;
                    label2 = (n.ToString()) * (cena.ToString());
                }
                else
                    label2 = (a.ToString());

                if (checkBox9.Checked)
                {
                    cena = (int)cd.Tag;
                    int n = (int)numericUpDown1.Value;
                    label3 = (n.ToString()) * (cena.ToString());
                }
                else
                    label3 = (a.ToString());

                if (checkBox12.Checked)
                {
                    cena = (int)cd.Tag;
                    int n = (int)numericUpDown1.Value;
                    label4 = (n.ToString()) * (cena.ToString());
                }
                else
                    label4 = (a.ToString());

                if (checkBox11.Checked)
                {
                    cena = (int)cd.Tag;
                    int n = (int)numericUpDown1.Value;
                    label5 = (n.ToString()) * (cena.ToString());
                }
                else
                    label5 = (a.ToString());
                
                if (checkBox10.Checked)
                {
                    cena = (int)cd.Tag;
                    int n = (int)numericUpDown1.Value;
                    label6 = (n.ToString()) * (cena.ToString());
                }
                else
                    label6 = (a.ToString());

                //itog+=groupBox4;
            }

            textBox1.Text += "\r\n\r\nИтог: \r\n" + itog;
        }

        private void Form1_Load(object sender, EventArgs e)
        {

        }

 

    }
}

[phomm]

Непонятно, что именно Вам надо, распишите от и до - открываем программу - вводим в таком-то порядке такие то данные , то и то нажимаем и должны увидеть то и то.
И естественно сам проект приложите, а то в этом коде невооружённым взглядом (без прогона в среде) я и разбираться не стану - т.к. сложная запутанная копипаста.

[simples]

Сообщение от Наталья Баева

Подскажите что исправить чтобы заработало

+1 к phomm
А что НЕ работает то? Телепаты как обычно в отпуске

[Наталья Баева]

проблема в том что у меня сумма итоговая не считается
нужно цену умножить на количество, но из Tag он цену не берет, соответственно я не могу из numericUpDown использовать число (количество товара)
к тому же желательно чтоб в столбец цена помещалась либо 0 либо если выбран товар то цена, а потом все label суммировались и итог помещался в переменную itog

[phomm]

Сообщение от phomm

И естественно сам проект приложите, а то в этом коде невооружённым взглядом (без прогона в среде) я и разбираться не стану - т.к. сложная запутанная копипаста.

Сами догадаетесь, чего в Вашем сообщении не хватает ?

[Наталья Баева]

это нужно?

[phomm]

Так сойдёт или слишком сложно ?
Если сложно, чуть позже поменяю Ваш код. У Вас там какие-то проблемы были при запуске при конверсии компонентов. Да вообще запутанный код, сложно сказать как он работает, не разобрав его досконально, сложно поправлять.
Мой посложнее, зато в нём легко модифицировать, т.к. разведены данные и компоненты их обслуживающие, добавить новую позицию в прайсе дело пары строк кода.

[simples]

for (int j = 0; j < groupBox3.Controls.Count; j++)
            {
                b = 0; // добавлено. иначе считает что попало
                CheckBox cd = groupBox3.Controls[j] as CheckBox;
                if (cd.Checked) // исправлено
                {
                    cena = int.Parse(cd.Tag.ToString()); // исправлено
                    n = (int)numericUpDown1.Value;
                    b = n * cena;
                }

                itog +=b;
            }

PS код индийский что-ли? переписали бы нормально...

[Наталья Баева]

считать начал но не верно я выбрала материнскую плату по 150 1 шт и оперативную память за 200 2 шт в итог пишет 1050

[simples]

Сообщение от simples

for (int j = 0; j < groupBox3.Controls.Count; j++)
            {
                b = 0; // добавлено. иначе считает что попало
                CheckBox cd = groupBox3.Controls[j] as CheckBox;
                if (cd.Checked) // исправлено
                {
                    cena = int.Parse(cd.Tag.ToString()); // исправлено
                    n = (int)numericUpDown1.Value;
                    b = n * cena;
                }

                itog +=b;
            }

Если Вы мне (квотирование используйте пж) - то я же Вам уже написал решение Вашей ошибки (жирный шрифт).