Не могу найти ошибку в блоке меню в tpl файла?

Question

[sergey]

Есть простая разметка на html/css/js - responsive меню https://jsfiddle.net/martynuk/5tk6q8ty/ необходимо его разместить в tpl файл.

Код tpl я написал, но он не срабатывает, не могу найти ошибку. Кто может помочь?

<?php if ($categories) { ?>
	<div class="container">
		<a class="toggleMenu" href="#">Menu</a>
	<ul class="nav_menu">
		<?php foreach ($categories as $category) { ?>
			<li><?php if ($category['children']) { ?><div class="parent"<?php } ?>
				<a href="<?php echo $category['href']; ?>"><?php echo $category['name']; ?></a>
				<?php if ($category['children']) { ?>
                <?php for ($i = 0; $i < count($category['children']);) { ?>
                    <ul>
		          		<?php $j = $i + ceil(count($category['children']) / $category['column']); ?>
		          		<?php for (; $i < $j; $i++) { ?>
		          		<?php if (isset($category['children'][$i])) { ?>
          				<li>
	                        <a href="<?php echo $category['children'][$i]['href']; ?>">
		                    <?php echo $category['children'][$i]['name']; ?></a>
                        </li>
          				<?php } ?>
          				<?php } ?>
	                    <?php } ?>
        			</ul>
                <?php } ?>
				<?php } ?>
			</li>
		<?php } ?>
	</ul>
<div>
<?php } ?>

Comments

[raiboon]

Какая же жуть ваш php...

[sergey]

Да я не специалист по php, писал как мог, и вот теперь это не срабатывает. На адаптивном размером 320px устройстве только первый пункт меню открывается, остальные нет.

Answer 1

[ChenZhen]

У вас лишняя закрывающая фигурная скобка. Используйте подсчёт открывающих-закрывающих скобок в редакторе. Открывающих у вас 7, закрывающих 8. И форматирование кода немного съехало - не понятно какая скобка что закрывает. Лишняя стоит перед последним /li . Я в коде её выделил комметом для наглядности.

<?php if ($categories) { ?>
  <div class="container">
    <a class="toggleMenu" href="#">Menu</a>
    <ul class="nav_menu">
      <?php foreach ($categories as $category) { ?>
      <li>
        <a href="<?php echo $category['href']; ?>"><?php echo $category['name']; ?></a>
        <?php if ($category['children']) { ?>
        <?php for ($i = 0; $i < count($category['children']); $i++) { ?>
        <ul>
          <?php $j = $i + ceil(count($category['children']) / $category['column']); ?>

          <?php for (; $i < $j; $i++) { ?>

          <?php if (isset($category['children'][$i])) { ?>
          <li>
            <a href="<?php echo $category['children'][$i]['href']; ?>">
              <?php echo $category['children'][$i]['name']; ?></a>
          </li>
          <?php } ?>


          <?php } //for  i < $j ?>

          <?php } //for ($i = 0; $i < count ?>

        </ul>
        <?php } // if ($category['children']) #8 ?>
      </li>
      <?php } // foreach ($categories as $category) ?>
    </ul>
    <div>
      <?php } ?>

Comments

[sergey]

Увидел спасибо, убрал лишнюю. Но выборка дочерних элементов не срабатывает... При клике на пункт меню должен открываться список подпунктов, но этого не происходит.

[ChenZhen]

sergey martynuk: Еще у div class="parent" у вас не хватает > у открывающего тега.

[sergey]

Тег добавил, но это не решает проблему

[ChenZhen]

sergey martynuk: еще циклы for проверьте, в первом $i++ не хватает, во втором $i=0

[sergey]

Да я в php плохо разбираюсь, копировал с разных источников. Цикли проверить это как?

Answer 2

[sergey]

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

<?php if ($categories) { ?>
	<div class="container">
		<a class="toggleMenu" href="#">Menu</a>
		<ul class="nav_menu">
			<?php foreach ($categories as $category) { ?>
			<li><?php if ($category['children']) { ?><?php } ?>
				<a href="<?php echo $category['href']; ?>"><?php echo $category['name']; ?></a>
				<?php if ($category['children']) { ?>
				<?php for ($i = 0; $i < count($category['children']);) { ?>
				<ul>
					<?php $j = $i + ceil(count($category['children']) / $category['column']); ?>

					<?php for (; $i < $j; $i++) { ?>

					<?php if (isset($category['children'][$i])) { ?>
					<li>
						<a href="<?php echo $category['children'][$i]['href']; ?>">
							<?php echo $category['children'][$i]['name']; ?></a>
					</li>
					<?php } ?>


					<?php } //for  i < $j ?>

					<?php } //for ($i = 0; $i < count ?>

				</ul>
				<?php } // if ($category['children']) #8 ?>
			</li>
			<?php } // foreach ($categories as $category) ?>
		</ul>
		<div>
			<?php } ?>

Comments

[ChenZhen]

Взял этот код, отредактировал и вставил в ответ выше. Попробуйте. Я поправил первый цикл for и во вложенном цикле заменил $i на $z

[sergey]

сейчас остальные пункты меню пропали вообще, есть только первый рабочий, при клике на него выпадает список подменю