Perchè non compila?

**Darèios89** · 20-05-2010, 00:07

Ho un esercizio, aldilà della corretteza logica ci deve essere un errore diverso:

codice:

import java.io.*;

public interface Stack <E>
{
	public int size();
	//Return numero elementi
	
	public boolean isEmpty();
	//Return true se lo stack è vuoto
	
	public void push(E object);
	//Pone in cima allo stack un elemento
	
	public E pop(); 
	//Elimina l'elemento in cima allo stack e lo restituisce
	
	public E top(); 
	//Indica qual è l'elemento in cima allo stack senza estrarlo
	
}

public class Node <E>{
	private E element;
	private E next;
	
	public Node(E element, E next)
	{
		this.element=element;
		this.next=next;
	}
	
	public Node <E> getElement() {return element;}
	
	public Node <E> getNext() {return next;}

	public void setElement(E object){ element=object;}
	
	public void setNext(Node <E> e){element.getNext()=e;}
}

public class NodeStack <E> implements Stack <E> {
	
	private Node <E> top;
	private int size;
	
	public NodeStack()
	{
		top=null;
		size=0;
	}
	
	public int size() {return size;}
	
	public boolean isEmpty(){return (top==null);}
	
	public void push(E element){
		Node <E> n=new Node <E>(element,top);
		top=n;
		size++;
	}
	
	public E pop(){
		if(E.isEmpty()) throw new EmptyStackException("Stack vuoto");
		E object=top.getElement();
		top=top.getNext();
		size--;
		return object;
	}
	
	public E top()
	{
		if(E.isEmpty()) throw new EmptyStackException("Stack vuoto");
		return top.getElement();
	}
	
	public String toString()
	{
		
}
		
	
	
}

public class EsercizioI{


public static void main(String []args)
{
	FileReader reader=null;
	BufferedReader buffer=null;
	PrintWirter out=null;
	
	try{
		reader=new Filereader("input.txt");
		buffer=new BufferedReader(reader);
		out=new PrintWriter("OUTPUT.txt");
		
		NodeStack <String> stack=new NodeStack(); 
		
		String sequenze="";
		
		while((sequenze=buffer.readLine())!=null)
			stack.push(sequenze);
			
			for(Node node=new Node(); node.getNext()!=null; node=node.getNext())
			{
				boolean parentesizzata=false;
				String element=node.getElement();
				
				for(int i=0; i<element.length(); i++)
				{
					if(
						(element.charAt(i)=="(" && element.charAt(element.length()-1-i)==")") || 
						(element.charAt(i)=="{" && element.charAt(element.length()-1-i)=="}")
					||(element.charAt(i)=="[" && element.charAt(element.length()-1-i)=="]")
					||(element.charAt(i)=="<" && element.charAt(element.length()-1-i)==">"))
					
					parentesizzata=true;
					
				
					else
					{
						for(int j=0; j<element.length()-1; j++)
						{
							if(
							(element.charAt(j)=="(" && element.charAt(j+1)==")") || 
							(element.charAt(j)=="{" && element.charAt(j+1)=="}")
							||(element.charAt(j)=="[" && element.charAt(j+1)=="]")
							||(element.charAt(j)=="<" && element.charAt(j+1)==">"))
					
							parentesizzata=true;
							
						}
						
						i=element.length()-1;
					}
				}
				
				if(parentesizzata==true)
					out.print("OK");
				else out.print("ERRORE");
				
			}
		}
		
		finally{
			
				if(reader!=null)reader.close();
			
				if(out!=null)out.close();
			}
}						
}

Gli errori già partono per l'interfaccia, che mi dice deve essere dichiarata a parte, e anche la classe Nodo, ma perchè mi dà quest'errore?

**antotan** · 20-05-2010, 07:01

Il file non viene compilato a causa della marea di errori che ci sono. Se vuoi scoprirli devi per forza mettere ogni classe nel suo file .java, così fai ordine.
Alcuni errori sono:

Ritorni un E e non un Node<E> (element è un E):

codice:

public Node <E> getElement() {return element;}
public Node <E> getNext() {return next;}

element, essendo un E, non ha il metodo getNext:

codice:

public void setNext(Node <E> e){element.getNext()=e;}

Questi non esistono:

codice:

PrintWirter
Filereader

Metti ordine, correggi gli errori di battitura e ricordati di intercettare le eccezioni.
Ti potrebbe convenire usare un IDE comunque, se non hai voglia di trovarti da solo gli errori.

**LeleFT** · 20-05-2010, 10:58

Non è questione di ordine, è questione di rispettare le regole previste da Java: ogni classe o interfaccia che sia dichiarata public deve stare dentro ad un file .java con lo stesso nome della classe o dell'interfaccia.

Capitolo 1 di qualunque libro su Java.

Ciao.

**Darèios89** · 20-05-2010, 14:57

Ah bene, al momento ho risolto non mettendole public.
Ma eventualmente dovrei dichiarare ogni classe in un file separato giusto?

Per esempio:

codice:

 public interface Stack<E>{
.
.
.

Salvata come?
Stack.java?

E poi per usarle basta che sono tutte nella stessa cartella e le importo nel main?
cosa scrivo..semplicemente:

codice:

import Stack
import Altreclassi

?

**Alex'87** · 20-05-2010, 15:11

Originariamente inviato da Darèios89
E poi per usarle basta che sono tutte nella stessa cartella e le importo nel main?
cosa scrivo..semplicemente:

codice:

import Stack
import Altreclassi

?

Se sono nello stesso package non serve usare import.

**Darèios89** · 20-05-2010, 15:45

Ok...bè..vedrò di provare questa cosa in un secondo momento, ho cercato di aggiustare il mio programma, mi dà solo un problema, che credo dipenda dai generics:

codice:

import java.io.*;

interface Stack <E>
{
	public int size();
	//Return numero elementi
	
	public boolean isEmpty();
	//Return true se lo stack è vuoto
	
	public void push(E object);
	//Pone in cima allo stack un elemento
	
	public E pop(); 
	//Elimina l'elemento in cima allo stack e lo restituisce
	
	public E top(); 
	//Indica qual è l'elemento in cima allo stack senza estrarlo
	
}

class Node <E>{
	private E element;
	private Node <E> next;
	
	public Node(E element, Node <E> next)
	{
		this.element=element;
		this.next=next;
	}
	
	public E getElement() {return element;}
	
	public Node <E> getNext() {return next;}

	public void setElement(E object){ element=object;}
	
	public void setNext(Node <E> e){next=e;}
}

class EmptyStackException extends RuntimeException{
	public EmptyStackException(String err){
		super(err);
	}
}

class NodeStack <E> implements Stack <E> {
	
	private Node <E> top;
	private int size;
	
	public NodeStack()
	{
		top=null;
		size=0;
	}
	
	public int size() {return size;}
	
	public boolean isEmpty(){return (top==null);}
	
	public void push(E element){
		Node <E> n=new Node <E>(element,top);
		top=n;
		size++;
	}
	
	public E pop() throws EmptyStackException {
		if(isEmpty()) throw new EmptyStackException("Stack vuoto.");
		E object=top.getElement();
		top=top.getNext();
		size--;
		return object;
	}
	
	public E top() throws EmptyStackException
	{
		if(isEmpty()) throw new EmptyStackException("Stack vuoto");
		return top.getElement();
	}
	
}

public class EsercizioI{


public static void main(String []args) throws IOException
{
	FileReader reader=null;
	BufferedReader buffer=null;
	PrintWriter out=null;
	
	try{
		reader=new FileReader("input.txt");
		buffer=new BufferedReader(reader);
		out=new PrintWriter("OUTPUT.txt");
		
		NodeStack <String> stack=new NodeStack(); 
		
		String sequenze="";
		int numero=0;
		
		while((sequenze=buffer.readLine())!=null)
		{
			stack.push(sequenze);
			numero++;
		}
			
				
				while(stack.isEmpty()!=true)
				{
					
					boolean parentesizzata=false;
					String element=(String)stack.top();
				
					
					for(int i=0; i<element.length(); i++)
					{
					
					if(
						(element.charAt(i)=='(' && element.charAt(element.length()-1-i)==')') || 
						(element.charAt(i)=='{' && element.charAt(element.length()-1-i)=='}')
					||(element.charAt(i)=='[' && element.charAt(element.length()-1-i)==']')
					||(element.charAt(i)=='<' && element.charAt(element.length()-1-i)=='>'))
					
					parentesizzata=true;
					
				
					else
					{
						for(int j=0; j<element.length()-1; j=j+2)
						{
							if(
							(element.charAt(j)=='(' && element.charAt(j+1)==')') || 
							(element.charAt(j)=='{' && element.charAt(j+1)=='}')
							||(element.charAt(j)=='[' && element.charAt(j+1)==']')
							||(element.charAt(j)=='<' && element.charAt(j+1)=='>'))
					
							parentesizzata=true;
							
						}
						
						i=element.length()-1;
					}
					
					}
					
					stack.pop();
				
				
					if(parentesizzata==true)
					out.print("OK");
					else out.print("ERRORE");
				}
				
		}
		
		
		finally{
			
				if(reader!=null)reader.close();
			
				if(out!=null)out.close();
			}
}						
}

L'errore è:

codice:

Note: EsercizioI.java uses unchecked or unsafe operations.
Note: Recompile with -Xlint:unchecked for details.
>Exit code: 0

**andbin** · 20-05-2010, 16:05

Originariamente inviato da Darèios89

codice:

Note: EsercizioI.java uses unchecked or unsafe operations.
Note: Recompile with -Xlint:unchecked for details.
>Exit code: 0

NodeStack <String> stack=new NodeStack<String>();

Da aggiungere la parte in rosso.

**Alex'87** · 20-05-2010, 16:09

Originariamente inviato da Darèios89
L'errore è:

codice:

Note: EsercizioI.java uses unchecked or unsafe operations.
Note: Recompile with -Xlint:unchecked for details.
>Exit code: 0

Non è un errore, ma un warning. C'è una bella differenza

**Darèios89** · 20-05-2010, 16:13

Ah bene....si una distrazione....ti ringrazio.
Bene di funzionare funziona, nel senso che scrive qualcosa

Però non come dovrebbe, praticamente nell'esercizio si dice:

codice:

Si realizzi un'implementazione dell'interfaccia Stack<E> (come mostrato a lezione)
e si utilizzi la classe ottenuta per la realizzazione del seguente problema legato alle
sequenza parentesizzate.
Una sequenza S si dice parentesizzata se
– è formata da una coppia di parentesi, una aperta e l'altra chiusa.
Esempio: S = ( )
– è la concatenazione di due sequenze parentesizzate, S1 ed S2.
Esempio: S1 S2
– è data da una sequenza parentesizza, S, racchiusa tra parentesi
Esempio: ( S )
Si fornisca un programma Java che prenda in input un file di testo contenente una
lista di sequenze di parentesi, una per riga, e permetta di controllare se tali
sequenze siano delle sequenza parentesizzate.
Input
Il file di input contiene una sequenza di sequenze, una per riga, terminanti con il
simbolo “ ;” (punto e virgola). Ciascuna sequenza contiene una lista di simboli di
parentesi tra: (, [, {, < >, }, ], )
Output
il file di output contiene una riga per ogni sequenza di input. La riga contiene la
stringa 'Ok' se la relativa sequenza è parentesizzata, altrimenti contiene la stringa
'Errore'.

Ora io ho fatto un controllo, ma non credo che funzioni perchè l'output stampa sempre ok,
il mio controllo è stato vedere se nella stringa ci fossero le stesse parentesi agli estremi, cioè indice 0 con l'ultimo indice della stringa, poi il secondo con il penultimo e così via.
Se la condizione non è verificata faccio un altro controllo secondo cui devo trovare parentesi aperta e chiusa passando da indice ad indice....

**andbin** · 20-05-2010, 16:30

Originariamente inviato da Darèios89
Bene di funzionare funziona, nel senso che scrive qualcosa

Però non come dovrebbe

Non ho letto tutta la parte di gestione di quei file che fai nel main() ma da una veloce prova del tuo codice mi pare che lo stack funzioni bene.

codice:

NodeStack<String> stack = new NodeStack<String>();

stack.push("One");
stack.push("Two");
stack.push("Three");

System.out.println(stack.size());
System.out.println(stack.pop());
System.out.println(stack.size());
System.out.println(stack.pop());
System.out.println(stack.size());
System.out.println(stack.pop());
System.out.println(stack.size());

try {
    stack.pop();
} catch (EmptyStackException e) {
    System.out.println("Corretto ... essendo vuoto!");
}

Output:
3
Three
2
Two
1
One
0
Corretto ... essendo vuoto!

Il che bene o male prova che almeno a livello basilare lo stack funziona come dovrebbe.

Discussione: Perchè non compila?

Strumenti discussione

Ricerca discussione

Visualizza

Perchè non compila?

Permessi di invio