Skip to content

Дз4#5

Open
RooshRoosh wants to merge 1 commit intocripi-interface-development:masterfrom
RooshRoosh:master
Open

Дз4#5
RooshRoosh wants to merge 1 commit intocripi-interface-development:masterfrom
RooshRoosh:master

Conversation

@RooshRoosh
Copy link

Результаты не очень, требуется пара комментариев.
строки 85, 93 - в функцию, обрабатывающую значение вместо "значения" передаётся строка "undefined+значение". Толи "var" где-то дописать надо, толи...

Результаты не очень, требуется пара комментариев.
@outring
Copy link
Contributor

outring commented Nov 14, 2012

Проблема в этом месте:

stack['elements'][stack['elements'].length] += element;

В данном случае мы обращаемся к элементу массива с индексом на 1 больше, чем индекс последнего элемента. Т.е. если в массиве 3 элемента с индексами 0, 1, 2 — мы обращаемся к элементу с индексом 3, которого нет, который undefined. Т.к. element в данном случае — строка (т.к. взята из строкового атрибута id), то undefined приводится к строке и склеивается.

Причём не очень понятно, почему изначально эта переменная зовётся value, а когда передаётся в forNumber становится element.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants