Problema con variables globales y parametros

[jlmn23]

Estoy dando visual c# con procedimientos y variables globales y no paro de liarme. Ahora el problema es que tengo que hacer un ejercicio tan sencillo como una tabla de multiplicar con una sola variable global.

El formulario tiene un textbox donde pongo el numero para que saque la tabla de multiplicar, un boton para hacer el calculo y el otro para limpiar y a si pòder hacer otra tabla de multiplicar y un listbox donde tengo que sacar los numeros multiplicados.

Al calcular primero me saca los numeros del 1 al 10 despues limpio y pongo otro numero y no hace nada. Pongo el codigo del primer boton:


int numero = 1;
private void button1_Click(object sender, EventArgs e)
{

int numero_introducido=int.Parse(textBox1.Text);

Tabla_Multiplicar(numero_introducido);
}
void Tabla_Multiplicar(int resultado)
{
while (numero <= 10)
{
resultado = resultado*numero;
listBox1.Items.Add(numero.ToString());
numero++;
};
}

Estes es el codigo que he puesto en el boton calcular. He cambiado el nombre de las variables y tod hasta que me ha dejado ejecutar el programa y el resultado es el comentado. Le pongo al textbox cualquier numero y me saca siempre 1,2,3, a si hasta 1 en el listbox despues le doy a limpiar pongo otro numero en el textbox le doy a calcular y no me hace nada.

Gracias. Que follon con los parametros de veras yo creia haber entendido esto y lo e las variables globales pero parece ser que no del todo.

Gracias de ante mano.

[caotz]

Al listbox estas ñadiendo la variable "numero" que la estas usando solo como contandor, en este caso tienes que añadir "resultado".

Saludos.

[jlmn23]

Iniciado por caotz

Al listbox estas ñadiendo la variable "numero" que la estas usando solo como contandor, en este caso tienes que añadir "resultado".

Saludos.

De eso me he dado cuenta despues de poner el mensaje por que aun que yo ponga los problemas no me paro ahi, intento resolverlos.

Al poner en el listbox la variable resultado me salen numeros pero no me sale 3,6,9 hasta 30 que es lo que tiene que salir si no da unos resultados que ni se parecen. Para poner un ejemplo cuando pongo en el listbox 3 sale:

3,6,18,72,360, no sigo por que ya los numeros son mas grandes.

El caso es que sigo sin que esto funcione bien.

Gracias.

[jlmn23]

Iniciado por jlmn23

De eso me he dado cuenta despues de poner el mensaje por que aun que yo ponga los problemas no me paro ahi, intento resolverlos.

Ya he solucionado el problema pero ahora tengo otros 2 que ya son faciles.

El primero es que quiero hacer un if para que compruebe si hay un numero en el textbox de lo contrario que de un mensaje diciendo que no hay nada y que tiene que poner un numero. Pongo el codigo que tengo ahora:

int numero = 1;
int numero2;
private void button1_Click(object sender, EventArgs e)
{

int numero_introducido;

if (textBox1.Text = "")
{ MessageBox.Show("Debe introducir un numero"); }
else
{
numero_introducido = int.Parse(textBox1.Text);
};




Tabla_Multiplicar(numero_introducido);
}
void Tabla_Multiplicar(int resultado)
{
while (numero <= 10)
{
numero2 = resultado;
resultado = resultado*numero;
listBox1.Items.Add(resultado.ToString());
resultado = numero2;
numero++;
};

}

Eso esta en el codigo que calcula la multiplicacion y me dice en el if que no se puede convertir explicitamente un string en bool.

Y el segundo fallo esta cuando limpio con otro boton el txtbox y la lista que al poner en el textbox otro numero no hace nada. Pongo codigo del segundo boton:

textBox1.Text = "";
listBox1.Items.Clear();

Pues eso fala el if, si quito el if va pero quiero que se compruebe eso antes si hay un numero y el boton de limpiar pues cuando le doy y vuelvo a poner otro numero no me hace el calculo.

Gracias chicos.
Gracias.

[alexg88]

El problema del if es por usar el operador de asignación ( = ) y no el de igualdad ( == ).

El segundo problema es que no reinicias el contador a 0 cada vez que llamas al método Tabla_Multiplicar.

Un consejo es que uses variables con nombres con sentido en el contexto, es decir, al método Tabla_Multiplicar le pasas una variable llamada resultado, cuando realmente no es el resultado sino el número del cual quieres obtener la tabla de multiplicar.

Código C#:

int contador = 1;
int resultado = 0;
 
private void button1_Click(object sender, EventArgs e)
{
 
int numero_introducido;
 
//Es ==, no =
 if (textBox1.Text == "") //Tambien puedes poner textBox1.Text.Equals("") o mejor 
//String.IsNullOrEmpty(textBox1.Text) que controla tanto que la cadena sea vacía como que sea nula
 { 
  MessageBox.Show("Debe introducir un numero"); 
 }
 else
 {
 numero_introducido = int.Parse(textBox1.Text);
 Tabla_Multiplicar(numero_introducido);
 }
 
}
void Tabla_Multiplicar(int numero_tabla)
{
 
//Reiniciamos el contador a 0 cada vez que se realiza una nueva tabla de multiplicar
contador= 0;
 
while (contador <= 10)
{
 resultado = contador * numero_tabla;
 listBox1.Items.Add(resultado.ToString());
 contador++;
}
}
 

Saludos.

[jlmn23]

Iniciado por alexg88

El problema del if es por usar el operador de asignación ( = ) y no el de igualdad ( == ).

El segundo problema es que no reinicias el contador a 0 cada vez que llamas al método Tabla_Multiplicar.

Un consejo es que uses variables con nombres con sentido en el contexto, es decir, al método Tabla_Multiplicar le pasas una variable llamada resultado, cuando realmente no es el resultado sino el número del cual quieres obtener la tabla de multiplicar.

Código C#:

int contador = 1;
int resultado = 0;
 
private void button1_Click(object sender, EventArgs e)
{
 
int numero_introducido;
 
//Es ==, no =
 if (textBox1.Text == "") //Tambien puedes poner textBox1.Text.Equals("") o mejor 
//String.IsNullOrEmpty(textBox1.Text) que controla tanto que la cadena sea vacía como que sea nula
 { 
  MessageBox.Show("Debe introducir un numero"); 
 }
 else
 {
 numero_introducido = int.Parse(textBox1.Text);
 Tabla_Multiplicar(numero_introducido);
 }
 
}
void Tabla_Multiplicar(int numero_tabla)
{
 
//Reiniciamos el contador a 0 cada vez que se realiza una nueva tabla de multiplicar
contador= 0;
 
while (contador <= 10)
{
 resultado = contador * numero_tabla;
 listBox1.Items.Add(resultado.ToString());
 contador++;
}
}
 
Ya he solucionado el problema, habia puesto tu ejercicio y no funcionaba lo habre copiado mal, el caso es que ya tengo el problema resuelto con mi codigo, he puesto la llamada al procedimiento en el else y el contador en vez de poner 0 he puesto 1 y va de maravilla. Todo esto pasa por que estoy estudiando c# en casa con un manual y no tengo un profesor que me ayude cuando tengo problemas de este tipo. Este tipo de ejercicios de parmetrs y variables globales y demas me etan dando unos quebraderos de cabeza que jamas imaginaba tener. Ahora me toca ver funciones, espero que se me de mejor.
 
Gracias.
 

Saludos.