- 01
- 02
- 03
- 04
- 05
- 06
- 07
- 08
- 09
- 10
- 11
- 12
- 13
- 14
- 15
if(
('node' == arg(0) ) &&
is_numeric(arg(1) ) &&
($node = node_load(array('nid' => arg(1), 'status' => 1 ) ) ) &&
(
('chapter' == $node->type) ||
('article' == $node->type) ||
('gall' == $node->type)
) &&
isset($node->field_parentchapter) &&
is_array($node->field_parentchapter) &&
count($node->field_parentchapter)
){
/// blah blah
}
telnet 06.12.2010 20:27 # −1
brainstorm 06.12.2010 20:35 # +1
может еще подрежу, хотя врядли.
gegMOPO4 07.12.2010 17:14 # 0
Если всё это внутри цикла, можно заменить на последовательные "if(!...) continue;". Но необходимости в этом нет, дело вкуса.
brainstorm 07.12.2010 17:20 # 0
у них часто неинициализированные переменные проверются if($a['b'] ) при том что логика может зависеть и от сущеставования $a как такового воообще.
gegMOPO4 07.12.2010 17:36 # 0
brainstorm 07.12.2010 17:40 # 0
Uchkuma 06.12.2010 21:19 # +4
brainstorm 06.12.2010 21:25 # −2
andremacareno 07.12.2010 12:04 # 0
3.14159265 07.12.2010 16:50 # +2
наверное так надо писать?
на мой взгяд код - отлично писан.
только за отступ надо еще один минус опу
gegMOPO4 07.12.2010 17:15 # 0
brainstorm 07.12.2010 17:20 # 0
istem 07.12.2010 06:44 # 0
--
reset($node->field_parentchapter) !== false, не?
gegMOPO4 07.12.2010 17:17 # +1