properly using setter