Почему не работает фильтр?

Question

[coldunox]

Создал форму поиска

<form action="users.tpl.php" method="post">
    <label class="col-md-4 control-label" for="newRole">Найти студента</label>
    <input type="text" name="valueToSearch" placeholder="Поиск студентов..." ng-model="searchText" class="form-control "/>
    <input type="submit" name="search"  class="btn btn-default" value="Фильтр"/>
</form>

Код фильтрации

<?php 
   if (isset($_POST['search'])){
       $valueToSearch = $_POST['valueToSearch'];
       $query = "SELECT * FROM `users` WHERE concat(`fullName`, `group_name`) like '%" . $valueToSearch . "%'";
       $search_result = filterTable($query);
       
   } else {
       $query = "select * from `users` where role_id=2 order by fullName";
       $search_result = filterTable($query);
   }
   
   function filterTable($query){
       $conn = mysqli_connect('XXX', 'XXX', 'XXX', 'XXX');
       $filter_Result = mysqli_query($conn, $query);
       return $filter_Result;
}
?>

Таблица

<table  id="myTable" class="table table-bordered table-hover table-striped">
<thead>
    <tr>
        <th>ФИО</th>
        <th>Группа</th>
        <th>Логин</th>
        <th>Email</th>
    </tr>
</thead>
<tbody>
<?php
    while ($row = mysqli_fetch_array($search_result)) { ?>
        <tr data-ng-click="showEditForm(); getUserData(<?php echo $row['id'];?>);" >
            <td><?php echo $row['fullName']; ?></td>
            <td><?php echo $row['group_name']; ?></td>
            <td><?php echo $row['login']; ?></td>
            <td><?php echo $row['email']; ?></td>
            
        </tr>
    <?php } ?>
</tbody>
</table>

Почему не работает mysqli_fetch_array ?

Comments

[Дмитрий Энтелис]

это прям яркий пример того как НЕ надо писать код.

[coldunox]

Дмитрий Энтелис, спасибо за критику. Она вдохновляет

[Андрей Николаев]

а какой $query формируется?

[coldunox]

Андрей Николаев, пока формируется $query = "select * from `users` where role_id=2 order by fullName";
а хотелось бы $query = "SELECT * FROM `users` WHERE concat(`fullName`, `group_name`) like '%" . $valueToSearch . "%'";

[Alexandr]

А вы кнопку "Фильтр" нажимаете? Может дело в том, что отправляете нажатием на enter в поле ввода

[coldunox]

Deexter, нет, на кнопку жму

[Alex]

я конечно не знаток PHP но чет мне кажется что инпут с типом сабмит сам себя не отправляет и не работает
isset($_POST['search']))
уберите имя у сабмит инпута, а добавьте в форму еще один
<input name="search" value="Фильтр"/>

[coldunox]

Alex, попробовал - не сработало

[Alex]

coldunox,

if (isset($_POST['search'])){

заменить на
if (isset($_POST['valueToSearch'])){

Answer 1

[Дмитрий Энтелис]

Ладно, отвечу серьезно.

По поводу самой проблемы:
Можно в режиме телепата гадать 100500 раз о том что происходит.
По факту вопрос слишком абстрактный, научитесь дебажить код.

Конкретный пример: у вас есть ваш "код фильтрации".
Идеально было бы поставить xdebug и ide которая умеет с ним работать, но на крайний случай вас спасут print_r и var_dump.
Прямо с начала, берете, и для каждой строчки по очереди проверяете - те ли данные вам пришли, что вы ожидали, те ли строки сформировались что вы ожидали, итд.

Скорее всего вы поймете в чем проблема сами.
Даже если не поймете - сможете задать уже конкретный вопрос.
Ну и хорошо бы вывод всех ошибок и нотисов включать.

<?php
ini_set('error_reporting', E_ALL);
ini_set('display_errors', 1);
ini_set('display_startup_errors', 1);

Теперь в целом по коду:
1. Почитайте про MVC. mysqli функции вперемешку с html версткой - недопустимы.
2. Вынесите всю работу с базой в отдельную либу. есть 100500 готовых решений на гитхабе.
sql connect в filterTable - очень плохо.
3. В базу нельзя отправлять user input без обработки! Как минимум нужно делать mysqli_real_escape_string
4. За запрос where concat(`fullName`, `group_name`) = ... в реальном проекте, увольняют сразу. Запрос не использует индексы и должен для каждой строки таблицы выполнить объединение 2х полей, а потом еще и сравнить. Это дико медленно.
Нужна такая строка для поиска - собирайте её отдельным полем в базе.

Comments

[coldunox]

А если так?

<table  id="myTable" class="table table-bordered table-hover table-striped">
    <thead>
        <tr>
            <th>ФИО</th>
            <th>Группа</th>
            <th>Логин</th>
            <th>Email</th>
        </tr>
    </thead>
    <tbody>
        <?php
        foreach ($pageData['usersList'] as $key => $value) { ?>
        <tr data-ng-click="showEditForm(); getUserData(<?php echo $value['id'];?>);" >
            <td><?php echo $value['fullName']; ?></td>
            <td><?php echo $value['group_name']; ?></td>
            <td><?php echo $value['login']; ?></td>
            <td><?php echo $value['email']; ?></td>
        
        </tr>
    <?php } ?>
    </tbody>
</table>

Функция получения всех пользователей

<?php
class UsersModel extends Model {
	public function getUsers() {
		$sql = "SELECT * 
		        FROM users 
		        WHERE role_id='2' 
		        ORDER BY fullName";
		$stmt = $this->db->prepare($sql);
		$stmt->execute();
		$result = array();
		while($row = $stmt->fetch(PDO::FETCH_ASSOC)) {
			$result[] = $row;
		}
		return $result;	
	}
}

[Дмитрий Энтелис]

Так гораздо лучше :)

1.

$stmt->execute();
    $result = array();
    while($row = $stmt->fetch(PDO::FETCH_ASSOC)) {
      $result[] = $row;
    }

нужно убрать внутрь класса db, иначе будет дикое дублирование кода, когда в model будет много функций.
Ну и я советую использовать PDO::FETCH_OBJ - это как-то идеологически правильнее.

2. Есть стандарт https://www.php-fig.org/psr/psr-1/
По нему (и с учетом предыдущего замечания) лучше писать вместо
<?php echo $value['fullName']; ?>
<?=$value->fullName;?>