Identificación y Refactorización de Código

¿Qué está mal con el siguiente código?

class UserManager {
    var users = mutableListOf<User>()
    
    fun addUser(name: String, age: Int, email: String) {
        if (name.length >= 2 && email.contains("@")) {
            val user = User(name, age, email)
            users.add(user)
            println("User added successfully")
        } else {
            println("Invalid user data")
        }
    }
    
    fun getUser(email: String): User? {
        for (user in users) {
            if (user.email == email) return user
        }
        return null
    }
    
    fun getUserAge(email: String): Int {
        val user = getUser(email)
        return user?.age ?: -1
    }
}

data class User(
    var name: String,
    var age: Int,
    var email: String
)