Instead of retriggering setTimeout() on every call to movesnow(), I suggest calling setInterval() setInterval() just once for the entire program.
You have some duplicated code between initsnow() and movesnow() (the snowingzone switch) that should be factored out. It appears that snowingzone is hard-coded to 3, though, so I'm not sure what your real intention is.
It's considered good practice to end every statement with an explicit semicolon, even though JavaScript doesn't require it.
I suggest renaming the snow array to flakes, and possibly renaming a few other variables in the same way. I'm not sure what the crds variable stands for — the abbreviation is too cryptic for me.
Instead of retriggering setTimeout() on every call to movesnow(), I suggest calling setInterval() just once for the entire program.
You have some duplicated code between initsnow() and movesnow() (the snowingzone switch) that should be factored out. It appears that snowingzone is hard-coded to 3, though, so I'm not sure what your real intention is.
It's considered good practice to end every statement with an explicit semicolon, even though JavaScript doesn't require it.
I suggest renaming the snow array to flakes, and possibly renaming a few other variables in the same way. I'm not sure what the crds variable stands for — the abbreviation is too cryptic for me.
Instead of retriggering setTimeout() on every call to movesnow(), I suggest calling setInterval() just once for the entire program.
You have some duplicated code between initsnow() and movesnow() (the snowingzone switch) that should be factored out. It appears that snowingzone is hard-coded to 3, though, so I'm not sure what your real intention is.
It's considered good practice to end every statement with an explicit semicolon, even though JavaScript doesn't require it.
I suggest renaming the snow array to flakes, and possibly renaming a few other variables in the same way. I'm not sure what the crds variable stands for — the abbreviation is too cryptic for me.
Instead of retriggering setTimeout() on every call to movesnow(), I suggest calling setInterval() just once for the entire program.
You have some duplicated code between initsnow() and movesnow() (the snowingzone switch) that should be factored out. It appears that snowingzone is hard-coded to 3, though, so I'm not sure what your real intention is.
It's considered good practice to end every statement with an explicit semicolon, even though JavaScript doesn't require it.
I suggest renaming the snow array to flakes, and possibly renaming a few other variables in the same way. I'm not sure what the crds variable stands for — the abbreviation is too cryptic for me.