Скрыть объявление
ВАШИ ПРАВА ОГРАНИЧЕНЫ!

Зарегистрируйтесь на форуме, чтобы стать полноценным участником сообщества!

Js-скрипт «живого» поиска по сайту для ucoz

Тема в разделе "Скрипты для Ucoz", создана пользователем iCooLER, 26 июн 2017.

26.06.17 в 07:51
27.06.17 в 07:19
49
9.107
4
  1. TopicStarter Overlay
    iCooLER

    Кодер

    Регистрация:
    13 сен 2016
    Сообщения:
    390
    Лучших ответов:
    6
    Рейтинги:
    +318 / 12 / -0
    Блин, тратить время и объяснять очевидное я не хочу. Я говорил и повторюсь еще раз: в коде все ок! Грамотно ли он написан — возможно нет, я это признаю. Но увы ошибок там нет — в данном скрипте и в данной реализации все на своих местах!

    Возможно функцию .match() я использовал в данном случае не совсем правильно. Тогда скажи как должно быть?! А то придираться ума хватает, а ты бы лучше свой вариант сделал и, если ты действительно гуру разработки — все это увидят. Не нужно для этого флудить в теме и говорить: «а вот это тут зачем? как прокоментируешь?» o_O
     
  2. LisER07

    VIP Кинотрафик v2 Кодер

    Регистрация:
    4 окт 2015
    Сообщения:
    630
    Лучших ответов:
    2
    Рейтинги:
    +116 / 3 / -0
    ок, на держи
    Код:
    $('form[action] input[name]').keyup(function() {
        var $this = $(this),
            query = $this.val(),
            form = $this.closest('form'),
            action = form.attr("action"),
            actionTest = new RegExp(/search|dir|publ|load|news|stuff|blog|video|photo/).test(action),
            search = new RegExp(/search/).test(action);
        if ( form.attr('id') == 'addEntForm' ) return false;
        if (query.length >= 3 && actionTest === true ) {
            $.ajax({
                url: ((search) ? action + '?q=' + query : action),
                type: ((search) ? 'GET' : 'POST'),
                data: ((search) ? {} : {query: $query, a: 2}),
                success: function(data){
                    SearchLiveInto(data, query, $this);
                }
            });   
        } else $('.search-live').remove();
    });
     
    • Не согласен Не согласен x 1
  3. TopicStarter Overlay
    iCooLER

    Кодер

    Регистрация:
    13 сен 2016
    Сообщения:
    390
    Лучших ответов:
    6
    Рейтинги:
    +318 / 12 / -0
    Ну вот — гораздо грамотнее! Нафиг целый день флуд разводил в теме? :confused:

    Я хз, что оказывается в js есть условие аналогично, как и в php : ((search) ? 'GET' : 'POST') — теперь буду знать.
    search = new RegExp(/search/).test(action) — new RegExp я не очень изучал, нашел первое что подошло.

    А это что блин?! var $this = $(this) :)
     
  4. TopicStarter Overlay
    iCooLER

    Кодер

    Регистрация:
    13 сен 2016
    Сообщения:
    390
    Лучших ответов:
    6
    Рейтинги:
    +318 / 12 / -0
    В функции $.ajax по умолчанию идет асинхронный запрос?
     
  5. LisER07

    VIP Кинотрафик v2 Кодер

    Регистрация:
    4 окт 2015
    Сообщения:
    630
    Лучших ответов:
    2
    Рейтинги:
    +116 / 3 / -0
    нечего! и так можно ! просто для удобство я всегда так делаю :D ) зато получил 6 дизлайков блин :eek:
     
    • Нравится Нравится x 2
  6. bezzubenko

    Забанен

    Регистрация:
    28 мар 2017
    Сообщения:
    19
    Лучших ответов:
    0
    Рейтинги:
    +3 / 0 / -0
    Это просто переменная такая же как и у тебя input
     
  7. LisER07

    VIP Кинотрафик v2 Кодер

    Регистрация:
    4 окт 2015
    Сообщения:
    630
    Лучших ответов:
    2
    Рейтинги:
    +116 / 3 / -0
    да, по умолчание оно true
     
  8. TopicStarter Overlay
    iCooLER

    Кодер

    Регистрация:
    13 сен 2016
    Сообщения:
    390
    Лучших ответов:
    6
    Рейтинги:
    +318 / 12 / -0
    Вот тебе лайк =)
     
  9. TopicStarter Overlay
    iCooLER

    Кодер

    Регистрация:
    13 сен 2016
    Сообщения:
    390
    Лучших ответов:
    6
    Рейтинги:
    +318 / 12 / -0
    Я в курсе — но он изначально к этому придрался, а потом в своем варианте также сделал)
     
  10. LisER07

    VIP Кинотрафик v2 Кодер

    Регистрация:
    4 окт 2015
    Сообщения:
    630
    Лучших ответов:
    2
    Рейтинги:
    +116 / 3 / -0
    не к этому ! к тому что ты засунул $(this) в переменную $input и на других повторно вытаскивал $(this)
     
  11. bezzubenko

    Забанен

    Регистрация:
    28 мар 2017
    Сообщения:
    19
    Лучших ответов:
    0
    Рейтинги:
    +3 / 0 / -0
    Вообщем и в этом коде есть изьяны, но они всегда есть и у всех, всегда есть то , что можно улучшить ) ну если только вообще там не 1 строка тривиального кода.
     
  12. LisER07

    VIP Кинотрафик v2 Кодер

    Регистрация:
    4 окт 2015
    Сообщения:
    630
    Лучших ответов:
    2
    Рейтинги:
    +116 / 3 / -0
    я не говорю что мой код ох..ый, да есть ошибки, даже сам несколько нашел, мой просто меньше жрет памяти (хотя это не заметно, но оно есть)
     
  13. Roman.U

    PRO Users

    Регистрация:
    13 ноя 2015
    Сообщения:
    6.192
    Лучших ответов:
    5
    Рейтинги:
    +1.037 / 117 / -0
    А поиск происходит по метатегам или по тайтлу?
     
  14. LisER07

    VIP Кинотрафик v2 Кодер

    Регистрация:
    4 окт 2015
    Сообщения:
    630
    Лучших ответов:
    2
    Рейтинги:
    +116 / 3 / -0
    Как обычный поиск укоз)
     
  15. kinouser

    PRO Users

    Регистрация:
    12 ноя 2015
    Сообщения:
    949
    Лучших ответов:
    0
    Рейтинги:
    +230 / 23 / -0
    молодцы,общими усилиями можете допиливать любой код и доводить до совершенства......
     
    • Нравится Нравится x 3
  16. TopicStarter Overlay
    iCooLER

    Кодер

    Регистрация:
    13 сен 2016
    Сообщения:
    390
    Лучших ответов:
    6
    Рейтинги:
    +318 / 12 / -0
    UPD: Слегка переписал скрипт!
     
    • Нравится Нравится x 2
  17. LisER07

    VIP Кинотрафик v2 Кодер

    Регистрация:
    4 окт 2015
    Сообщения:
    630
    Лучших ответов:
    2
    Рейтинги:
    +116 / 3 / -0
    Код проверил ? Работает ?
     
  18. LisER07

    VIP Кинотрафик v2 Кодер

    Регистрация:
    4 окт 2015
    Сообщения:
    630
    Лучших ответов:
    2
    Рейтинги:
    +116 / 3 / -0
    Исправь в коде, не == true, а === true
     
  19. TopicStarter Overlay
    iCooLER

    Кодер

    Регистрация:
    13 сен 2016
    Сообщения:
    390
    Лучших ответов:
    6
    Рейтинги:
    +318 / 12 / -0
    тестировал на одном сайте, на остальных будет работать тоже. твой код я не использовал, только взял такую хрень new RegExp :) но все же сделал в итоге проверку немного по-другому, хотя все тоже)
     
  20. TopicStarter Overlay
    iCooLER

    Кодер

    Регистрация:
    13 сен 2016
    Сообщения:
    390
    Лучших ответов:
    6
    Рейтинги:
    +318 / 12 / -0
    Знаю есть разница, но и так работает) Привычка писать только два ==. Поправил код...
     
Похожие темы
  1. Thor
    Ответов:
    15
    Просмотров:
    4.186
Загрузка...
Яндекс.Метрика