I've been told that it's not a good idea to keep calling randomSeed() like that. You should call randomSeed() one time only, maybe in setup(), and give it some genuinely random value such as reading a floating analog input pin or timing a human pressing a button.
By the way
can be shortened to
if (error)
because it's already a Boolean. Comparing a Boolean value with true has no effect. (true == true) is true and (false == true) is false 