-
Notifications
You must be signed in to change notification settings - Fork 18
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
Why is max word length 22 characters? #3
Comments
On their blog Spritz claim average human is only capable of perceiving about 13 letters at a time. I guess they assume longer words should be split... alternatively one might try to prolong time duration for such instances. |
Word should be splitted, but it crashes. |
I agree, the crash should be fixed. |
My fix - http://pastebin.com/Xqy9CqXA, but it doesn't solves problem fully. The fix check if word Index exists in array and is not undefined. If does, it skips. This way code doesn't crashes at "long word". |
Add guard of zero or negative num in String.repeat(num) so it won't crash on long words.
Max word length is 22 chars, this seems slightly low because there are plenty of longer words.
If you get a longer word app crashes. Uncaught RangeError: Invalid array length
The text was updated successfully, but these errors were encountered: