Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Дз4 #5

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Дз4 #5

wants to merge 1 commit into from

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