Primeiro, a cada vez que usa o mysqli_fetch_assoc
ele vai retornar o próximo resultado, ou seja no momento que faz isto:
while(mysqli_fetch_assoc($busca) != false){
array_push($rows,mysqli_fetch_assoc($busca));
}
Você executa dois mysqli_fetch_assoc
a cada loop, ou seja se a sua query só esta trazendo um resultado por loop do for($i = 0; $i < count($palavras_pesquisa) ; $i++){
então ele sempre vai fazer o merge de valores NULL
, outra coisa você comparou com FALSE
em mysqli_fetch_assoc($busca) != false
, o que não faz sentido, pois mysqli_fetch_assoc
retorna ou array
ou null
apenas, também use o mysqli_error
para detectar o erros.
O mais correto a fazer assim:
$busca = mysqli_query($conn,"SELECT * FROM usuarios WHERE nome REGEXP '$string'") or die(mysqli_error($conn));
//'$busca' sempre retorna false, fazendo com que o restante abaixo não seja executado.
while ($dados = mysqli_fetch_assoc($busca)) {
array_push($rows, $dados);
}
Resumindo, SEMPRE FAÇA EXATAMENTE COMO ESTÃO NOS EXEMPLOS DA DOCUMENTAÇÃO (geralmente adoc em inglês esta mais correta na maioria dos casos), mesmo que eles não sejam ótimos, ainda sim é melhor do que usar errado, ou usar aleatoriamente sem saber o que esta fazendo, o efeito do mysqli_fetch_assoc
é semelhante ao yeld
, pra cada vez que usar mysqli_fetch_assoc
ele vai trazer a próxima linha.
No entanto fazer um SELECT
pra cada loop pode ser desperdício, você poderia simplificar fazer isto:
for($i = 0; $i < count($palavras_pesquisa) ; $i++){
$string = '.*'.$palavras_pesquisa[$i].'*';
$busca = mysqli_query($conn,"SELECT * FROM usuarios WHERE nome REGEXP '$string'");
Prefira usar implode
e fazer apenas uma query, ficando assim:
<?php
session_start();
if(isset($_POST['pesquisa'])){
$palavras_pesquisa = explode(' ',$_POST['pesquisa']);
$conn = mysqli_connect('localhost','root','','secure_login');
$rows = array();
$ids = array();
if (!empty($palavras_pesquisa)) { //Verifica se tem palavras
foreach ($palavras_pesquisa as &$value) {
$value = '.*' . $value . '*';
}
$consulta = "nome REGEXP '" . implode("' OR nome REGEXP '", $palavras_pesquisa) . "'"; //Cria um OR pra cada palavra
$busca = mysqli_query($conn, "SELECT * FROM usuarios WHERE " . $consulta) or die(mysqli_error($conn));
while ($dados = mysqli_fetch_assoc($busca)) {
$rows[] = $dados;
}
}
for($i = 0;$i < count($rows); $i++){
...
Outra coisa, SEMPRE FAÇA UMA BOA INDENTAÇÃO DO CÓDIGO, isso ajuda a não se perder.
Todavia é recomendável também não passar dados diretamente na query, isso devido ao sqlinjection, se está usando o mysqli então prefira usar o prepared statement, ficaria assim:
<?php
session_start();
if(isset($_POST['pesquisa'])){
$palavras_pesquisa = explode(' ', $_POST['pesquisa']);
$conn = mysqli_connect('localhost', 'root', '', 'secure_login');
$rows = array();
$ids = array();
if (!empty($palavras_pesquisa))//Verifica se tem palavras
{
//Gera where
$consulta = rtrim(str_repeat('`nome` REGEXP ? OR ', count($palavras_pesquisa)), 'OR ');
//Gera tipos parâmetros
$paramsTypes = str_repeat('s', count($palavras_pesquisa));
//Gera referencias
$params = array();
$params[] = &$paramsTypes;
foreach ($palavras_pesquisa as $value) {
$value = '.*' . $value . '*';
$params[] = &$value;
}
$stmt = mysqli_prepare($conn, "SELECT * FROM `usuarios` WHERE " . $consulta);
//Adiciona o stmt como primeiro item no array
array_unshift($params, $stmt);
if ($stmt) {
//Passa array como parâmetros no bind_param
call_user_func_array('mysqli_stmt_bind_param', $params);
if(!mysqli_stmt_execute($stmt)) {
echo 'Erro no stmt:', mysqli_stmt_error($stmt);
exit;
}
$result = mysqli_stmt_get_result($stmt);
while ($dados = mysqli_fetch_assoc($result))
{
$rows[] = $dados;
}
mysqli_stmt_close($stmt);
} else {
echo 'Erro no prepare:', mysqli_error($conn);
exit;
}
}
for($i = 0;$i < count($rows); $i++){
...
Veja a documentação pra entender as funções que usei no último exemplo:
Mysqli:
Entenda as criticas aqui como sugestões pra você melhorar.
nome1
nome 2
e ai você faz um item por vez? Porque você não usa oOR
? Tipo:.*nome1*|.*nome2*
?