Duda Java

zoeshadow #1 Nov '11

El caso es que estamos empezando en clase con Java, el programa ha de pedirte 2 numeros por pantalla y devolverte el resultado de la suma, multiplicacion, resta y division.
El problema es que no soy capaz de pasar los numeros leidos a la otra clase, ya que lo del "getter" no me funciona dentro del static void main... blabla
Aqui el codigo:

spoiler

package ejercicio1;

import java.util.Scanner;

public class Main {

    int num1,num2;
  

public static void main(String[] args) {

 Scanner sc = new Scanner(System.in);

 System.out.println("Introduzca un numero");
 int num1 = sc.nextInt();
 System.out.println("Introduzca un numero");
 int num2 = sc.nextInt();

 operaciones op = new operaciones();

 System.out.println("La suma es "+op.res_suma );
 System.out.println("La resta es "+op.res_resta );
 System.out.println("La multiplicacion es "+op.res_mult );
 System.out.println("La division es "+op.res_div );

         }

public int getNum1() {
    return num1;
}

public int getNum2() {
    return num2;
}



}

El codigo de la clase operaciones, en donde están todas las operaciones escritas.

spoiler

package ejercicio1;

public class operaciones {

Main num = new Main();

int a = num.getNum1();
int b = num.getNum2();


int res_suma = a + b;
int res_resta = a - b;
int res_mult = a * b;
double res_div = a / b;




}

El get pasa un valor 0 a la otra clase, y si lo pongo dentro del corchete del void main, me da error, he estado buscando por inet pero estoy un poco perdido de que es lo que necesito exactamente.

Decir que si muestro por pantalla num1 y num2 justo despues de insertarlos, sale el numero correctamente.

PD: se que podria hacerlo todo en una clase y quitarme de problemas, pero es algo que me gustaria saber para mas adelante xD

Imperium #2 Nov '11

A mi me paso eso el otro dia y no lo solucione LOL

Yandr0s #3 Nov '11

Por una parte tienes que definir un constructor para la clase operaciones.

De todas formas, Si main llama a operaciones, y operaciones llama a main... tal como lo tienes seria un bucle infinito del horror XD

SkOi #4 Nov '11 Inocente

Pasa num1 y num2 como parámetros a la clase operaciones.

Yandr0s #5 Nov '11

#1

Main

package ejercicio1;

import java.util.Scanner;

public class Main {

    int num1,num2;
 

public static void main(String[] args) {

 Scanner sc = new Scanner(System.in);

 System.out.println("Introduzca un numero");
 int num1 = sc.nextInt();
 System.out.println("Introduzca un numero");
 int num2 = sc.nextInt();

 operaciones op = new operaciones(num1,num2);

 System.out.println("La suma es "+op.res_suma );
 System.out.println("La resta es "+op.res_resta );
 System.out.println("La multiplicacion es "+op.res_mult );
 System.out.println("La division es "+op.res_div );

         }

operaciones

    package ejercicio1;

public class operaciones {

    int a,b,res_suma,res_resta,res_mult;
    double res_div;
    public operaciones( int n1, int n2)
    {
    
    this.a=n1;
    this.b=n2;


    this.res_suma = a + b;
    this. res_resta = a - b;
    this.res_mult = a * b;
    this.double res_div = a / b;
   


}

#6 ya, el this no es necesario, aunque es recomendable, y te acabas acostumbrando xD

1 respuesta

zoeshadow #6 Nov '11

#5 Gracias! xD, al final no hay que poner el this ni nada, basta con

operaciones op = new operaciones(num1,num2);

Era una tonteria pero madre mia, que dolor de cabeza XD

1 respuesta

tOWERR #7 Nov '11

¿Por que no te creas los metodos suma, resta, multiplicacion y division en la clase operaciones y ha esos metodos les pasas los numeros como parametros. Mira te dejo como quedaria cada clase:

Clase Main:
spoiler
```
package ejercicio1;

import java.util.Scanner;

public class Main {

    int num1,num2;
  

public static void main(String[] args) {

 Scanner sc = new Scanner(System.in);

 System.out.println("Introduzca un numero");
 int num1 = sc.nextInt();
 System.out.println("Introduzca un numero");
 int num2 = sc.nextInt();

 operaciones op = new operaciones();

 System.out.println("La suma es " + op.suma(num1, num2));
 System.out.println("La resta es " + op.resta(num1, num2));
 System.out.println("La multiplicacion es " + op.multiplicacion(num1, num2));
 System.out.println("La division es " + op.division(num1, num2));

}
}
```
- Clase operaciones:
  spoiler
```
package ejercicio1;

public class operaciones {

int suma(int n1, int n2){
	return n1 + n2;
}

int resta(int n1, int n2){
	return n1 - n2;
}

int multiplicacion(int n1, int n2){
	return n1 * n2;
}

int division(int n1, int n2){
	return n1 / n2;
}
}
```
  No te compliques tanto la vida, es mas sencillo de lo que crees. Espero que te sirva.
  Si necesitas alguna ayuda MP.
  
  Un saludo.

3 respuestas

zoeshadow #8 Nov '11

#7 Entendido, sin duda tiene pinta de ser una forma mucho mas limpia y correcta de hacerlo, muchas gracias =)!

T

Tututavi #9 Nov '11

No compendo porque declaras num1 y num2 en la clase para redeclararlos en el main. Al hacer esto cuando modificas num1 y num2 dentro de tu main estas modificando las variables locales num1 y num2 de tu main y no num1 y num2 de tu clase. Tál y como tienes el código en #1 tus variables num1 y num2 de la clase no se inicializan por lo que cuando haces el get estas devolviendo "null" y tu función espera retornar un int .

Si declaras una variable en una clase esta será visible para todas las funciones de la clase. Si la declaras dentro de una función normalmente solo será visible dentro de esta función. Normlamente el ámbito de una función va de "dentro" para "afuera".

Personalmente la opción que encuentro más bonita es la de #7 .
Piensa también que un get o un set funciona como una función cualquiera, es decir que no tiene ninguna diferencia con tu función suma etc.

JuAn4k4 #10 Nov '11

El fallo no creo que sea una tonteria eh, va mas tirando a un fallo gordo.

#7 Entonces Operaciones es una clase Util y no pinta nada declarar un objeto Operaciones como tal.

Estas haciendo un metodo, que podria ser perfectamente estatico para sumar dos numeros, tiene mas sentido inicializar la clase con los dos valores y luego dejarla a ella que sepa que tiene que hacer con ellos.

Wasd #11 Nov '11

Este solo usa una clase, y en caso de error te informa:

spoiler

import java.io.*;
public class multis{
public static void main(String[] args){

InputStreamReader isr = new InputStreamReader(System.in);
BufferedReader buffer = new BufferedReader(isr);

try{
System.out.println("Introduce el primer numero");
String numero1 = buffer.readLine();
int numero11 = Integer.parseInt(numero1);

System.out.println("Introduce el segundo numero");
String numero2 = buffer.readLine();
int numero22 = Integer.parseInt(numero2);

System.out.println("Estas son distintas operaciones con los numeros introducidos:");
System.out.println("Suma: " + numero11 + " + " + numero22 + " = " + (numero11 + numero22));
System.out.println("Resta: " + numero11 + " - " + numero22 + " = " + (numero11 - numero22));
System.out.println("Multiplicacion: " + numero11 + " x " + numero22 + " = " + (numero11 * numero22));
System.out.println("Division " + numero11 + " / " + numero22 + " = " + (numero11 / numero22));

   }
catch(Exception e){
System.out.println("Error en la introduccion de datos por parte del usuario");
  }
 }
}

1 respuesta

tOWERR #12 Nov '11

La solución de #11 es más simple, yo le hice 2 clases porque el utilizaba 2 clases, pero en 1 sola clase lo puede hacer también perfectamente.

zoeshadow #13 Nov '11

Mas que nada queria hacerlo en dos clases para irme acostumbrando, cierto que para un ejercicio tan sencillo es casi mejor hacerlo en una clase pero bueno xD

Gnos1s #14 Nov '11

Eso de empezar con Java sin haber explicado antes qué es la OO...

1 respuesta

zoeshadow #15 Nov '11

#14 La teoria la hemos dado, lo que no habiamos dado es sintaxis y cosas así

Usuarios habituales