mysql Verificando dados no banco

[Helison Santos 0]

Olá pessoal tudo bem? Estou necessitando de uma força com um código aqui, vou tentar ser o mais claro possível. É o seguinte, tem esta tabela em meu banco de dados:

DROP TABLE IF EXISTS `pipeline_documentos`;
CREATE TABLE `pipeline_documentos` (
  `documento_id` int(11) NOT NULL auto_increment,
  `documento_nome` varchar(255) default NULL,
  `documento_nome_original` varchar(255) default NULL,
  `documento_type` varchar(255) default NULL,
  `documento_tamanho` varchar(255) default NULL,
  `documento_especie` varchar(255) default NULL,
  `documento_label` varchar(255) default NULL,
  `documento_data` varchar(255) default NULL,
  `user_id` int(11) default NULL,
  `segurado_id` int(11) default NULL,
  `negocio_id` int(11) default NULL,
  `objeto_id` int(11) default NULL,
  PRIMARY KEY  (`documento_id`)
) ENGINE=InnoDB DEFAULT CHARSET=latin1;

A função dela será armazenar documentos enviados no sistema, o que vai separa um documento do outro são as colunas "documento_especie" e "documento_especie", e seus ID relacionados (segurado_id,negocio_id,objeto_id), em media serão de 5 a 7 documentos enviados, variam por causa da CNH e/ou CPF e RG que podem ou não serem enviados juntos ou apenas um deles.

Então o que preciso é verificar se um documento já foi enviado ou não e retornar um erro para não deixar o usuário avançar caso algum documento não exista no sistema. Comecei a rascunha alguma coisa, porém não senti muita segurança nisso:

function verifica_documentos($segurado_id,$negocio_id,$objeto_id,$retonar=false){

	#Documenro Objeto
	#Declaração de Ofereta
	#Proposta Assinada
	#CNH
	#CPF e RG

	#$documentos = array('Documenro Objeto','Declaração de Ofereta','Proposta Assinada','CNH','CPF','RG');

	$erro = Array();
	$erros = '0';

	$sel_doc_1 = $this->seleciona("SELECT * FROM pipeline_documentos WHERE negocio_id='".$negocio_id."' AND documento_label='negocio'");
	$contar_negocio = mysql_num_rows($sel_doc_1);

	if($contar_negocio < '2'){
		$erro[] = 'Declaração de Ofereta e/ou Proposta Assinada';
		$erros++;
	}

	$sel_doc_2 = $this->seleciona("SELECT * FROM pipeline_documentos WHERE segurado_id='".$segurado_id."' AND documento_label='segurado'");
	$contar_segurado = mysql_num_rows($sel_doc_2);

	if($contar_segurado < '1'){
		$erro[] = 'CNH, CPF e/ou RG';
		$erros++;
	}

	$sel_doc_3 = $this->seleciona("SELECT * FROM pipeline_documentos WHERE negocio_id='".$negocio_id."' AND objeto_id='".$objeto_id."' AND documento_label='objeto'");
	$contar_objeto = mysql_num_rows($sel_doc_3);

	if($contar_objeto == '0'){
		$erro[] = 'Documenro Objeto Segurado';
		$erros++;
	}

	if($retonar == 'lista_erros'){
		
		$sql = implode( ' ),( ',$erro );
		return '('.$sql.')';

	}else if($retorna = 'numero_erros'){
		
		return $erros;

	}

}

Existe uma forma mais simplificada? Onde possa chegar ao mesmo resultado ou a um melhor resultado? Desde já agradeço a colaboração de todos que se prontificarem a ajudar.

[--The Matrix-- 1]

Fica meio complicado falar se existe uma forma mais simplificada pois não conheço a lógica completa de seu sistema, mas há alguns pontos que eu verifiquei.

1. O mysql_num_rows retorna um int e você está comparando com uma string. O correto mesmo seria você utilizar algo semelhante a isso:

if($contar_objeto === 0)

Os três iguais (===) comparam se o objeto possui o mesmo valor e o mesmo tipo.

2. A variável erros pode ser substituída pelo tamanho do array.

if($retornar = 'numero_erros'){
    return count($erro);
}

3. Esse é mais uma dica de lógica. Você faz duas comparações antes de retornar o valor, mas se você passar o $retornar como 'lista_erros' ele irá sair da função já que ela irá encontrar um return, assim não faz sentido usar o else.

4. Se você testar sua função passando o $retornar como 'numero_erros' ela não vai retornar nada, certo? Isso acontece porque você esqueceu do "R" no if está $retorna e não $retornar.

Espero ter ajudado!

[Helison Santos 0]

Obrigado pelo retorno, então eu percebi o erro de digitação depois, mas as outras dicas são pertinentes e e ja add em meu código, funcionando perfeitamente.

Ainda estou trabalhando em uma forma de retornar qual documento está faltando, o problema é que antes nem todos eram obrigatórios, por exemplo o utilizador poderia enviar tanto uma CHN quanto CPF+RG ou até os 3, o que eu fiz até agora, é que quando a CNH é enviada desabilito o envio dos outros dois e virce-versa.

Queria um resultado meio assim:

$sel_doc_2 = $this->seleciona("SELECT * FROM pipeline_documentos WHERE segurado_id='".$segurado_id."' AND documento_label='segurado'");
$contar_segurado = mysql_num_rows($sel_doc_2);

#Para quando nenhum documento for enviado;
if($contar_segurado < '1'){
    $erro[] = 'CNH, CPF e/ou RG';
}else{
   #Para quando somente o CPF ter sido enviado, que neste caso se torna obrigatório.
   if(...){
      $erro[] = 'Faltando o RG';
   }
}

#OU FORA?

#Para quando somente o CPF ter sido enviado, que neste caso se torna obrigatório.
if(...){
    $erro[] = 'Faltando o RG';
}

Esta função de verificar os erros implica no avanço do utilizador no sistema, sem os documentos ele fica impedido de finalizar a negociação.