поругайте на код php

И всё прочее, что касается HTML
Правила форума
Убедительная просьба юзать теги [code] при оформлении листингов.
Сообщения не оформленные должным образом имеют все шансы быть незамеченными.
Аватара пользователя
ProFTP
подполковник
Сообщения: 3388
Зарегистрирован: 2008-04-13 1:50:04
Откуда: %&й
Контактная информация:

поругайте на код php

Непрочитанное сообщение ProFTP » 2009-05-20 5:29:11

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

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

посмотрите первый код на php и на html (в html все вроде бы праивльно) остальное я привел для пример, или для интереса...

skiped...
Последний раз редактировалось ProFTP 2009-05-29 14:19:36, всего редактировалось 1 раз.
Pеrl FAQ
perl -e 'print join"",map $$_[rand@$_],([0..9,'a'..'z','A'..'Z'])x30'
ИзображениеИзображение

Хостинговая компания Host-Food.ru
Хостинг HostFood.ru
 

Услуги хостинговой компании Host-Food.ru

Хостинг HostFood.ru

Тарифы на хостинг в России, от 12 рублей: https://www.host-food.ru/tariffs/hosting/
Тарифы на виртуальные сервера (VPS/VDS/KVM) в РФ, от 189 руб.: https://www.host-food.ru/tariffs/virtualny-server-vps/
Выделенные сервера, Россия, Москва, от 2000 рублей (HP Proliant G5, Intel Xeon E5430 (2.66GHz, Quad-Core, 12Mb), 8Gb RAM, 2x300Gb SAS HDD, P400i, 512Mb, BBU):
https://www.host-food.ru/tariffs/vydelennyi-server-ds/
Недорогие домены в популярных зонах: https://www.host-food.ru/domains/

zg
полковник
Сообщения: 5845
Зарегистрирован: 2007-12-07 13:51:33
Откуда: Верх-Нейвинск

Re: поругайте на код php

Непрочитанное сообщение zg » 2009-05-20 7:32:11

ProFTP писал(а):нету ли групых ошибок?
так... многоскриптовые проекты надо выкладывать одинм архивом. Эти портянки читать невозможно, если хочешь совета, поправь пост и изложи суть проблемы.

Аватара пользователя
ProFTP
подполковник
Сообщения: 3388
Зарегистрирован: 2008-04-13 1:50:04
Откуда: %&й
Контактная информация:

Re: поругайте на код php

Непрочитанное сообщение ProFTP » 2009-05-20 16:01:04

сори, а синтаксических ошибок грубых нету в первом скрипте в самом вверху??
Pеrl FAQ
perl -e 'print join"",map $$_[rand@$_],([0..9,'a'..'z','A'..'Z'])x30'
ИзображениеИзображение

Аватара пользователя
ProFTP
подполковник
Сообщения: 3388
Зарегистрирован: 2008-04-13 1:50:04
Откуда: %&й
Контактная информация:

Re: поругайте на код php

Непрочитанное сообщение ProFTP » 2009-05-20 17:32:49

Код: Выделить всё

if($clean->strictDigit(@ $_GET['weekly']))
а ч почему там "собака" стоит??

в первые вижу...

это потомучто оно в функцию передается?
Pеrl FAQ
perl -e 'print join"",map $$_[rand@$_],([0..9,'a'..'z','A'..'Z'])x30'
ИзображениеИзображение

zg
полковник
Сообщения: 5845
Зарегистрирован: 2007-12-07 13:51:33
Откуда: Верх-Нейвинск

Re: поругайте на код php

Непрочитанное сообщение zg » 2009-05-20 17:42:41

ProFTP писал(а):а ч почему там "собака" стоит??
потому что программер поленился проверить наличие переменной
ProFTP писал(а):а синтаксических ошибок грубых нету в первом скрипте в самом вверху??
а пхп их выдаёт?

Аватара пользователя
ProFTP
подполковник
Сообщения: 3388
Зарегистрирован: 2008-04-13 1:50:04
Откуда: %&й
Контактная информация:

Re: поругайте на код php

Непрочитанное сообщение ProFTP » 2009-05-20 18:12:59

в логи писало в error.log я исправил несколько переменных, уже не пишет...
Pеrl FAQ
perl -e 'print join"",map $$_[rand@$_],([0..9,'a'..'z','A'..'Z'])x30'
ИзображениеИзображение

zg
полковник
Сообщения: 5845
Зарегистрирован: 2007-12-07 13:51:33
Откуда: Верх-Нейвинск

Re: поругайте на код php

Непрочитанное сообщение zg » 2009-05-20 19:07:26

ProFTP писал(а):все работает
тогда не парься :smile:

Аватара пользователя
ProFTP
подполковник
Сообщения: 3388
Зарегистрирован: 2008-04-13 1:50:04
Откуда: %&й
Контактная информация:

Re: поругайте на код php

Непрочитанное сообщение ProFTP » 2009-05-27 0:21:27

Есть сайт со следующей структурой:
/main.css
/index.php
/files/
/images/


как написать правило для mod_rewrite. При котором пользователь может напрямую
обращаться к файлу /main.css, а также к любым файлам в папках /files/ и
/images/ но только если файлы существуют. Все остальные запросы должны
перенаправляться на файл /index.php (не редирект)
Pеrl FAQ
perl -e 'print join"",map $$_[rand@$_],([0..9,'a'..'z','A'..'Z'])x30'
ИзображениеИзображение

Аватара пользователя
Alex Keda
стреляли...
Сообщения: 35420
Зарегистрирован: 2004-10-18 14:25:19
Откуда: Made in USSR
Контактная информация:

Re: поругайте на код php

Непрочитанное сообщение Alex Keda » 2009-05-27 0:32:31

Код: Выделить всё

RewriteEngine On
RewriteBase /

RewriteCond %{REQUEST_FILENAME} !-f
RewriteCond %{REQUEST_FILENAME} !-d
RewriteRule ^(.*)/(.*)/(.*)/(.*)/       index.php?town=$1&general_cat=$2&sub_cat=$3&page=$4 [NC]
Убей их всех! Бог потом рассортирует...

Аватара пользователя
ProFTP
подполковник
Сообщения: 3388
Зарегистрирован: 2008-04-13 1:50:04
Откуда: %&й
Контактная информация:

Re: поругайте на код php

Непрочитанное сообщение ProFTP » 2009-05-27 1:02:28

задание по-моиму не такое как ты написал...

что значит НЕ редирект никто не знает?
Pеrl FAQ
perl -e 'print join"",map $$_[rand@$_],([0..9,'a'..'z','A'..'Z'])x30'
ИзображениеИзображение

zg
полковник
Сообщения: 5845
Зарегистрирован: 2007-12-07 13:51:33
Откуда: Верх-Нейвинск

Re: поругайте на код php

Непрочитанное сообщение zg » 2009-05-27 7:40:41

lissyara писал(а):^(.*)/(.*)/(.*)/(.*)/
жесть :smile:
ProFTP писал(а):Все остальные запросы должныперенаправляться на файл
1. выставляешь 404 ошибку на index.php
2. кладёшь во все недвижковые папки .htaccess, в котором вырубаешь мод_реврайт
3. в индексной папке при помощи реврайта заворачиваешь всё и вся на индекс.пхп

Аватара пользователя
Alex Keda
стреляли...
Сообщения: 35420
Зарегистрирован: 2004-10-18 14:25:19
Откуда: Made in USSR
Контактная информация:

Re: поругайте на код php

Непрочитанное сообщение Alex Keda » 2009-05-27 8:42:44

zg писал(а):
lissyara писал(а):^(.*)/(.*)/(.*)/(.*)/
жесть :smile:
а как лучше?
Убей их всех! Бог потом рассортирует...

zg
полковник
Сообщения: 5845
Зарегистрирован: 2007-12-07 13:51:33
Откуда: Верх-Нейвинск

Re: поругайте на код php

Непрочитанное сообщение zg » 2009-05-27 9:04:47

lissyara писал(а):а как лучше?
при твоём варианте получается разрозненная логика, намного лучше, если реврайт будет пихать адрес в одну переменную, которую потом будет обрабатывать скрипт.

например так

Код: Выделить всё

RewriteRule ^(.*)$      index.php?q=$1 [NC]
...

index.php
<?
...
@list($town, $general_cat, $sub_cat, $page) = split('/', $_GET['q']);

...
?>
то есть вся логика обработки адреса переходит в index.php, а .htaccess становится более универсальным, что удобно при доработке или просто для понимания логики.

Но я использую немного другой код

Код: Выделить всё

## Запросы, которые идут к разделам и файлам php, направляем на индекс
RewriteCond %{REQUEST_URI} \.php$   [OR]
RewriteCond %{REQUEST_URI} /$       [OR]
RewriteCond %{REQUEST_URI} /[^\.]+$
RewriteRule ^(.*)$ index.php?q=$1&%{QUERY_STRING} [L]
в принципе очень удобно - если запрос к пхп или резделу, то идёт посылка на индекс, если нет, то ничё не происходит.