Não existe correto ou errado propriamente dito nesse caso. Existe o indicado e o mais eficiente. Se os campos são inteiros, trate-os como tal. Pode parecer que não tenha qualquer coisa o impedindo de usar os inteiros convertidos para string nessa situação, mas existe uma "regra" em desenvolvimento de sistemas que costumo seguir com muito sucesso: Faça a coisa menos surpreendente. Sempre.
Surpreendente, no caso, deve ser lido como "algo que gere menos surpresa".
Nesse caso, se o campo é inteiro (ou long
) no banco de dados, trate-o como tal no seu código também. Isso gerará menos surpresa a algum outro programador, ou até à você mesmo quando vier resolver um bug no código daqui a alguns anos. Ademais, usar Convert.ToInt32()
no lugar de ToString()
não terá qualquer diferença, mas apenas o fato de você se perguntar se tratar o inteiro como string no seu código é o certo, já mostra que o ideal seria nem precisar fazer essa pergunta e tratar inteiro como o inteiro que é.
Te garanto também que no DataTable
esse campo é inteiro. Se for comparar o ID do GridView com o id no dataset/datatable, pode vir a ter que converter de volta para inteiro de qualquer
Há ainda funcionalidades futuras que possam ser afetadas por essa conversão desnecessária feita hoje. Por exemplo, você pode vir a precisar colocar os Ids dentro de um Hashset
e se forem strings as operações serão mais lentas (não muito, pois o Hashset usa um hash para ser encontrado, mas há perda de performance). Você pode querer ordenar os Ids para saber qual foi criado antes no banco de dados (já que é identity) e a ordenação de strings, além de muito mais lenta, não segue a ordem numérica: "10" < "2"
enquanto que 10 > 2
.
Resumindo: enquanto existem motivos para usar inteiro como inteiro, com vantagens, não existem vantagens nem motivos (no seu caso) de tratá-los como string.
Não pregue peças em si mesmo (e nos outros que usem seu código). Faça o que gere menos surpresa. Se pergunte: O que é o mais esperado? Que um campo inteiro no banco seja tratado como string no seu código, ou que seja tratado como inteiro mesmo?