Помощник
|
Файловый менеджер — всё в одном файле |
Den1xxx
|
Сообщение
#1
|
||
|
|
||
|
|||
Degradator |
25.11.2014, 10:43;
Ответить: Degradator
Сообщение
#2
|
|
если не будет лень, оберну функции в нормальный класс на этих выходных.
|
|
|
Arks |
2.12.2014, 6:14;
Ответить: Arks
Сообщение
#3
|
|
есть замечания:
1. Локализованные строки вынесены в тот же файл 2. для сравнения с int в функции get_rights_string используется if/else вместо switch 3. в строках 50-150 невменяемое количетсво конкатенаций и операндов, processor die какой-то! Вы же понимаете что опкешер сгладит погрешности но так писать код не стоит? 4. функция download_file сразу отправляет заголовки. Сурово. Не стоит ли буферизировать вывод? 5. функция rcms_scandir может быть даже не вызвана.. но она парсится! Остальное.. тот поток сознания и html даже сложно комментировать, это какой-то треш. Для начала выделите html, перепишите на какой-нибудь другой шаблонный язык(лучше не связанный с php) Иначе Вам или вашим пользователя в файлменеджер загрузят кактус или другую бячку. Треш и угар. Стыдно такое выкладывать, разьве что как антиквариат без описания на память потомков. |
|
|
Arks |
2.12.2014, 6:49;
Ответить: Arks
Сообщение
#4
|
|
честно говоря, я вообще плохо понимаю что такое filemanager на php. Видимо это для избранных. Я не против файл-менеджера, но только когда он виртуализирует ФС в рамках например файла(как INNODB), а не когда он "ломай меня полностью" и ходит по директориям.
|
|
|
Arks |
2.12.2014, 6:54;
Ответить: Arks
Сообщение
#5
|
|
Кстати почему в проекте в одной ветке master 2 коммита? От rebase / squash должны остаться хоть какие-то следы?
|
|
|
Den1xxx
|
Сообщение
#6
|
|
Треш и угар. Стыдно такое выкладывать, разьве что как антиквариат без описания на память потомков. Вы правы, что код не причёсан. Там в начале просто надергано функций из реального проекта, по принципу «абы заработало». Выложил, т.к. возможно кому-то будет полезно как шелл на PHP, чтобы не писать самому. Про треш и угар знаю, потому и написал версия пока unstable 0.01. Правда насчет отправки заголовков функцией download_file() Вы не правы. Она создана чтобы отправить заголовки, закачиваемый файл и умереть — нафига там вывод буферизировать? Посмотрите внимательней — внутри download_file() там die() в 2 местах. Я не против файл-менеджера, но только когда он виртуализирует ФС в рамках например файла(как INNODB), а не когда он "ломай меня полностью" и ходит по директориям. Ну может он Вам просто не нужен вовсе — сервер стоит дома и вместо ФМ на PHP у Вас bash. Например, как я его использую, правда как модуль внутри системы. Приходишь к Заказчику и показываешь что написал и как работает. И тут заметил какую-то досадную оплошность. Зашёл в ФМ — исправил за 1 минуту, подписал акт выполненных работ и отчалил. Всё лучше, чем бежать домой, исправлять, и приходить в следующий раз. //PS Может кому будет ещё интересно Уже есть подобный проект — http://phpfm.sourceforge.net/ Проекту уже несколько лет, там есть и exec, и закачка/распаковка архивов и прочие вещи, в том числе локализация. Что характерно, внутри треш ещё хуже чем у меня (например хидеры отправляют сразу первыми строками), потому проект мне откровенно не понравился. И размер файла больше в 8 раз, и авторизация — вообще нафиг не нужно. Если к моему нужно прикрутить админку — прикручивается в полпинка. В начале файла написать чтобы скрипт умирал если юзер не авторизован админом и всё! Для моей ЦМС, например, достаточно написать: [PHP]if (!$system->checkforright('FILES')) die('Bye');[/PHP] Файловый менеджер не должен быть заменой ЦМС, это просто доп. удобный инструмент и всё. |
|
|
Witu |
2.12.2014, 17:21;
Ответить: Witu
Сообщение
#7
|
|
http://sourceforge.net/projects/ajaxplorer/ и не нужно изобретать велосипед! как самостоятельный менеджер самое то.
Если уж из своего проекта выдергиваете, то хоть прикрутите что-то уникальное, например манипуляции с картинками (повороты, изменение размеров, обрезание) на стороне клиента... |
|
|
Degradator |
2.12.2014, 18:59;
Ответить: Degradator
Сообщение
#8
|
|
|
|
|
Degradator |
2.12.2014, 18:59;
Ответить: Degradator
Сообщение
#9
|
|
|
|
|
Arks |
8.12.2014, 6:43;
Ответить: Arks
Сообщение
#10
|
|
Den1xxx,
Хорошо, часть фраз Вы выбрали из контекста. Посмотрите на остальную. |
|
|
|
Похожие темы
Тема | Ответов | Автор | Просмотров | Последний ответ | |
---|---|---|---|---|---|
Быстрый старт - Всё о накрутке пф за 7 минут | 6 | waterworld | 2361 | 13.2.2024, 20:08 автор: Думающий |
|
Требуется контент-менеджер | 1 | Grimov | 1160 | 8.2.2024, 9:54 автор: projectadmin |
|
Контент-менеджер в ссылочные биржи | 3 | Natalia22 | 1669 | 30.5.2023, 9:19 автор: Natalia22 |
|
Требуется контент-менеджер для интернет-магазинов по игровой тематике | 1 | slavalist | 894 | 27.3.2023, 1:51 автор: slavalist |
|
RedBox - все нужные инструменты в одном месте | 0 | fantomofdarkness | 794 | 15.2.2023, 14:50 автор: fantomofdarkness |
Текстовая версия | Сейчас: 19.4.2024, 15:34 |