1

Tenho uma tabela com vários registros e um link para exclusão, como eu faço para ao clicar nesse link passar o valor do id do registro em que eu quero deletar ? Código da minha view:

@model IList<Financas.Entidades.Usuario>

@Html.ActionLink("Novo Usuário", "Form")
<table class="table table-hover">
    <thead>
        <tr>
            <th>Id</th>
            <th>Nome</th>
            <th>E-mail</th>
        </tr>
    </thead>
        <tbody>
            @foreach(var usuario in Model)
            {
                <tr>
                    <td>@usuario.Id</td>
                    <td>@usuario.Nome</td>
                    <td>@usuario.Email</td>
                    <td>@Html.ActionLink("Excluir","Excluir","Usuario")</td>
                </tr>
            }
        </tbody>
</table>

Controller:

    public ActionResult Excluir(int? usuario)
    {
            try
            {
                IList<Usuario> dados = usuarioDAO.GetById(usuario);
                return View(dados);

            }
            catch (Exception)
            {

                throw;
            }
        }

2 Respostas 2

0

Vamos tratar primeiro dos erros em seu código.

O erro da pergunta é simples, você não está passando nada como parâmetro à Action. Basta alterar para o seguinte:

   <td>@Html.ActionLink("Excluir","Excluir","Usuario", new{usuario = usuario.Id})</td>

Dessa forma você estará passando um ID para o parâmetro usuario em sua Action.

Agora vamos fazer algumas melhorias em seu código?

A primeira:

Não sei o que está em seu usuarioDAO, mas aparentemente você está apenas listando uma lista de usuários por ID. Em nenhum momento em seu código existe uma opção de excluir.

public ActionResult Excluir(int? usuario)
        {
                try
                {
                    IList<Usuario> dados = usuarioDAO.GetById(usuario);
                    return View(dados);

                }
                catch (Exception)
                {

                    throw;
                }
            }

Segunda:

Se você não possui uma View de confirmação de exclusão, porque deixar a opção de passar um ID null? Se você vai excluir um usuário, tem que ser passado algum dado, ou seja, você pode alterar o parâmetro de int? usuario para int usuario. Dessa forma estará garantindo que sempre terá um valor.

Terceira:

Eu mudaria sua lógica toda apenas por isso:

public ActionResult Excluir(int id)
            Usuario usuario= db.Usuarios.Find(id);

            if(usuario == null) return new HttpStatusCodeResult(HttpStatusCode.BadRequest);//Um erro aqui, pois o usuário não foi encontrado

            db.Usuarios.Remove(usuario);
            db.SaveChanges();
            return RedirectToAction("Indice");
}
3
  • Quem faz q exclusao é minha classe DAO, onde implementei: public void excluir(Usuario usuario) { context.Usuario.Remove(usuario); context.saveChanges(); }
    – Desalex
    Commented 21/06/2016 às 13:00
  • @Desalex Não existe razão para isso. Esse tipo de implementação possui várias respostas aqui no site, basta procurar para entender melhor. Entretanto, o foco da pergunta não é essa. Basta usar apenas a solução postada no início da resposta que irá resolver seu problema, isso é, levando em consideração que sua classe DAO esteja correta.
    – Randrade
    Commented 21/06/2016 às 13:06
  • entendi, só quis dizer quer quem faz o controle de inserção e deleção da minha aplicação é a camada DAO. Mas obrigado pelo esclarecimento
    – Desalex
    Commented 22/06/2016 às 1:23
0

Só passar assim:

Html.ActionLink("Excluir", "Excluir", "Usuario", new {id = usuario.Id}, null)
10
  • fiz do jeito que falou, porem ainda vem valor null para o meu controller...
    – Desalex
    Commented 21/06/2016 às 3:06
  • passa a assinatura da action no controller Commented 21/06/2016 às 3:07
  • editei e coloquei o codigo do controller no post
    – Desalex
    Commented 21/06/2016 às 3:10
  • muda para public ActionResult Excluir(int id) Commented 21/06/2016 às 3:11
  • O parametro passado chama Id. Então você tem que ter um parametro chamado Id na action para o mvc fazer o bind. e eu tirei o nullable do tipo. Commented 21/06/2016 às 3:12

Você deve fazer log-in para responder a esta pergunta.

Esta não é a resposta que você está procurando? Pesquise outras perguntas com a tag .