good point, will change for a forEach