VPF::require_once в цикле - Форум программистов

flash32

Дата 20.1.2010, 13:01 (ссылка)

(нет голосов)

Загрузка ...

Новичок

Профиль
Группа: Участник
Сообщений: 16
Регистрация: 20.1.2010

Репутация: нет
Всего: нет

есть код:

Код


$sql="SELECT * FROM `modules` WHERE `active` = '1'";
$result = mysql_query($sql) or die($die_sel . mysql_error());
while ($row = mysql_fetch_array($result)) {
if(file_exists($home_dir.$mod_dir.'/'.$row['dir_name'].'/'.$row['name'].'.php')) {
    require_once($home_dir.$mod_dir.'/'.$row['dir_name'].'/'.$row['name'].'.php');
} else {
    $is_close = '1';
    $error = 'невозможно открыть модуль '.$row['name'];
}
}

в таблице 4 строки, значит должно подключаться 4 файла, но при отрабатывании подключается только первый!!! .. если сделать так:

Код


$sql="SELECT * FROM `modules` WHERE `active` = '1'";
$result = mysql_query($sql) or die($die_sel . mysql_error());
while ($row = mysql_fetch_array($result)) {
if(file_exists($home_dir.$mod_dir.'/'.$row['dir_name'].'/'.$row['name'].'.php')) {
    echo ($home_dir.$mod_dir.'/'.$row['dir_name'].'/'.$row['name'].'.php');
} else {
    $is_close = '1';
    $error = 'невозможно открыть модуль '.$row['name'];
}
}

то при отрабатывании я вижу 4 полных пути к файлам!!!

если просто писать 4 раза require_once с путями к файлам, то всё прекрасно работает!!!

в чём может быть проблема(`active` везде = 1)???
мож кто знает альтернативные методы подключения???

DikShv

Дата 20.1.2010, 13:31 (ссылка)

(голосов:3)

Загрузка ...

Новичок

Профиль
Группа: Участник
Сообщений: 23
Регистрация: 19.1.2010

Репутация: нет
Всего: нет

Во первых прочти тут
http://php.su/functions/?f=include&choice=info

Ну а во вторых используй проста include
и втретьих конструкции с "_once" обробатываются медленее чем без тут лучше проследить что бы небыло совпадений

Simpliest

Дата 20.1.2010, 13:51 (ссылка)

(нет голосов)

Загрузка ...

Опытный

Профиль
Группа: Участник
Сообщений: 625
Регистрация: 1.9.2009

Репутация: 1
Всего: 3

Цитата(flash32 @ 20.1.2010, 12:01

)

я вижу 4 полных пути к файлам!!!

пути то, ты видишь.
А вот есть ли по этим путям файлы? smile

Код


var_dump(realpath($home_dir.$mod_dir.'/'.$row['dir_name'].'/'.$row['name'].'.php'));

Добавлено через 2 минуты и 34 секунды
http://simpliest.co.cc/sample/require_once/require.php

http://simpliest.co.cc/sample/require_once/require.phps
http://simpliest.co.cc/sample/require_once/1.phps
http://simpliest.co.cc/sample/require_once/2.phps
http://simpliest.co.cc/sample/require_once/3.phps
http://simpliest.co.cc/sample/require_once/4.phps

--------------------

flash32

Дата 20.1.2010, 13:54 (ссылка)

(нет голосов)

Загрузка ...

Новичок

Профиль
Группа: Участник
Сообщений: 16
Регистрация: 20.1.2010

Репутация: нет
Всего: нет

забыл написать ... с include то же самое (((

Simpliest

Дата 20.1.2010, 13:59 (ссылка)

(нет голосов)

Загрузка ...

Опытный

Профиль
Группа: Участник
Сообщений: 625
Регистрация: 1.9.2009

Репутация: 1
Всего: 3

Проверь.
http://ua2.php.net/is_readable
возможно просто нет прав на их чтение.

--------------------

flash32

Дата 20.1.2010, 14:02 (ссылка)

(нет голосов)

Загрузка ...

Новичок

Профиль
Группа: Участник
Сообщений: 16
Регистрация: 20.1.2010

Репутация: нет
Всего: нет

Цитата(Simpliest @ 20.1.2010, 13:59)

Проверь.
http://ua2.php.net/is_readable
возможно просто нет прав на их чтение.

при подключении руками всё ок!!!

Simpliest

Дата 20.1.2010, 14:08 (ссылка)

(нет голосов)

Загрузка ...

Опытный

Профиль
Группа: Участник
Сообщений: 625
Регистрация: 1.9.2009

Репутация: 1
Всего: 3

Так не бывает smile

Я пример кода приводил - он работает smile

Файла или нет или он не может быть прочитан.

--------------------

flash32

Дата 20.1.2010, 14:20 (ссылка)

(нет голосов)

Загрузка ...

Новичок

Профиль
Группа: Участник
Сообщений: 16
Регистрация: 20.1.2010

Репутация: нет
Всего: нет

Цитата(Simpliest @ 20.1.2010, 14:08)

Так не бывает smile

Я пример кода приводил - он работает smile

Файла или нет или он не может быть прочитан.

блин, реально оно так!!! ...
посмотрел то что ты мне писал и изобразил следующее

Код


$files = array();
$sql="SELECT * FROM `modules`";
$result = mysql_query($sql) or die($die_sel . mysql_error());
while ($row = mysql_fetch_array($result)) {
    $files[] = $home_dir.$mod_dir.'/'.$row['dir_name'].'/'.$row['name'].'.php';
}

foreach ($files as $name) { 
    require_once $name; 
}

вот так всё работает как надо ...
вот только понять не могу почему в foreach всё работает, а в while - нет .. хотя что тут, что там разбирается массив ..

Добавлено через 3 минуты и 57 секунд
2 Simpliest
СПАСИБО!!! ... помог ... сам бы я начал писать в файл и потом читать из него ... мега-костыль

s0lman

Дата 20.1.2010, 14:28 (ссылка)

(нет голосов)

Загрузка ...

Белый и лохматый

Профиль
Группа: Участник
Сообщений: 637
Регистрация: 1.11.2007
Где: Ukraine

Репутация: 9
Всего: 29

Может быть, фенкции должен передаваться уже сформированный путь к файлу? smile

типа такого

Код


$fname=$home_dir.$mod_dir.'/'.$row['dir_name'].'/'.$row['name'].'.php';
require_once($fname);

--------------------

Когда я уже выучусь на волшебника? :(

DikShv

Дата 20.1.2010, 14:36 (ссылка)

(нет голосов)

Загрузка ...

Новичок

Профиль
Группа: Участник
Сообщений: 23
Регистрация: 19.1.2010

Репутация: нет
Всего: нет

Пробывал убирать скобки?
Вот рабочая версия

Код


<?
$home_dir="D:/Servers/ABOCMS/home/test1.ru/www";
$mod_dir="";
$rows=array(
    array('dir_name'=>'dd','name'=>1),
    array('dir_name'=>'dd','name'=>2),
    array('dir_name'=>'dd','name'=>3),
    array('dir_name'=>'dd','name'=>4),
);
$i=0;
while ($i<count($rows)) {
    $row=$rows[$i];
    if(file_exists($home_dir.$mod_dir.'/'.$row['dir_name'].'/'.$row['name'].'.php')) {
        include $home_dir.$mod_dir.'/'.$row['dir_name'].'/'.$row['name'].'.php';
    } else {
        $is_close = '1';
        $error = 'невозможно открыть модуль '.$row['name'];
    }
    ++$i;
}
?>

и не нужно использовать 2 цикла. Это глупо

Это сообщение отредактировал(а) DikShv - 20.1.2010, 14:49

sTa1kEr

Дата 20.1.2010, 15:24 (ссылка)

(нет голосов)

Загрузка ...

9/10 программиста

Профиль
Группа: Завсегдатай
Сообщений: 1553
Регистрация: 21.2.2007

Репутация: 56
Всего: 146

flash32, все что вам было нужно, это:
а) Включить вывод всех ошибок
б) Избавится от дублирования кода

Код


// ...
$file = $home_dir.$mod_dir.'/'.$row['dir_name'].'/'.$row['name'].'.php';
if (file_exists($file)) {
    require_once ($file);
} 
// ...

flash32

Дата 20.1.2010, 15:31 (ссылка)

(нет голосов)

Загрузка ...

Новичок

Профиль
Группа: Участник
Сообщений: 16
Регистрация: 20.1.2010

Репутация: нет
Всего: нет

разобрался, всем спасибо за помощь!!!
p.s. даж не ожидал, что так оперативно кто-либо отреагирует!!!)))

nerezus

Дата 20.1.2010, 18:52 (ссылка)

(нет голосов)

Загрузка ...

Вселенский отказник

Профиль
Группа: Участник
Сообщений: 3330
Регистрация: 15.6.2005

Репутация: 12
Всего: 43

Цитата

Во первых прочти тут
http://php.su/functions/?f=include&choice=info

Лучше не тут, а в верной и актуальной документации(php.net).

Цитата

и втретьих конструкции с "_once" обробатываются медленее чем без тут лучше проследить что бы небыло совпадений

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

--------------------

Сообщество художников Artsociety.ru

DikShv

Дата 20.1.2010, 19:19 (ссылка)

(нет голосов)

Загрузка ...

Новичок

Профиль
Группа: Участник
Сообщений: 23
Регистрация: 19.1.2010

Репутация: нет
Всего: нет

Цитата(nerezus @ 20.1.2010, 18:52

)

Ага, лучше заказать унитаз когда жопа не влезает и уже некуда ее стягивать.

Я не зря так написал из личного опыта знаю когда скрипт криво написанный держал 50 онлай но после такой на первый взгляд нелепой оптимизации его пропус возрос до 100 это хостинге, а на сервере с 500 до 1000

конечно это только 1 пункт из списка нелепых заметок.

nerezus

Дата 20.1.2010, 19:32 (ссылка)

(нет голосов)

Загрузка ...

Вселенский отказник

Профиль
Группа: Участник
Сообщений: 3330
Регистрация: 15.6.2005

Репутация: 12
Всего: 43

DikShv, я привел пример, т.к. твой комент не только не имеет смысла(экономия на спичках), но и реально медленнее:

Твой вариант:

Код


if (!in_array($filename, $GLOBALS['already_required'])) {
    require $filename;
    array_push($GLOBALS['already_required'], $filename);
}

Вариант PHP:

Код


require_once $filename;

А еще я умею пользоваться профайлером, поэтому мой камент про бумагу там и появился(будем считать пока что, что твой вариант мог быть быстрее).
Я сомневаюсь, что заказчик готов платить в 2 раза больше ради прироста производительности 2%.
Ему выгоднее сделать второй проект и получить прирост дохода 100%(считая затраты на раскрутку и т.д. данная цифра будет конечно же меньше)

--------------------

Сообщество художников Artsociety.ru

0 Пользователей читают эту тему (0 Гостей и 0 Скрытых Пользователей)
0 Пользователей:
« Предыдущая тема \| PHP: Общие вопросы \| Следующая тема »