Всем привет!
Кто может подсказать?
Как грамотно дополнить нижеприведённый код?
Код должен узнавать какой фаил/url выбран.
<?php
if ($url == index.php)
{
echo "<img src=\"bilder/visa.jpg\" width=\"400\" height=\"646\" border=\"0\">";
}
if ($url == соор.php)
{
echo "<img src=\"bilder/соор.jpg\" width=\"400\" height=\"646\" border=\"0\">";
}
?>
Заранее спасибо!

Честно говоря, не понимаю вопроса.
Тот же код, только чище и эффективней в плане нагрузки на процессор.

<?php
if ($url == 'index.php')
{
echo '<img src="bilder/visa.jpg" width="400" height="646" border="0">';
}
elseif ($url == 'соор.php')
{
echo '<img src="bilder/соор.jpg" width="400" height="646" border="0">';
}
?> 

если каждому файлу обязательно сопоставлена своя картинка и у всех одинаковые размеры, то можно и так:
<?php $image = array( "index.php" => "visa.jpg", "соор.php" => "соор.jpg"); ?>
<img src="bilder/<?=$image[$url]?>" width="400" height="646" border="0">

Если подходить с точки зрения компактности и конкретно такого подхода с двумя возможными $url, то project33, дал лучшее решение из приведённых до него. Данный вариант подойдёт для вывода большого количества разных картинок, привязанных к разным страницам. ИМХО!
Если же брать конкретный пример с условием без использования массива, при условии что на главной будет одна, а на остальных другая картинка, то я бы так написал.
echo "<img src=\"bilder/".( ( $url == index.php ) ? "visa.jpg" : "соор.php" )."\" width=\"400\" height=\"646\" border=\"0\">";
или так, если быть совсем точным в отношении проверки, и того случая, что картинки выводятся только на двух страницах и не больше.
echo "<img src=\"bilder/".( ( $url == index.php ) ? "visa.jpg" : ( ( $url == соор.php ) ? "соор.php" : "" ) )."\" width=\"400\" height=\"646\" border=\"0\">";
При этом правильней писать не html в РНР, а РНР в html, таким образом.
< img src="bilder/"<?php echo ( $url == index.php ) ? "visa.jpg" : "соор.php"; ?>" width="400" height="646" border="0">
По крайней мере так пишут, с точки зрения современного подхода правильного написания, и быстродействия обработки.
Приведённый мной вариант не совсем удобочитаемый в сравнении со стандартным if(xx) { xx } else { xx }, но намного короче, и по мне намного проще для простых проверок на 2-3 условия, так как не требует дублирования строк(и).

не совсем так, мой пример универсален для любого количества $url, их нужно только вписать в массив. кроме того в нем нет условного оператора, который прилично так грузит проц при большой выборке. да и сам код с условным оператором при большом количестве страниц разрастется до гиганских размеров, а в моем случае только добавится минимальный код "индекс" => "картинка". за отзыв спасибо

project33, ты не совсем понял что я имел в виду, а я имел в виду что в твоём варианте с этими двумя страницами это самый оптимальный вариант. Я так же следующим предложением указал на то что такой вариант хорош для большого количества страниц.
Мой же вариант рассчитан как раз на вывод 2-3 страниц и не больше, конкретно по примеру топик писателя, на два условия. И если учитывать что таких страниц будет более 3-х, то данный вариант не оптимален. А для двух условий, он проще некуда, и массив тут не обязателен.
В любом случае, каждый пишет так как считает лучше, я лишь привёл свой вариант, который лишь в написании более компактный чем стандартный, но время обработки его такое же как и стандартного. Я такой вариант использую что бы сократить код на странице, убрав лишние строки там где это не нужно. К примеру зачем делать 4 строки(в примере автора вообще 10), когда всё вместится в одну строку. По крайней мере я такого подхода придерживаюсь, так как кодю время от времени ток для себя, и не придерживаюсь каких то стандартов, типа PEAR разрабов или ещё кого..
Сейчас прочёл ещё раз топик атора темы, и понял что "смотрел в книгу, а видел фигу". Не разглядел что автор спрашивает про дополнить код, а тут только два простых варианта, либо пример project33 с массивом, либо switch - > case проверка, типа так:
switch ($url) {
case "index.php": $img = "visa.jpg"; break;
case "соор.php": $img = "соор.jpg"; break;
default: $img ="default.jpg"; //Не обязательно
}
echo "<img src=\"bilder/".$img."\" width=\"400\" height=\"646\" border=\"0\">";
Но switch уже для более сложных конструкций, к примеру если на условие требуется больше действий, к примеру несколько переменных. В конкретном случае с расширением по вопросу автора, массив применять лучше, будет более компактно.

Очень рекомендую Вам почитать про разницу между одинарными и двойными кавычками в php.

тогда все ок . вариант со свич-кейс мне тоже в голову пришел сразу, конструкции типа elseif я немного недолюбливаю, но я все же предпочитаю вариант совсем без выборки когда это возможно

Что касается двойных кавычек и одинарных, я знаю, я просто оставил кавычки из примера, не хотел всё переправлять.

Не всегда можно решить просто без выборки, особенно кода на каждое условие нужно выполнить несколько действий, но в данном случае да, тут либо массив, либо как я привёл с короткой записью if. Когда ток первый раз начал читать по пыху, я тоже строил как автор на elseif, притом сложные многоуровневые, пока не узнал другие варианты условий...
Но да лан..., и так уже кучу написал, учитывая что я не программист, пущай отвечают профи, а то глядишь кроме кавычек я ещё чего не захотел исправлять...

Поговаривают что три равно работает быстрее.

Я делал что то подобное. Делал это так:
<img src="bilder/<?=$url?>" width="400" height="646" border="0">

мильторг, ты вообще понял что написал/предложил?

$url содержит название файла открытой страницы, в данном случае index.php или соор.php, это не картинки, а PHP файлы. Ты случаем не чего не перепутал?

Ганж, херня это всё и бессмысленные оптимизации, я перестал (оно не только в пхп) кашу делать из одинарных / двойных кавычек, юзаю двойные и переменные внутри них, не пишу больше:

$foo = 'string' . $value . 'string';

И то что он три строчки в двойных кавычках сделал не ухудшает ничего никак вообще.

На 100% согласен, и даже больше, так как такой подход позволяет так же делать код более удобочитаемым, если строка слишком длинная, к примеру если этой строкой идёт обращение к базе, то там иногда все 10 строк может быть, если разделить..
$foo = 'string'
. $value
. 'string';
Потом.., насколько я помню, по современным стандартам, выражение $foo = "string $value string"; считается уже неправильным(устаревшим) и не рекомендуется к применению.

Я написал идею,а не код.
$url конечно нужно обрабатывать. Хотя бы для того чтоб "$url содержит название файла открытой страницы,"
В данном случае. Если название файла уже в $url. На перле это так:
s/php/jpg/
На php есть тоже подобная функция. Но. Повторю. Я пишу идею, а не готовый код

В 5 PHP расширили возможности парсинга переменных в строках и оно при этом устаевшее?
Конкатенация строк в двойных кавычках с переменными ничуть не лучше конкатенации строк с одинарными кавычками с переменными, всё та же дерьмовая каша на выходе.

От лукавого это. Мне такое месиво из данных, которое позволяют скриптовые языки, совсем не нравится.

Для того, чтобы не было месива, придумали MVC/MVT, месиво это не проблема языков, это проблема специфики веба.

Ну а если нет - упаковать все щ отдельный класс со встроенным селектором и нужными пропертями...
П.С. Ох уж етот веб...