Sim, está errado.
Quando você declara sua função no controller, você diz que irá receber um parâmetro, nesse caso data
- isso não está errado.
$scope.Cadastrar = function(data) { ... }
O problema está no ng-click
na tua view, você está chamando a função Cadastrar
, mas sem passar parâmetro algum.
ng-click="Cadastrar()"
Quando você tenta acessar o objeto data
no controller, ele está vazio, não possui nenhuma das propriedades que voc~evocê tentou acessar.
Como corrigir?
Simples, primeiro, no ng-model
dos inputs
crie um objeto, por exemplo pessoa
.
<input type="text" class="form-control" ng-model="pessoa.nome">
<input type="text" class="form-control" ng-model="pessoa.sexo">
E no ng-click
passe o objeto esperado.
ng-click="Cadastrar(pessoa)"
Dessa maneira, você irá receber o objeto no controller e não terá o erro.