Javascript-форум (https://javascript.ru/forum/)
-   Ваши сайты и скрипты (https://javascript.ru/forum/project/)
-   -   Оптимизация работы галереи (https://javascript.ru/forum/project/12070-optimizaciya-raboty-galerei.html)

RaskolnikOFF 30.09.2010 16:43

Gozar
Мне упрямым смысла быть нету. Я помощи ищу, а не правоту свою стремлюсь доказать. Если в логике кода есть ошибка, то укажите на нее.
Может вы имеете ввиду кусок с прелоадом? Там я действительно поначалу не разобрался и не дописал немного. Но он никак по идее не должен влиять на работу остальной части кода. Можете его проигнорировать. А можете посоветовать, что там править надо. Или ссылкой поделиться, как ентой штукой пользоваться правильно, а то что-то гугл не шибко помогает.
Peace, bro.

Gozar 30.09.2010 16:45

RaskolnikOFF, кликните 10-15 раз на стрелку вправо/влево.

RaskolnikOFF 30.09.2010 17:03

Если быстро кликать, рисунки исчезают.Через некоторое время они появляются снова. Я это знал. Вот поэтому и встал вопрос оптимизации. Складывается чувство, что браузер просто не успевает отрисовывать их. Или что то вроде. Может просто сделать как в IE, чтобы на слишком быстрые нажатия скрипт просто не реагировал. Т.е. срабатывал раз в полсекунды например?

Gozar 30.09.2010 17:29

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

ps: а вообще это ГОВНОКОД, т.к. криво написан с применением jquery(не в обиду будет сказано этой маленькой либе).

RaskolnikOFF 30.09.2010 17:38

Gozar
Откуда ему быть ровным, если я неделю назад первый раз jquery увидел? А Вы бы вместо "комплиментов" моему коду дали бы хоть один дельный совет. А то "пальцы гнуть" каждый может.

Gozar 30.09.2010 17:49

Цитата:

Сообщение от RaskolnikOFF (Сообщение 72532)
дали бы хоть один дельный совет.

я вам его дал, пишите так чтобы работало в ie,ff,opera,chrome.

jquery нужна для того чтобы писать кросс-браузерный код, а не быстро написать какой-нибудь.

RaskolnikOFF 30.09.2010 17:59

А конкретней? Я же уже говорил, что у меня работает. Кликал в опере 18 раз, все норм было. Вы еще не сказали ничего того, чтобы я не знал. Судя по карме у Вас достаточно опыта, чтобы сказать что-нибудь по существу, а не просто: "Все неправильно, сделай правильно". Если бы для меня ошибки были очевидны, я бы советов не спрашивал.

Gozar 30.09.2010 18:16

Цитата:

Сообщение от RaskolnikOFF (Сообщение 72541)
Если бы для меня ошибки были очевидны, я бы советов не спрашивал.

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

HelpeR 06.10.2010 16:21

RaskolnikOFF, что бы не происходил глюк прокрутки, после клика на стрелку отключайте обработчик onclick и только после того, как прокрутка закончится включайте его обратно. Можете посмотреть как это сделал я http://jsprime.fuzaylov.net/preview/PhotoLine/

Gozar 06.10.2010 20:41

HelpeR,
с чего вы взяли что вы сделали хорошо, я далеко ходить не буду:

photos : ['1.jpg','2.jpg','3.jpg'], // названия рисунков ленты
    big_photos : ['1b.jpg','2b.jpg','3b.jpg'], // название больших рисунков
    pic_dir : '/img/', // директория хранения рисунков
    titles : ['Описание1','Описание2','Описание3'], //Описание к фотографиям

так делать неудобно, потому что если идет выборка из базы, то нужно сложить аж три массива чтобы подставить потом в ленту, я молчу про расширяемость. Гораздо проще кормить в ленту объект, а не клепать массивы превращая хорошую идею в говно. И определитесь наконец какой у вас стиль написания переменных:
такой: pic_padding
такой: paddingBot
или такой: Blackout

а то вот это в js выглядит как порнография: Blackout_src


Часовой пояс GMT +3, время: 23:31.