навигационный провал PHP+js

[mrgrudge]

вообще дело такое, есть что то вроде меню

в хтмл вот такого виду

<a> 1</a>
<ul>
  <li>1-1</li>
  <li>1-2</li>
  <li>1-3</li>
</ul>

<a> 2</a>
<ul>
  <li>2-1</li>
  <li>2-2</li>
  <li>2-3</li>
</ul>

<a> 3</a>
<ul>
  <li>3-1</li>
  <li>3-2</li>
  <li>3-3</li>
</ul>

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

PHP код:

<script>
function showhide(el) {
el = document.getElementById(el);
if(el.style.display!='none') {
el.style.display='none';
} else {
el.style.display='block';
}
}
</script>

   
<? 
$result6 = mysql_query ("SELECT * FROM tablic1 ORDER BY id ",$db);
if (mysql_num_rows($result6) > 0 )
{
    $myrow6 = mysql_fetch_array ($result6);
      do 
      {
      printf("<a class='tovar_element_0' OnClick='showhide('%s')'>%s</a><br><ul id='%s' style='display:none;'>",$myrow6['id'],$myrow6['rub_name'],$myrow6['id']);
      $rub=$myrow6['rub'];
      $result7 = mysql_query ("SELECT * FROM tablica2 WHERE rub='$rub'",$db);
          if (mysql_num_rows($result7) > 0 )
          {
         $myrow7 = mysql_fetch_array ($result7);
          do {  printf("<li><a class='tovar_cat' href='showtovarcat.php?id=%s'> %s</a></li>",$myrow7['id'],$myrow7['name']);}
          while ($myrow7 = mysql_fetch_array($result7));
          echo "</ul>";
          }
        else {echo "<p class='sysinfo'>записей нет в таблице </p>";}
      
 
     }
     while ($myrow6 = mysql_fetch_array($result6));
}
else {echo "<p class='sysinfo'>записей нет в таблице</p>";}
?>

при чем если без php т.е. только js+html то как нистранно работает....
вот работающая версия js+html

<script>
function showhide(el) {
el = document.getElementById(el);
if(el.style.display!='none') {
el.style.display='none';
} else {
el.style.display='block';
}
}
</script>
<a OnClick="showhide('1')"> 1</a><br>
<ul id='1' style='display:none;'>
  <li>3-1</li>
  <li>3-2</li>
  <li>3-3</li>
</ul>

<a OnClick="showhide('2')"> 3</a><br>
<ul id='2' style='display:none;'>
  <li>3-1</li>
  <li>3-2</li>
  <li>3-3</li>
</ul>

<a OnClick="showhide('3')"> 3</a><br>
<ul id='3' style='display:none;'>
  <li>3-1</li>
  <li>3-2</li>
  <li>3-3</li>
</ul>/li>
</ul>

в чем дело понять никак не могу

[mrgrudge]

подозреваемый найден....
чтоб работало php в коде то что должно делаться при онклике должно стоять в одинарных ковычках из за копризной printf....т.е. OnClick='showhide('%s')' и ниче не работает...
просто в хтмл коде с двойными ковчками все рабит....OnClick="showhide('2')"
кто подскажет как поправить?

[Виталий Желтяков]

Либо использовать вместо ' - \', либо

амперсант#39;

[(IICuX)]

 printf("<a class='tovar_element_0' OnClick='showhide('\'%s\'')'>%s</a><br><ul id='\'%s\'' style='display:none;'>",$myrow6['id'],$myrow6['rub_name'],$myrow6['id']);

[dekameron]

printf("<a class=\"tovar_element_0\" OnClick=\"showhide('%s')\">%s</a><br><ul id=\"%s\" style=\"display:none;\">",$myrow6['id'],$myrow6['rub_name'],$myrow6['id']);

Для удачного прохождения проверки на валидность кода рекомендую названия классов, ИД, ифункций обработки событий заключать в двойные кавычки

[(IICuX)]

Сообщение от dekameron

printf("<a class=\"tovar_element_0\" OnClick=\"showhide('%s')\">%s</a><br><ul id=\"%s\" style=\"display:none;\">",$myrow6['id'],$myrow6['rub_name'],$myrow6['id']);

Для удачного прохождения проверки на валидность кода рекомендую названия классов, ИД, ифункций обработки событий заключать в двойные кавычки

ну тогда уже вместо OnClick onclick что уж )

[dekameron]

Сообщение от (IICuX)

ну тогда уже вместо OnClick onclick что уж )

ага)))
профтыкал