Поиск в файле всех пунктов меню (проверка кода)

@Wegik · Регистрация: 23.08.2014

Студворк — интернет-сервис помощи студентам

С регулярными выражениями работаю впервые, вот возникла необходимость написать редактор меню, написал, хотелось бы узнать мнение более опытных программыстов. Заранее спасибо)
Взял часть кода:

PHP
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
 case 'cms':
            $kontent="";
            $file=file ('../../../shablon/cms.php');
            for ($i=0; $i<count ($file); $i++){
                if (preg_match ('/.*<div class="meny_verh">/i', $file[$i])){
                    $kontent.="Верхнее меню:";
                    for ($y=$i; $y<count($file); $y++){
                        if (preg_match ('/.*<a href="(.*)">/i', $file [$y], $str_link)){
                            preg_match ('/.*">(.*)<\/a>/i', $file [$y], $str_name);
                            $kontent.="</br>Ссылка: <input type='text' size='40' value='".$str_link['1']."'></input>
                            Имя: <input type='text' size='25' value='".$str_name ['1']."'></input>";
                        }
                        if (preg_match ('/.*<\/div>/', $file [$y])){
                            break;
                        }
                    }
                }
                if (preg_match ('/.*<div class="meny_text">/i', $file[$i])){
                    $kontent.="</br></br>Левое меню:";
                    for ($y=$i; $y<count ($file); $y++){
                        if (preg_match ('/.*<a href="(.*)">/i', $file[$y], $str_link)){
                            preg_match ('/.*">(.*)<\/a>/i', $file[$y], $str_name);
                            $kontent.="</br>Ссылка: <input type='text' size='40' value='".$str_link['1']."'></input> 
                            Имя: <input type='text' size='25' value='".$str_name ['1']."'></input>";
                        }
                        if (preg_match ('/.*<\/div>/i', $file [$y])){
                            break;
                        }
                    }
                }
                if (preg_match ('/.*<div id="podval">/', $file [$i])){
                    $kontent.="</br></br>Подвал:";
                    for ($y=$i; $y<count ($file); $y++){
                        if (preg_match ('/.*<a href="(.*)">/i', $file [$y], $str_link)){
                            preg_match ('/.*">(.*)<\/a>/i', $file [$y], $str_name);
                            $kontent.="</br>Ссылка: <input type='text' size='40' value='".$str_link ['1']."'></input>
                            Имя: <input type='text' size='25' value='".$str_name ['1']."'></input>";
                        }
                    }
                }
            }
            break;

@AmsTaFFix · 30.09.2014, 17:46

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

Но к примеру не понятно что за переменная $file без обращения к коду, который её использует. Да и даже тогда не очень понятно. К примеру, если взять кусок кода

PHP
1
$file=file ('../../../shablon/cms.php');

то вот как понять что вы имели ввиду? Переменная должна быть понятной на столько, чтобы можно было понять её предназначение по одному лишь названию

PHP
1
$file

Не используйте этот поганый транслит "$kontent", никогда. Это считается плохим тоном и обычно означает, что программист ещё новичок

Не используйте $y как счетчик, для этого есть i,j,k. Т.к. $y и $x ассоциируются у всего населения с координатами, а не счетчиком.

расскажите что означает данная строка без обращения к контексту, где она появляется:

PHP
1
$str_link ['1']

что это за единица? что она означает? почему не 2, а есть 3? а они последовательны? Не проще ли сделать более понятно:

PHP
1
$linkHref = $str_link ['1'];

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

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

Добавлено через 5 минут
Ещё хочу добавить, что у вас явный копипаст, от которого необходимо избавиться

NEbO · 01.10.2014, 10:16

а я б пристрелил за использование транслита, конструкций вида

PHP
1
preg_match ('/.*<a href="(.*)">/i', ...)

или

PHP
1
preg_match ('/.*">(.*)<\/a>/i', ...)

Не по теме:

попробуйте две ссылки запихать в одну строку, чтоб заматчились в этом условии, и поймете о чем я

, а еще за вот такую, понатыканную везде, штуку:

HTML5
1
</br>

А вообще, мне думается, что вы используете регулярки неэффективно, и многое можно было бы объединить в одной. как говорится,

Code
1
cat ../../../shablon/cms.php

for more details

Новые блоги и статьи Все статьи Все блоги /
SDL3 для Web (WebAssembly): Обработчик клика мыши в браузере ПК и касания экрана в браузере на мобильном устройстве 8Observer8 02.02.2026 Содержание блога Для начала пошагово создадим рабочий пример для подготовки к экспериментам в браузере ПК и в браузере мобильного устройства. Потом напишем обработчик клика мыши и обработчик. . .	Философия технологии iceja 01.02.2026 На мой взгляд у человека в технических проектах остается роль генерального директора. Все остальное нейронки делают уже лучше человека. Они не могут нести предпринимательские риски, не могут. . .	SDL3 для Web (WebAssembly): Вывод текста со шрифтом TTF с помощью SDL3_ttf 8Observer8 01.02.2026 Содержание блога В этой пошаговой инструкции создадим с нуля веб-приложение, которое выводит текст в окне браузера. Запустим на Android на локальном сервере. Загрузим Release на бесплатный. . .	SDL3 для Web (WebAssembly): Сборка C/C++ проекта из консоли 8Observer8 30.01.2026 Содержание блога Если вы откроете примеры для начинающих на официальном репозитории SDL3 в папке: examples, то вы увидите, что все примеры используют следующие четыре обязательные функции, а. . .
SDL3 для Web (WebAssembly): Установка Emscripten SDK (emsdk) и CMake для сборки C и C++ приложений в Wasm 8Observer8 30.01.2026 Содержание блога Для того чтобы скачать Emscripten SDK (emsdk) необходимо сначало скачать и уставить Git: Install for Windows. Следуйте стандартной процедуре установки Git через установщик. . . .	SDL3 для Android: Подключение Box2D v3, физика и отрисовка коллайдеров 8Observer8 29.01.2026 Содержание блога Box2D - это библиотека для 2D физики для анимаций и игр. С её помощью можно определять были ли коллизии между конкретными объектами. Версия v3 была полностью переписана на Си, в. . .	Инструменты COM: Сохранение данный из VARIANT в файл и загрузка из файла в VARIANT bedvit 28.01.2026 Сохранение базовых типов COM и массивов (одномерных или двухмерных) любой вложенности (деревья) в файл, с возможностью выбора алгоритмов сжатия и шифрования. Часть библиотеки BedvitCOM Использованы. . .	SDL3 для Android: Загрузка PNG с альфа-каналом с помощью SDL_LoadPNG (без SDL3_image) 8Observer8 28.01.2026 Содержание блога SDL3 имеет собственные средства для загрузки и отображения PNG-файлов с альфа-каналом и базовой работы с ними. В этой инструкции используется функция SDL_LoadPNG(), которая. . .