PDA

View Full Version : [JAVA] Problema classe che usa Iterator


Helldron
18-04-2009, 09:59
Ciao a tutti..Ho un problema abbastanza fastidioso :confused: , magari è banale però non so se si tratta di codice o di importare qualche libreria per questa mia classe che usa Iterator. Quando compilo il main() il compilatore mi tira fuori:


C:\Enrico\università\esercizi java>javac LCITest.java
.\LCI.java:16: cannot find symbol
symbol : class Iterator
location: class LCI
public Iterator enumera(){
^
.\LCI.java:21: cannot find symbol
symbol : class Iterator
location: class LCI
private class LCIIterator implements Iterator{
^
LCITest.java:14: cannot find symbol
symbol : class Iterator
location: class LCITest
Iterator it = lista.enumera();
^
3 errors

La classe LCI è questa:
public class LCI{
/**OVERVIEW: la classe creao oggetti ListaConcatenataDiInteri che sanno aggiungere oggetti NodoInt in testa e che hanno un iteratore per gestirli*/

private NodoInt testa;

public LCI(){
testa = null;
}

public void addInTesta(NodoInt x){
x.setSuccessivo(testa);
testa = x;
}

public Iterator enumera(){
/**REQUIRES: non modificare la lista quando l'Iteratore è in uso*/
return new LCIIterator();
}//il metodo generatore

private class LCIIterator implements Iterator{

private NodoInt dove;

public LCIIterator(){
dove = testa;
}

public boolean hasNext(){
return dove != null;
}

public Object next(){
return new Integer(dove.getValore()); //wrapping del tipo che restituisce getValore(cioè un int) in uno di tipo non primitivo(Integer)
}

public void remove(){
dove = dove.getSuccessivo();
}
}
}

la classe NodoInt serve solo per creare un oggetto nodo:
public class NodoInt{

/**OVERVIEW: classe che crea oggetti di tipo NodoInt con 1 valore intero e puntatore al successivo elemento di una lista
fA: <intero,riferimento nodo successivo> -> <val,succ>
IR: val != int && succ != NodoInt*/

private int val;
private NodoInt succ;

public NodoInt(int n){
val = n;
succ = null;
}

public int getValore(){
/** @return: this.val*/
return val;
}

public NodoInt getSuccessivo(){
/**MODIFIES: restituisce il riferimento a this (effetto collaterale)*/
return succ;
}

public void setSuccessivo(NodoInt x){
/**EFFECTS: modifica il valore di this (variabile succ) che diventa il riferimento a x
MODIFIES: modifica il valore di this (variabile succ) che diventa il riferimento a x (effetto collaterale)
@param: NodoInt*/

succ = x;
}
}
La classe del main è questa:
public class LCITest{
public static void main(String[] args){

LCI lista = new LCI();

NodoInt a = new NodoInt(12);
NodoInt b = new NodoInt(20);
NodoInt c = new NodoInt(-30);

a.setSuccessivo(b);
b.setSuccessivo(c);

lista.addInTesta(a);

Iterator it = lista.enumera();
int i = 0;

while(it.hasNext()){
int k = ((Integer)it.next().intValue()); //wrapping da Integer a int per recuperare il valore dell-elemento i-esimo della lista

System.out.println("valore "+i+" della lista: "+k+"\n");

it.remove();

i++;
}
}
}

Grazie!!!

wingman87
18-04-2009, 13:16
Hai importato java.util.Iterator?

Helldron
19-04-2009, 11:34
Hai importato java.util.Iterator?
Ecco non sapevo di doverla importare quella libreria, ci provo e vediamo se funge! Grazie :)

Helldron
20-04-2009, 16:03
Ok problema risorlto..solo ora il compilatore mi da errore all'istruzione:

int k = ((Integer)it.next()).intValue();
---------------------------^
//wrapping da Integer a int per recuperare il valore dell-elemento i-esimo della lista

E' sempre del tipo cannot find symbol, insomma poche info utili

Ho controllato l'API di Integer e così si usa, a quanto pare, però mi sembra molto strano, a livello di semantica dovrebbe funzionare :confused:

banryu79
20-04-2009, 16:37
E' sempre del tipo cannot find symbol, insomma poche info utili


Hum, la classe Integer fa parte del package java.lang e questo package viene importato di default dal compilatore quindi non può essere Integer il simbolo che non trova.

Magari se posti tutto il messaggio, come hai fatto nel primo post:

C:\Enrico\università\esercizi java>javac LCITest.java
.\LCI.java:16: cannot find symbol
symbol : class Iterator
location: class LCI

scopriamo a che simbolo si riferisce ;)

Helldron
23-04-2009, 18:46
Hum, la classe Integer fa parte del package java.lang e questo package viene importato di default dal compilatore quindi non può essere Integer il simbolo che non trova.

Magari se posti tutto il messaggio, come hai fatto nel primo post:

scopriamo a che simbolo si riferisce ;)Ho risolto era un problema di grammatica però adesso ho scoperto un'altra cosa un pò strana nell'output.
Sembra che qualcosa non vada a buon fine.

Ho dichiarato 4 oggetti di tipo NodoInt e gli ho passato i rispettivi valori, li ho concatenati tramite il metodo setSuccessivo() mentre la lista concatenata ha in testa il nodoInt "a". Ora però mi stampa solo il valore di "a" e non degli altri elementi, ho controllato l'iteratore della classe LCI però mi pare vada tutto bene nella semantica..voi vedete qualche errore??

/**
output:
valore 0 della lista: 12
//qui andrebbero gli altri valori della lista e cioè di b,c,d
*/

PGI-Bis
23-04-2009, 19:07
Io darei un'occhiata a next perchè secondo me non "nexta"...

Helldron
24-04-2009, 10:53
Io darei un'occhiata a next perchè secondo me non "nexta"...
Ho ricontrollato il metodo next() e gli altri moduli che coinvolge. Mi sembra che vada liscio, non capisco. Se fosse addInTesta() il problema non mi farebbe visualizzare il primo elemento, sebbene sembra next() il punto crtico, a me pare scritto bene :confused:

wingman87
24-04-2009, 12:52
Puoi ripostare il codice completo, magari tra i tag code?

Helldron
25-04-2009, 15:19
Eccolo:
public class LCI{
/**OVERVIEW: la classe creao oggetti ListaConcatenataDiInteri che sanno aggiungere oggetti NodoInt in testa e che hanno un iteratore per gestirli*/

private NodoInt testa;

public LCI(){
testa = null;
}

public void addInTesta(NodoInt x){
x.setSuccessivo(testa);
testa = x;
}

public Iterator enumera(){
/**REQUIRES: non modificare la lista quando l'Iteratore è in uso*/
return new LCIIterator();
}//il metodo generatore

private class LCIIterator implements Iterator{

private NodoInt dove;

public LCIIterator(){
dove = testa;
}

public boolean hasNext(){
return dove != null;
}

public Object next(){
return new Integer(dove.getValore()); //wrapping del tipo che restituisce getValore(cioè un int) in uno di tipo non primitivo(Integer)
}

public void remove(){
dove = dove.getSuccessivo();
}
}
}


//la classe NodoInt serve solo per creare un oggetto nodo:

public class NodoInt{

/**OVERVIEW: classe che crea oggetti di tipo NodoInt con 1 valore intero e puntatore al successivo elemento di una lista
fA: <intero,riferimento nodo successivo> -> <val,succ>
IR: val != int && succ != NodoInt*/

private int val;
private NodoInt succ;

public NodoInt(int n){
val = n;
succ = null;
}

public int getValore(){
/** @return: this.val*/
return val;
}

public NodoInt getSuccessivo(){
/**MODIFIES: restituisce il riferimento a this (effetto collaterale)*/
return succ;
}

public void setSuccessivo(NodoInt x){
/**EFFECTS: modifica il valore di this (variabile succ) che diventa il riferimento a x
MODIFIES: modifica il valore di this (variabile succ) che diventa il riferimento a x (effetto collaterale)
@param: NodoInt*/

succ = x;
}
}

//La classe del main è questa:

public class LCITest{
public static void main(String[] args){

LCI lista = new LCI();

NodoInt a = new NodoInt(12);
NodoInt b = new NodoInt(20);
NodoInt c = new NodoInt(-30);

a.setSuccessivo(b);
b.setSuccessivo(c);

lista.addInTesta(a);

Iterator it = lista.enumera();
int i = 0;

while(it.hasNext()){
int k = ((Integer)it.next().intValue()); //wrapping da Integer a int per recuperare il valore dell-elemento i-esimo della lista

System.out.println("valore "+i+" della lista: "+k+"\n");

it.remove();

i++;
}
}
}

wingman87
25-04-2009, 19:16
Ciao, l'errore è nel modo in cui hai usato l'oggetto LCI e i NodoInt.
Hai prima settato i nodi in modo che l'uno puntasse al successivo, praticamente hai
a->b->c
Poi però hai usato il metodo addInTesta su a. addInTesta setta a come nuova testa ma setta anche il successore di a alla vecchia testa e quindi ottieni
a->null
Per risolvere devi usare solo addInTesta per inserire i vari nodi e non usare direttamente setSuccessivo:
lista.addInTesta(c);
lista.addInTesta(b);
lista.addInTesta(a);