Publicidade

Rodrigo5468

Verificar campos e DB

Patrocínio:

Olá,

 

Estou fazendo um sistema de cadastro em PDO OO.

 

Cadastro.php

public function insert() {
	if(isset($usuario) && isset($nomesobrenome) && isset($email) && isset($senha)) {
		if(empty($usuario) OR empty($nomesobrenome) OR empty($email) OR empty($senha)) {
			echo 'Existe campos vazios.</div>';
		}
	}else {
		$verifica = DB::prepare("SELECT * FROM {$this->table} WHERE usuario=:usuario OR nomesobrenome=:nomesobrenome OR email=:email");
		$verifica->bindValue(':usuario',$this->usuario,PDO::PARAM_STR);
		$verifica->bindValue(':nomesobrenome',$this->nomesobrenome,PDO::PARAM_STR);
		$verifica->bindValue(':email',$this->email,PDO::PARAM_STR);
		$verifica->execute();
		
		if($verifica->rowCount() == 0) {
			$sql  = "INSERT INTO $this->table (usuario, nomesobrenome, email, senha) VALUES (:usuario, :nomesobrenome, :email, :senha)";
			$stmt = DB::prepare($sql);
			$stmt->bindParam(':usuario', $this->usuario);
			$stmt->bindParam(':nomesobrenome', $this->nomesobrenome);
			$stmt->bindParam(':email', $this->email);
			$stmt->bindParam(':senha', $this->senha);
			return $stmt->execute();
		}else {
			return false;
		}
	}
}

 

 

register.php

			<?php
				$usuarios = new Usuarios();
				
				if(isset($_POST['cadastrar'])) {
					$usuario					= $_POST['usuario'];
					$nomesobrenome				= $_POST['nomesobrenome'];
					$email						= $_POST['email'];
					$senha						= $_POST['senha'];
					
					$usuarios->setUsuario($usuario);
					$usuarios->setNomesobrenome($nomesobrenome);
					$usuarios->setEmail($email);
					$usuarios->setSenha($senha);
					
					#Insert
					if($usuarios->insert()) {
						echo 'Conta registrada com sucesso! Você deverá confirmar a conta em seu endereço de e-mail.</div>';
					}
				}
			?>

Quando eu registro a conta aparece a seguinte frase "Conta registrada com sucesso! Você deverá confirmar a conta em seu endereço de e-mail.". Mas já existe um cadastro no banco de dados não aparece nenhum erro, informando que já existe o usuário, o nome e sobrenome ou o e-mail. E quando existe campos vazios o script cadastra a conta do mesmo jeito. O que  preciso mudar para corrigir o meu código?

 

 

Obrigado desde já.

0

Compartilhar este post


Link para o post
Compartilhar em outros sites

@Rodrigo5468, existe muitos problemas no seu código, vou falar apenas do problema em questão e outras basicas
 

public function insert() 
{
  if(empty($this->usuario) OR empty($this->nomesobrenome) OR empty($this->email) OR empty($this->senha)) {
    echo 'Existe campos vazios.</div>'; # Dar echo dentro da classe não é bom!
    return false;
  }

  $stmt = DB::prepare("
    SELECT * 
    FROM {$this->table} 
    WHERE usuario=:usuario 
    OR nomesobrenome=:nomesobrenome 
    OR email=:email"
  );

  $stmt->bindValue(':usuario',$this->usuario,PDO::PARAM_STR);
  $stmt->bindValue(':nomesobrenome',$this->nomesobrenome,PDO::PARAM_STR);
  $stmt->bindValue(':email',$this->email,PDO::PARAM_STR);
  $stmt->execute();

  $usuarioExiste = $stmt->rowCount();

  if($usuarioExiste) {
    return false;
  }

  $sql = "INSERT INTO $this->table (usuario, nomesobrenome, email, senha) 
  VALUES (:usuario, :nomesobrenome, :email, :senha)";

  $stmt = DB::prepare($sql);
  $stmt->bindParam(':usuario', $this->usuario);
  $stmt->bindParam(':nomesobrenome', $this->nomesobrenome);
  $stmt->bindParam(':email', $this->email);
  $stmt->bindParam(':senha', $this->senha);

  return $stmt->execute();
		
}

# Esses isset não é necessario, já que as propriedades sempre vão existir
# Você esta esquecendo o  $this-> na validação do campos preenchido
# Você não precisa desse monte de if, else
# Quando quiser encerrar o codigo, use o return

 

12 horas atrás, Rodrigo5468 disse:

$verifica = DB::prepare("SELECT * FROM {$this->table} WHERE usuario=:usuario OR nomesobrenome=:nomesobrenome OR email=:email");

 

Não sei qual é a sua regra de negócio, mas esta certo de que deseja um OR, e não um END?

 

1

Compartilhar este post


Link para o post
Compartilhar em outros sites

Pow cara!!! O código fincou bem simples e funciona perfeitamente.

Você disse "mas esta certo de que deseja um OR, e não um END?" Não entendi muito bem o end, seria and? Estou começando agora, não sei qual é o melhor para usar o or, end ou and. Se poder me dá uma força.

 

echo 'Existe campos vazios.</div>'; # Dar echo dentro da classe não é bom!

Você também escreveu isso no código. Também não acho legal dar echo dentro da classe. Eu queria puxar um echo e colocar as informações na página register.php como posso fazer isso?

Exemplo:

			<?php
				$usuarios = new Usuarios();
				
				if(isset($_POST['cadastrar'])) {
					$usuario					= $_POST['usuario'];
					$nomesobrenome				= $_POST['nomesobrenome'];
					$email						= $_POST['email'];
					$senha						= $_POST['senha'];
					
					$usuarios->setUsuario($usuario);
					$usuarios->setNomesobrenome($nomesobrenome);
					$usuarios->setEmail($email);
					$usuarios->setSenha($senha);
					
					#Insert
					if($usuarios->insert()) {
						echo 'Conta registrada com sucesso! Você deverá confirmar a conta em seu endereço de e-mail.</div>';
					}
                  	#Dar echo aqui falando que existe campos vazios.
                  	#Dar echo aqui falando que algum dado que ele tentou registrar já existe no banco de dados.
				}
			?>

 

 

Obrigado por tudo cara, muito obrigado mesmo.

0

Compartilhar este post


Link para o post
Compartilhar em outros sites

 

25 minutos atrás, Rodrigo5468 disse:

echo 'Existe campos vazios.</div>'; # Dar echo dentro da classe não é bom!

Você também escreveu isso no código. Também não acho legal dar echo dentro da classe.

Você pode gerar exceções...

 

Ex dentro do seu contexto:

<?php
public function insert() 
{
	if(empty($this->usuario) OR empty($this->nomesobrenome) OR empty($this->email) OR empty($this->senha)) {	
		throw new Exception('Existe campos vazios.');
	}
	....

Ai na hora de executar algo +/- assim:

<?php
try {

	$usuario					= $_POST['usuario'];
	$nomesobrenome				= $_POST['nomesobrenome'];
	$email						= $_POST['email'];
	$senha						= $_POST['senha'];
	$usuarios->setUsuario($usuario);
	$usuarios->setNomesobrenome($nomesobrenome);
	$usuarios->setEmail($email);
	$usuarios->setSenha($senha);

	if($usuarios->insert()){
		echo '<div class="success">Conta registrada com sucesso! Você deverá confirmar a conta em seu endereço de e-mail.</div>';
	}
    
} catch (Exception $e) {
	echo '<div class="error">'. $e->getMessage() .'</div>';    
}

 

Ps: foi um exemplo simples, aprofundar é preciso.

 

Leia mais:

http://php.net/manual/pt_BR/language.exceptions.php

 

1

Compartilhar este post


Link para o post
Compartilhar em outros sites

Princípio orientado à objeto denominado Tell, don't ask. BasicamenteVocê deve mandar/dizer para que uma classe/objeto faça algo e não perguntar a ela sobre o que, ou se, ela realizou. Exceto se o nome do método é uma pergunta (get, is, has, can, etc..).

 

Em um método denominado insert espera-se que o método insira um registro. Se tudo ocorrer certo, um registro será inserido. Se um registro não foi inserido, algo que não estava esperado ocorreu. Se inesperado ocorreu, é uma exceção.

1

Compartilhar este post


Link para o post
Compartilhar em outros sites

Postado (editado)

5 horas atrás, Rodrigo5468 disse:

if($usuarios->insert()) { echo 'Conta registrada com sucesso! Você deverá confirmar a conta em seu endereço de e-mail.</div>'; } #Dar echo aqui falando que existe campos vazios. #Dar echo aqui falando que algum dado que ele tentou registrar já existe no banco de dados.


Você esta muito viciado em if, else rsrs (brincadeira)

Melhorando mais um pouco, ficaria assim:

public function insert() 
{
  if(empty($this->usuario) OR empty($this->nomesobrenome) OR empty($this->email) OR empty($this->senha)) {
    return 'Existe campos vazios!';
  }

  $stmt = DB::prepare("
    SELECT * 
    FROM {$this->table} 
    WHERE usuario=:usuario 
    OR nomesobrenome=:nomesobrenome 
    OR email=:email"
  );

  $stmt->bindValue(':usuario',$this->usuario,PDO::PARAM_STR);
  $stmt->bindValue(':nomesobrenome',$this->nomesobrenome,PDO::PARAM_STR);
  $stmt->bindValue(':email',$this->email,PDO::PARAM_STR);
  $stmt->execute();

  $usuarioExiste = $stmt->rowCount();

  if($usuarioExiste) {
    return 'Usuario já existe!';
  }

  $sql = "INSERT INTO $this->table (usuario, nomesobrenome, email, senha) 
  VALUES (:usuario, :nomesobrenome, :email, :senha)";

  $stmt = DB::prepare($sql);
  $stmt->bindParam(':usuario', $this->usuario);
  $stmt->bindParam(':nomesobrenome', $this->nomesobrenome);
  $stmt->bindParam(':email', $this->email);
  $stmt->bindParam(':senha', $this->senha);

  if ($stmt->execute()) {
	return '<div class="success">Conta registrada com sucesso! Você deverá confirmar a conta em seu endereço de e-mail.</div>';
  }

  return 'Falha ao cadastrar!';	
		
}
<?php
        $usuario = $_POST['usuario'];
        $nomesobrenome = $_POST['nomesobrenome'];
        $email = $_POST['email'];
        $senha = $_POST['senha'];
      
        $usuarios->setUsuario($usuario);
        $usuarios->setNomesobrenome($nomesobrenome);
        $usuarios->setEmail($email);
        $usuarios->setSenha($senha);
      
        echo $usuarios->insert();


Note como o uso do return evita o uso de else, e deixa o código mais simples, como da mesma forma em 

echo $usuarios->insert();

Não é necessário o uso de if elseif. Não deixe de ver a ótima dica do  Tell, don't ask que o Gabriel sugeriu

 

5 horas atrás, Rodrigo5468 disse:

Você disse "mas esta certo de que deseja um OR, e não um END?" Não entendi muito bem o end, seria and?


Sim, um and, escrevi errado :)

Por exemplo:
Considere que esse registro existe no banco: joao, joao da silva, joao@joao.com

Se tentar cadastrar joao, joao dos santos, joao@joao.com
Com o OR, sera cadastrado, com o AND não

De uma pesquisa por "logica de programação, operadores logicos"

Editado por EdCesar
Refatoração: Retirado try catch
1

Compartilhar este post


Link para o post
Compartilhar em outros sites
2 horas atrás, EdCesar disse:

Você esta muito viciado em if, else rsrs (brincadeira)

Essa técnica pode se enquadrar como 'early return' 

 

o @EdCesar esqueceu de lançar a exceção...


Não faz sentido ter blocos de try sem uso de 

 

throw new Exception('Oppss.. isso não deveria acontecer.');



Tem alguns artigos para exemplificar o uso do early return independente de usar blocos de try/catch

http://blog.alura.com.br/quanto-mais-simples-melhor/

https://softwareengineering.stackexchange.com/a/18473

 

 

 

 

1

Compartilhar este post


Link para o post
Compartilhar em outros sites
47 minutos atrás, gabrieldarezzo disse:

o @EdCesar esqueceu de lançar a exceção...


@gabrieldarezzo,  bem observado, atualizei e retirei o try cath.
Nesse caso caso dele, acho que dar apenas o return com a mensagem de falha ou sucesso, deixa o código mais simples

Tem outras coisas também que é importante tratar, também como:

$nome = (isset($_POST['usuario'])) ? $_POST['usuario'] : '';

ou no PHP 7:

$usuario = $_POST['usuario'] ?? '';

Foge do tópico, mas fica a dica :)

1

Compartilhar este post


Link para o post
Compartilhar em outros sites

Entendi tudo por enquanto, mas estou com uma pequena dúvida, por favor me esclareça.

@EdCesarpode perceber que você citou esses dois últimos códigos.

"Tem outras coisas também que é importante tratar, também como:

$nome = (isset($_POST['usuario'])) ? $_POST['usuario'] : '';

ou no PHP 7:

$usuario = $_POST['usuario'] ?? '';

"

 

 

Eu tenho meus três "private" que irei utilizar para mostrar o usuário os dados incorretos, e que ele precisa corrigir para cadastrar no sistema.

private

private $usuario;
private $nomesobrenome;
private $email;

Aqui nesses seguintes códigos eu quero especificar os campos, usarei o código acima de exemplo que você deixou? Eu testei aqui mas não obtive resultados, se poder me deixar um exemplo.

if(empty($this->usuario) OR empty($this->nomesobrenome) OR empty($this->email) OR empty($this->senha)) {
	return 'Existe campos vazios!';
}

#####################

if($usuarioExiste) {
	return 'Usuario já existe!';
}

Eu quero especificar se é o usuário, nomesobrenome ou e-mail que está vazio, ou se algum desses dados já existem. Exemplo:

"O campo usuário está vazio, por favor, preencha!"

"O endereço de e-mail que você forneceu já existe em nosso banco de dados, tente usar outro."

 

 

Muito obrigado por tudo mais uma vez.

0

Compartilhar este post


Link para o post
Compartilhar em outros sites

Olá novamente!

 

Fiz aqui para ter uma validação um pouco melhor, mas acho que estou fazendo errado, pois não está funcionando perfeitamente.

Verificando se existe campos em brancos, e quais deles estão em branco.

if(empty($this->usuario) OR empty($this->nomesobrenome) OR empty($this->email) OR empty($this->senha)) {
	if(empty($this->usuario)) {
		return 'Usuário está em branco.';
	}elseif(empty($this->nomesobrenome)) {
		return 'Nome/Sobrenome está em branco.';
	}elseif(empty($this->email)) {
		return 'E-mail está em branco está em branco.';
	}elseif(empty($this->senha)) {
		return 'Senha está em branco.';
	}else{
		return 'Usuário, Nome/Sobrenome, E-mail, Senha, estão em branco.';
	}
	
	return 'Preencha todos os campos em branco.';
}

Verificando se existe algum campo registrado no banco de dados, e quais deles são.

public function insert() {
	if($this->usuario) {
		return 'Campo Usuário já está cadastrado em nosso banco de dados.';
	}elseif($this->nomesobrenome) {
		return 'Campo Nome/Sobrenome  já está cadastrado em nosso banco de dados.';
	}elseif($this->email) {
		return 'Campo E-mail já está cadastrado em nosso banco de dados.';
	}else{
		return 'Algum desses campos já estão cadastrado em nosso banco de dados: Usuário, Nome/Sobrenome ou E-mail.';
	}
	
	return 'Algum dado que você tentou inserir já consta em nosso banco de dados.';
}

 

 

Quando chega para verificar se a senha está em branco ou não, ele prossegue e cadastra o usuário com senha nula. E a mensagem de erro para verificar se existe algum dado já em nosso banco de dados, ele sempre mostra a opção Usuário. Por favor, como faço para arrumar este pequeno detalhe?

 

 

Obrigado mais uma vez.

0

Compartilhar este post


Link para o post
Compartilhar em outros sites

Seu código que está com a complexidade ciclomática muito alta. Use early returns para reduzí-la.

 

Quanto a validação, tudo que não permitir a persistência no SGBD é uma exceção, ou seja:

if($this->usuario) {
    throw new RuntimeException('Campo Usuário já está cadastrado em nosso banco de dados.');
}

if($this->nomesobrenome) {
    throw new RuntimeException('Campo Nome/Sobrenome  já está cadastrado em nosso banco de dados.');
}

/** demais validações **/

 

1

Compartilhar este post


Link para o post
Compartilhar em outros sites

Bom, tentei fazer do seu método mas já erro.

4NAYqUM.png

E como eu posso diminuir ou até tirar a complexidade ciclamática?

 

 

Desde já, obrigado mais uma vez.

0

Compartilhar este post


Link para o post
Compartilhar em outros sites

o

1 hora atrás, Rodrigo5468 disse:

E como eu posso diminuir ou até tirar a complexidade ciclamática?

 

O me chara disse no post ué:

5 horas atrás, Gabriel Heming disse:

Use early returns para reduzí-la.

 

 

Chegou a abrir os links abaixo?

Em 2017-5-17 at 17:29, gabrieldarezzo disse:

Tem alguns artigos para exemplificar o uso do early return independente de usar blocos de try/catch

http://blog.alura.com.br/quanto-mais-simples-melhor/

https://softwareengineering.stackexchange.com/a/18473

1

Compartilhar este post


Link para o post
Compartilhar em outros sites

Sim, eu já fiz os try/catch e apliquei os exemplos dos links.

Verificar se existe campos em brancos, e quais são

if(empty($this->usuario) OR empty($this->nomesobrenome) OR empty($this->email) OR empty($this->senha)) {
	if(empty($this->usuario))
		return 'Campo Usuário em branco.';
	
	if(empty($this->nomesobrenome))
		return 'Campo Nome/Sobrenome em branco.';
	
	if(empty($this->email))
		return 'Campo E-mail em branco';
	
	if(empty($this->senha))
		return 'Campo Senha em branco.';
}

Verificar quais campos estão cadastrado no banco de dados, e quais são

if($usuarioExiste) {
	if($this->usuario)
		return 'Usuario já existe!';
	
	if($this->nomesobrenome)
		return 'Nome/Sobrenome já existe!';
	
	if($this->email)
		return 'E-mail já existe!';
}

 

Para verificar se os campos em branco, ele funciona perfeitamente, mas quando deixa o campo senha em branco ele não apresenta o erro. E o usuário sempre, sempre mostra o primeiro return que é "Usuário já existe". Estou pesquisando em diversos fóruns, artigos, youtube para conseguir arrumar mas não consigo. Se poder me explicar um pouco mais eu agradeço de coração.

0

Compartilhar este post


Link para o post
Compartilhar em outros sites

@Rodrigo5468, legal que você esta se esforçando! Melhorando um pouco mais o seu código, na questão das validações, poderia ficar assim:

 

Para a questão dos campos inválidos

Troque:

if(empty($this->usuario) OR empty($this->nomesobrenome) OR empty($this->email) OR empty($this->senha)) {
	if(empty($this->usuario))
		return 'Campo Usuário em branco.';
	
	if(empty($this->nomesobrenome))
		return 'Campo Nome/Sobrenome em branco.';
	
	if(empty($this->email))
		return 'Campo E-mail em branco';
	
	if(empty($this->senha))
		return 'Campo Senha em branco.';
}

Por isso:

$propriedades = get_class_vars(get_class($this)); 

if (!filter_var($this->email, FILTER_VALIDATE_EMAIL)) {
  throw new \Exception('E-mail invalido!');
}

foreach ($propriedades as $propriedade => $valor) {
  if (empty($this->$propriedade)) {
    throw new \Exception('o campo '. $propriedade . ' não foi preenchido!');
  }
}


Se na sua classe você tem mais propriedades do que usuario, nomesobrenome,email e senha, então troque

$propriedades = get_class_vars(get_class($this)); 

do código acima, por:
 

$propriedades = [
  'usuario' => $this->usuario, 
  'nomesobrenome' => $this->nomesobrenome, 
  'email' => $this->email, 
  'senha' => $this->senha
];

 

Para a validação dos dados existentes
 

Troque esse trecho:

$usuarioExiste = $stmt->rowCount();

  if($usuarioExiste) {
    return 'Usuario já existe!';
  }

Por esse:

$resultado = $stmt->fetch(\PDO::FETCH_ASSOC);

$dadosJaExistem = array_intersect([
  'usuario' => $this->usuario, 
  'nomesobrenome' => $this->nomesobrenome, 
  'email' => $this->email], 
  $resultado
);

if ($dadosJaExistem) {
   $mensagem = 'O(s) valor(es): ' . implode(',', $dadosJaExistem)  . ' ja existem no banco de dados, por favor escolha outro(s)';
   throw new Exception($mensagem);
}

 

2

Compartilhar este post


Link para o post
Compartilhar em outros sites
Em 2017-5-17 at 18:16, EdCesar disse:
Em 2017-5-17 at 17:29, gabrieldarezzo disse:

o @EdCesar esqueceu de lançar a exceção...


@gabrieldarezzo,  bem observado, atualizei e retirei o try cath.
Nesse caso caso dele, acho que dar apenas o return com a mensagem de falha ou sucesso, deixa o código mais simples

Tem outras coisas também que é importante tratar, também como:


$nome = (isset($_POST['usuario'])) ? $_POST['usuario'] : '';

ou no PHP 7:


$usuario = $_POST['usuario'] ?? '';

Foge do tópico, mas fica a dica :)

 

Gostei.  Eu vou estudar isso também.  Bom domingo a todos.

1

Compartilhar este post


Link para o post
Compartilhar em outros sites

Tive alguns problemas pessoais e não conseguir acessar o fórum.

 

Campos em brancos:

uQkghDO.png

Usando o seguinte código dá esse erro.

$propriedades = [
	'usuario' => $this->usuario, 
	'nomesobrenome' => $this->nomesobrenome, 
	'email' => $this->email, 
	'senha' => $this->senha
];
			
if(!filter_var($this->email, FILTER_VALIDATE_EMAIL)) {
	throw new \Exception('E-mail invalido!');
}
			
foreach($propriedades as $propriedade => $valor) {
	if(empty($this->$propriedade)) {
		throw new \Exception('o campo '. $propriedade . ' não foi preenchido!');
	}
}

Mas se eu trocar o trecho "throw new \Exception" por "return" o código funciona. Mas se eu preencher todos os campos, e deixar a senha em branco ele insere um cadastro no banco de dados com a senha em branco. Não dá erro nenhum.

 

 

Já a baste para identificar quais os campos existem já cadastros no banco de dados sempre da um erro que o e-mail está invalido. Já tentei substituir o "throw new Exception" pelo "return" e colocar uma barra invertida no Exception mas nada.

 

Já pesquisei em diversões fóruns, vídeos mas nada não tô conseguindo consertar o código. Poderia me explicar como eu posso conserta-lo?

 

 

Desde já obrigado.

0

Compartilhar este post


Link para o post
Compartilhar em outros sites

Esta certo, isso é um erro esperado, que só vai ocorrer quando você enviar um e-mail invalido, provavelmente você testou com um e-mail que não contem um formato valido, certo? Para parar de exibir essa tela laranja, e exibir apenas a mensagem, você precisa colocar um try/catch no  $usuarios->insert()

Pare um pouco para entender melhor como funciona os tratamentos de erros, validações, calculo de interseção entre outras coisas que foram colocadas aqui,  perca umas duas horinhas e ganhe dezenas! Tentar "aprender na força" não é um bom caminho, já fiz muito isso, não faço mais :)

Feito isso, estamos aqui para auxiliar. Não deixa de ver:

http://php.net/manual/pt_BR/function.array-intersect.php
http://php.net/manual/en/filter.filters.validate.php
http://php.net/manual/pt_BR/language.exceptions.php

1

Compartilhar este post


Link para o post
Compartilhar em outros sites

Desculpe a demora, estava sem internet.

@EdCesar eu li todos os links e mais alguns por fora e, cheguei a conclusão que o código está correto (eu acho).

 

uQkghDO.png

Esse erro está falando que o e-mail é invalido, está certo. Mas esse código ele ta em

classes/

 

E tenho o index que deveria aparecer esse erro como um diálogo (pop-up), informando que algum desses dados são inválidos ou já existem no banco de dados.

 

Eu preciso ignorar esse erro, para ele não redirecionar para essa tela e colocar em uma div, como um diálogo. Preciso puxar o erro para o index. Como eu posso fazer isso? Já pesquisei, tentei usar um try/catch e puxar a variável $erro para o index, assim mostrando o erro diagnosticado no arquivo Usuários.php da pasta classes/, mas não conseguir, poderia me dá uma força? Desde já muito obrigado.

0

Compartilhar este post


Link para o post
Compartilhar em outros sites
58 minutos atrás, Rodrigo5468 disse:

[...], tentei usar um try/catch e puxar a variável $erro para o index, assim mostrando o erro diagnosticado no arquivo Usuários.php da pasta classes/, mas não conseguir[...]

 

Mostre-nos como fez e não deu certo.

1

Compartilhar este post


Link para o post
Compartilhar em outros sites

Olá!

 

Estrutura do Projeto.

4FvM6lI.png

 

Usuarios.php

		public function insert() {
			try {
				$propriedades = [
					'usuario' => $this->usuario, 
					'nomesobrenome' => $this->nomesobrenome, 
					'email' => $this->email, 
					'senha' => $this->senha
				];
				
				if(!filter_var($this->email, FILTER_VALIDATE_EMAIL)) {
					throw new \Exception('E-mail invalido!');
				}
				
				foreach($propriedades as $propriedade => $valor) {
					if(empty($this->$propriedade)) {
						throw new \Exception('o campo '. $propriedade . ' não foi preenchido!');
					}
				}
			}catch(Exception $e) {
				$error_message = $e->getMessage();
			}
			
			$stmt = DB::prepare("SELECT * FROM {$this->table} WHERE usuario=:usuario OR nomesobrenome=:nomesobrenome OR email=:email");
			$stmt->bindValue(':usuario',$this->usuario,PDO::PARAM_STR);
			$stmt->bindValue(':nomesobrenome',$this->nomesobrenome,PDO::PARAM_STR);
			$stmt->bindValue(':email',$this->email,PDO::PARAM_STR);
			$stmt->execute();
			
			$resultado = $stmt->fetch(\PDO::FETCH_ASSOC);
			
			$dadosJaExistem = array_intersect([
				'usuario' => $this->usuario, 
				'nomesobrenome' => $this->nomesobrenome, 
				'email' => $this->email], 
				$resultado
			);
			
			if($dadosJaExistem) {
				$mensagem = 'O(s) valor(es): ' . implode(',', $dadosJaExistem)  . ' ja existem no banco de dados, por favor escolha outro(s)';
				throw new Exception($mensagem);
			}
			
			$sql = "INSERT INTO $this->table (usuario, nomesobrenome, email, senha) VALUES (:usuario, :nomesobrenome, :email, :senha)";
			$stmt = DB::prepare($sql);
			$stmt->bindParam(':usuario', $this->usuario);
			$stmt->bindParam(':nomesobrenome', $this->nomesobrenome);
			$stmt->bindParam(':email', $this->email);
			$stmt->bindParam(':senha', $this->senha);
			
			if ($stmt->execute()) {
				return 'Conta registrada com sucesso! Você deverá confirmar a conta em seu endereço de e-mail.';
			}
			
			return 'Falha ao cadastrar!';
		}

 

index.php

			<?php
				$usuarios = new Usuarios();
				
				if(isset($_POST['cadastrar'])) {
					$usuario = $_POST['usuario'];
					$nomesobrenome = $_POST['nomesobrenome'];
					$email = $_POST['email'];
					$senha = $_POST['senha'];
					
					$usuarios->setUsuario($usuario);
					$usuarios->setNomesobrenome($nomesobrenome);
					$usuarios->setEmail($email);
					$usuarios->setSenha($senha);
					
					echo $usuarios->insert();
					
					if(isset($error_message)) {
						echo $error_message;
					}
				}
			?>

 

 

Já tentei de tudo e ainda estou tentando. Obrigado mais uma vez.

0

Compartilhar este post


Link para o post
Compartilhar em outros sites

Acredito que toda Exception deve estar isolada dentro de um bloco try{}.

 

 

Errado:

throw new Exception('Opss.. aqui tem coisa errada');

Certo:

try {
	throw new Exception('Opss.. aqui tem coisa errada');
}
 catch(Exception $ex){
	$ex->getMessage();
	die();
}

 

 

Uma opção para cobrir 100% do seu try/exception:

 

<?php

public function insert() {
	
	$propriedades = [
		'usuario' => $this->usuario, 
		'nomesobrenome' => $this->nomesobrenome, 
		'email' => $this->email, 
		'senha' => $this->senha
	];
	
	if(!filter_var($this->email, FILTER_VALIDATE_EMAIL)) {
		throw new \Exception('E-mail invalido!');
	}
	
	foreach($propriedades as $propriedade => $valor) {
		if(empty($this->$propriedade)) {
			throw new \Exception('o campo '. $propriedade . ' não foi preenchido!');
		}
	}
	

	$stmt = DB::prepare("SELECT * FROM {$this->table} WHERE usuario=:usuario OR nomesobrenome=:nomesobrenome OR email=:email");
	$stmt->bindValue(':usuario',$this->usuario,PDO::PARAM_STR);
	$stmt->bindValue(':nomesobrenome',$this->nomesobrenome,PDO::PARAM_STR);
	$stmt->bindValue(':email',$this->email,PDO::PARAM_STR);
	$stmt->execute();

	$resultado = $stmt->fetch(\PDO::FETCH_ASSOC);

	$dadosJaExistem = array_intersect([
		'usuario' => $this->usuario, 
		'nomesobrenome' => $this->nomesobrenome, 
		'email' => $this->email], 
		$resultado
	);

	if($dadosJaExistem) {
		$mensagem = 'O(s) valor(es): ' . implode(',', $dadosJaExistem)  . ' ja existem no banco de dados, por favor escolha outro(s)';
		throw new Exception($mensagem);
	}

	$sql = "INSERT INTO $this->table (usuario, nomesobrenome, email, senha) VALUES (:usuario, :nomesobrenome, :email, :senha)";
	$stmt = DB::prepare($sql);
	$stmt->bindParam(':usuario', $this->usuario);
	$stmt->bindParam(':nomesobrenome', $this->nomesobrenome);
	$stmt->bindParam(':email', $this->email);
	$stmt->bindParam(':senha', $this->senha);

	if ($stmt->execute()) {
		return 'Conta registrada com sucesso! Você deverá confirmar a conta em seu endereço de e-mail.';
	}

	return 'Falha ao cadastrar!';
}



$usuarios = new Usuarios();

if(isset($_POST['cadastrar'])) {
	
	
	$usuario = $_POST['usuario'];
	$nomesobrenome = $_POST['nomesobrenome'];
	$email = $_POST['email'];
	$senha = $_POST['senha'];
	
	$usuarios->setUsuario($usuario);
	$usuarios->setNomesobrenome($nomesobrenome);
	$usuarios->setEmail($email);
	$usuarios->setSenha($senha);
	
	try {
		echo $usuarios->insert();	
	} catch(Exception $ex){
		$ex->getMessage();
		die();
	}
		
		
	
	if(isset($error_message)) {
		echo $error_message;
	}
}

Outro coisa que você pode fazer é tentar isolar o problema.....

 

Ex: está com erro de E-mail?

 

Verificou oq está chegando no parametro ?

-> var_dump($email);die();

 

Fez um teste individual?

Algo simples assim:


$emails = array(
	 'gabrieldarezzo@yahoo.com.br'
	,'darezzo.gabriel@gmail.com'
	,'EmailNaovalido.com'
	,'quasevalido@'
);


foreach($emails as $email){
	if(!filter_var($email, FILTER_VALIDATE_EMAIL)) {
		echo "Email: {$email} NOT VALID";
	} else {
		echo "Email: {$email} IS VALID";
	}
	
	echo "\n <br />";
}

 

Enfim vai isolando o problema sempre.

1

Compartilhar este post


Link para o post
Compartilhar em outros sites

Crie uma conta ou entre para comentar

Você precisar ser um membro para fazer um comentário

Criar uma conta

Crie uma nova conta em nossa comunidade. É fácil!


Crie uma nova conta

Entrar

Já tem uma conta? Faça o login.


Entrar Agora

  • Próximos Eventos

  • Conteúdo Similar

    • Por fogazza
      Boa noite a todos.
      Queria tirar uma dúvida.
      Possui um formulário php de cadastro de clientes, e outro formulário onde eu cadastro os empréstimos destes clientes.
      Minha pergunta é:
      Existe alguma forma de eu ao digitar o cpf, ele preencher automaticamente o campo nome do formulário empréstimos de acordo com o nome que está cadastrado na tabela clientes?
    • Por lucasvbl
      Estou usando o foreach e ele aparece esse erro, estou usando PHP PDO
       
       
      <? // Esse é o formulario ?> <form action="uploads.php" method="post" enctype="multipart/form-data" > <input type="file" name="pictures[]" multiple /> <? // Aqui seria a estrutura de repetição que usaria, mas ele apresenta aquele erro foreach ($_FILES["pictures"]["error"] as $key => $error) { if ($error == UPLOAD_ERR_OK) { $tmp_name = $_FILES["pictures"]["tmp_name"][$key]; $name = $_FILES["pictures"]["name"][$key]; $foto_nome=time()."_".$name; move_uploaded_file($tmp_name, "contrato_anexos/$foto_nome"); $insere = $_base->prepare("INSERT INTO contrato_anexos SET contrato='$codigo', anexo= '$foto_nome' "); $insere->execute(); } } ?>  
    • Por thbomfim
      Voltei a programar a pouco tempo e estou com um probleminha, peguei um Script de comunidade Wap bem simples e básico para atualizar ele é assim relembrar mas o problema é que eu fiz a conexão com pdo e está me dando este erro :
      Fatal error: Call to a member function query() on null
       
      O erro vem dessa linha:
      $norm =$db->query("SELECT COUNT(*) FROM fun_users")->fetch(); eu já botei global $db; já coloquei a variável de conexão no arquivo mas não muda nada e este erro da em qualquer arquivo do Script, alguém sabe como resolver este problema ?
    • Por Blacksnake
      Olá,
       
      Sou novo na linguagem c# e estou a estudar o novo ef core.
       
      Quando comecei instalei o ef core 1.0.1 agora fiz update para 2.0.0 bem como do vs2015 para o vs2017 comunity.
      Após este update surge-me o seguinte erro:
      System.IO.FileNotFoundException: 'Não foi possível carregar o ficheiro ou assemblagem 'MySqlConnector, Version=0.25.1.0, Culture=neutral, PublicKeyToken=null' ou uma das respectivas dependências. O sistema não conseguiu localizar o ficheiro especificado.'
       
      O meu código é:
      using ERP.Models;
      using ERP.Models.Options;
      using Microsoft.EntityFrameworkCore.Diagnostics;
      using System;
      using System.Collections.Generic;
      using System.Linq;
      using System.ComponentModel.DataAnnotations;
      using System.ComponentModel.DataAnnotations.Schema;
      using Microsoft.EntityFrameworkCore;
       
      namespace ERP.Datasource
      {
          public class Mycontext: DbContext
          {
              protected override void OnConfiguring(DbContextOptionsBuilder optionsBuilder)
                  => optionsBuilder
                      .UseMySql(@"Server=localhost;database=ef;uid=root;pwd=******;")
                      .ConfigureWarnings(warnings => warnings.Throw(RelationalEventId.QueryClientEvaluationWarning));
       
       
              public DbSet<User> User { get; set; }
              public DbSet<Entity> Entities { get; set; }
              public DbSet<Infoapp> Infoapp { get; set; }
              public DbSet<CountryCoin> CountryCoin { get; set; }
       
          }
      }
    • Por sabrina.teles
      create database copa20143; use copa20143;   create table grupo(
      id_grupo varchar(45) primary key,
      grupo character,
      selecao varchar(150),
      timeVencedor varchar(45),
      timePerdedor varchar(45),
      Empate varchar(45));   create table jogo(
      id_jogo int primary key,
      dataJogo date,
      horario time,
      placar varchar(45),
      locall varchar(45));   alter table jogo
      add grupo_id varchar(45);   alter table jogo
      add foreign key(grupo_id) references grupo(id_grupo);   insert into jogo (id_jogo,dataJogo,horario,placar,locall, grupo_id)
      values(01,"2014/06/12","17:00",'3x1', 'São Paulo- Arena Corinthians ','A');   0       Error Code: 1452. Cannot add or update a child row: a foreign key constraint fails 0.094 sec