Почему выдает ошибку GET и как исправить?

Question

[Ingernirated]

Есть файлы под названием news1, news2, news1-2
Подскажите, что я неправильно написал, что выдает ошибку?
И какова причина ошибки?

Notice: Undefined index: num2 in
$sec = $_GET['num2'];

$fir = $_GET['num'];
	$sec = $_GET['num2'];
	if(!isset($sec)) {
	$text = file_get_contents("news/news$fir.txt");
	} else {
	$text = file_get_contents("news/news$fir-$sec.txt");
	}
	echo $text;

при запросе http://localhost/12.php?num=2
выдает ошибку Notice: Undefined index: num2 in, но после ошибки файл news2 всё же показан.
при http://localhost/12.php?num=1&num2=2 всё чётко

Answer 1

[Shaks]

Notice: Undefined index: num2 in
Индекс num2 в переменной ($_GET) не определен..

Вы делайте запрос на localhost/12.php?num=2 и не передаете num2, отсюда и notice ошибка.

но после ошибки файл news2 всё же показан.

а вот не врите.. показан файл news1

$fir = $_GET['num']; # $fir  у вас берется из переменной $_GET['num']
$sec = $_GET['num2']; # num2 не передан, он udnefined
if(!isset($sec)) { # срабатывает этот иф, т.к $sec у вас undefined
  $text = file_get_contents("news/news$fir.txt"); # отображается переменная $fir

Comments

[Ingernirated]

добавил скрины

[Ingernirated]

в этом и вопрос, как избежать этой ошибки, если есть файлы с одним числом, но есть и с двумя, как быть?

[Shaks]

както так

<?php
$num = isset($_GET['num']) && preg_match('#^[\d\-]+$#', $_GET['num']) ? $_GET['num'] : null;
$default_file = 'news/default.txt';

$file_path = file_exists("news/news{$num}.txt") ? "news/news{$num}.txt" : $default_file;
$text = file_exists($file_path);

echo $text;

[Shaks]

$text = file_exists($file_path);
опечатался.. тут file_get_contents()

[Ingernirated]

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

if($_GET['numeric'] == null) {
		$num = 1;
	}
	else {
	$num = $_GET['numeric'];
	};
	
	$file = file_get_contents("news/news$num.txt");
	
	echo nl2br($file);

Как проверить
$num = $_GET['numeric'];
на его наличие-отсутствие?
Что бы избежать ошибки - Notice: Undefined index: numeric in, которая вопреки проверки

if($_GET['numeric'] == null) {
		$num = 1;
	}

всё ровно вылетает

[Дмитрий]

Ingernirated: Вам подсказали, куда уж проще?
Вам надо инициализировать переменную, для этого используйте тернарный оператор. Всё расписали.
Чтобы упростить - уберите регулярное выражение.
О тернарном операторе читайте в документации.

[Ingernirated]

slo_nik: ясн, а почему

if($_GET['numeric'] == null) {
    $num = 1;
  }

не считает как за полноценную проверку и выдает ошибку?

[Shaks]

Ingernirated
вот это (убрал регулярку для упрощения):
$num = isset($_GET['num']) ? $_GET['num'] : null;
переводится как

if(isset($_GET['num']))
    $num=$_GET['num'];
else
    $num=null;

Чтоб не ловить notice ошибки, перед присвоением нужно проверять переменные на присутствие. Я сначала проверяю присутствует ли $_GET[num] а потом уже присваиваю его.
--------------

ясн, а почему

if($_GET['numeric'] == null) {
    $num = 1;
  }

Та потому что блин!! ты пользуешься НЕ ОБЪЯВЛЕННОЙ ПЕРЕМЕННОЙ $_GET['numeric']

[Shaks]

как ты можешь сравнить $_GET['numeric'] с null если $_GET['numeric'] не существует? Не думал об этом?
Нужно проверить на его существование, для этого есть оператор isset. Ты им пользовался в своем примере, но не там где надо :) и над не тем чем надо )

[Ingernirated]

Shaks: от души просто, спасибо! )

[Ingernirated]

Shaks: тысяча извинений, что ещё раз беспокою этим чудесным вечером.
а как проверить на отсутствие файла?

if(isset($_GET['numeric'])) {
		$num = $_GET['numeric'];
		
		if($num > 3) {
			echo "Такой статьи не существует!";
		}
		
		else {
		$file = file_get_contents("news/news$num.txt");
	
		echo nl2br($file);
		}

если введут localhost/12.php?numeric=wdad , вместо цифр
Warning: file_get_contents(news/newswdad.txt): failed to open stream: No such file or directory
is_string($num) закрыло эту ошибку, но и первые три статьи тоже перестали открываться и начало писать, что их нет.

[Shaks]

file_exists() функция есть такая. Но учти, ты с таким подходом легко словишь php инъекцию ) смотри внимательно мой пример.

[Ingernirated]

Shaks: да я так вскрытие вен скоро с этим программированием словлю.
подскажи пожалуйста
preg_match('#^[\d\-]+$#', $_GET['num']) ? $_GET['num'] : null;
Как называются разделы в пхп, где можно почитать про preg_match('#^[\d\-]+$#
и про "или" - "?"
а то читал про экранирование, но там шла речь исключительно о sql и html иньекциях, даже не подозревал о иньекциях пхп