Question

Sorry for the long code, but my environment does not support script tags.

http://pastebin.ca/2105293

I try to call loadbang (line 976) and get

_r.Multistrokes[z].name is undefined

I thought I defined it right there in the function, no?

Answer 1

Your Multistroke constructor function (it’s not a “class”) sets the Name property on the object (line 103):

this.Name = name;

But on line 1,000, you’re trying to use the name property (note the difference in capitalization):

strokeName[z]=_r.Multistrokes[z].name.toString

JavaScript is case-sensitive.

---

Side note: Line 1,000 also has the issue that you’re not actually calling toString, you’re assigning the function reference to strokeName[z]. To actually call it, add () to the end of toString.

Side note 2: The code relies heavily on the abomination that is JavaScript’s automatic semicolon insertion. Strongly recommend not relying on it, and putting in all necessary semicolons explicitly. Relying on ASI makes minification difficult, and can introduce subtle bugs.