Verificar se um registro já existe

[luanda 0]

Fiz um sistema bem simples de cadastro, onde o usuário digita as informações e o código envia esses dados para o banco. Acontece que esse código não impede que várias pessoas cadastrem-se com o mesmo username e e-mail, mesmo eu tendo declarado unique para estes na tabela.

<?php
    include "conexao.php";
?>

<?php
    
    // captura dos dados digitados no formulário //
	$nome=$_POST ['nome'];
    $sobrenome=$_POST ['sobrenome'];
	$username=$_POST ['username'];
	$email=$_POST ['email'];
    $senha=$_POST ['senha'];
    $telefone=$_POST ['telefone'];
	
    $sql = mysql_query("INSERT INTO cadastro (nome,sobrenome,username,email,senha,telefone)  VALUES ('$nome', '$sobrenome','$username','$email','$senha','$telefone')") or die(mysql_error());
    
    $resultado = mysql_query ($sql);
   header ("Location:../usuario/index.php");
?>

 

[Matheus Tavares 167]

Olá, seja bem-vindo.

 

Independente do seu código PHP, se as chaves estão definidas corretamente você não deveria conseguir cadastrar registros duplicados, mas o mais correto seria você fazer essa validação também na aplicação antes de inserir o registro, pois assim você consegue avisar o usuário sobre a duplicidade dos dados.

 

Em outras palavras, você deveria antes fazer um SELECT com WHERE email = 'email@email.com', por exemplo. Apenas após um retorno vazio desse select você executa o INSERT.

 

Exporte sua tabela e poste aqui para vermos como suas chaves UNIQUE estão definidas.

[luanda 0]

8 horas atrás, Matheus Tavares disse:

Olá, seja bem-vindo.

 

Independente do seu código PHP, se as chaves estão definidas corretamente você não deveria conseguir cadastrar registros duplicados, mas o mais correto seria você fazer essa validação também na aplicação antes de inserir o registro, pois assim você consegue avisar o usuário sobre a duplicidade dos dados.

 

Em outras palavras, você deveria antes fazer um SELECT com WHERE email = 'email@email.com', por exemplo. Apenas após um retorno vazio desse select você executa o INSERT.

 

Exporte sua tabela e poste aqui para vermos como suas chaves UNIQUE estão definidas.

Matheus, obrigada pela ajuda. Consegui achar um código que consegue avisar sobre a duplicata para o usuário, mas no caso ele só funciona com o username (tentei usando or e-mail no select mas não foi, com certeza estou fazendo errado). 

-- phpMyAdmin SQL Dump
-- version 4.1.4
-- http://www.phpmyadmin.net
--
-- Host: 127.0.0.1
-- Generation Time: 02-Nov-2018 às 13:20
-- Versão do servidor: 5.6.15-log
-- PHP Version: 5.4.24

SET SQL_MODE = "NO_AUTO_VALUE_ON_ZERO";
SET time_zone = "+00:00";


/*!40101 SET @OLD_CHARACTER_SET_CLIENT=@@CHARACTER_SET_CLIENT */;
/*!40101 SET @OLD_CHARACTER_SET_RESULTS=@@CHARACTER_SET_RESULTS */;
/*!40101 SET @OLD_COLLATION_CONNECTION=@@COLLATION_CONNECTION */;
/*!40101 SET NAMES utf8 */;

--
-- Database: `mao_na_massa`
--

-- --------------------------------------------------------

--
-- Estrutura da tabela `cadastro`
--

CREATE TABLE IF NOT EXISTS `cadastro` (
  `id` int(11) NOT NULL AUTO_INCREMENT,
  `nome` varchar(50) COLLATE utf8_unicode_ci NOT NULL,
  `sobrenome` varchar(50) COLLATE utf8_unicode_ci NOT NULL,
  `username` varchar(50) COLLATE utf8_unicode_ci NOT NULL,
  `email` varchar(50) COLLATE utf8_unicode_ci NOT NULL,
  `senha` varchar(50) COLLATE utf8_unicode_ci NOT NULL,
  `telefone` varchar(50) COLLATE utf8_unicode_ci NOT NULL,
  `nivel` varchar(20) COLLATE utf8_unicode_ci NOT NULL DEFAULT 'usuario',
  PRIMARY KEY (`id`),
  UNIQUE KEY `username` (`username`,`email`)
) ENGINE=MyISAM  DEFAULT CHARSET=utf8 COLLATE=utf8_unicode_ci AUTO_INCREMENT=12 ;

--
-- Extraindo dados da tabela `cadastro`
--

INSERT INTO `cadastro` (`id`, `nome`, `sobrenome`, `username`, `email`, `senha`, `telefone`, `nivel`) VALUES
(3, 'luanda', 'oliveira', 'estrabso', 'kkk@gmail.com', '123', '22221111', 'admin'),
(9, 'luanda', 'anne', 'kiminseok', 'akakka@jakak', '111111', '(22) 22222-2222', 'usuario'),
(8, 'luann', 'ssss', 'akka', 'akakka@jakak', '123', '22222222222', 'usuario'),
(7, 'poxa', 'poxa', 'poxa', 'poxa@poxa.poxa', 'poxa', '(99) 9999-9999', 'usuario');

/*!40101 SET CHARACTER_SET_CLIENT=@OLD_CHARACTER_SET_CLIENT */;
/*!40101 SET CHARACTER_SET_RESULTS=@OLD_CHARACTER_SET_RESULTS */;
/*!40101 SET COLLATION_CONNECTION=@OLD_COLLATION_CONNECTION */;

 

[Matheus Tavares 167]

Bem como imaginei... a definição das suas chaves UNIQUE estão incorretas.

Você está definindo apenas uma chave dupla, dessa forma:

UNIQUE KEY `username` (`username`,`email`)

Com isso você está dizendo ao banco que você não deseja registros com o mesmo username e e-mail simultaneamente.

Para funcionar da forma que você quer, você precisa definir duas chaves UNIQUE separadas, uma para o username e outra para o e-mail.

 

Assim:

CREATE TABLE `cadastro` (
  `id` int(11) NOT NULL AUTO_INCREMENT,
  `nome` varchar(50) COLLATE utf8_unicode_ci NOT NULL,
  `sobrenome` varchar(50) COLLATE utf8_unicode_ci NOT NULL,
  `username` varchar(50) COLLATE utf8_unicode_ci NOT NULL,
  `email` varchar(50) COLLATE utf8_unicode_ci NOT NULL,
  `senha` varchar(50) COLLATE utf8_unicode_ci NOT NULL,
  `telefone` varchar(50) COLLATE utf8_unicode_ci NOT NULL,
  `nivel` varchar(20) COLLATE utf8_unicode_ci NOT NULL DEFAULT 'usuario',
  PRIMARY KEY (`id`),
  UNIQUE KEY `username` (`username`),
  UNIQUE KEY `email` (`email`)
) ENGINE=MyISAM AUTO_INCREMENT=12 DEFAULT CHARSET=utf8 COLLATE=utf8_unicode_ci;

 

[Omar~ 87]

SELECT username, email FROM cadastro WHERE username = 'joao' OR email = 'mail@mail.com'

Ou seja se existir um username joao não cadastra, mas se houver também um mail mail@mail.com;  Mesmo que não seja no mesmo registro onde está joão também não cadastra.

<?php
$conn = new mysqli('endereco_do_banco', 'login', 'senha', 'nome_do_banco_de_dados');
if ($conn->connect_errno) {
    die("Não conectado por causa disso: %s\n", $conn->connect_error);
} else {
    $sql = "SELECT username, email FROM cadastro WHERE username = '{$username}' OR email = '{$email}'";
    $result = $conn->query($sql);
    if ($result->num_rows) {
        echo ("Não pode cadastrar porque existe um username ou email cadastrado");
    } else {
       //// ---- Aqui entra o código do cadastro :)
    }
}
mysqli_close($conn);

 

Observações.

* Detesto esse treco de mysqli, por isso não testei o código acima, mas parece está correto.

* Esqueça disso: mysql se for usar use mysqli, melhor use PDO.

* Seu código ou o que mostrei não possui nenhum tipo de segurança, pois apenas pega um valor e passa para execução, ou seja nada impede que por exemplo no username venha um código malicioso. Então filtre os dados antes de enviar-los para o banco.

 

Aqui um pequeno guia para começar a manipular o banco de dados de uma forma coerente: https://forum.imasters.com.br/topic/565800-crud-introdução-a-pdo/

 

[luanda 0]

2 horas atrás, Omar~ disse:

SELECT username, email FROM cadastro WHERE username = 'joao' OR email = 'mail@mail.com'

Ou seja se existir um username joao não cadastra, mas se houver também um mail mail@mail.com;  Mesmo que não seja no mesmo registro onde está joão também não cadastra.

<?php
$conn = new mysqli('endereco_do_banco', 'login', 'senha', 'nome_do_banco_de_dados');
if ($conn->connect_errno) {
    die("Não conectado por causa disso: %s\n", $conn->connect_error);
} else {
    $sql = "SELECT username, email FROM cadastro WHERE username = '{$username}' OR email = '{$email}'";
    $result = $conn->query($sql);
    if ($result->num_rows) {
        echo ("Não pode cadastrar porque existe um username ou email cadastrado");
    } else {
       //// ---- Aqui entra o código do cadastro :)
    }
}
mysqli_close($conn);

 

Observações.

* Detesto esse treco de mysqli, por isso não testei o código acima, mas parece está correto.

* Esqueça disso: mysql se for usar use mysqli, melhor use PDO.

* Seu código ou o que mostrei não possui nenhum tipo de segurança, pois apenas pega um valor e passa para execução, ou seja nada impede que por exemplo no username venha um código malicioso. Então filtre os dados antes de enviar-los para o banco.

 

Aqui um pequeno guia para começar a manipular o banco de dados de uma forma coerente: https://forum.imasters.com.br/topic/565800-crud-introdução-a-pdo/

 

 

Omar, acabei dando um jeito aqui que consegui fazer com que ele não aceitasse o registro duplicado. O código ficou bem maior (por burrice minha mesmo), mas funcionou.

 

<?php
    
    // captura dos dados digitados no formulário //
	$nome=$_POST ['nome'];
    $sobrenome=$_POST ['sobrenome'];
	$username=$_POST ['username'];
	$email=$_POST ['email'];
    $senha=$_POST ['senha'];
    $telefone=$_POST ['telefone'];

    $verifica_user = mysql_query("SELECT * FROM cadastro WHERE  username='$username' LIMIT 1") or die(mysql_error());
    $verifica_user = mysql_num_rows($verifica_user);

    $verifica_email = mysql_query("SELECT * FROM cadastro WHERE  email='$email' LIMIT 1") or die(mysql_error());
    $verifica_email = mysql_num_rows($verifica_email);

    if($verifica_user > 0){echo "Este user já está sendo utilizado!" ;
    }elseif($verifica_email > 0){echo "Esse e-mail já está sendo utilizado!";}
    else{
	
    $sql = mysql_query("INSERT INTO cadastro (nome,sobrenome,username,email,senha,telefone)  VALUES ('$nome', '$sobrenome','$username','$email','$senha','$telefone')") or die(mysql_error());
    
    $resultado = mysql_query ($sql);
   header ("Location:../usuario/index.php");
    }
?>

E não estou tão preocupada com a segurança dos dados porque isso é pra um trabalho do colégio, nada profissional. Além disso, sobre o PDO, minha professora só nos ensinou com o uso do mysql, então não sei usá-lo. Mas obrigada pela ajuda!!