(need help) Simple problem in C#

If i type 6 into the textbox and click on the button, the label will turn into 6(so far good). But when i want to add 4 to the 6 it doesn't change to 10.

Can someone tell me what i'm doing wrong here? I've looked the net for many hours without luck.

Code:

---

private void button_Click(object sender, RoutedEventArgs e)
        {

            int aantaltextBox = Convert.ToInt32(textBox.Text);
            int limiet = Convert.ToInt32(textBox1.Text);
            int aantallabel = Convert.ToInt32(label.Content=0);
            while (aantallabel < limiet)
            {
                label.Content = aantallabel + aantaltextBox;
            }   
        }

---

'Max'(limiet) means the label cant go over 20.
http://i.imgur.com/Wu3iBxg.png
http://i.imgur.com/FEQC15D.png
http://i.imgur.com/Gci4UGm.png

this looks so broken, can't even see it.


ps: dave can easily solve this for u

Why are you converting a value to integer? You inputting the value into an integer variable.

Quote:

---

Originally Posted by Paul

Why are you converting a value to integer? You inputting the value into an integer variable.

---

Thats why im posting it here, i only can do what i learnt.. as you can guess i am a beginner in this.

Probably easier to understand and make it work if you just have two separate textboxes for the values that you want to add together. Then you can just add textBox1.text + textBox2.Text and display it in the label.

Unless you have to do it the way you're doing.

Post more than just that function. That function should be working fine, only thing that was wrong with it is that you're converting to int when you're assigning it to int.

Quote:

---

Originally Posted by Resort

Probably easier to understand and make it work if you just have two separate textboxes for the values that you want to add together. Then you can just add textBox1.text + textBox2.Text and display it in the label.

Unless you have to do it the way you're doing.

---

Yea i know that would be easy.. what i'm trying to do is replacing textBox2 with a label.

- - - Updated - - -

Quote:

---

Originally Posted by Paul

Post more than just that function. That function should be working fine, only thing that was wrong with it is that you're converting to int when you're assigning it to int.

---

I don't get what you saying about the 'int' can you please explain with a example?

Code:

---

private void button_Click(object sender, RoutedEventArgs e)
        {

            int aantaltextBox = textBox.Text;
            int limiet = textBox1.Text;
            int aantallabel = label.Content=0;
            while (aantallabel < limiet)
            {
                label.Content = aantallabel + aantaltextBox;
            }

---

If both text boxes contain a number, there is no need to convert to integer because it has assigned itself as an int (or so it should be). You are also assigning it to an int variable.

Post the whole code at least, because these function aren't telling us where or how they're being called/used.

Also you should add an "Else statement" because the code has nothing to return if you write a value too high.

And you don't even have a function where the previous value is saved, it just gets replaced.

You're using "ConvertToInt" which is useless because that only works if you input something like "3.14" it will output either 314 or 3 (rounded), I haven't used C# in a while. But even with that you're using "int" at the beginning so that makes even less sense to use "ConvertToInt"

Right now aantallabel is your textbox where the user can input a value that's anything within an int.

So what did you do? You made textbox1 and user value output, but you did not make a function for the 2nd value that you want to use. Right now your code is literally saying "print in the users input, plus the textbox itself".

Quote:

---

Originally Posted by Paul

Code:

---

private void button_Click(object sender, RoutedEventArgs e)
        {

            int aantaltextBox = textBox.Text;
            int limiet = textBox1.Text;
            int aantallabel = label.Content=0;
            while (aantallabel < limiet)
            {
                label.Content = aantallabel + aantaltextBox;
            }

---

If both text boxes contain a number, there is no need to convert to integer because it has assigned itself as an int (or so it should be). You are also assigning it to an int variable.

---

[nvm, now i have to use just other name thanks for the tip] No it doesn't work.. if you do for example textbox 1(40) + textbox 2(50)= 4050(you get this because its a string) you still need to convert.

Quote:

---

Originally Posted by yung

Post the whole code at least, because these function aren't telling us where or how they're being called/used.

Also you should add an "Else statement" because the code has nothing to return if you write a value too high.

And you don't even have a function where the previous value is saved, it just gets replaced.

You're using "ConvertToInt" which is useless because that only works if you input something like "3.14" it will output either 314 or 3 (rounded), I haven't used C# in a while

Right now aantallabel is your textbox where the user can input a value that's anything within an int.

So what did you do? You made textbox1 and user value output, but you did not make a function for the 2nd value that you want to use. Right now your code is literally saying "print in the users input, plus the textbox itself".

---

There is no whole code(i've been stuck on this and i kept searching for an solution did not had time for the other things), i've put this apart to make it easy as possible. And YES i need a function to save the previous value i don't know what that function is, i thought it had something to do with loops. And float or double is "3.14" int is rounded.

- - - Updated - - -

This is the end result i need to get:
I know how to do the rest, the only thing i don't know is [see thread]. That's all and because i know how to do i haven't worked on the other things only the design is completely done.

http://i.imgur.com/ijdB5SX.png

You will have to learn how to debug properly.
Dump your variables to the console and see what's going wrong.

It's hard to say what's going wrong by looking at the code you posted, but the while loop should never end because aantallabel (which is 0) will always be smaller than limiet.

Quote:

---

Originally Posted by Dave

You will have to learn how to debug properly.
Dump your variables to the console and see what's going wrong.

It's hard to say what's going wrong by looking at the code you posted, but the while loop should never end because aantallabel (which is 0) will always be smaller than limiet.

---

I dont even know if my code is the right or the wrong one.. Thought the images were easy to understand. Well thanks anyways guys :)

Quote:

---

Originally Posted by qais187

This is the end result i need to get:
I know how to do the rest, the only thing i don't know is [see thread]. That's all and because i know how to do i haven't worked on the other things only the design is completely done.

---

Dude that's way to complicated for you. It's way too early. If you want to store it or something you need to learn a little bit more, just do easy shit like this

Code:

---

public void button1_Click(object sender, RoutedEventArgs e){       
        int firstbox = Convert.ToInt32(textBox1.Text); //here you input first value
        int firstvalue = textBox1.Text; // this is to store the first value you write in textbox
        textBox1.Text=""; //this is so that the textbox clears after the click
}


public void button2_Click(object sender, RoutedEventArgs e){
        int secondbox = Convert.ToInt32(textBox1.Text);  //second input!
        int secondvalue = textBox.Text; //this is to store the second value that you input after buttonclick
        int result = firstvalue + secondvalue; //the sum
        int limitbox = Convert.ToInt32(textBox2.Text); //limit!
        if (result <= limitbox) { //if you only write < and the MAX is 20, it will give you Invalid because 20 is not less than 20
            textBox1.Text=result.tostring();
        }else{
            MessageBox.Show("Invalid input");
        }


}

---

This is horribly written and I don't even know C#, but you can use this kind of method, you need database knowledge like asp.net or something in order to store a value properly, or maybe something else. Either way I don't see the point of doing something that's way too hard while you barely know the basics, do simpler tasks instead of going for what looks easy but that you have no idea what functions are required.

But for this program to work you need to make a button for Button1_Click that says "Add" or something, then another one for Button2_Click that says "Result" or something.

Quote:

---

Originally Posted by yung

Dude that's way to complicated for you. It's way too early. If you want to store it or something you need to learn a little bit more, just do easy shit like this

Code:

---

public void button1_Click(object sender, RoutedEventArgs e){       
        int firstbox = Convert.ToInt32(textBox1.Text); //here you input first value
        int firstvalue = textBox1.Text; // this is to store the first value you write in textbox
        textBox1.Text=""; //this is so that the textbox clears after the click
}


public void button2_Click(object sender, RoutedEventArgs e){
        int secondbox = Convert.ToInt32(textBox1.Text);  //second input!
        int secondvalue = textBox.Text; //this is to store the second value that you input after buttonclick
        int result = firstvalue + secondvalue; //the sum
        int limitbox = Convert.ToInt32(textBox2.Text); //limit!
        if (result <= limitbox) { //if you only write < and the MAX is 20, it will give you Invalid because 20 is not less than 20
            textBox1.Text=result.tostring();
        }else{
            MessageBox.Show("Invalid input");
        }


}

---

This is horribly written and I don't even know C#, but you can use this kind of method, you need database knowledge like asp.net or something in order to store a value properly, or maybe something else. Either way I don't see the point of doing something that's way too hard while you barely know the basics, do simpler tasks instead of going for what looks easy but that you have no idea what functions are required.

But for this program to work you need to make a button for Button1_Click that says "Add" or something, then another one for Button2_Click that says "Result" or something.

---

I get this(tried to put 'firstvalue' also by the 'button1_click1' but that didnt work :
first error is: Cannot implicitly convert type 'string' to 'int'
2nd: The name 'textBox' does not exist in the current context
3rd: The name 'firstvalue' does not exist in the current context

http://i.imgur.com/OWaAfP5.png
And yea i know but that's my homework we did not learn it how to do this little part of the homework is research on the internet.. the closest related thing to this is that we learnt to make a calculator.

Well one problem is

Code:

---

int resultat = textBox1.Text; // It's supposd to say textBox1.Text that's a typo

---

.
But I can't use Visual Studio on Mac and I am too lazy to boot and build my top meme windows, so you have to debug/build/compile for me...

But I don't think that works, so now I know that C# can't store two values in the same textbox unless it's saved in a txt file or some database asp.netshit I don't know, but it can at least not be put in a single function or anything-
But you should do this...

Remove all of int firstvalue and int secondvalue.

Change

Code:

---

int result = firstvalue + secondvalue;

---

to

Code:

---

int result = firstbox + secondbox;

---

Quote:

---

Originally Posted by yung

Well one problem is

Code:

---

int resultat = textBox1.Text; // It's supposd to say textBox1.Text that's a typo

---

.
But I can't use Visual Studio on Mac and I am too lazy to boot and build my top meme windows, so you have to debug/build/compile for me...

But I don't think that works, so now I know that C# can't store two values in the same textbox unless it's saved in a txt file or some database asp.netshit I don't know, but it can at least not be put in a single function or anything-
But you should do this...

Remove all of int firstvalue and int secondvalue.

Change

Code:

---

int result = firstvalue + secondvalue;

---

to

Code:

---

int result = firstbox + secondbox;

---

---

There is only 2 textbox one for input the other for the limit, by saying 'secondvalue or secondbox' do you mean the label? Cause i tried but it doesn't make sense.. And i really appreciate that you do this :) ... if you don't want to reply anymore i'll understand it, its very frustrating ._.

Try this.

Code:

---

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

namespace WindowsFormsApplication1
{
    public partial class Form1 : Form
    {

        private readonly int firstbox;

        public Form1()
        {
            InitializeComponent();
        }

        private void Form1_Load(object sender, EventArgs e)
        {

        }

        public void button1_Click(object sender, EventArgs e)
        {
            int firstbox = Int32.Parse( textBox1.Text ); //here you input first value
            int firstvalue = Int32.Parse( textBox1.Text ); // this is to store the first value you write in textbox
            textBox1.Text = ""; //this is so that the textbox clears after the click
        }


        public void button2_Click(object sender, EventArgs e)
        {
            int secondbox = Int32.Parse( textBox1.Text );  //second input!
            int secondvalue = Int32.Parse( textBox1.Text ); //this is to store the second value that you input after buttonclick
            int result = firstbox + secondbox; //the sum
            int limitbox = Int32.Parse( textBox2.Text ); //limit!

            if (result <= limitbox)
            { //if you only write < and the MAX is 20, it will give you Invalid because 20 is not less than 20
                textBox1.Text = result.ToString();
            }
            else
            {
                MessageBox.Show("Invalid input");
            }


        }
    }
}

---

I made firstbox an instance variable otherwise it would have been out of context.

Remember to edit the namespace. You may also need to remove readonly.

Quote:

---

Originally Posted by Paul

Try this.

Code:

---

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

namespace WindowsFormsApplication1
{
    public partial class Form1 : Form
    {

        private readonly int firstbox;

        public Form1()
        {
            InitializeComponent();
        }

        private void Form1_Load(object sender, EventArgs e)
        {

        }

        public void button1_Click(object sender, EventArgs e)
        {
            int firstbox = Int32.Parse( textBox1.Text ); //here you input first value
            int firstvalue = Int32.Parse( textBox1.Text ); // this is to store the first value you write in textbox
            textBox1.Text = ""; //this is so that the textbox clears after the click
        }


        public void button2_Click(object sender, EventArgs e)
        {
            int secondbox = Int32.Parse( textBox1.Text );  //second input!
            int secondvalue = Int32.Parse( textBox1.Text ); //this is to store the second value that you input after buttonclick
            int result = firstbox + secondbox; //the sum
            int limitbox = Int32.Parse( textBox2.Text ); //limit!

            if (result <= limitbox)
            { //if you only write < and the MAX is 20, it will give you Invalid because 20 is not less than 20
                textBox1.Text = result.ToString();
            }
            else
            {
                MessageBox.Show("Invalid input");
            }


        }
    }
}

---

I made firstbox an instance variable otherwise it would have been out of context.

Remember to edit the namespace. You may also need to remove readonly.

---

It still says "The name 'firstbox' does not exist in the current context" im using xaml if its making a difference

Quote:

---

Originally Posted by qais187

It still says "The name 'firstbox' does not exist in the current context" im using xaml if its making a difference

---

Oh, you're using WPF. Yes it does, make a difference. Thought you were using winforms.

It shouldn't be out of context. I never receoeved that error in my debugger.

Quote:

---

Originally Posted by Paul

This C#/.NET. It's Windows programming, not web based. ;/

---

Well I started my developer edu just a month ago. We prob learn this also

int aantaltextBox = Convert.ToInt32(textBox.Text);
int limiet = Convert.ToInt32(textBox1.Text);
int aantallabel = Convert.ToInt32(label.Content=0);
while (aantallabel < limiet)
{
label.Content = aantallabel + aantaltextBox;
}

i think the problem with this code is that every time you click this button, it resets label to 0. so everytime you click the button, it does label.Content = numTextbox (user input) + numLabel (which is always 0). you only want the label to output 0 once, thats when the program starts.


label.Content = 0;

private void method1{
int numTextbox = Int32.Parse(numBox.Text);
int limit = Int32.Parse(limitBox.Text);
int numLabel = int32.Parse(label.Text);

if (numTextbox <= limit) label.Content = numTextbox + numLabel;
}

Quote:

---

Originally Posted by ZAIN

int aantaltextBox = Convert.ToInt32(textBox.Text);
int limiet = Convert.ToInt32(textBox1.Text);
int aantallabel = Convert.ToInt32(label.Content=0);
while (aantallabel < limiet)
{
label.Content = aantallabel + aantaltextBox;
}

i think the problem with this code is that every time you click this button, it resets label to 0. so everytime you click the button, it does label.Content = numTextbox (user input) + numLabel (which is always 0). you only want the label to output 0 once, thats when the program starts.


label.Content = 0;

private void method1{
int numTextbox = Int32.Parse(numBox.Text);
int limit = Int32.Parse(limitBox.Text);
int numLabel = int32.Parse(label.Text);

if (numTextbox <= limit) label.Content = numTextbox + numLabel;
}

---

Ye i know, had already tried it before but it says: "The name 'label.Content' does not exist in the current context" and "The name 'label' does not exist in the current context. And i get a error by 'label.Text' i changed it to 'label.Content' but then i get a different error: "Argument 1: cannot convert from 'object' to 'string' "

http://i.imgur.com/WWWEwzr.png

- - - Updated - - -

Quote:

---

Originally Posted by Paul

It's not that bad, lol. I would choose it over Java. java is just annoying.

Also, if you're on OS X, why haven't you downloaded Xamarin to test?


Qais, if it's out of context, usually means one of the following things.

- Initialize the variable from another class (which you don't have).
- Make it an instance variable like I did.
- Declare the variable within' the same function. int firstbox;

---

Hmm even when it works, i still need 1 button though.. don't know how i could put that all into 1 button.

Quote:

---

Originally Posted by yung

Java is even worse than C# why are you comparing shit to ebola...~
C++, C, Rust and the new Swift are the only good programming languges.



Ok, you're going way too far now, go and google the fkn shit because you don't know anything about C# at all wtf, this can't be your homework you're beyond helpless, if you can't do it on your own why are you asking for spoonfeed here? We've helped you more than enough to fix the problem which you write in the topic, now you want us to do the rest because you can't even use google or do your own homework.

From topic: "Can someone tell me what i'm doing wrong here? I've looked the net for many hours without luck."

Took me less than 5 mins to find a decent answer while I have almost 0 knowledge of C#, I learned more in those 5 minutes than you did in "many hours" and apparently even school. Your assignment was to copy a picture of a GUI? Without instructions or an example code to guide you? Or even the teachers example? So much bullshit dude...

I hope this thread gets closed because you're basically asking for your homework to be done by this forum, but you're pretending there are unsolvable problems in the code, but in reality can be fixed within a minute just by googling.

---

You really think i just gonna post some homework in a GAMEFORUM because you think im lazy? I've looked days/hours into this problem without luck. And like i said it a small problem of my hw(see example) i never said do the whole shit because i know the rest. What you did was using 2 buttons to solve(still had errors) it go ahead try to find it while using only 1 then talk. Like i said if you dont want to help i inderstand that but i dont need this shit talking from someone who thinks he all knows whats going on.

Just to confirm. It works with two buttons, yes?

Quote:

---

Originally Posted by Paul

Just to confirm. It works with two buttons, yes?

---

Dont know will try later(on phone now), ill let u know

Well this worked for me.

Code:

---

        public void button1_Click(object sender, EventArgs e)
        {
            int firstbox = Int32.Parse( textBox1.Text ); //here you input first value
            int firstvalue = Int32.Parse( textBox1.Text ); // this is to store the first value you write in textbox
            int secondbox = Int32.Parse(textBox1.Text);  //second input!
            int secondvalue = Int32.Parse(textBox1.Text); //this is to store the second value that you input after buttonclick
            int result = firstbox + secondbox; //the sum
            int limitbox = Int32.Parse(textBox2.Text); //limit!

            textBox1.Text = ""; //this is so that the textbox clears after the click

            if (result <= limitbox)
            { //if you only write < and the MAX is 20, it will give you Invalid because 20 is not less than 20
                textBox1.Text = result.ToString();
            }
            else
            {
                MessageBox.Show("Invalid input");
            }

        }

---

Quote:

---

Originally Posted by Paul

Well this worked for me.

Code:

---

        public void button1_Click(object sender, EventArgs e)
        {
            int firstbox = Int32.Parse( textBox1.Text ); //here you input first value
            int firstvalue = Int32.Parse( textBox1.Text ); // this is to store the first value you write in textbox
            int secondbox = Int32.Parse(textBox1.Text);  //second input!
            int secondvalue = Int32.Parse(textBox1.Text); //this is to store the second value that you input after buttonclick
            int result = firstbox + secondbox; //the sum
            int limitbox = Int32.Parse(textBox2.Text); //limit!

            textBox1.Text = ""; //this is so that the textbox clears after the click

            if (result <= limitbox)
            { //if you only write < and the MAX is 20, it will give you Invalid because 20 is not less than 20
                textBox1.Text = result.ToString();
            }
            else
            {
                MessageBox.Show("Invalid input");
            }

        }

---

---

YESSS it worked! :D i changed it a bit. Thank you very much Paul and yes you too yung and the other people for replying and trying to help :) /request lock

Code:

---

private void button_Click(object sender, RoutedEventArgs e)
        {
            int firstbox = Int32.Parse(textBox1.Text); //here you input first value
            int secondbox = Convert.ToInt32(label.Content);  //second input!           
            int result = firstbox + secondbox; //the sum
            int limitbox = Int32.Parse(textBox2.Text); //limit!

            textBox1.Text = ""; //this is so that the textbox clears after the click

            if (result <= limitbox)
            { //if you only write < and the MAX is 20, it will give you Invalid because 20 is not less than 20
                label.Content = result.ToString();
            }
            else
            {
                MessageBox.Show("Invalid input");
            }
        }

---