Использование цикла foreach с картинками

Question

Я пытаюсь сделать свой код максимально коротким, и я намерен рандомизировать изображения из выбора из 6 изображений для 3 различных графических блоков через определенные промежутки времени.Вместо того, чтобы трижды копировать и вставлять переключатель и регистр для каждой картинки, я пытаюсь вместо этого изменить один цикл foreach.Я очень плохо знаком с C # и окнами, поэтому любая помощь приветствуется.РЕДАКТИРОВАТЬ: Извините за плохую формулировку, но проблема заключалась в том, что программа говорит, что элемент не является допустимым Picturebox, и мне интересно, почему и как это исправить, если это возможно, но спасибо за альтернативные решения.

private void timer1_Tick(object sender, EventArgs e)
    {

        Random random = new Random();
        int picture = random.Next(1, 7);
        var pictures = new List<PictureBox> { pictureBox1, pictureBox2, pictureBox2 };
        foreach (PictureBox element in pictures)
        {
            switch (picture)
            {
                case 1:
                    this.element.Image = Image.FromFile("C:\\Users\\seanb\\OneDrive\\Pictures\\cherry.jpg");
                    break;
                case 2:
                    this.element.Image = Image.FromFile("C:\\Users\\seanb\\OneDrive\\Pictures\\bell.jpg");
                    break;
                case 3:
                    this.element.Image = Image.FromFile("C:\\Users\\seanb\\OneDrive\\Pictures\\lemon.jpg");
                    break;
                case 4:
                    this.element.Image = Image.FromFile("C:\\Users\\seanb\\OneDrive\\Pictures\\orange.jpg");
                    break;
                case 5:
                    this.element.Image = Image.FromFile("C:\\Users\\seanb\\OneDrive\\Pictures\\star.jpg");
                    break;
                case 6:
                    this.element.Image = Image.FromFile("C:\\Users\\seanb\\OneDrive\\Pictures\\skull.jpg");
                    break;
            }
        }

Answer 1

private void timer1_Tick(object sender, EventArgs e)
        {
            Random random = new Random();
            List<string> picPaths = new List<string>();
            picPaths.Add("C:\\Users\\seanb\\OneDrive\\Pictures\\cherry.jpg");
            picPaths.Add("C:\\Users\\seanb\\OneDrive\\Pictures\\bell.jpg");
            picPaths.Add("C:\\Users\\seanb\\OneDrive\\Pictures\\lemon.jpg");
            picPaths.Add("C:\\Users\\seanb\\OneDrive\\Pictures\\orange.jpg");
            picPaths.Add("C:\\Users\\seanb\\OneDrive\\Pictures\\star.jpg");
            picPaths.Add("C:\\Users\\seanb\\OneDrive\\Pictures\\skull.jpg");

            pictureBox1.Image = Image.FromFile(picPaths[random.Next(picPaths.Count)]);
            pictureBox2.Image = Image.FromFile(picPaths[random.Next(picPaths.Count)]);
            pictureBox3.Image = Image.FromFile(picPaths[random.Next(picPaths.Count)]);
        }

Answer 2

Подход с Dictionary

int picture = new Random().Next(1, 7);
Dictionary<int, string> dictionary = new Dictionary<int, string>()
{
    {1, "cherry.jpg"},
    {2, "bell.jpg"},
    {3, "lemon.jpg"},
    {4, "orange.jpg"},
    {5, "star.jpg"},
    {6, "skull.jpg"}
};
string res = "default.jpg";
var pictures = new List<PictureBox> { pictureBox1, pictureBox2, pictureBox2 };
string path = System.IO.Path.Combine("C:\\Users\\seanb\\OneDrive\\Pictures\\" + dictionary.TryGetValue(picture, out res));
pictures.ForEach(x => x.Image = Image.FromFile(path));

Answer 3

Итак, у вас есть фотографии:

static Image[] s_Images = new string[] {
     "cherry.jpg",
     "bell.jpg",
     "lemon.jpg", 
     "orange.jpg",
     "star.jpg", 
     "skull.jpg"}
  .Select(file => Path.Combine(@"C:\Users\seanb\OneDrive\Pictures", file))
  .Select(file => Image.FromFile(file)) 
  .ToArray();  

static Random random = new Random();

И вы хотите назначить эти изображения для графических блоков случайным образом:

private void timer1_Tick(object sender, EventArgs e) {
  foreach (PictureBox box in new PictureBox[] { pictureBox1, pictureBox2, pictureBox2 }) {
    box.Image = s_Images[random.Next(s_Images.Length)];
  }
}

Answer 4

Random random = new Random();
int picture = random.Next(1, 7);
var pictureBoxs = new List<PictureBox> { pictureBox1, pictureBox2, pictureBox2 };
string basePath = "C:\\Users\\seanb\\OneDrive\\Pictures\\";
string[] pictures = new string[]{ "cherry.jpg", "bell.jpg", "lemon.jpg", "orange.jpg", "star.jpg", "skull.jpg" };
foreach (PictureBox element in pictureBoxs)
{
    this.element.Image = Image.FromFile(basePath+ pictures[picture]);
}

Answer 5

Как насчет использования словаря , например:

Dictionary<int, Image> dictionary = new Dictionary<int, Image>()
{
    {1,  Image.FromFile("C:\\Users\\seanb\\OneDrive\\Pictures\\cherry.jpg")},
    {2,  Image.FromFile("C:\\Users\\seanb\\OneDrive\\Pictures\\bell.jpg")},
};

dictionary.TryGetValue(picture, out value);
if (value != null)
{
    this.element.Image = value;
}