c# - 检查现有文件

Question

我需要一种方法来检查图片是否已经在 lbl_Dias 中。如果没有添加图片，如果是，则转到下一张图片。

这是为了制作一个随机顺序的图像列表，但没有重复。

到目前为止我得到的是这个

protected void DiasShow()
    {
        string[] getFiles = Directory.GetFiles(HttpContext.Current.Server.MapPath("/CSS/Design/Page_Design/Dias/1920x1080/"));

        for (int i = 0; i <= GetFiles.Length; i++)
        {
            Random FindRandom = new Random();
            string RandomFileName = getFiles[FindRandom.Next(getFiles.Length)];

            FileInfo ImageName = new FileInfo(RandomFileName);
            string FileType = ImageName.Name.Substring(ImageName.Name.Length - 4);

            if ((FileType.ToUpper() == ".JPG") || (FileType.ToUpper() == "JPEG"))
            {
                lbl_Dias.Text += "<img src=\"CSS/Design/Page_Design/Dias/1920x1080/" + ImageName.Name + "\" />";
            }
        }
    }

希望大家帮帮忙，我有点股票^^

Answer 1

首先，摆脱 NumberOfImages，因为它毫无意义。 整个 foreach 循环很糟糕，因为您在不需要它的地方遍历集合。

其次，您可以使用 FileInfo 的扩展属性来检查文件的扩展字符串 - 无需子字符串等。

第三，你到底想在这里做什么？

您确实意识到您可能会在循环结束时得到随机重复项，因为您没有从集合中删除使用过的图像。

最后，您根本不需要检查文件是否存在，因为您是从返回文件存在的函数中获取的。

protected void DiasShow()
{
    var mapPath = HttpContext.Current.Server.MapPath("/CSS/Design/Page_Design/Dias/1920x1080/");

    var images =
        Directory.GetFiles(mapPath).Select(
            file => new FileInfo(file)).Where(fi =>
                fi.Extension.EndsWith("jpg", StringComparison.OrdinalIgnoreCase) ||
                fi.Extension.EndsWith("jpeg", StringComparison.OrdinalIgnoreCase)).ToList();

    var rand = new Random();

    while (images.Count > 0)
    {
        var i = rand.Next(images.Count);
        lbl_Dias.Text += "<img src=\"CSS/Design/Page_Design/Dias/1920x1080/" + images[i].Name + "\"/>";
        images.RemoveAt(i);
    }
}