[SOLUCIONADO] Java - Guardar String en un array de URL[]

[Paniic]

Quiero almacenar en un array de tipo URL, una linea obtenida de un fichero TXT, pero me devuelve error por ser tipos incompatibles. (Soy nuevo en esto del POO)

El error salta en la linea 12.

Código (java) [Seleccionar] Expandir



String urlsFile = "C:\\urls.txt";
String errorException = "Error de lectura en fichero"+urlsFile;
URL arrayList[];

  public void getURLfromTXTfile() {
       try {
           BufferedReader br = new BufferedReader(new FileReader(urlsFile));
           String line;
           Integer i = 0;
           while ((line = br.readLine())!=null){
               arrayList[i] = new URL[line];
               i++;
           }
           br.close();
       } catch (Exception e) {
           System.out.println(errorException);
       }

   }



Gracias de antemano.

[Usuario Invitado]

Hola,

Veo que no inicializas tu array arrayList que por cierto tiene un identificador malísimo. Una variable en Java debe describir su contenido. Si yo veo arrayList, ¿qué me dice eso?. Otra cosa, arrayList es una clase en Java.

Si no quieres un tamaño fijo, debes usar ArrayList:

Código (java) [Seleccionar] Expandir


public List<URL> getURLsFromFile(String directory, String fileName) throws IOException, IllegalAccessException {
      Path path = Paths.get(directory, fileName);
      Charset charset = Charset.forName("UTF-8");
      List<String> allLines = Files.readAllLines(path, charset);
      if(allLines == null || allLines.isEmpty()) {
        throw new IllegalAccessException("El archivo está vacío");
      }
      List<URL> urls = new ArrayList<>();
      for(String line : allLines) {
        urls.add(new URL(line));
      }
      return urls;
  }

Y lo llamarías así:

Código (java) [Seleccionar] Expandir


UrlHelper helper = new UrlHelper();
try {
    List<URL> urls = helper.getURLsFromFile("D://", "urls.txt");
    for(URL current : urls) {
        System.out.println(current);
    }
} catch(IOException | IllegalAccessException e) {
    System.err.println(e.getMessage());
}


En caso el fichero estuviera vacío, se lanza un IllegalAccessException con el mensage "El archivo está vacío", para que se pueda apreciar la causa exacta del problema. Es importante el manejo de excepciones, no olvidarlo.


Saludos.

[Paniic]

• En primer lugar muchas gracias, pues me solucionaste un error de antes al indicarme que no lo había inicializado, y por tomarte tiempo en la respuesta

Respecto al identificador del arrayList, suelo cambiarles los nombres para concretar al final, pero si es cierto que debo corregirlo y nombrarlos correctamente desde el principio.

• En segundo lugar, usando tu código me surge un único problema (antes de intentarlo con arrays también lo intenté con un arrayList).

Código (java) [Seleccionar] Expandir

Path path = Paths.get(directory,fileName);

Me indica que son tipos incompatibles debido a que necesita de javafx.scene.shape.Path y recibe un valor de java.nio.file.Path..


• En ultimo lugar, es un programa corto y de prueba para mi y para aprender Java, por ello uso Exception en lugar de excepciones específicas, que como dices, ayudan a identificar mucho antes un problema.

Gracias.

[Usuario Invitado]

Estás importando javafx.scene.shape.Path en lugar de java.nio.file.Path. Fíjate siempre es los mensajes que te dice el compilador.

Me indica que son tipos incompatibles debido a que necesita de javafx.scene.shape.Path y recibe un valor de java.nio.file.Path..

[Paniic]

Estás importando javafx.scene.shape.Path en lugar de java.nio.file.Path. Fíjate siempre es los mensajes que te dice el compilador.

Perdón -.-
Intenté usar java.nio.file.Path pero el auto-import de Maven no me cogía ese paquete.

Metiéndoselo a mano me lo cogió ya.

Muchas gracias Gus.