GET, с базой данных

@taewo · Регистрация: 27.03.2017

Студворк — интернет-сервис помощи студентам

Привет, видимо совсем туплю уже. Нужно вывести при клике на ссылку с параметрами GET инфу из БД соответствующую полю.

PHP
1
2
3
4
5
6
7
8
9
10
11
12
<?php 
  if ($_GET['id']){
  global $db;
  $ids = $_GET['id'];
  $sql = "SELECT * FROM `article` WHERE id={$row['ids']}";
  $row = $result->fetch_assoc();
      echo '
     <p>'.$row['title'].'</p>
     <p>'.$row['type'].'</p>
     ';
}
?>

запрос делаю такой: http://localhost/views/article.php?id=0

При данном коде не работает ничего.

При таком запросе, без ГЕТ все работает

PHP
1
2
3
4
5
6
7
8
9
10
11
     <?php
     global $db;
     $sql = "SELECT * FROM `article` WHERE id = 0";
     $result = $db->query($sql);
     for($i = 0; $i < $result->num_rows; $i++)
     {
     $row = $result->fetch_assoc();
     echo '
     <p>'.$row['title'].'</p>
     <p>'.$row['type'].'</p>
     ';}

@miketomlin · 27.03.2017, 16:36

Вот и устраняйте все прочие различия. И при выполнении запроса, которого кстати у вас нет, $row еще не определена – вы перетащили идентификатор в $ids, а не в $row['ids'], – исправляйте (небезопасно, но хоть будет работать).

@taewo · 27.03.2017, 18:28 **[ТС]**

А как тогда сделать безопасным? Задумка думаю понятна, это вывод страниц с БД по айди.

@miketomlin · 27.03.2017, 20:39

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

Например, нулевой идентификатор можете считать служебным и на входе считать корректными только натуральные числа (чтобы не появлялось ведущих нолей), причем в определенном ограниченном диапазоне и использовать для валидации значения маску ^[1-9]\\d{0,5}$. Также в php есть функции-фильтры валидации данных.

Добавлено через 7 минут
P.S. Результат запроса нужно проверять. Строки обычно в цикле while формируют:

PHP
1
while ($row = $result->fetch_assoc()) {

@taewo · 27.03.2017, 21:20 **[ТС]**

PHP
1
2
3
4
5
6
7
8
9
10
11
12
13
     <?php 
     if ($_GET['id']){
     $sql = "SELECT * FROM `article` WHERE id = {$_GET['id']}";
     $result = $db->query($sql);
     while ($row = $result->fetch_assoc()) {
     echo '
     <p>'.$row['title'].'</p>
     <p>'.$row['type'].'</p>
     ';
     }
         
      }
?>

В чем тут ошибка? Вроде бы параметр взял, поставил условие в выдаче результатов.

@miketomlin · 27.03.2017, 21:27

$result проверяйте.

Добавлено через 2 минуты
$db проинициализирован?

@taewo · 27.03.2017, 21:40 **[ТС]**

Да, глобалдб вставлен и он работает, если без запроса с get.

@miketomlin · 27.03.2017, 21:58

Это, видимо, из-за нулевого идентификатора. Блок if (0) { не будет выполняться.

Добавлено через 1 минуту
Нулевые идентификаторы обычно используются в служебных целях. Используйте в качестве публичных идентификаторов натуральные числа, о чем ранее писал.

@taewo · 27.03.2017, 22:00 **[ТС]**

Да, с единичкой заработало. Спасибо. Завтра буду проверять все остальное.
У такого метода какие недостатки есть, кроме тех, что вы описали ниже, с проверками. И что лучше, выводить по айди или по названию допустим, есть ли разница?
Спасибо

@miketomlin · 27.03.2017, 22:09

Использование ЧПУ конечно считается более вменяемым подходом. Но для начала можно использовать и числовые id. Если вы хотите услышать от меня какую-то серьезную рекомендацию, я бы посоветовал вам использовать концепт единой точки входа.

Новые блоги и статьи Все статьи Все блоги /
YAFU@home — распределённые вычисления для математики. На CPU Programma_Boinc 20.01.2026 YAFU@home — распределённые вычисления для математики. На CPU YAFU@home — это BOINC-проект, который занимается факторизацией больших чисел и исследованием aliquot-последовательностей. Звучит. . .	http://iceja.net/ математические сервисы iceja 20.01.2026 Обновила свой сайт http:/ / iceja. net/ , приделала Fast Fourier Transform экстраполяцию сигналов. Однако предсказывает далеко не каждый сигнал (см ограничения http:/ / iceja. net/ fourier/ docs ). Также. . .	http://iceja.net/ сервер решения полиномов iceja 18.01.2026 Выкатила http:/ / iceja. net/ сервер решения полиномов (находит действительные корни полиномов методом Штурма). На сайте документация по API, но скажу прямо VPS слабенький и 200 000 полиномов. . .	Расчёт переходных процессов в цепи постоянного тока igorrr37 16.01.2026 / * Дана цепь постоянного тока с R, L, C, k(ключ), U, E, J. Программа составляет систему уравнений по 1 и 2 законам Кирхгофа, решает её и находит: токи, напряжения и их 1 и 2 производные при t = 0;. . .
Восстановить юзерскрипты Greasemonkey из бэкапа браузера damix 15.01.2026 Если восстановить из бэкапа профиль Firefox после переустановки винды, то список юзерскриптов в Greasemonkey будет пустым. Но восстановить их можно так. Для этого понадобится консольная утилита. . .	Сукцессия микоризы: основная теория в виде двух уравнений. anaschu 11.01.2026 https:/ / rutube. ru/ video/ 7a537f578d808e67a3c6fd818a44a5c4/	WordPad для Windows 11 Jel 10.01.2026 WordPad для Windows 11 — это приложение, которое восстанавливает классический текстовый редактор WordPad в операционной системе Windows 11. После того как Microsoft исключила WordPad из. . .	Classic Notepad for Windows 11 Jel 10.01.2026 Old Classic Notepad for Windows 11 Приложение для Windows 11, позволяющее пользователям вернуть классическую версию текстового редактора «Блокнот» из Windows 10. Программа предоставляет более. . .

@miketomlin 930 / 846 / 190 Регистрация: 28.11.2013 Сообщений: 3,621
	27.03.2017, 16:36
	Вот и устраняйте все прочие различия. И при выполнении запроса, которого кстати у вас нет, $row еще не определена – вы перетащили идентификатор в $ids, а не в $row['ids'], – исправляйте (небезопасно, но хоть будет работать). 0

@taewo 0 / 0 / 0 Регистрация: 27.03.2017 Сообщений: 5
	27.03.2017, 18:28 [ТС]
	А как тогда сделать безопасным? Задумка думаю понятна, это вывод страниц с БД по айди. 0

@miketomlin 930 / 846 / 190 Регистрация: 28.11.2013 Сообщений: 3,621
	27.03.2017, 21:27
	$result проверяйте. Добавлено через 2 минуты $db проинициализирован? 0

@taewo 0 / 0 / 0 Регистрация: 27.03.2017 Сообщений: 5
	27.03.2017, 21:40 [ТС]
	Да, глобалдб вставлен и он работает, если без запроса с get. 0

@miketomlin 930 / 846 / 190 Регистрация: 28.11.2013 Сообщений: 3,621
	27.03.2017, 21:58
	Это, видимо, из-за нулевого идентификатора. Блок `if (0) {` не будет выполняться. Добавлено через 1 минуту Нулевые идентификаторы обычно используются в служебных целях. Используйте в качестве публичных идентификаторов натуральные числа, о чем ранее писал. 0

@taewo 0 / 0 / 0 Регистрация: 27.03.2017 Сообщений: 5
	27.03.2017, 22:00 [ТС]
	Да, с единичкой заработало. Спасибо. Завтра буду проверять все остальное. У такого метода какие недостатки есть, кроме тех, что вы описали ниже, с проверками. И что лучше, выводить по айди или по названию допустим, есть ли разница? Спасибо 0

@miketomlin 930 / 846 / 190 Регистрация: 28.11.2013 Сообщений: 3,621
	27.03.2017, 22:09
	Использование ЧПУ конечно считается более вменяемым подходом. Но для начала можно использовать и числовые id. Если вы хотите услышать от меня какую-то серьезную рекомендацию, я бы посоветовал вам использовать концепт единой точки входа. 0