From 131701fbf34261b2c758bd00379274ebd42ee56b Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Jos=C3=A9=20Henrique=20Honjoya?= <ti.honjoya@gmail.com>
Date: Thu, 28 Feb 2019 18:31:48 -0300
Subject: [PATCH] =?UTF-8?q?Corrigindo=20problema=20e=20coment=C3=A1rios?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Erro citado

Severity: Notice
Message: Undefined variable: count
Filename: controllers/Mypet.php
Line Number: 98
Backtrace:
File: /home/u363333504/public_html/application/controllers/Mypet.php
Line: 98
Function: _error_handler
File: /home/u363333504/public_html/index.php
Line: 315
Function: require_once
---
 application/controllers/Mypet.php | 13 ++++++++++++-
 1 file changed, 12 insertions(+), 1 deletion(-)

diff --git a/application/controllers/Mypet.php b/application/controllers/Mypet.php
index dd16ae3..1b7765f 100644
--- a/application/controllers/Mypet.php
+++ b/application/controllers/Mypet.php
@@ -89,12 +89,22 @@ public function cadastrar(){
 			//Verificar se já existe e-mail cadastrado
 			$clientes = $this->Generic_model->readAll($table);
 
-			$cont = 0;
+			// Realizei a coreção do nome da variável
+			$count = 0;
+			
+			// Aqui você terá um problema de performace, pois imagine uma listagem de milhares de clientes, você perdera performace com o trafego de rede e lentidão na intereção de um por um
+			// Seria melhor e mais performatico a seguinte queries:
+			// SELECT count(1) from clientes where email = LOWER('email@docliente.com');
+			
 			foreach ($clientes as $cliente) {
 				if($cliente['email'] == $data['email'])
 					$count++;
 			}
+			
+			
 			//Caso já exista email cadastrado
+			// Nuca adicione tag HTML no controller, isso é responsábilidade da View
+			
 			if( $count > 0 ){
 				$this->session->set_flashdata('msg', 
 							'<span style="color: red; font-weight: 600">Este e-mail já foi cadastrado</span>');	
@@ -123,6 +133,7 @@ public function cadastrar(){
 
 		//Método para cadastrar um novo agendamento
 		public function agendar(){
+			// Nunca deixe qualquer tipo de saida no controller, primeiro seria uma falha de segurança, segundo poderia causar algum problema de session, em códigos legados
 			var_dump($this->input->post());
 			$data = $this->input->post();