VB.Net中从DataGridView生成下一个连续员工ID的问题

嘿，我来帮你排查这个问题～你的代码现在生成ID的逻辑有两个容易踩的小坑，这就是导致异常的原因：

问题根源 #

1. 默认新行被统计在内：DataGridView默认开启AllowUserToAddRows=True，这会在最后一行显示一个空白的新行，dg.Rows.Count会把这一行也算进去。比如你有2条有效记录，Rows.Count实际是3，所以生成的ID是3+1=4，而不是你预期的3。
2. ID不依赖行数的场景失效：如果之后删除过记录，或者员工ID不是严格按行数连续的（比如中间有ID被删除），用行数+1的逻辑会生成重复或不连续的ID，不符合业务需求。

修复方案 #

方案1：基于现有最大ID生成（推荐，适配所有场景）

如果你的员工ID列是整数类型（假设列名为EmployeeID），直接取现有数据里的最大ID再加1，这样不管有没有删除行、有没有新行，都能生成正确的连续ID：

Sub GetEmployeeiD()
    Dim id As Integer = 1
    Dim dg As DataGridView = dgvdata
    
    ' 筛选出所有非新行的有效数据
    Dim validRows = dg.Rows.Cast(Of DataGridViewRow)().Where(Function(r) Not r.IsNewRow)
    
    If validRows.Any() Then
        ' 获取所有有效行的EmployeeID最大值，再加1
        id = validRows.Max(Function(r) Convert.ToInt32(r.Cells("EmployeeID").Value)) + 1
    End If
    
    Me.txtid.Text = id.ToString()
End Sub

方案2：仅排除默认新行（适合无删除操作的简单场景）

如果你的数据不会有删除操作，只是要去掉默认的空白新行，可以简化逻辑：

Sub GetEmployeeiD()
    Dim id As Integer
    Dim dg As DataGridView = dgvdata
    
    ' 计算有效行数：总条数减去1（如果允许添加新行）
    Dim validRowCount = dg.Rows.Count - If(dg.AllowUserToAddRows, 1, 0)
    
    id = If(validRowCount = 0, 1, validRowCount + 1)
    Me.txtid.Text = id.ToString()
End Sub

验证一下 #

比如你现在有2条有效记录，用方案1的话，会取这两条记录的最大ID（假设是2），生成3；用方案2的话，validRowCount是2，生成3，都符合你的预期。

内容的提问来源于stack exchange，提问作者Salman Khan