Пишу для себя программу, которая разгребает каталог(например: рабочий стол) и раскидывает файлы по каждому формату в заданный пользователем каталог (например: jpg кидает в папку "Изображения", mp3 в папку "музыка", zip и rar в каталог архивы и т.д.). Программа работает, для полного завершения осталось написать сервис чтобы работала автономно (например как антивирус). Но для того чтобы приступить к разработке сервиса, хочу попросить Вас сделать код-ревью, т.к. показать, к сожалению, некому. Все коллеги разрабатывают ETL и далеки от данной тематике. Буду Вам очень признателен, т.к. очень хочу быть профессиональным разработчиком.
Класс, где хранятся все параметры:
namespace Transporter
{
public class Data
{
public Data()
{
string fileName = settingsFile;
if (File.Exists(fileName) && !File.ReadAllText(fileName).Equals("") && File.ReadAllText(fileName) != null)
{
XElement xElem2 = XElement.Load(fileName);
formats = xElem2.Descendants("item")
.ToDictionary(x => (string)x.Attribute("format"), x => (string)x.Attribute("path"));
}
else
{
formats = new Dictionary<string, string>();
}
}
private static string settingsFile = "settings.xml";
private readonly Dictionary<string, string> formats;
public Dictionary<string, string> Formats
{
get { return formats; }
}
private string path = null;
public string Path
{
get { return path; }
set { path = value; }
}
public void Add(string format, string path)
{
if (!formats.ContainsKey(format))
formats.Add(format, path);
else
{
formats.Remove(format);
formats.Add(format, path);
}
}
public void Save()
{
string fileName = settingsFile;
if (File.Exists(fileName)) File.Delete(fileName);
using (FileStream fileStream = new FileStream(fileName, FileMode.Create))
{
XElement xElem = new XElement
(
"items",
formats.Select(x => new XElement("item", new XAttribute("format", x.Key), new XAttribute("path", x.Value)))
);
xElem.Save(fileStream);
fileStream.Close();
}
}
public void RemoveSettingsFile()
{
if (File.Exists(settingsFile)) File.Delete(settingsFile);
}
}
public class item
{
[XmlAttribute]
public string path;
[XmlAttribute]
public string format;
}
}
Класс, который обрабатывает параметры класса Data:
namespace Transporter
{
class Handler
{
private Data data = new Data();
public void AddFormat(string format,string path)
{
data.Add(format, path);
}
public void AddPath(string path)
{
data.Path = path;
}
public void DelAllFormats()
{
data.Formats.Clear();
data.RemoveSettingsFile();
}
public string GetParameters()
{
StringBuilder builder = new StringBuilder();
builder.AppendLine("Исходный каталог:");
builder.AppendLine(data.Path == null ? "исходный каталог не указан" : data.Path);
builder.AppendLine();
builder.AppendLine("Форматы файлов и каталоги, куда файлы будут перенесены:");
if (data.Formats.Count == 0)
builder.AppendLine("Форматы файлов и каталоги не указаны");
else
{
foreach (var format in data.Formats)
{
builder.AppendLine(string.Format("{0} => {1}", format.Key, format.Value));
}
}
return builder.ToString();
}
public void MoveFiles(CancellationToken token)
{
while (!token.IsCancellationRequested)
{
try
{
var files = Directory.GetFiles(data.Path);
foreach (string file in files)
{
foreach (var format in data.Formats)
{
var fileInfo = new FileInfo(file);
if (fileInfo.Extension.ToLower().Contains(format.Key.ToLower()))
{
fileInfo.MoveTo(format.Value + @"\" + DateTime.Now.ToString("yyyyMMdd_hhmmss_") + fileInfo.Name);
}
}
}
Thread.Sleep(5000);
}
catch (IOException) { }
catch (KeyNotFoundException) { }
}
}
public void SaveSetting()
{
data.Save();
}
public bool checkParametrsForStart()
{
return data.Path != null && data.Formats.Count > 0;
}
}
}
Форма:
namespace Transporter
{
public partial class Transporter : Form
{
private Handler handler = new Handler();
private CancellationTokenSource cancellationTokenSource;
public Transporter()
{
InitializeComponent();
StringBuilder builder = new StringBuilder();
builder.AppendLine("Для начала работы укажите:");
builder.AppendLine("1) исходный каталог - откуда бы Вы хотели перенести файлы;");
builder.AppendLine("2) форматы файлов с каталогами - куда бы Вы xотели переместить файлы.");
builder.AppendLine();
builder.AppendLine("Кнопка 'Start' запускает перенос файлов.");
builder.AppendLine("Кнопка 'Stop' останавливает перенос файлов.");
builder.AppendLine("Кнопки 'Add' добавлют параметры.");
builder.AppendLine("Кнопка 'Reset' сбрасывает параметры переноса.");
builder.AppendLine("Кнопка 'View' выводит текущие параметры в консоль.");
builder.AppendLine("Кнопка 'Clear' очищает консоль.");
builder.AppendLine("Кнопка 'Save' сохраняет текущие параметры.");
builder.AppendLine();
builder.AppendLine("Автор программы: Rumeet94 (https://vk.com/evgengorb)");
tbMessage.Text = builder.ToString();
}
private void BtnAddDiir_Click(object sender, System.EventArgs e)
{
if (Regex.IsMatch(tbDir.Text, @"^.:\w*", RegexOptions.IgnoreCase))
{
handler.AddPath(tbDir.Text);
tbMessage.Text = "Путь исходного каталога успешно добавлен";
}
else
{
tbMessage.Text = "Путь исходного каталога указан некорректно";
}
}
private void BtnAddFormatAndDir_Click(object sender, System.EventArgs e)
{
if (Regex.IsMatch(tbFormatDir.Text, @"^.:\w*", RegexOptions.IgnoreCase) && !tbFormat.Text.Equals(""))
{
handler.AddFormat(tbFormat.Text, tbFormatDir.Text);
tbMessage.Text = "Формат и путь каталога для переноса файлов успешно добавлены";
}
else
{
tbMessage.Text = "Не указан формат или путь каталога для переноса файлов указан некорректно";
}
}
private void BtnStop_Click(object sender, System.EventArgs e)
{
tbMessage.Text = "Перенос файлов остановлен.";
cancellationTokenSource?.Cancel();
tbDir.ReadOnly = false;
tbFormat.ReadOnly = false;
tbFormatDir.ReadOnly = false;
btnStart.Enabled = true;
btnAddDiir.Enabled = true;
btnAddFormatAndDir.Enabled = true;
btnResPar.Enabled = true;
}
private void BtnStart_Click(object sender, System.EventArgs e)
{
if (handler.checkParametrsForStart()) {
tbDir.ReadOnly = true;
tbFormat.ReadOnly = true;
tbFormatDir.ReadOnly = true;
btnStart.Enabled = false;
btnAddDiir.Enabled = false;
btnAddFormatAndDir.Enabled = false;
btnResPar.Enabled = false;
cancellationTokenSource = new CancellationTokenSource();
tbMessage.Text = "Выполняется перенос файлов";
Task.Run(() => handler.MoveFiles(cancellationTokenSource.Token));
}
else
{
tbMessage.Text = "Невозможно запустить перенос файлов, т.к. исходный каталог " +
"или форматы и каталоги для переноса файлов не указаны";
}
}
private void BtnViewPar_Click(object sender, System.EventArgs e)
{
tbMessage.Text = handler.GetParameters();
}
private void BtnResPar_Click(object sender, System.EventArgs e)
{
handler.DelAllFormats();
}
private void BtnClearCon_Click(object sender, System.EventArgs e)
{
tbMessage.Text = "";
}
private void BtnSave_Click(object sender, System.EventArgs e)
{
handler.SaveSetting();
}
}
}