Java query

saraciao

il 09 feb 2020, 12:02

13 risposte

saraciao

Utente Attivo

Iscritto da: gen, 2019

Messaggi:: 295

13 Risposte

andbin
Super Famoso
Iscritto da
ott, 2013
Messaggi:
6272
Re: Java query
09 feb 2020, 14:23
saraciao ha scritto:
volevo chiedervi se questa query per le recensioni è corretta
No, purtroppo le parti di SQL inserite nelle stringhe NON sono corrette.

Si vede ad esempio:

......, A.dataRecensione" + "from RecensioneHotel .......

Quando la stringa viene composta tu ottieni

......, A.dataRecensionefrom RecensioneHotel .......

che ovviamente è sbagliato come sintassi. Quando si compone del SQL in questo modo bisogna essere attentissimi agli spazi, virgole, ecc...

Inoltre nel SQL hai "aperto" due parentesi tonde (e tra l'altro la prima NON serve affatto) che poi NON "chiudi":

String url="(SELECT A.nomeHotel, .......
and exists(select * "

queste parentesi non le hai chiuse!!

Poi nelle query hai usato 1 "?" ma nel PreparedStatement setti 2 parametri. E il secondo "?" dove va messo??

No, devi prima provare "a mano" le query con un tool di query (anche grafico) per il tuo DB, poi le trasporti in Java.

Riguardo la parte più Java, ci sono diverse cose che non vanno:

1) catturare SQLException e poi restituire null è poco appropriato. C'è ben poco che possono fare questi metodi (a parte al limite "loggare" l'eccezione). Meglio far USCIRE fuori una eccezione da questi metodi.
Hai la scelta: o fai uscire direttamente il SQLException ma è "checked" e quindi deve essere presa in considerazione altrove ... oppure definisci una tua eccezione "unchecked" che incapsula il SQLException in modo almeno da non dover obbligare a dichiarare/gestire troppo le eccezioni.

2) Hai fatto il executeQuery ed ottenuto il ResultSet ma poi devi ancora scansionarlo ed estrarre i dati.

3) Hai come tipo di ritorno dei ArrayList<String> ma le query tirano fuori 4 colonne. Quindi cosa vuoi fare? Cosa tiri fuori dai metodi? Una lista di stringhe che contiene cosa??

4) Il ResultSet e il PreparedStatement dovresti SEMPRE chiuderli. Quindi serve usare appropriatamente il costrutto try-finally

Insomma, è tutto da rivedere e scrivere più accuratamente.

saraciao

Utente Attivo

Iscritto da: gen, 2019

Messaggi:: 295

Re: Java query

09 feb 2020, 14:45

andbin ha scritto:

saraciao ha scritto:
volevo chiedervi se questa query per le recensioni è corretta
No, purtroppo le parti di SQL inserite nelle stringhe NON sono corrette.

Si vede ad esempio:

......, A.dataRecensione" + "from RecensioneHotel .......

Quando la stringa viene composta tu ottieni

......, A.dataRecensionefrom RecensioneHotel .......

che ovviamente è sbagliato come sintassi. Quando si compone del SQL in questo modo bisogna essere attentissimi agli spazi, virgole, ecc...

Inoltre nel SQL hai "aperto" due parentesi tonde (e tra l'altro la prima NON serve affatto) che poi NON "chiudi":

String url="(SELECT A.nomeHotel, .......
and exists(select * "

queste parentesi non le hai chiuse!!

Poi nelle query hai usato 1 "?" ma nel PreparedStatement setti 2 parametri. E il secondo "?" dove va messo??

No, devi prima provare "a mano" le query con un tool di query (anche grafico) per il tuo DB, poi le trasporti in Java.

Riguardo la parte più Java, ci sono diverse cose che non vanno:

1) catturare SQLException e poi restituire null è poco appropriato. C'è ben poco che possono fare questi metodi (a parte al limite "loggare" l'eccezione). Meglio far USCIRE fuori una eccezione da questi metodi.
Hai la scelta: o fai uscire direttamente il SQLException ma è "checked" e quindi deve essere presa in considerazione altrove ... oppure definisci una tua eccezione "unchecked" che incapsula il SQLException in modo almeno da non dover obbligare a dichiarare/gestire troppo le eccezioni.

2) Hai fatto il executeQuery ed ottenuto il ResultSet ma poi devi ancora scansionarlo ed estrarre i dati.

3) Hai come tipo di ritorno dei ArrayList<String> ma le query tirano fuori 4 colonne. Quindi cosa vuoi fare? Cosa tiri fuori dai metodi? Una lista di stringhe che contiene cosa??

4) Il ResultSet e il PreparedStatement dovresti SEMPRE chiuderli. Quindi serve usare appropriatamente il costrutto try-finally

Insomma, è tutto da rivedere e scrivere più accuratamente.
ho rivisto la parte di sqlpackage DaoTripInTravel;
import java.sql.Connection;
import java.sql.PreparedStatement;
import java.sql.ResultSet;
import java.sql.SQLException;
import java.util.ArrayList;

public class daoRecensioni {
	
	public ArrayList<String> queryRecensioneHotel(String RecensioneHotel ,String servizi)
	{
		Connection conn = DBconnection.getConnection();
		String url="(SELECT A.nomeHotel, A.prezzo, A.descrizione , A.dataRecensione" 
				"from RecensioneHotel A "  
				"where A.nomeHotel=? and exists(select * 
				"			  from Recensione A 
				where  A.idHotel=c.idHotel))";
			try 
			{
				PreparedStatement stm=conn.prepareStatement(url);
				stm.setString(1, RecensioneHotel);
				ResultSet sr=stm.executeQuery(query);
			}
			catch(SQLException e)
			 System.err.println("sql exception");
			 return null;	
			}
	}
}


public ArrayList<String> queryRecensioneAlloggio(String RecensioneAlloggi ,String servizi)
{
	Connection conn = DBconnection.getConnection();
	String url="(SELECT A.nomeAlloggio, A.prezzo, A.descrizione , A.dataRecensione" + 
			"from RecensioneAlloggio A " + 
			"where A.RecensioneAlloggio=? and exists(select * " + 
			"			  from Recensione A where  A.idAlloggio=c.idAlloggio))";
		try 
		{
			PreparedStatement stm=conn.prepareStatement(url);
			stm.setString(1, RecensioneAlloggi);
			ResultSet sr=stm.executeQuery(query);
			
		}
		catch(SQLException e){
              System.err.println("sql exception");
			 return null;	
			}
	}
}
public ArrayList<String> queryRecensioneAttrazioni(String RecensioneAttrazioni ,String servizi)
{
	Connection conn = DBconnection.getConnection();
	String url="(SELECT A.nomeAttrazione, A.prezzo, A.descrizione , A.dataRecensione" + 
			"from Recensione A " + 
			"where A.RecensioneAttrazione=? and exists(select * " + 
			"			  from Recensione A where  A.idAttrazione=c.idAttrazione))";
		try 
		{
			PreparedStatement stm=conn.prepareStatement(url);
			stm.setString(1, RecensioneAttrazioni);
			ResultSet sr=stm.executeQuery(query);
			sr.next()
		}
		catch(SQLException e)
		System.err.println("sql exception");
			 return null;	
			}
	}
}
ma potresti aiutarmi con la parte in java?
in ogni caso grazie mille

andbin
Super Famoso
Iscritto da
ott, 2013
Messaggi:
6272
Re: Java query
09 feb 2020, 14:54
Scusa ma hai "rivisto" che cosa? Mi pare di non vedere modifiche/miglioramenti ....
EDIT: no ok, ho visto bene, hai aggiunto dei System.err.println. Ma questo è il MENO (e comunque serve a poco).

Comunque ho già detto (quasi) tutto nel post precedente. Ci sono delle questioni che devi valutare e stabilire tu.

Es. Una query tira fuori 4 colonne ma poi gli oggetti nella lista sono dei String. Cosa intendi fare? Comporre una singola stringa?
Sì, si può fare ma è ben poco sensato e poco pratico (ma dipende dall'uso che ne devi poi fare).

saraciao

Utente Attivo

Iscritto da: gen, 2019

Messaggi:: 295

Re: Java query

09 feb 2020, 15:13

andbin ha scritto:
Scusa ma hai "rivisto" che cosa? Mi pare di non vedere modifiche/miglioramenti ....
EDIT: no ok, ho visto bene, hai aggiunto dei System.err.println. Ma questo è il MENO (e comunque serve a poco).

Comunque ho già detto (quasi) tutto nel post precedente. Ci sono delle questioni che devi valutare e stabilire tu.

Es. Una query tira fuori 4 colonne ma poi gli oggetti nella lista sono dei String. Cosa intendi fare? Comporre una singola stringa?
Sì, si può fare ma è ben poco sensato e poco pratico (ma dipende dall'uso che ne devi poi fare).

no voglio ottenere una recensione dove ce data, descrizione e il prezzo


ho rivisto la parte di sqlpackage DaoTripInTravel;
import java.sql.Connection;
import java.sql.PreparedStatement;
import java.sql.ResultSet;
import java.sql.SQLException;
import java.util.ArrayList;

public class daoRecensioni {
	
	public ArrayList<String> readRecensioniHotel()	{
	ArrayList<String> arrRecensioniHotel=new ArrayList<String>(String città, String nomeHotel);		
	
	        Connection conn = DBconnection.getConnection();
		String url="(SELECT R.nomeHotel , R.prezzo, R.descrizione , R.dataRecensione", +
				"from RecensioneHotel R "  +
				"where R.nome=? and exists(select * +
				"			  from città C
				where  R.nomeHotel=?  R.idHotel=C.idHotel))";
			try 
			{
				PreparedStatement stm=conn.prepareStatement(url);
				stm.setString(1, RecensioneHotel);
				stm.setString(2, città);				
				ResultSet sr=stm.executeQuery(query);
				while(rs.next())
								{
									String recensioneHotel = rs.getString(1);	

								}
						    return arrRecHotel;
									
					} catch (SQLException e) {
						System.out.println(e.getMessage());
					}
								
			}

andbin
Super Famoso
Iscritto da
ott, 2013
Messaggi:
6272
Re: Java query
09 feb 2020, 15:28
saraciao ha scritto:
no voglio ottenere una recensione dove ce data, descrizione e il prezzo
Ok, vedo ora che hai fatto delle modifiche, chiuso le parentesi ecc..
Ma c'è ancora molta roba che non va. Ora nella prima query hai tolto i + per la concatenazione ... ma ci vogliono!!
Poi devi sistemare le spaziature nel sql. E noto ora che hai usato A come "alias" sia per la query principale che per la subquery di exists (cosa poco furba usare lo stesso alias ...). Poi vedo es. where A.idHotel=c.idHotel Questa "c" cosa sarebbe??
E comunque manca ancora tutto il resto. Insomma stai facendo un po' di "pasticci" (troppi).

Quale è la relazione tra RecensioneHotel e Recensione?
S
saraciao
Utente Attivo
Iscritto da
gen, 2019
Messaggi:
295
Re: Java query
09 feb 2020, 15:34
andbin ha scritto:

saraciao ha scritto:
no voglio ottenere una recensione dove ce data, descrizione e il prezzo
Ok, vedo ora che hai fatto delle modifiche, chiuso le parentesi ecc..
Ma c'è ancora molta roba che non va. Ora nella prima query hai tolto i + per la concatenazione ... ma ci vogliono!!
Poi devi sistemare le spaziature nel sql. E noto ora che hai usato A come "alias" sia per la query principale che per la subquery di exists (cosa poco furba usare lo stesso alias ...). Poi vedo es. where A.idHotel=c.idHotel Questa "c" cosa sarebbe??
E comunque manca ancora tutto il resto. Insomma stai facendo un po' di "pasticci" (troppi).

Quale è la relazione tra RecensioneHotel e Recensione?
ho aggiornato sopra

andbin

Super Famoso

Iscritto da: ott, 2013

Messaggi:: 6272

Re: Java query

09 feb 2020, 15:41

saraciao ha scritto:


	public ArrayList<String> readRecensioniHotel()	{
	ArrayList<String> arrRecensioniHotel=new ArrayList<String>(String città, String nomeHotel);		
	
	        Connection conn = DBconnection.getConnection();
		String url="(SELECT R.nomeHotel , R.prezzo, R.descrizione , R.dataRecensione", +
				"from RecensioneHotel R "  +
				"where R.nome=? and exists(select * +
				"			  from città C
				where  R.nomeHotel=?  R.idHotel=C.idHotel))";
			try 
			{
				PreparedStatement stm=conn.prepareStatement(url);
				stm.setString(1, RecensioneHotel);
				stm.setString(2, città);				
				ResultSet sr=stm.executeQuery(query);
				while(rs.next())
								{
									String recensioneHotel = rs.getString(1);	

								}
						    return arrRecHotel;
									
					} catch (SQLException e) {
						System.out.println(e.getMessage());
					}
								
			}

No mi spiace non ha molto senso.

new ArrayList<String>(String città, String nomeHotel)
è sbagliato, non ha proprio senso, questi due String non c'entrano proprio nulla.

E da dove arriva adesso il valore RecensioneHotel che passi al stm.setString(1, RecensioneHotel); ??

La composizione del SQL è comunque ancora sbagliata (ora hai pure messo una virgola prima di un + ).

No, fai un bel respiro ... o fai una pausa. Così non vai molto avanti ....

saraciao

Utente Attivo

Iscritto da: gen, 2019

Messaggi:: 295

Re: Java query

09 feb 2020, 15:59

andbin ha scritto:

saraciao ha scritto:


	import java.sql.Connection;
import java.sql.PreparedStatement;
import java.sql.ResultSet;
import java.sql.SQLException;
import java.util.ArrayList;

public class daoRecensioni {
	
	public ArrayList<String> queryRecensioneHotel(String RecensioneHotel ,String servizi)
	{
		Connection conn = DBconnection.getConnection();
		String url="(SELECT A.nomeHotel, A.prezzo, A.descrizione , A.dataRecensione" +
				"from RecensioneHotel RH"  +
				"where A.nomeHotel=? and exists(select * +
				"			  from Recensione R 
				where servizi= ? RH.idHotel=R.idHotel))";
			try 
			{
				PreparedStatement stm=conn.prepareStatement(url);
				stm.setString(1, RecensioneHotel);
				stm.setString(2,servizi);
				ResultSet sr=stm.executeQuery(query);
				sr.next=(close);
			}
			catch(SQLException e)
			 System.err.println("sql exception");
			 return null;	
			}
	}
}


public ArrayList<String> queryRecensioneAlloggio(String RecensioneAlloggi ,String servizi)
{
	Connection conn = DBconnection.getConnection();
	String url="(SELECT A.nomeAlloggio, A.prezzo, A.descrizione , A.dataRecensione" + 
			"from RecensioneAlloggio RA" + 
			"where A.RecensioneAlloggio=? and exists(select * " + 
			"			  from recensioni R where = ? servizi  RA.idAlloggio=R.idAlloggio))";
		try 
		{
			PreparedStatement stm=conn.prepareStatement(url);
			stm.setString(1, RecensioneAlloggi);
			ResultSet sr=stm.executeQuery(query);
			stm.setString(2,servizi);
			ResultSet sr=stm.executeQuery(query);
			sr.next=(close);			
			
		}
		catch(SQLException e){
              System.err.println("sql exception");
			 return null;	
			}
	}
}
public ArrayList<String> queryRecensioneAttrazioni(String RecensioneAttrazioni ,String servizi)
{
	Connection conn = DBconnection.getConnection();
	String url="(SELECT A.nomeAttrazione, A.prezzo, A.descrizione , A.dataRecensione" + 
			"from RecensioneAttrazione  RAt" + 
			"where A.RecensioneAttrazione=? and exists(select * " + 
			"			  from  recensioni R where servizi=?  RAt.idAttrazione=R.idAttrazione))";
		try 
		{
			PreparedStatement stm=conn.prepareStatement(url);
			stm.setString(1, RecensioneAttrazioni);
			stm.setString(2,servizi);
			ResultSet sr=stm.executeQuery(query);
			sr.next()
		}
		catch(SQLException e)
		System.err.println("sql exception");
			 return null;	
			}
	}
}

saraciao

Utente Attivo

Iscritto da: gen, 2019

Messaggi:: 295

Re: Java query

09 feb 2020, 16:00

saraciao ha scritto:

andbin ha scritto:

saraciao ha scritto:


	import java.sql.Connection;
import java.sql.PreparedStatement;
import java.sql.ResultSet;
import java.sql.SQLException;
import java.util.ArrayList;

public class daoRecensioni {
	
	public ArrayList<String> queryRecensioneHotel(String RecensioneHotel ,String servizi)
	{
		Connection conn = DBconnection.getConnection();
		String url="(SELECT A.nomeHotel, A.prezzo, A.descrizione , A.dataRecensione" +
				"from RecensioneHotel RH"  +
				"where A.nomeHotel=? and exists(select * +
				"			  from Recensione R 
				where servizi= ? RH.idHotel=R.idHotel))";
			try 
			{
				PreparedStatement stm=conn.prepareStatement(url);
				stm.setString(1, RecensioneHotel);
				stm.setString(2,servizi);
				ResultSet sr=stm.executeQuery(query);
				sr.next=(close);
			}
			catch(SQLException e)
			 System.err.println("sql exception");
			 return null;	
			}
	}
}


public ArrayList<String> queryRecensioneAlloggio(String RecensioneAlloggi ,String servizi)
{
	Connection conn = DBconnection.getConnection();
	String url="(SELECT A.nomeAlloggio, A.prezzo, A.descrizione , A.dataRecensione" + 
			"from RecensioneAlloggio RA" + 
			"where A.RecensioneAlloggio=? and exists(select * " + 
			"			  from recensioni R where = ? servizi  RA.idAlloggio=R.idAlloggio))";
		try 
		{
			PreparedStatement stm=conn.prepareStatement(url);
			stm.setString(1, RecensioneAlloggi);
			ResultSet sr=stm.executeQuery(query);
			stm.setString(2,servizi);
			ResultSet sr=stm.executeQuery(query);
			sr.next=(close);			
			
		}
		catch(SQLException e){
              System.err.println("sql exception");
			 return null;	
			}
	}
}
public ArrayList<String> queryRecensioneAttrazioni(String RecensioneAttrazioni ,String servizi)
{
	Connection conn = DBconnection.getConnection();
	String url="(SELECT A.nomeAttrazione, A.prezzo, A.descrizione , A.dataRecensione" + 
			"from RecensioneAttrazione  RAt" + 
			"where A.RecensioneAttrazione=? and exists(select * " + 
			"			  from  recensioni R where servizi=?  RAt.idAttrazione=R.idAttrazione))";
		try 
		{
			PreparedStatement stm=conn.prepareStatement(url);
			stm.setString(1, RecensioneAttrazioni);
			stm.setString(2,servizi);
			ResultSet sr=stm.executeQuery(query);
			sr.next()
		}
		catch(SQLException e)
		System.err.println("sql exception");
			 return null;	
			}
	}
}

Ho modificato qualcosa...so che ancora non va bene ma va meglio di prima?

andbin
Super Famoso
Iscritto da
ott, 2013
Messaggi:
6272
Re: Java query
09 feb 2020, 16:21
saraciao ha scritto:
so che ancora non va bene ma va meglio di prima?
No purtroppo.

Facciamo così. Mostrami come sono fatte le tabelle RecensioneHotel e Recensione e cosa vuoi ottenere nel ArrayList e ti scrivo io un pezzo di codice da cui prendere spunto per le altre.
S
saraciao
Utente Attivo
Iscritto da
gen, 2019
Messaggi:
295
Re: Java query
09 feb 2020, 16:28
andbin ha scritto:

saraciao ha scritto:
so che ancora non va bene ma va meglio di prima?
No purtroppo.

Facciamo così. Mostrami come sono fatte le tabelle RecensioneHotel e Recensione e cosa vuoi ottenere nel ArrayList e ti scrivo io un pezzo di codice da cui prendere spunto per le altre.
Non ho due tabelle ho solo una tabella recensione che deve contenere prezzo data recensione e descrizione e devo realizzare una query che mostra ad esempio tutte le recensioni di un determinato hotel
andbin
Super Famoso
Iscritto da
ott, 2013
Messaggi:
6272
Re: Java query
09 feb 2020, 16:30
saraciao ha scritto:
Non ho due tabelle ho solo una tabella recensione che deve contenere prezzo data recensione e descrizione
E perché fai from RecensioneHotel e poi from Recensione ???
Scusa ma allora di cosa stiamo discutendo??

saraciao

Utente Attivo

Iscritto da: gen, 2019

Messaggi:: 295

Re: Java query

10 feb 2020, 10:03

andbin ha scritto:

saraciao ha scritto:
Non ho due tabelle ho solo una tabella recensione che deve contenere prezzo data recensione e descrizione
E perché fai from RecensioneHotel e poi from Recensione ???
Scusa ma allora di cosa stiamo discutendo??

Si in effetti è inutile potrei fare per la query:
Select*
Form recensioni
Where id_hotel= Id_Hotel

cosi va bene?



package DaoTripInTravel;
import java.sql.Connection;
import java.sql.PreparedStatement;
import java.sql.ResultSet;
import java.sql.SQLException;
import java.util.ArrayList;

public class daoRecensioni {
	public ArrayList<String> readRecensioni1;
	
	{
		Connection conn = DBconnection.getConnection();
		String url="(SELECT *" + 
				"from recensioni " + 
				"where id_hotel = id_hotel";
			try 
			{
				PreparedStatement stm=conn.prepareStatement(url);
				ResultSet sr=stm.executeQuery(url);
				sr.next();
			}
			
			catch(SQLException e) {
			System.err.println("sql exception");
	      }
      }


public ArrayList<String> readRecensioni2;
{
	Connection conn = DBconnection.getConnection();
	String url= "(SELECT * " + 
			"from recensioni " + 
			"where id_Alloggio = id_Alloggio";
		try 
		{
			PreparedStatement stm=conn.prepareStatement(url);
			ResultSet sr=stm.executeQuery(url);
			sr.next();
		}
		catch(SQLException e) {
			System.err.println("sql exception");
		}
}
public ArrayList<String> readRecensione2;
{
	Connection conn = DBconnection.getConnection();
	String url="(SELECT *" + 
			"from recensioni " + 
			"where id_Attrazioni = id_Attrazioni";
		try 
		{
			PreparedStatement stm=conn.prepareStatement(url);
			ResultSet sr=stm.executeQuery(url);
			sr.next();
		}
		catch(SQLException e) {
			System.err.println("sql exception");
		}
		
  }

}

Devi accedere o registrarti per scrivere nel forum

13 risposte