ایراد برنامه ... ؟

[cheraghe_jado]

سلام میشه ببینید ایراد این برنامه چیه ؟

کد: انتخاب همه

           Double na, b, ma, kh;
            
           
            na = Convert.ToInt32(textBox1.Text);
            
            if (radioButton1.Checked)
            
                b = (7 / 100) * na;
            else if (radioButton2.Checked)
                b = (9 / 100) * na;
            else if (radioButton3.Checked)
                b = (10 / 100) * na;

             
            if (na >= 0 & na <= 2000000)
                
                ma = 0;
            
            else if (na >= 2000001 & na <= 4000000)
                ma = (na - 2000000) * 0.10;
            else if (na >= 4000001 & na <= 999999999)
                ma = (4000000 - 2000000) * 0.1 + (na - 40000001) * 0.15;

            kh = (na - b) - ma; 
                
           label1.Text = (" حقوق خالص :" + kh.ToString());
           label2.Text = (" مالیات : " + ma.ToString());
           label3.Text = (" بیمه :" + b.ToString());
        }

        private void Form1_Load(object sender, EventArgs e)
        {
            label1.ResetText();
            label2.ResetText();
            label3.ResetText();

این 2 ارور رو میده :
Error 2 'Use of unassigned local variable 'b'

Error 1 'Use of unassigned local variable 'ma'

[پیر کاردانی]

سلام میشه ببینید ایراد این برنامه چیه ؟
این 2 ارور رو میده :
Error 2 'Use of unassigned local variable 'b'

Error 1 'Use of unassigned local variable 'ma'

سلام
همین جوری که نگاه کردم مشکل از تعریف متغیرت هست.
باید b,ma رو بدون علامت بگیری.
یا بالعکس.

یا علی

[cheraghe_jado]

یعنی چی ؟

[felfel_namaki]

شما باید ma و b رو مقدار دهی کنین
مثلا مساوی با 0 قرار بدید

[cheraghe_jado]

ارههههههههههه مرسی فلفلی درست شد

میگم اگه بخوام از سویچ استفاده کنم واسه تیکه اول برنامه ببین این درسته :

کد: انتخاب همه

 switch (b)
            {
                case  (radioButton1.Checked==true):
                    b= (na*7)/100;
                    break;
                case (radioButton2.Checked == true): 
                    
                    b=(na*9)/100;
                    break;
                case (radioButton3.Checked == true):
                    
                    b=(na*10)/100;
                    break;
            }

[cheraghe_jado]

بچه ها بگید دیگه کجای این کد دومی switch رو میگم ایراد داره افرین بگید

[پیر کاردانی]

شما باید ma و b رو مقدار دهی کنین
مثلا مساوی با 0 قرار بدید

سلام
وای پسر
من و یاد اشتباه همیشگیم انداختی یادش بخیر.
راست میگه تو اون خاطره های دور الان داره یه جیزایی از صفر کردن یادم میاد.

یا علی

[cheraghe_jado]

ای بابا پیر کاردانی به جای یاد آوری دوران جوانیت

ایراد switch منو بگوو

[cheraghe_jado]

دوستان یکی لطف کنه بگه این سوچ من چه ایرادی داره !
اصلا میشه به این نحوی که گفتم استفاده کنم یا نه ؟
فلفلی ؟
پیر کاردانی ؟

[cheraghe_jado]

ای بابا چرا کسی جواب نمیده ؟

[felfel_namaki]

ببین دوست من به دستور سوییچ گیر دادی هاااا ... دستور سویچ واسه مقایسه تساوی است فقط و لاغیر . من که هر چی فک میکنم نمیدونم چی رو با سویچ بشه نوشت شاید باقی دوستان بدونن ، برنامه خودت صاف و واضح است نیازی به سوییچ هم نیست . اما برنامه شما یه ایراداتی هم داره :
1. از ورودی شاید غیر عدد بگیره ! اونوقت چی میشه ؟ ( پس باید کاری کنین که فقط عدد بگیره )
2. من نمیدونم این دستور شما

کد: انتخاب همه

 b=(na*9)/100;

چطور اجرا میشه .! شما باید اولا از نوع اعشاری معرفی میکردید یا اونو ضرب در float یا double میکردید مثلا اینجوری :

کد: انتخاب همه

b=(float)(na*9)/100

امیدوارم کمکی کرده باشم .

راستی واسه مشکل اول هم کدشو میزارم اگه بلد نبودی نخوای 5 -6 تا پست دیگه هم واسه اون بدی

ببیین دوست من تو event تکست باکست تو keyprese این دستور رو بنویسی اکتفا میکنه

کد: انتخاب همه

 private void textBox1_KeyPress(object sender, KeyPressEventArgs e)
        {
            if (char.IsLetter(e.KeyChar))
            {
                e.Handled = true;
            }