Desquiciado con un problema :S

[Kev_fva]

Hola!

Tengo el siguiente método, que dado un Array de Strings con nombre de películas te devuelve un ArrayList con tantos objetos película como tenga el array.

public static ArrayList<Pelicula> createPelicula(String[] nom)throws Exception{

ArrayList<Pelicula>movieList = new ArrayList<Pelicula>();

Pelicula p;
for (int i = 0; i < nom.length; i++){
p = new Pelicula(nom[i],getGenere(nom[i]),getDuracion(nom[i]),getDirector(nom[i]));
movieList.add(p);
}


return movieList;

}

####### MAIN #######

String[]pelis = {"Death+Proof","Salvar+al+soldado+Ryan","Little+Ni cky","Argo"};
ArrayList<Pelicula> lista = createPelicula(pelis);

Iterator<Pelicula> it = lista.iterator();
while(it.hasNext())
System.out.println(it.next().toString());

Pues bien, no se porque solo me reconoce la primera película del Array a la hora de crear los objetos.

Esta es su ejecución:

Pelicula [nom=Death+Proof, genere=terror, duracio=95, director=Quentin+Tarantino]
Pelicula [nom=Salvar+al+soldado+Ryan, genere=terror, duracio=95, director=Quentin+Tarantino]
Pelicula [nom=Little+Nicky, genere=terror, duracio=95, director=Quentin+Tarantino]
Pelicula [nom=Argo, genere=terror, duracio=95, director=Quentin+Tarantino]

Muchas gracias :D

[chuidiang]

Pues posiblemente el problema esté en estos métodos que te devuelven siempre lo de la primera película.

getGenere(nom[i]),getDuracion(nom[i]),getDirector(nom[i])

pero claro, esos no los has puesto.....

Se bueno.

[Malenko]

Iniciado por chuidiang

Pues posiblemente el problema esté en estos métodos que te devuelven siempre lo de la primera película.

getGenere(nom[i]),getDuracion(nom[i]),getDirector(nom[i])

pero claro, esos no los has puesto.....

Se bueno.

Pues sí, porque de hecho, los nombres de las películas son los 4 diferentes. Tiene más pinta de que algo falla en las funciones de duración y director.

[Kev_fva]

Pues si, teníais razón. El fallo estaba en esos métodos...

Tenia esto:

Código:

Pattern dir = Pattern.compile(patron); 
Matcher encaja = dir.matcher(res);
if(encaja.find()) director = encaja.group(1);
return director;

y lo he solucionado cambiando el if por:

Código:

while(encaja.find()) director = encaja.group(1);

Lo que no se por que antes no funcionaba, ya que al llamar cada vez a los metodos lo haces con un nombre de película y debería sobrescribir las variables