如何从保存在设置中的stringCollection中删除字符串

Question

我是这个社区的新手。 我有一个问题：我对如何在Windows窗体中保存在运行时创建的控件和代码的工作方式非常满意，但是当我想从stringCollection中删除字符串时遇到了问题。 我使用了方法stringcollection.Remove（“ string”）插入刚刚存储的有效字符串，并且还使用settings.default.save（）保存了所有字符串，但是该字符串并未从字符串集合中删除。 为什么不起作用？ 请有人帮我！ :)

这是我的代码：

public Form1()
{
    InitializeComponent();


    if (Properties.Settings.Default.StringCollection == null)
        Properties.Settings.Default.StringCollection = new System.Collections.Specialized.StringCollection();

}


private void make_BookButtonAndStore(int x, int y, string name)
{
    make_Book(x, y, name);
    Properties.Settings.Default.StringCollection.Add(String.Format("{0};{1};{2}", book1.Location.X, book1.Location.Y, book1.Name));
    Properties.Settings.Default.Save();
}

private void make_Book(int x, int y, string name)
{
    // this code is initializing the book(button)
    book1 = new Button();
    //Image img = button1.Image;
    //book1.Image = img;
    book1.Name = name;
    //book1.Height = img.Height;
    //book1.Width = img.Width;
    book1.Location = new Point(44 + x, 19 + y);
    book1.MouseDown += new MouseEventHandler(book1_MouseDown);
    book1.MouseMove += new MouseEventHandler(book1_MouseMove);
    book1.MouseUp += new MouseEventHandler(book1_MouseUp);
    groupBox1.Controls.Add(book1);
}



void book1_MouseDown(object sender, MouseEventArgs e)
{
    activeControl = sender as Control;
    previousLocation = e.Location;
    Cursor = Cursors.Hand;
}

void book1_MouseMove(object sender, MouseEventArgs e)
{
    if (activeControl == null || activeControl != sender)
        return;

    var location = activeControl.Location;
    location.Offset(e.Location.X - previousLocation.X, e.Location.Y - previousLocation.Y);
    activeControl.Location = location;


}

void book1_MouseUp(object sender, MouseEventArgs e)
{
    activeControl = null;
    Cursor = Cursors.Default;

    Button btnPremuto = (Button)sender;
                Properties.Settings.Default.StringCollection.Remove(previousLocation.X+";"+previousLocation.Y+";"+btnPremuto.Name);
    Properties.Settings.Default.StringCollection.Add(String.Format("{0};{1};{2}", btnPremuto.Location.X, btnPremuto.Location.Y, btnPremuto.Name));
    Properties.Settings.Default.Save();

}

private void Form1_Load(object sender, EventArgs e)
{
    foreach (string line in Properties.Settings.Default.StringCollection)
    {
        if (!String.IsNullOrWhiteSpace(line))
        {
            // The line will be in format x;y;name
            string[] parts = line.Split(';');
            if (parts.Length >= 3)
            {
                int x = Convert.ToInt32(parts[0]);
                int y = Convert.ToInt32(parts[1]);

                make_Book(x, y, parts[2]);
            }
        }
    }
}

Answer 1

我暂时没有时间进行测试，但快速浏览一下，就好像您是从e.Location中的book1_MouseDown获取previousLocation的e.Location book1_MouseDown 。 这将是鼠标的位置，而不是控件的位置？

在我看来，您好像将控件的位置存储在StringCollection ，并且我假设它具有某些尺寸，因此在激发MouseDown时，鼠标可能不在控件的左上角。

我建议您考虑从sender那里获取位置，而不是从e处获取该控件的先前位置。

Answer 2

那里有很多有点奇怪的代码！ 我认为您要通过设置来解决的问题是，将StringCollections保存到设置时会有一些稍微有些奇怪的行为。 尽管您可以在集合中添加/删除内容，然后调用save，但实际上并不会做任何事情。

您可以看到settings对象在属性中具有正确的元素，但是它不起作用。 您还必须重新设置属性，例如：

public class Config
{
    private readonly Settings _settings;
    private readonly ACollectionOfStrings _selectedCategories;

    internal Config(Settings settings)
    {
        _settings = settings;
        if(_settings.SelectedCategories == null)
            _settings.SelectedCategories = new StringCollection();

        _selectedCategories = new ACollectionOfStrings(_settings.SelectedCategories);
    }

    public IEnumerable<string> SelectedCategories
    {
        get { return _selectedCategories; }
    }

    private void ModifySettings(Action<Settings> modification)
    {
        modification(_settings);
        Save();
    }

    private void Save()
    {
        _settings.Save();
    }

    public void AddCategory(string category)
    {
        _selectedCategories.Add(category);
        SaveList();
    }

    private void SaveList()
    {
        ModifySettings(s => s.SelectedCategories = _selectedCategories.List);
    }

    public void Remove(string category)
    {
        _selectedCategories.Remove(category);
        SaveList();
    }
}

我略过的代码有些复杂-ACollectionOfStrings是一个小的包装器类，基本上通过对StringCollection的调用传递，但也是IEnumerable（有点困惑，事实并非如此，而且对String的GetEnumerator调用集合不会返回IEnumerator，因此您还必须将其包装-多么奇怪的类！）