Supervisork

Рейтинг
60
Регистрация
27.02.2007

Ну че вы, разобрались? :)

Да нет там никаких особых подходов. По крайней мере, один раз делал под шаблоны смерти. Вот. Те же шары, только вид сбоку. Что в phpBB вставляешь в нужные места {переменные}, что в движки под шаблонизатором смерти. То есть, макет верстается, как обычно, потом, в зависимости от конкретной системы режется на куски и начиняется переменными. Для того, чтобы сделать это правильно, в большенстве случаев, мне кажется, достаточно просто смотреть на пример готового стандартного шаблона этой системы. Рыться в доках приходится только если нужно добавить новый функционал.

У тя $full_title скорей всего пустой. Писать так:

$sql = mysql_query("SELECT title, id FROM news ORDER BY id DESC LIMIT $full_title");

Стратегическая ошибка.

Пиши так:

$sql = "SELECT title, id FROM news ORDER BY id DESC LIMIT $full_title";

$sql = mysql_query($sql);

И тогда при желании, можно поставить между этими двумя строками

echo '<p>'.$sql.'</p>';

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

bearman:
я не про авторские права, а про ошибки и их кетчинг, а не if'ы с проверкой условий на браузеры. просто к слову сказал

Ну, тут просто глупо анализировать браузер. Здесь сам бог велел анализировать метод поддержки. А как обзовут браузер, скрипту по-барабану. У меня ж стаж яваскрипта уже седьмой год начался. В 2004 (если не в 2003) создавал на нем раскрывающиеся древовидные меню и меню из выпадающих списков. Сейчас они стали популярны. Тогда это было чем-то из ряда вон выходящим. Так что, некоторый опыт яваскрипта есть. Однако мастерства большого нет, так как случаются большие перерывы. По-этому, где-то могут быть "не изящные" решения. В первой, черновой версии скрипта они всегда есть. Когда пищу, конец предлставляю в общих чертах. По мере написания иногда возникает понимание, что вначале нужно было не так... А после написания все осмысливаю заново и, часто, переписываю еще раз, чтобы получить отшлифованный бриллиант. Но мастерства маловато, так что, ляпусы могут остаться незамеченными.

Так что, если видишь изящный ход, не удивляйся - это опыт. Если видишь ляпус, имей в виду: долгие перерывы в практике не могут не сказываться. Да и "новые веяния" могут по-долгу оставаться мной незамечены. Так как яваскрипт мне нужен только от случая к случаю, я не "держу руку на его пульсе".

Надеюсь, это поясняет особенности стиля в моем яваскрипт-коде :)

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

Когда я последний раз заглядывал в данные, тут была очередь порядка 190 изображений. Если человек не хочет смотреть все (оно же утомительно), зачем по-напрасну гонять трафик? Или имеешь в виду, подгружать только следующий экран (следующую "страницу")?

bearman:
альтернативу жукери? не смеши =)) реально не смеши, посмотри какие там решения существуют.

То есть, с твоей точки зрения, какашка вышла? :( А что тут тебе так не понравилось?

увы, для страницы http://4dle.ru/forum/7558 гугл не дает кеша :(

Мне-то все равно. У меня таких проблем нет. Просто, если пытаешся помочь человеку, стоило бы сделать копипаст. ИМХО

bearman:
ненавижу такое в коде, где точки с запятой? или это packer съел

Изначально-то я даже табуляции ставлю :) Это пакер кушал

bearman:
Непонятные имена переменных, тем более глобальные (могут перекрыть чужие скрипты) это имхо плохо

непонятные, для сокращения объема (мне-то понятно, что я писал). Изначально-то я думал просто посокращать имена и пробелы для компактности. Только потом пришла идея пропустить через пакер. А чужие скрипты не перекроют - там чужего нет :)

bearman:
Не позволит без правки кода вставить 10 галерей на одну страницу... Зашитые урлы яваскрипт бекенда не позволит без правки кода перенести "прикрутить"(ненавижу это слово) к другому сайту/движку

Ты мыслишь слишком универсально. Тут не будет несколько галерей на странице. И скрипт не нужно прикручивать к другим движкам. Менее универсальный код получается более компактным. Исходи из того, что это заточено под конкретную задачу, а не библиотека стандартных процедур на все случаи жизни :)

bearman:
Она используется в сотнях кусков кода с аяксом.

Многие конструкции где-то еще используются. Это агрумент за что, или прочив чего?

for(i=0;i<a;i++) - думаю, найдется в миллионе скриптов, разве это отменяет все авторские права на скрипт ;)

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

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

Сам фрагмент проверяет, готов ли галериТекст. Если по каким-то причинам не готов - не пытаться его отобразить (это я по-памяти отвечаю, не заглядывая в код).

bearman:
ЗЫ: раз Вы на Ты, то ТЫ уверен что всегда будет body.clientWidth или window.innerWidth ? я уверен что еще бывает document.documentElement

У меня принцип - будь проще. Мне нравится обращение на "ты". Я уверен, что всегда будет body.clientWidth. Я проверял в Опере, Мазиле и Эксплорере. Если смотрел скрипт resirer.js, который писался в 2005 (или 2004) году для другого сайта, но очень кстати пришелся здесь, то там идет проверка. А тут, для краткости, я не стал этого делать, так как видел, что все работает. Возможно это имело бы значение для Сафари... Но поскольку ее у меня нет, все равно не проверю...

bearman:
Ой, а размеры то зачем прямо жестко вшил в код...

Дык, скрипт НЕ универсальный, заточенный под задачу :) Я ж не утверждал, что создал универсальную замену jquery :)

bearman:
Как некрасиво ... а вдруг я для какой то цели напишу <script>a = false</script>?
бадабум ... и пипипи следующей логике, будет error который искать будешь часов 2*n. а переменную "a" переобозначить проще простого, поверь

А я тебя туда не пущу ;) Там же никто кроме меня не рулит скриптами. А я помню что и где делал. Аргументы по поводу уникальности переменных понятны. В следующий раз, наверное, буду следить за этим для упрощения расширяемости и переносимости.

bearman:
Где стырил, признавайся? в жукери подсматрел?)

Дык это ж, опять же, стандартная фича. Я ее с 2006 точно юзаю. Жкуери тогда еще, наверное, не существовало.

bearman:
твердая 4ка

Обижаешь начальник. Для данной задачи лучше и не сделать :) ИМХО.

Твоя критика была полезна. Но общее впечатление не достаточно четко выражено. Не придавая большого внимания коду, как сама галерея в целом? Какое впечатление? В том числе, как альтернатива подобным на jquery?

Brand from Amber:
Массив можно и и без аякса, в контенте или отдельным src подключать.
Я и не спорю - метод подгрузки данных через Аякс вполне пригоден.
Не возражения, а варианты:
<script src="скрипт">
<script src="данные">

Как вариант - да. Но использование аякса исключает вероятность попытки отобразить галерею раньше, чем загрузятся ее данные. Хотя, и в этих вариантах это можно было предусмотреть...

Brand from Amber:
Сворачиваем окно до небольших размеров. Открываем в этом окне фотку. Разворачиваем окно. получаем:

Спасибо. Не обратил внимание. Поработаю над функцией getPageSizeWithScroll.

Supervisork добавил 01.01.2010 в 17:30

подрихтовал getPageSizeWithScroll

ewg777, по твоей ссылке видно "У вас нет доступа к этой странице или функции. Если вы не авторизованы, можете сделать это, используя форму ниже, а затем попробовать снова". Если там есть что-то поинтересней, ты б скопировал сюда...

Тебя так же. Ваще, всех с праздником!

Ок. Примерно понял.

Всего: 186