Есть следующий код:
public AttachedFile SaveAttachment(NewAttachment attachment)
{
var tmpFile = Path.GetTempFileName();
try
{
using (var f = File.Open(tmpFile, FileMode.Append))
{
attachment.Stream.CopyTo(f);
}
string filePath = string.Empty;
string dst = string.Empty;
do
{
filePath = generatePathForAttachment();
dst = Path.Combine(archiveDirectory, filePath);
} while (File.Exists(Path.Combine(archiveDirectory, filePath)));
Directory.CreateDirectory(Path.GetDirectoryName(dst));
var attachedFile = new AttachedFile
{
OriginalTitle = attachment.OriginalTitle,
Path = filePath,
Size = attachment.Stream.Length,
AuthorId = attachment.AuthorId,
Created = DateTime.Now
};
File.Move(tmpFile, dst);
_context.Set<AttachedFile>().Add(attachedFile);
_context.SaveChanges();
return attachedFile;
}
catch (Exception)
{
throw;
}
}
его назначение сохранение файла на сервере, код рабочий.
Помогите модифицировать данный код что бы свести последствия возникающих Exception
к минимуму, т.е. мне необходимо что бы выполнилось либо все либо ничего.
Моей фантазии хватило только на оборачивании каждого проблемного участка в try
, т.е. что то типа:
try
{
if (!Directory.Exists(Path.GetDirectoryName(dst)))
Directory.CreateDirectory(Path.GetDirectoryName(dst));
}
catch(Exception)
{
//обработка исключения
}
try
{
if (File.Exists(dst))
File.Move(dst, src);
}
catch(Exception)
{
//обработка исключения
}
ну и т.д., но я не уверен в правильности подхода. Был бы премного благодарен за советы.
Реализация метода генерации пути для файла
private string generatePathForAttachment(params string[] args)
{
var dstPath = string.Empty;
var srcString = string.Empty;
if(args.Length==0)
{
srcString = DateTime.Now.ToString();
}
else
{
srcString = string.Concat(args.Select(x=>x));
}
var hash = getHashString(srcString);
dstPath = Path.Combine(hash.Substring(0, 2), hash.Substring(2, 2), hash.Substring(4));
return dstPath;
}
private string getHashString(string s)
{
string hash = string.Empty;
//переводим строку в байт-массим
byte[] bytes = Encoding.Unicode.GetBytes(s);
//создаем объект для получения средст шифрования
MD5CryptoServiceProvider CSP =
new MD5CryptoServiceProvider();
//вычисляем хеш-представление в байтах
byte[] byteHash = CSP.ComputeHash(bytes);
//формируем одну цельную строку из массива
foreach (byte b in byteHash)
hash += string.Format("{0:x2}", b);
return hash;
}
Фактически сервером выступает виртуальная машина, доступ к каталогу по мимо программы есть только у меня(как у разработчика) и администратор(но он туда не ходит)