why not make these two parameters as String?