Question

我为列表视图项创建提供了非常简单的逻辑。我将所有col头名称从dataGridView存储到 ColNamesForm1 数组中，然后仅比较最后5个字符是否与（num）或（cat）字符串。对于这两个选项，我将不同的图片添加到listview lvMoveFrom 中，使用存储在静态类 OtherFunctions 中的 Populate 函数。

但是我的代码出了问题，因为在最后一次迭代之后它会附加最后一列的图像 - 如果第一列是（num）而最后一列是（cat），listview中的图像都是相同的 - 来自cat图像。

如何解决此问题？我想知道为每个列创建新的ImageList对象，但我不知道，我怎么能动态地做到这一点，例如使用循环索引i。

拜托，您能帮忙解决我的问题。

private void Form1_Load(object sender, EventArgs e)
{
    for (int i = 0; i < ColNamesForm1.Length; i++)
    {
        if (ColNamesForm1[i].ToString().Substring(0, 4).ToUpper() != "COL_" && Regex.IsMatch(ColNamesForm1[i].ToString().Substring(4, 1), @"^\d+$") == false)
        {
            if (ColNamesForm1[i].ToString().Substring(ColNamesForm1[i].ToString().Length - 5) == "(num)")
            {
                OtherFunctions.Populate(lvMoveFrom, ColNamesForm1[i].ToString(), @"C:\pictures\num.png");
            }
            else
            {
                OtherFunctions.Populate(lvMoveFrom, ColNamesForm1[i].ToString(), @"C:\pictures\cat.png");
            }
        }
    }
}

public static void Populate(ListView lv, string itemName, string pathToPicture)
{
    ImageList img = new ImageList();
    img.ImageSize = new Size(30, 30);
    img.Images.Add(Image.FromFile(pathToPicture));
    lv.SmallImageList = img;
    lv.Items.Add(itemName, 0);
}

Answer 1

问题

基本上就是这样：

ImageList img = new ImageList();
img.ImageSize = new Size(30, 30);
img.Images.Add(Image.FromFile(pathToPicture));
lv.SmallImageList = img;
lv.Items.Add(itemName, 0);

正在向ListView添加新图片列表。你每次都传递了同样的ListView（所以你有效地覆盖了它）。其次，行：

lv.Items.Add(itemName, 0);

第二个参数是图像列表中的索引（您指定给ListView）。所以给它0会要求ListView基本上从lv.SmallImageList[0]（伪代码）中选择图像。

解决方案

要删除覆盖，我将图像设置逻辑拉出Populate并将其放回主方法中。我将分解设置逻辑：

ImageList img = new ImageList();
img.ImageSize = new Size(30, 30);

var paths = new List<string> { @"C:\pictures\num.png", @"C:\pictures\cat.png" };
paths.ForEach(path => img.Images.Add(MediaTypeNames.Image.FromFile(path)));

lvMoveFrom.SmallImageList = img;

我将所有图片路径放入List<string>，然后使用LINQ ForEach迭代每个图像路径，将其添加到ImageList img。与原始代码没有区别，除了我将所有图像添加到ListView而我只执行一次。

然后，为了让你的代码更容易理解，我做了一些简单的重构。

首先是if if语句：

if (ColNamesForm1[i].ToString().Substring(0, 4).ToUpper() != "COL_" && Regex.IsMatch(ColNamesForm1[i].ToString().Substring(4, 1), @"^\d+$") == false)

要：

if (ColNamesForm1[i].ToString().Substring(0, 4).ToUpper() == "COL_"
                || Regex.IsMatch(ColNamesForm1[i].ToString().Substring(4, 1), @"^\d+$"))
{
    continue;
}

这几乎就像一个保护条款，它说如果我们不满足这些最低条件，那么转到下一个项目。

然后我通过减少重复来简化你的方法执行：

if (ColNamesForm1[i].ToString().Substring(ColNamesForm1[i].ToString().Length - 5) == "(num)")
{
     OtherFunctions.Populate(lvMoveFrom, ColNamesForm1[i].ToString(), @"C:\pictures\num.png");
}
else
{
      OtherFunctions.Populate(lvMoveFrom, ColNamesForm1[i].ToString(), @"C:\pictures\cat.png");
}

要：

var image = ColNamesForm1[i].ToString().EndsWith("(num)")
                ? 0 // corresponds with the position of the image in the ImageList
                : 1;

OtherFunctions.Populate(lvMoveFrom, ColNamesForm1[i].ToString(), image);

最后，您会看到我更改了您的Populate方法。首先，我们使用图像预先填充ListView，然后使用该三元运算符选择要显示的图像索引。

所有代码都是：

private void Form1_Load(object sender, EventArgs e)
{
    ImageList img = new ImageList();
    img.ImageSize = new Size(30, 30);

    var paths = new List<string> { @"C:\pictures\num.png", @"C:\pictures\cat.png" };
    paths.ForEach(path => img.Images.Add(Image.FromFile(path)));

    lvMoveFrom.SmallImageList = img;

    for (int i = 0; i < ColNamesForm1.Length; i++)
    {
        if (ColNamesForm1[i].ToString().Substring(0, 4).ToUpper() == "COL_"
            || Regex.IsMatch(ColNamesForm1[i].ToString().Substring(4, 1), @"^\d+$"))
        {
            continue;
        }

        var image = ColNamesForm1[i].ToString().EndsWith("(num)")
                        ? 0 // corresponds with the position of the image in the ImageList
                        : 1;


        OtherFunctions.Populate(lvMoveFrom, ColNamesForm1[i].ToString(), image);
    }
}

public static void Populate(ListView lv, string itemName, int imageIndex)
{

    lv.Items.Add(itemName, imageIndex);
}

现在你可以进一步简化：

private void Form1_Load(object sender, EventArgs e)
{
    ImageList img = new ImageList();
    img.ImageSize = new Size(30, 30);

    var paths = new List<string> { @"C:\pictures\num.png", @"C:\pictures\cat.png" };
    paths.ForEach(path => img.Images.Add(Image.FromFile(path)));

    lvMoveFrom.SmallImageList = img;

    for (int i = 0; i < ColNamesForm1.Length; i++)
    {
        if (ColNamesForm1[i].ToString().Substring(0, 4).ToUpper() == "COL_"
            || Regex.IsMatch(ColNamesForm1[i].ToString().Substring(4, 1), @"^\d+$"))
        {
            continue;
        }

        var image = ColNamesForm1[i].ToString().EndsWith("(num)")
                        ? 0 // corresponds with the position of the image in the ImageList
                        : 1;

        lvMoveFrom.Items.Add(ColNamesForm1[i].ToString(), image);    
    }
}

在循环中创建ImageList对象

1 个答案:

问题

解决方案