Discussoes de Programacao e Tecnologia

ResultSet me tirando o sono

22 respostas
D
danielswater

Fala pessoal blz?
E o seguinte, preciso fazer um sistema web aqui que verifica se o login e o email da ja existe cadastrado no banco, se não existir, faz o cadastro normalmente. O problema e que eu nao sei o que esta acontecendo com o resultset nao me retorna nem null nem “” e nem aparece erro nenhum. O meu banco está vazio, justamente por isso coloquei as condições de verificar se e null ou string vazia. Segue me cod:

package br.com.desenvolvimento;

import java.io.IOException;
import java.io.PrintWriter;
import java.sql.Connection;
import java.sql.ResultSet;
import java.sql.SQLException;
import java.sql.Statement;

import javax.servlet.http.HttpServlet;
import javax.servlet.http.HttpServletRequest;
import javax.servlet.http.HttpServletResponse;

public class GravaContato extends HttpServlet {

	public void doGet(HttpServletRequest request, HttpServletResponse response) throws IOException{

		PrintWriter out = response.getWriter();

		Connection conexao = new Conexao().abreConexao();

		String sql = "SELECT login, email FROM usuario";

		try{
			Statement st = conexao.createStatement();
			ResultSet rs = st.executeQuery(sql);
			String login = "";
			String email = "";

			while(rs.next()){
				login = rs.getString("login");
				email = rs.getString("email");

				if(login == "" || login == null || email == "" || login == null
						|| login != request.getParameter("login") || email != request.getParameter("email")){
					
					Contato cont = new Contato();
					
					cont.setNome(request.getParameter("nome"));
					cont.setEmail(request.getParameter("email"));
					cont.setSexo(request.getParameter("sexo"));
					cont.setIdade(Integer.parseInt(request.getParameter("idade")));
					cont.setNews(request.getParameter("news"));
					cont.setLogin(request.getParameter("login"));
					
					ComandoDAO dao = new ComandoDAO();
					dao.insereContato(cont);
				}
				else{
					out.println("NÃO FOI POSSÍVEL CADASTRAR. EMAIL OU LOGIN JÁ EXISTE");
				}
			}

		}
		catch(Exception e){
			out.println("ERRO: " + e.getMessage());
		}			
	}

}

Alguma luz?

Alura Desenvolvimento Front-End React Do primeiro componente à liderança técnica! HTML/CSS/JavaScript fundamental, React com hooks e...

22 Respostas

romarcio
romarcio

No if, use equals para testar string.

if(login.equals("") || login == null || email.equals("") || login == null  
                        || !login.equals(request.getParameter("login")) || !email.equals(request.getParameter("email"))){
D
danielswater

Tinha esquecido do equals, mas mesmo assim continua com o mesmo problema, não verifica e nem grava no banco :shock:

romarcio
romarcio

Mas entra no if, no else ou cai no catch ?

Consegue debugar para verificar isso?

D
danielswater

Entao, ele nao entra nem no if nem no else, e também acredito que nao va para o catch pois ele nao me da nenhuma saida, nem que foi gravado, nem que nao foi gravado e nem a mensagem de erro do catch

romarcio
romarcio

Que estranho, tem certeza que essa classe está sendo executada?

É mais aconselhavel ustilizar PreparedStatement a Statement, mude essa parte:

...
try{  
            PreparedStatement stmt = prepareStatement(sql); 
            ResultSet rs = stmt.executeQuery(); 
...
//antes de terminar o bloco try, feche a conexão
rs.close();
stmt.close();
D
danielswater

Entao cara, fiz o que vc falou e continua nao acontecendo nada. Alias já tinha feito estes testes antes.
Uma coisa que percebi e que se por exemplo eu coloco algo no banco, ele faz a inclusao dos dados, porém ele não faz a comparação desta linha aqui:

login != request.getParameter("login") || email != request.getParameter("email"))
romarcio
romarcio
danielswater:
Entao cara, fiz o que vc falou e continua nao acontecendo nada. Alias já tinha feito estes testes antes. Uma coisa que percebi e que se por exemplo eu coloco algo no banco, ele faz a inclusao dos dados, porém ele não faz a comparação desta linha aqui: 
login != request.getParameter("login") || email != request.getParameter("email"))

Estranho, tenta passar para variáveis locais então:

String reqLogin = request.getParameter("login");
String reqEmail = request.getParameter("email");

//E depois faz o teste no if pelas variaveis
 ...
 login.equals(reqLogin) || email.equals(reqEmail));

Mas ta estranho isso, faz um System.out.print(); de todas a variaveis neste método, e ve se retorna tudo certo.

D
danielswater

Fiz o teste, ele mostra os request certinho, so que continua nao fazendo a gravação no banco e nem a comparaçao
Ja to ficando louco com isso

Priuli
Priuli

eu não sei confirmar neste momento mais acho que o response.getWriter() não irá imprimir nada na tela simplements irá imprimir no response ai caberia ter um tratamento javascript ou algo para pegar este erro…
Tente dar uns System.out… ou então debugar no catch…

Priuli
Priuli

romarcio:
No if, use equals para testar string.

if(login.equals("") || login == null || email.equals("") || login == null  
                        || !login.equals(request.getParameter("login")) || !email.equals(request.getParameter("email"))){

Verifique a ordem pois se o login ou email for null vai dar erro neste codigo.
Correto:

if( login == null || login.equals("")  || email == null ||  email.equals("") 
                        || !login.equals(request.getParameter("login")) || !email.equals(request.getParameter("email"))){
D
danielswater

Então, o problema e que nem erro da, nao me mostra absolutamente nada.
Eu mudei a classe aqui, ficando assim:

package br.com.desenvolvimento;  

import java.io.IOException;
import java.io.PrintWriter;
import java.sql.Connection;
import java.sql.PreparedStatement;
import java.sql.ResultSet;

import javax.servlet.http.HttpServlet;
import javax.servlet.http.HttpServletRequest;
import javax.servlet.http.HttpServletResponse;

public class GravaContato extends HttpServlet {  

	public void doGet(HttpServletRequest request, HttpServletResponse response) throws IOException{  

		PrintWriter out = response.getWriter();  

		Connection conexao = new Conexao().abreConexao();     
		String sql = "SELECT login, email FROM usuario"; 
		String login = "";  
		String email = "";
		String reqEmail = request.getParameter("email");
		String reqLogin = request.getParameter("login");

		try{  
			PreparedStatement pstm = conexao.prepareStatement(sql);
			ResultSet rs = pstm.executeQuery();
			
			while(rs.next()){  
				login = rs.getString("login");  
				email = rs.getString("email");  

				if(login.equals(reqLogin) || email.equals(reqEmail)){
					out.println("NAO FOI POSSIVEL CADASTRAR");
				}
				
				else{  

					Contato cont = new Contato();  

					cont.setNome(request.getParameter("nome"));  
					cont.setEmail(request.getParameter("email"));  
					cont.setSexo(request.getParameter("sexo"));  
					cont.setIdade(Integer.parseInt(request.getParameter("idade")));  
					cont.setNews(request.getParameter("news"));  
					cont.setLogin(request.getParameter("login"));  

					ComandoDAO dao = new ComandoDAO();  
					dao.insereContato(cont);  

					out.println("DADOS GRAVADOS COM SUCESSO"); 
				}  
			}rs.close(); 
			pstm.close();

		}  
		catch(Exception e){  
			out.println("ERRO: " + e.getMessage());  
		}
	
		out.println(login);
		out.println(email);
		out.println(reqEmail);
		out.println(reqLogin);
	}  

}

Mesmo assim só continua cadastrando no banco se já tiver algo la, agora ele compara se existe ou nao, porem ele cadastra do mesmo jeito, sendo igual ou nao

lucianotome
lucianotome

Cara, se não estou errado, vc faz um select no banco vazio, OK, este resultset te retorna nada, ai vc faz um

while(rs.next()){ enquanto tiver proximo ele entra no while, mas vc disse que o banco esta vazio, como ele vai entrar no while?

DZR
DZR

Exatamente como o Luciano disse

O while do rs.next() não vai entrar se o seu banco estiver vazio.
Ou seja, a sua comparação com banco vazio não irá funcionar.

lucianotome
lucianotome
Talvez vc possa fazer assim: 
if (rs.next()) {  
     do {   
         //fazer algo

     } while (rs.next());  
 } else {  
     //banco vazio
 }
D
danielswater

Ok pessoal, realmente acho que e esse mesmo o problema
Vou fazer os testes aqui logo mais e jaja volto pra dar o resultado

D
danielswater

Sem chance, continua dando a mesma coisa

lucianotome
lucianotome

Posta seu novo codigo ai

D
danielswater 
package br.com.desenvolvimento;

import java.io.IOException;
import java.io.PrintWriter;
import java.sql.Connection;
import java.sql.ResultSet;
import java.sql.SQLException;
import java.sql.Statement;

import javax.servlet.http.HttpServlet;
import javax.servlet.http.HttpServletRequest;
import javax.servlet.http.HttpServletResponse;

public class GravaContato extends HttpServlet {

	public void doGet(HttpServletRequest request, HttpServletResponse response) throws IOException{

		PrintWriter out = response.getWriter();

		Connection conexao = new Conexao().abreConexao();

		String sql = "SELECT login, email FROM usuario";

		try{
			Statement st = conexao.createStatement();
			ResultSet rs = st.executeQuery(sql);
			String login = "";
			String email = "";

			if(rs.next()){
				do{
				login = rs.getString("login");
				email = rs.getString("email");

					Contato cont = new Contato();
					
					cont.setNome(request.getParameter("nome"));
					cont.setEmail(request.getParameter("email"));
					cont.setSexo(request.getParameter("sexo"));
					cont.setIdade(Integer.parseInt(request.getParameter("idade")));
					cont.setNews(request.getParameter("news"));
					cont.setLogin(request.getParameter("login"));
					
					ComandoDAO dao = new ComandoDAO();
					dao.insereContato(cont);
				}
				while(rs.next());
			}
			else{
				out.println("NÃO FOI POSSÍVEL CADASTRAR. EMAIL OU LOGIN JÁ EXISTE");
			}

		}
		catch(Exception e){
			out.println("ERRO: " + e.getMessage());
		}			
	}
}

Aqui no caso eu ate tirei a comparaçao, so que a unica coisa que ele me da e que o login ou email já existe. Isso porque meu banco esta vazio :frowning:

lucianotome
lucianotome

Quanto o banco esta vazio
isto aqui sempre retorna false if(rs.next()){
ou seja, sempre vai dar

else{ out.println("NÃO FOI POSSÍVEL CADASTRAR. EMAIL OU LOGIN JÁ EXISTE"); }

Vc tem que rever a sua logica, nao tem erro nenhum no codigo, e sim na logica, OK!!!

lucianotome
lucianotome 
package br.com.desenvolvimento;

import java.io.IOException;
import java.io.PrintWriter;
import java.sql.Connection;
import java.sql.ResultSet;
import java.sql.SQLException;
import java.sql.Statement;

import javax.servlet.http.HttpServlet;
import javax.servlet.http.HttpServletRequest;
import javax.servlet.http.HttpServletResponse;

public class GravaContato extends HttpServlet {
    public void doGet(HttpServletRequest request, HttpServletResponse response) throws IOException{

		PrintWriter out = response.getWriter();

		Connection conexao = new Conexao().abreConexao();

		String sql = "SELECT login, email FROM usuario";

		try{
			Statement st = conexao.createStatement();
			ResultSet rs = st.executeQuery(sql);
			String login = "";
			String email = "";

			if(rs.next()){
				do{
				login = rs.getString("login");
				email = rs.getString("email");

				            if(login == "" || login == null || email == "" || login == null
						|| login != request.getParameter("login") || email != request.getParameter("email")){

					                Contato cont = new Contato();
					
					                cont.setNome(request.getParameter("nome"));
					                cont.setEmail(request.getParameter("email"));
					                cont.setSexo(request.getParameter("sexo"));
					                cont.setIdade(Integer.parseInt(request.getParameter("idade")));
					                cont.setNews(request.getParameter("news"));
					                cont.setLogin(request.getParameter("login"));
					
					                ComandoDAO dao = new ComandoDAO();
					                dao.insereContato(cont);
                                             }else{  
                                                       out.println("NÃO FOI POSSÍVEL CADASTRAR. EMAIL OU LOGIN JÁ EXISTE");  
                                             }  
		            }while(rs.next());
		}else{
					Contato cont = new Contato();
					
					cont.setNome(request.getParameter("nome"));
					cont.setEmail(request.getParameter("email"));
					cont.setSexo(request.getParameter("sexo"));
					cont.setIdade(Integer.parseInt(request.getParameter("idade")));
					cont.setNews(request.getParameter("news"));
					cont.setLogin(request.getParameter("login"));
					
					ComandoDAO dao = new ComandoDAO();
					dao.insereContato(cont);
			}

		}
		catch(Exception e){
			out.println("ERRO: " + e.getMessage());
		}			
	}
}
D
danielswater

Cara, simplesmente copiei seu codigo e agora cadastra toda vez, nao verifica as condições e tambem nao me da a mensagem que foi cadastrado com sucesso

Nao sei mais o que pode ser

eduardo.exs
eduardo.exs

porra… estou tendo o mesmo problema e incrívelmente comecei a depurar pra tentar entender e ví que ele não compara

não me pergunte o pq? mas tentei comprar a string enviada pelo formulario com a que está no banco

utilizei (.equals)

(==)

(!=)

e até tentativa de comparação estática… tipo: var == “string”
Criado 13 de outubro de 2010
Ultima resposta 5 de set. de 2011
Respostas 22
Participantes 6

Topicos relacionados

Compilar e executar uma classe java pelo prompt do windows 13 respostas [RESOLVIDO] Como executar função JavaScript após página carregada 13 respostas Java.lang.OutOfMemoryError: Java heap space - Como resolver? 11 respostas Como consultar CPF na Receita Federal usando Web Service? 22 respostas Como acessar localhost de forma externa? [RESOLVIDO] 13 respostas
Alura POO: o que é programação orientada a objetos? Aprenda os conceitos básicos da programação orientada a objetos, como classes, objetos, herança, encapsulamento e polimorfismo, com exemplos.
Casa do Codigo JavaScript Assertivo: Testes e qualidade de codigo em... Por Gabriel Ramos — Casa do Codigo
Fiap Pos-Tech: Front-end Engineering Pós-Graduação para desenvolver habilidades avançadas em tecnologias front-end, com foco em...