Для многих IF's. Есть ли способ лучше?

Question

У меня есть этот код:

If CheckBox1.Checked = True Then 
      My.Computer.FileSystem.RenameFile("C:\Users\mario\Desktop\Dominio\1\Pc - S.txt", "Pc - S - A.txt") 
      'Sleep(2000) ' to sleep for 2 second 
      System.Threading.Thread.Sleep(2000) 
      My.Computer.FileSystem.RenameFile("C:\Users\mario\Desktop\Dominio\1\Pc - S - A.txt", "Pc - S.txt") 
     End If 

     If CheckBox2.Checked = True Then 
      My.Computer.FileSystem.RenameFile("C:\Users\mario\Desktop\Dominio\2\Pc - S.txt", "Pc - S - A.txt") 
      'Sleep(2000) ' to sleep for 2 second 
      My.Computer.FileSystem.RenameFile("C:\Users\mario\Desktop\Dominio\2\Pc - S - A.txt", "Pc - S.txt") 
     End If 

И это продолжается еще 24, если х. Мне нужно, чтобы они изменили имя в одно и то же время, а затем подождали две секунды и снова изменили его все время. Будет ли у меня 48 (24 + 24) IF?

Объяснение:

Im создавая супер простой способ shuting вниз компьютеры в моей школе. Вот моя идея (надеюсь, что вы можете понять)

я поставил программу на каждом компьютере в школе (в Regestry при старте), который выполняет следующие действия: (Atencion на самом деле не в языке программирования)

для (1) {

sleep 1000  // wait 1 sec 

if.FILE.EXISTS(\\domain\folder1\shutdown.txt) then{ 

    do: shutdown pc // if file exist it shutdowns the pc 

} 

}

Так что это делает, если есть файл в определенной директории в домене, shutsdown ПК. Я делаю это для 24 шт, поэтому мне нужно отключить их индивидуально, поэтому много папок. Так что я мог бы просто переименовать файлы вручную, но для 24 шт. Это большая работа. Поэтому я разрабатываю заявку, которая спрашивает меня, какие ПК я хочу отключить, а затем:

1: Переименовать файл.

2: Подождите 2 или более секунд (так что компьютеры в сети есть время, чтобы проверить, если есть такой файл)

3: переименовать файл обратно так, когда компьютер снова перезагрузится оно не выключится.

Вы получили то, что я пытаюсь сделать? И да, есть более ортодоксальные способы сделать это. но это действительно просто, и это скорее испытание, чем что-либо. (Позже мы будем использовать более подходящие способы сделать это и сделать другие варианты по сети)

Answer 1

С той лишь разницей, что единственное число в пути к файлу, почему бы не сделать что-то вроде этого?

var dict = new Dictionary<CheckBox, string>(); 
dict.Add(CheckBox1, "1"); 
dict.Add(CheckBox2, "2"); 
//etc 

foreach (var kvp in dict) 
{ 
    if(kvp.Key.Checked == true) 
    { 
     My.Computer.FileSystem.RenameFile(String.Format("C:\Users\mario\Desktop\Dominio\{0}\Pc - S.txt", kvp.Value), "Pc - S - A.txt") 
    } 
} 

Мы создаем общий словарь для хранения ассоциации флажков в строке/номер, а затем цикл по словарю, проверяя значения. Мы используем строку формата (вместо этого вы можете использовать конкатенацию строк), чтобы вставить строку/число в строку пути к файлу.

Обратите внимание, что это на C#, поскольку я не программист VB. Синтаксис, вероятно, будет несколько отличаться.

Answer 2

Поскольку вы намерены удаленно отключать ПК, почему бы не закрыть их из своего кода, а не хакеры «создать файлы, а затем подождать 2 секунды»?

Опять же, это C#, но перевод должен быть достаточно прост:

var ComputerNames = new List<string>() {"Computer1", "Computer2", "Computer3"}; 

foreach(var pc in ComputerNames) 
{ 
    Process.Start("shutdown","-s -m \\\\" + pc); 
}