Как прекратить приращение переменной при падении в том же дропсоте

Question

Как прекратить приращение переменной при падении в том же дропсоте

0

У меня есть 7 капель и 7 перетаскиваемых элементов. Я могу использовать один и тот же элемент для всех 7 droppots, или я могу использовать все перетаскиваемые элементы на той же droppot. Я имею переменную contor с приростом при падении, но, если я перетаскиваю все элементы на той же droppot, эта переменная достигает 7 и что-то происходит. Мне нужно, чтобы увеличить эту переменную только один раз за каплей, даже если я сброшу и измените src 7 раз в той же капельке. Мой код выглядит так:

jsFiddle link

drop: function( event, ui )
{
    dropTarget = event.target.id;
    indexDrops = dropTarget.charAt(dropTarget.length-1);
    $("#dropspot" + indexDrops).css("opacity", 0);
    $("#dropspot" + indexDrops).children("img").prop('src', dragTarget.src); 
    interval = setInterval(showDrop, 1);
    $(ObjectsToHide[indexDrops]).css('visibility', 'visible');    

    contor++;

    if(contor == 7){
        resetBtn.disabled = false;
        $(dragObjects).draggable("disable");
        updateBubble(2);
    };
}

user3459377 28 апр. 2014, в 14:55

Источник

Теги:

javascript

jquery

1 ответ

Ещё вопросы

rafaelcastrocouto · Accepted Answer · 2014-04-28T11-50-00.000Z

РЕШЕНИЕ

Вам действительно нужно прочитать jquery API... так много неправильных вещей в вашем коде...

1 - Не выбирать группы с идентификатором:

ObjectsToHide.push(document.getElementById("toChange" + i));

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

ObjectsToHide = $('.class');

2 - Не сохранять индекс:

indexDrags = dragObjects.indexOf(event.currentTarget);

вместо этого сохраните сам элемент в var, поэтому вам не придется его снова выбирать.

dragsEl = event.currentTarget;

3 - Не выполняйте повторяющиеся запросы:

$("#dropspot" + indexDrops).css("opacity", 0);
$("#dropspot" + indexDrops).children("img").prop('src', dragTarget.src);

вместо этого сохраните запрос в var, затем повторно используйте его или просто соедините.

var q = $("#dropspot" + indexDrops)
.css("opacity", 0).children("img")
.prop('src', dragTarget.src);

4 - Если вам нужен только один таймер работы, используйте setTimeout. Не используйте setInterval, если вы не хотите, чтобы операция выполнялась повторно.

5 - Вы должны ссылаться на свои библиотеки при создании скрипта. В вашем случае: jquery и jqueryUI.

подробнее http://doc.jsfiddle.net/basic/introduction.html#external-resources

6 - Вам нужна абсолютная ссылка на ваши изображения, иначе они не будут загружены:

src="images/clouds.svg"

вместо

src="http://your.site/images/clouds.svg"

Вы правы, я начинающий :(, но проблема, с которой я столкнулся, не решена ... Мне нужно, чтобы contor увеличивал один раз на каждую точку, даже если точка все еще активна, и я удаляю другой элемент
Во всяком случае, я узнал кое-что из твоего объяснения
ваш счетчик работает .. проверить это jsbin.com/cukov/2/edit
Да, работает, но он увеличивается, даже если я снова перетаскиваю элементы в том же дропсоте ... проверьте ur jsbin с предупреждением для contor и попробуйте перетащить элементы в первый дропсот много раз, это увеличивается :( Мне нужно увеличить только один раз для каждая капля
Блин, еще 1 вещь @rafaelcastrocouto, как очистить данные после сброса, чтобы снова посчитать? я установил contor = 0; но он не увеличивается снова
Добавлено много комментариев, чтобы вы понимали, что происходит в коде ... jsbin.com/cukov/6/edit
Еще раз спасибо за ваш ответ! Это именно то, что я искал :)