Помогите понять принцип действия

[antonP]

Помогите понять вот этот код:

<? if (@$top_menu):?>
<? foreach ($top_menu as $sm):?>

<? if($sm->path == "main" & $_SERVER['REQUEST_URI'] == "/".$sm->path."/"){
$sm->path="/";?>
<li class="activ"><a href="<?=$sm->path?>"><?=$sm->name?></a></li>

<? } elseif ($_SERVER['REQUEST_URI'] == "/".$sm->path."/"){ ?>
	<li class="activ"><a href="/<?=$sm->path?>/"><?=$sm->name?></a></li>
		<? } else{ ?> 
		<li><a href="/<?=$sm->path?>/"><?=$sm->name?></a></li>
<? };?>

<? endforeach?>
<? endif?>

Тут вроде всё просто и понятно, но конструкция типа $sm->name или $sm->path мне не совсем понятна. Что это обозначает. Пытался гуглить, не помогло. Это типо перенаправление к вложенному массиву чтоли?

[vvsh]

ооо. читайте что такое ООП

[Veseloff]

Кошмар. Автора кода — расстрелять.

[vvsh]

да мало тут расстрелять.

[antonP]

И почему же Вы хотите меня расстрелять ?

[vvsh]

головой апп стену. код ужасен.

и как вы писали этот код если вы его не понимаете?

[Veseloff]

И почему же Вы хотите меня расстрелять ?

1. Логику надо отделить от представления. Это значит, что вёрстка должна быть отдельно, а php-код — отдельно.

2. Надо хоть как-то «стандартизировать» написание кода — хотя бы в рамках одного проекта. То есть, если вы пишете

foreach ($top_menu as $sm):
endforeach

то делайте так везде, а не в части случаев так, а в части с фигурными скобками ({ и })

3. Короткие php-теги «<?» лучше не использовать, используйте рекомендуемый вариант «<?php»

4. Не надо закрывать тег там, где это не нужно:

<?php if (@$top_menu):
foreach ($top_menu as $sm):?>

[antonP]

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

Спасибо за разжёвывание Veseloff. Если есть ещё какие то недостатки с радостью выслушаю

Edited October 11, 2010 by antonP

[antonP]

Люди помогите ещё одно сделать, или даже понять как сделать. Этим скриптом:

<? if (@$top_menu):?>
<? foreach ($top_menu as $sm):?>

<? if($sm->path == "main" & $_SERVER['REQUEST_URI'] == "/".$sm->path."/"){$sm->path="/";?>
<li class="activ"><a href="<?=$sm->path?>"><?=$sm->name?></a></li>

<? } elseif ($_SERVER['REQUEST_URI'] == "/".$sm->path."/"){ ?>
	<li class="activ"><a href="/<?=$sm->path?>/"><?=$sm->name?></a></li>
		<? } else{ ?> 
		<li><a href="/<?=$sm->path?>/"><?=$sm->name?></a></li>
<? };?>

<? endforeach?>
<? endif?>

выводится меню сайта, где акривная ссылка как то выделяется. Так вот можно ли каким то образом прямо здесь добавить ещё одну ссылку в обход ситемы.

Получается мне в член класса нужно добавить ещё одну ссылку, но как это осуществить я пока не допираю. Мож знающие кинут подсказку, где копать...

[Veseloff]

А что мешает сделать в конце кода:

<? endforeach?>
 <li><a href="/address/">Текст ссылки</a></li>
<? endif?>

[antonP]

Да я так то и сделал, но все-таки для практики хотелось бы сделать так, чтобы она добавляла себя к общему списку ссылок, но думаю тут всё сложней, чем кажется... так просто наверно не добавить, так как всё это создаётся объектом, там целая "тьму таракань" в системе, чтож будемс искать описание объектов, и изучать ...