Не работает цикл для массива (причина в замыкании)

@number1985 · Регистрация: 08.02.2015

Студворк — интернет-сервис помощи студентам

Добрый день.

Не могу разобраться с замыканиями в JS/Jquery.

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

Код ниже. Проблема в reward = function()

JavaScript
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
$(document).ready(function ()
{
    var messages = [
                    "Nice Job!",
                    "Excellent clickin'!",
                    "That was Awesome!",
                    "Man are you good!",
                    "Boom!",
                    "You're a pro!",
                    "Unbelievable!",
                    "Insanity!",
                    "You're on fire!",
                    "That was crazy!",
                    "You are blowin' my mind!"
    ]
    
    
    reward = function()
    {   
        
        $('.ball-container').css('display','none');
        
        $('.message_container').css('position', 'relative');
        
        for(var levels = 1; levels <= messages.length; levels++)
        {
                
        $('#text').css('visibility','visible').html(messages[levels]);
        
        $('#next_level').css('visibility','visible').html('Next Level: ' + levels);
        };
        window.setTimeout(hideText, 2000);
        };
    
    
    hideText = function()
    {
        $('.message_container').css('position', 'absolute');
        $('#text').css('visibility','hidden');
        $('#next_level').css('visibility','hidden');
        $('.ball-container').css('display','block');
        $('.ball').click(reward);
        };
 
    $('.ball').click(reward);
    
});

Как исправить? Спасибо

@Давран · 08.02.2015, 20:39

number1985, зачем делать такую запись?

Сообщение от number1985

reward = function()

не лучше так

JavaScript
1
2
3
4
5
function revard(){
 
}
 
$('.ball').click(reward());

@number1985 · 08.02.2015, 20:48 **[ТС]**

Как научили )

Код, однако, работать не станет

@Давран · 08.02.2015, 20:57

number1985, а сообщения выводите как?

Добавлено через 6 минут

JavaScript
1
2
3
$.each(messages , function(){
$('#text').html(this);
});

вот например можно так пустить цикл и вывести его внутри блока text

@number1985 · 08.02.2015, 21:04 **[ТС]**

Давран, спасибо. Увы, не работает

@Давран · 08.02.2015, 21:07

number1985, так, вы хотите, что бы по клику на какое то место, сообщения выводились в другом месте. сообщения из массива? если да, то сообщения должны заменять друг друга, то есть первое вышло, стерлось и на его место второе и так далее или все должны напечататься?

@number1985 · 08.02.2015, 21:10 **[ТС]**

Давран, должно быть как здесь http://jenniferdewalt.com/bouncing_ball.html.

Собственно, я это пытаюсь воссоздать. после нажатия на мяч, появляется сообщение. С каждым нажатие новое сообщение и так 11 раз. Однако у меня появляется только последнее сообщение массива из-за замыкания в JS. так как замыкания тема для меня незнакомая, переписать правильно не получается. В исходный код оригинальной страницы смотрела, но он у нас совсем по-разному написан, поэтому там ответа нет.

@Давран · 08.02.2015, 21:21

в общем надеюсь то, что нужно.
тут только пример, нужное сами доделаете

HTML5
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
<html>
<head>
        <meta http-equiv="content-type" content="charset=utf-8" />
<script type="text/javascript" src="js/jquery.js"></script>
<script type="text/javascript">
$(document).ready(function(){
var strings = ['Первая строка','Вторая строка','Третья строка'];
var count=0;
$('#block').click(function(){
    $('#mini').html(strings[count]);
    count++;
})
});
</script>
</head>
<body>
    <div id="block" style="width:300px; height:200px; border:#00CC00 solid 2px; padding:10px;"></div>
    <p id="mini"></p>
</body>
</html>

Добавлено через 54 секунды
при нажатии на зеленый квадрат, снизу сообщения выводятся по очереди

@number1985 · 09.02.2015, 12:26 **[ТС]**

Давран, спасибо большое! Заработало. Осталось разобраться, почему мой вариант не работает.

Новые блоги и статьи Все статьи Все блоги /
SDL3 для Desktop (MinGW): Создаём пустое окно с нуля для 2D-графики на SDL3, Си и C++ 8Observer8 10.03.2026 Содержание блога Финальные проекты на Си и на C++: hello-sdl3-c. zip hello-sdl3-cpp. zip Результат:	Установка CMake и MinGW 13.1 для сборки С и C++ приложений из консоли и из Qt Creator в EXE 8Observer8 10.03.2026 Содержание блога MinGW - это коллекция инструментов для сборки приложений в EXE. CMake - это система сборки приложений. Здесь описаны базовые шаги для старта программирования с помощью CMake и. . .	Как дизайн сайта влияет на конверсию: 7 решений, которые реально повышают заявки Neotwalker 08.03.2026 Многие до сих пор воспринимают дизайн сайта как “красивую оболочку”. На практике всё иначе: дизайн напрямую влияет на то, оставит человек заявку или уйдёт через несколько секунд. Даже если у вас. . .	Модульная разработка через nuget packages DevAlt 07.03.2026 Сложившийся в . Net-среде способ разработки чаще всего предполагает монорепозиторий в котором находятся все исходники. При создании нового решения, мы просто добавляем нужные проекты и имеем. . .
Модульный подход на примере F# DevAlt 06.03.2026 В блоге дяди Боба наткнулся на такое определение: В этой книге («Подход, основанный на вариантах использования») Ивар утверждает, что архитектура программного обеспечения — это структуры,. . .	Управление камерой с помощью скрипта OrbitControls.js на Three.js: Вращение, зум и панорамирование 8Observer8 05.03.2026 Содержание блога Финальная демка в браузере работает на Desktop и мобильных браузерах. Итоговый код: orbit-controls-threejs-js. zip. Сканируйте QR-код на мобильном. Вращайте камеру одним пальцем,. . .	SDL3 для Web (WebAssembly): Синхронизация спрайтов SDL3 и тел Box2D 8Observer8 04.03.2026 Содержание блога Финальная демка в браузере. Итоговый код: finish-sync-physics-sprites-sdl3-c. zip На первой гифке отладочные линии отключены, а на второй включены:. . .	SDL3 для Web (WebAssembly): Идентификация объектов на Box2D v3 - использование userData и событий коллизий 8Observer8 02.03.2026 Содержание блога Финальная демка в браузере. Итоговый код: finish-collision-events-sdl3-c. zip Сканируйте QR-код на мобильном и вы увидите, что появится джойстик для управления главным героем. . . .

@number1985 0 / 0 / 0 Регистрация: 08.02.2015 Сообщений: 5
	08.02.2015, 20:48 [ТС]
	Как научили ) Код, однако, работать не станет 0

@number1985 0 / 0 / 0 Регистрация: 08.02.2015 Сообщений: 5
	08.02.2015, 21:04 [ТС]
	Давран, спасибо. Увы, не работает 0

@Давран 203 / 152 / 44 Регистрация: 24.11.2013 Сообщений: 1,106
	08.02.2015, 21:07
	number1985, так, вы хотите, что бы по клику на какое то место, сообщения выводились в другом месте. сообщения из массива? если да, то сообщения должны заменять друг друга, то есть первое вышло, стерлось и на его место второе и так далее или все должны напечататься? 0

@number1985 0 / 0 / 0 Регистрация: 08.02.2015 Сообщений: 5
	08.02.2015, 21:10 [ТС]
	Давран, должно быть как здесь http://jenniferdewalt.com/bouncing_ball.html. Собственно, я это пытаюсь воссоздать. после нажатия на мяч, появляется сообщение. С каждым нажатие новое сообщение и так 11 раз. Однако у меня появляется только последнее сообщение массива из-за замыкания в JS. так как замыкания тема для меня незнакомая, переписать правильно не получается. В исходный код оригинальной страницы смотрела, но он у нас совсем по-разному написан, поэтому там ответа нет. 0

@number1985 0 / 0 / 0 Регистрация: 08.02.2015 Сообщений: 5
	09.02.2015, 12:26 [ТС]
	Давран, спасибо большое! Заработало. Осталось разобраться, почему мой вариант не работает. 0

Не работает цикл для массива (причина в замыкании)

Решение