C#删除下拉列表选中项失败，求代码问题排查

嘿，我一眼就发现你代码里的几个关键问题，这就是你删除操作失败的原因：

1. 误用RemoveAt方法 #

List<T>.RemoveAt()的参数是整数类型的索引值，用来指定要删除元素在列表中的位置，但你却传入了一个lambda表达式b => b.id == id，这完全不符合该方法的用法，编译器应该已经抛出错误了吧？

如果你想根据条件（比如book的id）删除元素，有两种正确的方式：

• 使用RemoveAll()：它接受一个谓词（lambda表达式），会删除所有符合条件的元素
• 先找到目标元素的索引/对象，再调用RemoveAt()或Remove()

2. 逻辑矛盾：删除后修改元素属性 #

你在删除book之后，又去修改这个book的title、year等属性——这完全没有意义，因为这个book已经从catalogueInstance.books列表中移除了，修改它不会对列表产生任何影响，反而和你“删除”的目标相悖。

3. 潜在的空引用问题 #

你声明了Catalogue catalogueInstance;但没有看到初始化代码，如果这个实例是null，调用catalogueInstance.books时会直接抛出空引用异常，导致整个操作失败。你需要确保在执行删除前，catalogueInstance已经被正确初始化（比如从JSON文件中读取并反序列化）。

修正后的代码示例 #

下面是调整后的代码，我们用两种方式实现正确的删除逻辑：

方式一：使用RemoveAll删除符合条件的元素 #

public partial class DeleteBook : System.Web.UI.Page 
{
    // 假设FILENAME是你定义的常量路径
    private const string FILENAME = "your-catalogue-path.json";
    Catalogue catalogueInstance;

    // 建议在Page_Load中初始化catalogueInstance
    protected void Page_Load(object sender, EventArgs e)
    {
        if (!IsPostBack)
        {
            // 从文件读取并反序列化，避免空引用
            if (File.Exists(FILENAME))
            {
                string json = File.ReadAllText(FILENAME);
                catalogueInstance = JsonConvert.DeserializeObject<Catalogue>(json);
            }
            else
            {
                // 如果文件不存在，初始化一个空的Catalogue
                catalogueInstance = new Catalogue { books = new List<Book>() };
            }
        }
    }

    protected void btnDelete_Click(object sender, EventArgs e) 
    {
        if (!int.TryParse(txtID.Text, out int id))
        {
            txtSummary.Text = "请输入有效的Book ID！";
            return;
        }

        // 使用RemoveAll删除所有id匹配的元素，返回删除的数量
        int deletedCount = catalogueInstance.books.RemoveAll(b => b.id == id);
        
        if (deletedCount > 0)
        {
            // 序列化并保存到文件
            string jsonText = JsonConvert.SerializeObject(catalogueInstance);
            File.WriteAllText(FILENAME, jsonText);
            txtSummary.Text = $"成功删除 {deletedCount} 本ID为 {id} 的书籍！{Environment.NewLine}";
        }
        else
        {
            txtSummary.Text = $"未找到ID为 {id} 的书籍！{Environment.NewLine}";
        }
    } 
}

方式二：先找到元素再删除（适合确保唯一id的场景） #

protected void btnDelete_Click(object sender, EventArgs e) 
{
    if (!int.TryParse(txtID.Text, out int id))
    {
        txtSummary.Text = "请输入有效的Book ID！";
        return;
    }

    // 找到目标book对象
    Book targetBook = catalogueInstance.books.FirstOrDefault(b => b.id == id);
    
    if (targetBook != null)
    {
        // 从列表中移除该对象
        catalogueInstance.books.Remove(targetBook);
        // 保存修改
        string jsonText = JsonConvert.SerializeObject(catalogueInstance);
        File.WriteAllText(FILENAME, jsonText);
        txtSummary.Text = $"Book ID为 {id} 的书籍已从目录中删除！{Environment.NewLine}";
    }
    else
    {
        txtSummary.Text = $"未找到ID为 {id} 的书籍！{Environment.NewLine}";
    }
}

另外，我还添加了int.TryParse来处理用户输入无效的情况，避免因输入非数字导致的异常，同时在Page_Load中初始化catalogueInstance，解决潜在的空引用问题。

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