Depende do tipo de problema que você espera ter. Se for só erro de lógica me parece que esta é a melhor forma mesmo. Ela já diz se alguma operação persistiu ou não.
Mas pode não ter persistido por uma série de razões que o próprio EF informação através de exceção, aí só capturando uma exceção para saber.
Sempre capture as exceções esperadas e que pode fazer algo a respeito. Nunca capture por capturar e menos ainda capture Exception
. Leia a documentação para saber todas exceções possíveis neste método. Você não precisa tratar aí todas as exceções. É muito comum em quase todos os códigos que vejo aqui o uso errado de exceções.
Algo assim seria interessante:
public bool IncluirRegistro(Pessoa tEntEF) {
try {
db.Pessoa.Add(tEntEF);
int DadosSalvos = ;
return (db.SaveChanges() > 0);
} catch (DbUpdateException e) {
return false;
}
}
Coloquei no GitHub para referência futura.