SoloCodigo

Hola franZn, bueno das muy poca informacion para poder ayudarte, en el codigo que publicaste estas usando la clase Cliente, pero no veo su definicion, sospecho que el error esta en que el miembro saldo de la clase Cliente no es de tipo entero, digo sospecho porque si no pones el codigo es imposible de saber.

Ahora respecto a esto:

porque tengo ke definir 2 veces esto: Cajero ca = new Cajero(), intente ponerlo fuera de los metodos pero no funciona.

Al decir "no me funciona" no podemos saberlo sin mas datos, disculpa si parezco algor rudo, pero no somos adivinos. Para saber por que no funciona, necesitamos saber exactamente cual es el error que te da, ¿es error de sintaxis?, ¿o acaso es error de logica?, por lo tanto deber colocar cual es el error EXACTO que te marca, si es error de compilador, o es un error en el resultado, ademas debes colocar EXACTAMENTE el codigo tal como lo tienes, y la linea en que te esta marcando el error.

De esa manera se puede analizar y ver que estas haciendo mal, de otra forma, es solo adivinar.

Saludos

Hola de nuevo.

Bueno ahora si publicaste tu codgio completo y analizandolo me doy cuenta que cometes un error de logica en el siguiente fragmento:

                for (i=0;i<=1;i++)
                       
                {
                        if (clientes[i].dni == dnis)
                                r=clientes[i].saldo;
                        else
                                r=0;
                               
                               
                       
                }

Utilizas un ciclo for para buscar el DNI, pero nunca sales del ciclo cuando lo encuentras, ademas en el else pones r=0, de esa forma aun cuando lo encuentras, vuelves a poner el saldo en cero. Asi como esta si por ejemplo tienes 10 DNI, solo en una pasada del ciclo haces r=clientes[ i ].saldo;, pero en las otras nueve pasadas haces r=0.

En mi opinion lo mas correcto es eliminar el else, esta de mas, ya que de todas formas antes de entrar al ciclo haces r=0, ademas debes agregar una forma de salir del ciclo una vez que encuentras el DNI, lo mejor es utilizar break, que te permite salir del for antes de que se ejecute n numero de veces.

Sin embargo veo otro problema, ¿que pasa si algun cliente tiene un saldo de cero? No tienes forma de distinguir cuando no encuentras el DNI, y cuando el saldo es cero, ya que en ambos casos te dara cero el resultado. En este supuesto yo sugeriria que en lugar de hacer r=0, hagas r=-1, ya que un saldo nunca podra tener un valor negativo, de esa manera separarias cuando el DNI no se encuenta de cuando alguien tiene un saldo de cero.

Dicho eso, el codigo corregio seria algo como:

import java.util.*;
 
 
public class Cajero   {
 
       
       
       
        public Cliente[] clientes = new Cliente[2];
       
       
        public static void main(String[] args) {
                int valor;
                int dnisearch;
                int i;
                Scanner s = new Scanner(System.in);
                Cajero ca = new Cajero();
                               
                for (i=0;i<=1;i++)
                {
                ca.clientes[i] = new Cliente();
                System.out.println("Ingrese Cliente, DNI y Saldo");
                ca.clientes[i].setData();
                       
                }
               
       
               
                for (i=0;i<=1;i++)//compruebo q los datos estan bien ingresados
                {
                        System.out.println("Cliente: "+ca.clientes[i].nombre);
                        System.out.println("Saldo: "+ca.clientes[i].saldo);
                }
               
               
                System.out.println("ingrese dni para obtener saldo");
                dnisearch = s.nextInt();
               
                valor= ca.getSaldo(dnisearch);
                if (valor == -1)
                System.out.println("DNI no encontrado");
                else
                System.out.println("saldo "+valor);
 
        }
 
       
        public int getSaldo (int dnis)
       
        {
               
                int i;
                // r = -1 para indicar que no encontro el dni
                int r=-1;
                for (i=0;i<=1;i++)
                       
                {
                        if (clientes[i].dni == dnis)
                        {
                                r=clientes[i].saldo;
                                // Salir del for si encuentra el DNI
                                break;
                              }
// Eliminamos esto, que es innecesario
//                        else
//                                r=0;
                               
                               
                       
                }
       
                        return(r);             
                       
               
               
        }
}

Ahora respecto a:

Supongan que tengo ke leer un entero en el metodo getSaldo(), deberia definir como lo hice en el main Scanner s = new Scanner(System.in); y ahi justamente es donde radica mi duda, porqué lo tengo que definir de nuevo??? si ya lo he hecho en el main.

El que lo definas en el main no significa que sera vaildo para todo el programa, ya que la variable "Scanner s" al estar definida dentro del main, solo es valida para ese metodo, entra en el ambito de lo que son "variables locales". Para que puedas utilizar esa variable en toda la clase debes definirla como tu mismo dices en los atributos de la clase, y ademas me parece que debes definirla como static, ya que de lo contrario tendrias que instanciar la clase Cajero para poder utilizar la variable.

Bueno espero con esto haber aclarado tus dudas.

Saludos =)