Как правильно добавить параметр ID в php скрипт?

Question

[wwwnotna]

В ссылке передается ID

?id=47169

Мне нужно чтобы в файле подставлялся этот номер

<?php

include ("article_тутНОМЕРid");
?>

Как правильно вставить параметр в данном случае вместо "тутНОМЕРid"?

Comments

[wwwnotna]

Спасибо! Подскажите, как лучше сделать если два параметра могут по отдельности передеваться?

Т.е.

/forum/?message_id=28328 и /forum/?for_id=88427

Тогда общий код просто дублируем и меняем ID или это не гуд получится?

$filename = "article_".((int)($_REQUEST['message_id']));
if (is_file($filename))
include $filename;

$filename = "article_".((int)($_REQUEST['for_id']));
if (is_file($filename))
include $filename;

Answer 1

[Rsa97]

$filename = "article_".((int)($_REQUEST['id']));
if (is_file($filename))
    include $filename;

Comments

[avalak]

лучше использовать is_readable

[wwwnotna]

Спасибо! Подскажите, как лучше сделать если два параметра могут по отдельности передеваться?

Т.е.

/forum/?message_id=28328 и /forum/?for_id=88427

Тогда общий код просто дублируем и меняем ID или это не гуд получится?

$filename = "article_".((int)($_REQUEST['message_id']));
if (is_file($filename))
include $filename;

$filename = "article_".((int)($_REQUEST['for_id']));
if (is_file($filename))
include $filename;

[Rsa97]

@wwwnotna А система нумерации единая? Если "article_{$id}" - корректное имя файла для обоих параметров, то можно и дублировать код.

[wwwnotna]

@Rsa97 да, более того я немного не точно указал... article_ эти префиксы тоже разные.

[wwwnotna]

@Rsa97 а вы берете заказы?)

[wwwnotna]

@Rsa97 и еще вопрос возник( как написать, что если файла не существует либо id не задан, не знаю на что лучше сделать проверку (по сути есть места где ID не передается) то инклудим другой файл?

[Rsa97]

@wwwnotna Заказы я не беру. Проверка делается очень просто
if (isset($_REQUEST['id']) && is_readable(($filename = 'article_'.((int)($_REQUEST['id']))))
include $filename;
else
include 'something_else';

[wwwnotna]

@Rsa97 большое спасибо за помощь очень-очень помогли! С новым годом всего наилучшего!

Answer 2

[Roman Zakharov]

if(isset($_GET["id"]) and (int)$_GET["id"] > 0){
	$id = $_GET["id"];
	$file_name = "article_".$id;
	if(is_readable($file_name)) {
		include($file_name);
	}
}

Comments

[Roman Zakharov]

этот код можно бесконечно оптимизировать

[wwwnotna]

Первый вариант не более простой-ли?

[wwwnotna]

Да и тут лишние проверки, ведь если первый пункт не проходит, значит файл не существует, значит is_readable сработает.

[Roman Zakharov]

простой, но мне больше нравится расписывать все проверки, документировать и оптимизировать =)