Homework 4 Completed #3

Open
wants to merge 6 commits into
base: master
from

Conversation

Projects
None yet
2 participants
@tmandel2

tmandel2 commented Dec 14, 2018

No description provided.

splitStr.splice(i, 1);
i--;
}
}

This comment has been minimized.

@reuben

reuben Dec 31, 2018

Contributor

not good practice to splice an array while you're iterating over it

This comment has been minimized.

@reuben

reuben Dec 31, 2018

Contributor

i mean i see you adjusted i to compensate, but... a cleaner alternative might be just to concatenate each character that isn't a space on to your string(s) as you as you loop

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