forEach makes more sense than map for refresh