No encuentro el error en este codigo de Java

B

[Borrado] #1 Dic '09

Buenas!Toy aqui practicando para un examen de java que tengo esta tarde, y me da este error y no le veo por ningun lado.Os dejo el codigo y os dejo el error que me da:

-Codigo

spoiler

import java.io.*;
public class palin_prueba{
public static void main(String[]args)throws IOException{
BufferedReader in = new BufferedReader(new InputStreamReader(System.in));

    //VARIABLES
    String cad;
    String cad3 = "";
    String cad2 = "";
    //DATOS INTRODUCIDOS POR EL USUARIO
    System.out.println("Introduce la palabra para comprobar si es un palindromo");
    cad = in.readLine();
    
    //DATOS DE SALIDA
   // palindromo obj2 = new palindromo();
   // System.out.println(obj2.comprobarPalindromo(palabra));
do{
   for(int i=cad.length()-1;i>=0;i--){
        char c = cad.charAt(i);
        cad3 = cad3 + c;
        for(int k=0;cad.length()>=k;k++){
            char f = cad.charAt(k);
            cad2= cad2 + f;
        }//FIN FOR#2
        if(cad3.equalsIgnoreCase(cad2)){
            System.out.println("Son palindromo");
        }
        else{
            System.out.println("No son palindromo");
        }
   }//FIN FOR#1
}while(cad3.equalsIgnoreCase(cad2));
}//FIN MAIN

}//FIN CLASS

-Error

spoiler

Espero una ayudita rapida.
Un Saludo.

D

donkokito #2 Dic '09

No sé java, pero por qué pones "for(int i=cad.length()-1;i>=0;i--){"

Si en la condición ya estás especificando i >= que 0?

Lo mismo se te sale de rango por eso.

Shendraf #3 Dic '09

En

char f = cad.charAt(k);

cuando k sea cad.length() dará un error. El bucle que recorre k debe tener la condición

 (k < cad.length())

Recomendación: para la próxima utiliza las etiquetas [ code] [ /code] (sin espacios) para el código

PiTaGoRaS #4 Dic '09 Spoon Bender

El error viene de lo que dice #3, pero aún así tu algoritmo de reconocer palíndromos no funciona ni de broma.

no hace falta complicarse la vida

public class Palindromo
{
    public static void main(String[] args) throws IOException
    {
        BufferedReader in = new BufferedReader(new InputStreamReader(System.in));

    // VARIABLES
    String cad;

    // DATOS INTRODUCIDOS POR EL USUARIO
    System.out.println("Introduce la palabra para comprobar si es un palindromo");
    cad = in.readLine();

    int i = 0;
    int j = cad.length() - 1;
    
    while (i < j && cad.charAt(i) == cad.charAt(j))
    {
        i++;
        j--;
    }        
    
    if (i >= j)
    {
        System.out.println("palindromos");
    }
    else
    {
        System.out.println("no es palindromo");
    }
}
}

Shendraf #5 Dic '09

Mi propuesta:

BufferedReader in = new BufferedReader(new InputStreamReader(System.in));
boolean errorEnc = false;
String palabra = in.readLine();
for (int i=0, j=palabra.length()-1; (i<j) && (!errorEnc); i++, j--) 
      if (palabra.charAt(i) == palabra.charAt(j)) errorEnc = true;
    

 System.out.println("La palabra " + palabra + " " + (errorEnc?"":"no ") + "es palíndromo");

JuAn4k4 #6 Dic '09

Tu propuesta es lo mismo que escribieron antes pero con el PlanE, le has pasado una apisonadora de asfalto y eso ha quedado. Algo nada recomendable vamos.

bLaKnI #7 Dic '09 Inocente

Que mania teneis algunos macho...

Sabeis de lo ÚNICO que sirve escribir código de esta forma? No??

Se nota que no trabajais de ello. O almenos, no durante demasiado tiempo.

Eso solo sirve para porcular a todo el pobre que viene detrás a tocar vuestro magnífico código de 3 lineas que hace lo mismo que uno escrito con sus 20 líneas.
Lo mismo que la gente que le tiene tirria a los enters y espacios...
Se trata de facilitarse la vida entre compañeros de trabajo! Separad las líneas y comentad, y os convertireis en los mejores programadores del mundo. Ya habrá tiempo para hacer estas gilipolleces virtuosas en otro momento...

Madre mia...

Shendraf #8 Dic '09

Vaya! No sabía que mi propuesta la van a usar en el trabajo. Si lo sé me la curro mucho más, como no tengo otra cosa mejor que hacer.....

#7 ¿No sabes lo que es el PMD y el Checkstyle? Seguro que sí ¬¬

No encuentro el error en este codigo de Java

Usuarios habituales

Tags