Каталог программ

Статус
Закрыто для дальнейших ответов.

Tsatur

Well-Known Member
Регистрация
15.05.2005
Сообщения
96
Оцените дизайн сайта http://www.winnux.net/test/
Пока что ничего не работает, по мере добавления функций буду просить о тестировании, кто умеет ломать сайты, официально разрешаю поиздеваться над ним (над самим сайтом, но не над сервером :))...
 

Creator

Administrator
Команда форума
Регистрация
03.03.2005
Сообщения
1 745
Дизайн сайта мне понравился... за исключением черной полосы сверху.
Так же посоветовал уменьшить разрыв между блоками меню, т.е. не делать вообще разрыва строки(<br>).
Еще мне не очень понравился подобранный цвет для верхнего навигационного меню.

Это что касается дизайна сайта... что касается реализации... советую не выдавать, что "запрос не выполнен" (например: "http://www.winnux.net/test/authors/add.php"), а отрабатывать неверные запросы и выдавать сообщение, что файл не найден или не существует страница.

Это мое субъективное мнение.
 

Tsatur

Well-Known Member
Регистрация
15.05.2005
Сообщения
96
Creator сказал(а):
Дизайн сайта мне понравился... за исключением черной полосы сверху.
Так же посоветовал уменьшить разрыв между блоками меню, т.е. не делать вообще разрыва строки(<br>).
Еще мне не очень понравился подобранный цвет для верхнего навигационного меню.

Это что касается дизайна сайта... что касается реализации... советую не выдавать, что "запрос не выполнен" (например: "http://www.winnux.net/test/authors/add.php"), а отрабатывать неверные запросы и выдавать сообщение, что файл не найден или не существует страница.

Это мое субъективное мнение.
[post="902"][/post]​
Большое спасибо... учту... по поводу "запрос не выполнен" - исправлю, просто сделал временно для отладки...
 

Creator

Administrator
Команда форума
Регистрация
03.03.2005
Сообщения
1 745
Забыл дать небольшой совет: всегда проверяй тип получаемых параметров из адресной строки, т.е. (http://www.winnux.net/test/list.php?sec=inet&n=0abra katabra) устанавливай принужденно "n" - integer, т.к. у тебя поиск идет в числовом поле (поле в БД), а там нельзя ничего кроме цифровых значений искать.
Еще что найду сообщу.
 

DnAp

self::setDebugMode(true);
Регистрация
30.04.2005
Сообщения
490
Что-то мне подборка цветов не понравилась и полоса наверху, может его резиновым сделать? Хотя я не знаю пока не увижу что выйдет не пойму...
Чуть про функциональность существующих размеров:
Объем программы (Kb) - ну раз ты проверяеш на битость ссылку(и надеюсь скриптом) то почему бы не получить сразу размер файла?
Да и никогда не знал что разер программы может выглядеть "5ds" например!
Сайт программы * - почему обязательное? я знаю несколько полезных программ у которых нету сата...
Хех, правда некоторые из них не насчитывают и 50 копий...
ссылки только на рисунки (*.gif, *.jpg...); - незнаю кто будет вводить новые программы, но удобней будет складывать картинки себе на сервер(как аваторы в форуме) чтоб битых картинок не было, т.е. сделать бутону отправить...
Ломать пока что рано... Ничегож нету...
ОЙ чет я как-то раскритиковался..
 

v0rbis

selfcoded
Регистрация
02.05.2005
Сообщения
923
вполне приличный дизайн :drinks_cheers:

вот только логотипчик забацай :)
 

Tsatur

Well-Known Member
Регистрация
15.05.2005
Сообщения
96
DnAp сказал(а):
Что-то мне подборка цветов не понравилась и полоса наверху, может его резиновым сделать?
C полосой разберусь... резиновый - не хочу...
DnAp сказал(а):
Объем программы (Kb) - ну раз ты проверяеш на битость ссылку(и надеюсь скриптом) то почему бы не получить сразу размер файла?
Да и никогда не знал что разер программы может выглядеть "5ds" например!
Не проверяю я ссылку на битость :) Свой опыт позволяет сказать, что часто, то переезжаешь с сервера на сервер, то забываешь изменить адрес ссылки... здесь даю волю авторам: хотят, чтобы их прогу скачивали - пусть заботятся :)

Про 5ds - не понял о чем ты...
DnAp сказал(а):
Сайт программы * - почему обязательное? я знаю несколько полезных программ у которых нету сата...
Хех, правда некоторые из них не насчитывают и 50 копий...
Долго ломал голову над этим вопросом и решил, что данное требование - хорошая профилактика от появления в базе всякого мусора... а вот серьезные программы в конце концов обзаводятся своими сайтами :)
DnAp сказал(а):
ссылки только на рисунки (*.gif, *.jpg...); - незнаю кто будет вводить новые программы, но удобней будет складывать картинки себе на сервер(как аваторы в форуме) чтоб битых картинок не было, т.е. сделать бутону отправить...
Место на сервере забивать... не хочу... кто решить ввести линк рисунка - позаботится о его "небитости" (ввод информации-то необязательный, значит, если вводишь, то тебе это важно, а если важно, постараешься ввести нужный адрес)... проверить битость ссылок - не так уж сложно - раз в месяц скриптик запустить и все....
DnAp сказал(а):
Ломать пока что рано... Ничегож нету...
ОЙ чет я как-то раскритиковался..
[post="906"][/post]​
Ломать уже и щас можно :) А вот насчет критики - спасибо! Так и надо, иначе откуда прогресс будет?
 

Tsatur

Well-Known Member
Регистрация
15.05.2005
Сообщения
96
v0rbis сказал(а):
вполне приличный дизайн :drinks_cheers:

вот только логотипчик забацай :)
[post="925"][/post]​
Спасибо... с логотипчиком поработаю попозже...
 

Tsatur

Well-Known Member
Регистрация
15.05.2005
Сообщения
96
Creator сказал(а):
Забыл дать небольшой совет: всегда проверяй тип получаемых параметров из адресной строки, т.е. (http://www.winnux.net/test/list.php?sec=inet&n=0abra katabra) устанавливай принужденно "n" - integer, т.к. у тебя поиск идет в числовом поле (поле в БД), а там нельзя ничего кроме цифровых значений искать.
Еще что найду сообщу.
[post="904"][/post]​
В to-do файле это у меня уже давно висит :) времени нет :) Притом запись-то с восклицательным знаком, то бишь повышенная важность!

А вот запись в тудушнике:
-! Осуществить проверку всех данных, которые вводятся через строку запроса
:)
Все равно спасибо! Вижу, что взялись за сайт с пристрастием и уже вглубь копать начинаете :thumbup: :yes:
 

Tsatur

Well-Known Member
Регистрация
15.05.2005
Сообщения
96
Здравствуйте, уважаемые... времени у меня не хватает, на скорую руку дописал сайт, очень сырой правда, т.н. Alpha версия...
Очень прошу потестить и обратить внимание теперь на работу сайта, эргономику, удобность и т.д. Если не сложно, добавьте пожалуйста хотя бы по одной программе (конечно лучше новые версии :))...
Теперь адрес без "теста", т.е. www.winnux.net.
Заранее благодарю...

ЗЫ
Чуть не забыл, некоторые пункты меню не работают, об этом я знаю :)
 

Creator

Administrator
Команда форума
Регистрация
03.03.2005
Сообщения
1 745
В to-do файле это у меня уже давно висит :) времени нет :) Притом запись-то с восклицательным знаком, то бишь повышенная важность!
Так... видел сделал проверку http://www.winnux.net/list.php?sec=inet абра кадабра&n=0, но надо так же и в http://www.winnux.net/list.php?sec=inet&n=0 абра кадабра. Говорит, что не может выполнить запрос. Хорошо, что не показывает в чем конкретно ошибка, а то sql-инъекцию можно было бы произвести. А это вообще что за переменная $n? Я не увидел в ней необходимости... Но она явно используется в запросах.

Потом, что я заметил в просмотре прог... урл: http://www.winnux.net/pri.php?id=0, дает форму в которой просто ничего нет, т.е. есть поля в которых должна быть инфа проги, но они пусты:((( Т.е. там тоже нет никакой проверки на верность типа переменной, если я ввиду id=проверка, то можно улицезреть нашу любимую страничку "Не могу выполнить запрос!";)

Так, а что будет если я нажму скачать прогу? Урл: http://www.winnux.net/download.php?id=&uri=main

Вот результат:

Warning: mysql_fetch_row(): supplied argument is not a valid MySQL result resource in /home/winnuxn/work/stat.php on line 100

Warning: mysql_fetch_row(): supplied argument is not a valid MySQL result resource in /home/winnuxn/work/stat.php on line 7

Warning: mysql_fetch_row(): supplied argument is not a valid MySQL result resource in /home/winnuxn/public_html/download.php on line 26
Нда... я думаю тебе еще много над чем придется поработать;)

Видно ты забыл про запись в "тудушнике" со знаком вопроса;)

Далее дизайн: дизайн мне и раньше нравился, но полоса сверху огорчает:(((
Вот так вот:)
 

Tsatur

Well-Known Member
Регистрация
15.05.2005
Сообщения
96
2 Creator
Большое спасибо!!! Буду исправлять!
 

Creator

Administrator
Команда форума
Регистрация
03.03.2005
Сообщения
1 745
Рад был помочь :thumbsup:
 

Tsatur

Well-Known Member
Регистрация
15.05.2005
Сообщения
96
Так, вродебы исправил, проверяйте. Перенаправляю на главную страницу, для другого варианта пока что нет времени...
 

Creator

Administrator
Команда форума
Регистрация
03.03.2005
Сообщения
1 745
Так, вижу сайт встал на путь исправления;)
Из найденных мною багов остался только: http://www.winnux.net/list.php?sec=sec&poi...D0%BB%D1%8F&n=0

Так же нет проверки переменной, на нецифровое значение. А так все цивильно!
НО, почему не исправил черную полосу?
 

Tsatur

Well-Known Member
Регистрация
15.05.2005
Сообщения
96
Уважаемые! Мой сайт начал шевелиться :) Мне нужна ваша помощь: я прошу вас помочь мне заолнять базу программ. Проверка на наличие программы в базе скоро будет доделана, но т.к. на этот момент в базе всего 9 программ, которые видно на главной странице, то можете спокойно заполнять данные, не боясь повториться :)
Заранее всех благодарю!
 
Статус
Закрыто для дальнейших ответов.
Верх Низ