В данном примере вначале генерируется список из 50 элементов со значениями от 0 до 49, после чего из него достаются случайные значения. Если файл уже существует, мы берём следующее, и так, пока не исчерпаем все доступные.
Если производительность критична, то вместо List<Int32>
стоит реализовать другую коллекцию, с возможность быстрого удаления элементов.
При высокой вероятности коллизии и небольшим диапазоном доступных значений, мне кажется, что это оптимальное решение.
Если диапазон будет шире, то эффективнее использовать HashSet
и запоминать уже сгенерированные номера, а так же те, на которых мы получили ошибку - файл уже существует.
Если в качестве целевой папки использовать %temp%, то лучше подойдёт стандартный метод Path.GetTempFileName
.
Наконец, если читабельность имён не критична, я бы вообще заменил рандомный хвост на Guid.NewGuid()
, который гарантирует вам уникальность за счёт стремящейся к нулю вероятности коллизии.
RandomNumbers random = new RandomNumbers(maxValue: 50);
Directory.CreateDirectory(Path.Combine(PathForCreate, "NameFolder"));
for (Int32 i = 0; i < ListFiles.Count; i++)
{
String sourceFileName = ListFiles[i];
String destFileName = Path.Combine(PathForCreate, "NameFolder", Path.GetFileName(ListFiles[i]));
CopyFileWithRandomName(random, destFileName, sourceFileName);
}
private static void CopyFileWithRandomName(RandomNumbers random, String destFileName, String sourceFileName)
{
using (IEnumerator<Int32> randomEnumerator = random.EnumerateUniqueValues().GetEnumerator())
{
while (randomEnumerator.MoveNext())
{
try
{
destFileName += randomEnumerator.Current;
File.Copy(sourceFileName, destFileName, false);
return;
}
catch (IOException ex)
{
const Int32 alreadyExistsErrorCode = -2147024816;
if (ex.HResult != alreadyExistsErrorCode)
throw;
}
}
throw new EndOfStreamException("Все доступные номера заняты.");
}
}
private sealed class RandomNumbers
{
private readonly Random _random = new Random();
private readonly List<Int32> _numbers;
public RandomNumbers(Int32 maxValue)
{
List<Int32> numbers = new List<Int32>(maxValue);
for (Int32 i = 0; i < maxValue; i++)
numbers.Add(i);
_numbers = numbers;
}
public IEnumerable<Int32> EnumerateUniqueValues()
{
Int32 randomIndex = _random.Next(_numbers.Count);
yield return _numbers[randomIndex];
List<Int32> tempList = _numbers.ToList();
tempList.RemoveAt(randomIndex);
while (tempList.Count > 0)
{
randomIndex = _random.Next(_numbers.Count);
yield return tempList[randomIndex];
tempList.RemoveAt(randomIndex);
}
}
}
File.Exist
почему не используете? Или, лучше, просто запоминайте все сгенерированные числа и проверяйте при генерации следующего, что он не повторяется – Андрей NOP 22 янв '18 в 11:39int r; do { r = s.Next(50); } while (!myHashSet.Add(r));
- на выходе имеем неповторяющиеся r. ЗдесьmyHashSet = new HashSet<int>();
– Андрей NOP 22 янв '18 в 12:04