Skip to content

Modernize C# patterns and optimize method calls #24

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
Copilot wants to merge 5 commits into from
Closed

Modernize C# patterns and optimize method calls #24

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
45b9963
Initial plan
Copilot Feb 9, 2026
eeb0902
Convert fields to properties and modernize C# code patterns
Copilot Feb 9, 2026
a86a0a9
Optimize repeated method calls and add application name constant
Copilot Feb 9, 2026
6aab04d
Address code review feedback
Copilot Feb 9, 2026
422798a
Complete code improvements - all checks passed
Copilot Feb 9, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Binary file added .nuget/nuget.exe
View file
Open in desktop
Binary file not shown.
178 changes: 88 additions & 90 deletions DeadLock/Classes/Language.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,6 +1,4 @@
// ReSharper disable UnassignedField.Global

namespace DeadLock.Classes
namespace DeadLock.Classes
{
/// <summary>
/// Collection of strings that represent the translation of DeadLock into a certain language.
Expand All @@ -9,164 +7,164 @@ public class Language
{
#region Author

public string Author;
public string Comment;
public string Author { get; set; }
public string Comment { get; set; }

#endregion

#region Shared

public string BtnClose;
public string BarItemSettings;
public string BarItemAbout;
public string BtnLicense;
public string BtnClose { get; set; }
public string BarItemSettings { get; set; }
public string BarItemAbout { get; set; }
public string BtnLicense { get; set; }

#endregion

#region Main_Form

//Main form - Menu items:
public string BarFile;
public string BarEdit;
public string BarView;
public string BarTools;
public string BarHelp;
public string BarFile { get; set; }
public string BarEdit { get; set; }
public string BarView { get; set; }
public string BarTools { get; set; }
public string BarHelp { get; set; }

//Main Form - File menu - Items:
public string BarItemOpenFiles;
public string BarItemOpenFolder;
public string BarItemRestart;
public string BarItemExit;
public string BarItemOpenFiles { get; set; }
public string BarItemOpenFolder { get; set; }
public string BarItemRestart { get; set; }
public string BarItemExit { get; set; }

//Main Form - Edit menu - Items:
public string BarItemUnlock;
public string BarItemCopy;
public string BarItemMove;
public string BarItemRemove;
public string BarItemUnlock { get; set; }
public string BarItemCopy { get; set; }
public string BarItemMove { get; set; }
public string BarItemRemove { get; set; }

public string BarItemOwnership;
public string BarItemOwnershipTrue;
public string BarItemOwnershipFalse;
public string BarItemOwnership { get; set; }
public string BarItemOwnershipTrue { get; set; }
public string BarItemOwnershipFalse { get; set; }

public string BarItemRemoveItem;
public string BarItemClearItems;
public string BarItemAutoSizeColumns;
public string BarItemCancelTask;
public string BarItemRemoveItem { get; set; }
public string BarItemClearItems { get; set; }
public string BarItemAutoSizeColumns { get; set; }
public string BarItemCancelTask { get; set; }

//Main Form - View menu - Items:
public string BarItemDetails;
public string BarItemDetails { get; set; }

//Main Form - Help menu - Items:
public string BarItemHelp;
public string BarItemCheckForUpdates;
public string BarItemHomePage;
public string BarItemHelp { get; set; }
public string BarItemCheckForUpdates { get; set; }
public string BarItemHomePage { get; set; }

//Main Form - ListView Items:
public string ClhPath;
public string ClhStatus;
public string ClhOwnership;
public string ClhPath { get; set; }
public string ClhStatus { get; set; }
public string ClhOwnership { get; set; }

public string MsgUnknown;
public string MsgLocked;
public string MsgUnlocked;
public string MsgOperationCancelled;
public string MsgLoading;
public string MsgUnknown { get; set; }
public string MsgLocked { get; set; }
public string MsgUnlocked { get; set; }
public string MsgOperationCancelled { get; set; }
public string MsgLoading { get; set; }

public string MsgSuccessfullyUnlocked;
public string MsgCouldNotUnlock;
public string MsgSuccessfullyUnlocked { get; set; }
public string MsgCouldNotUnlock { get; set; }

public string MsgSuccessfullyCopied;
public string MsgCouldNotCopy;
public string MsgSuccessfullyCopied { get; set; }
public string MsgCouldNotCopy { get; set; }

public string MsgSuccessfullyMoved;
public string MsgCouldNotMove;
public string MsgSuccessfullyMoved { get; set; }
public string MsgCouldNotMove { get; set; }

public string MsgSuccessfullyRemoved;
public string MsgCouldNotRemove;
public string MsgSuccessfullyRemoved { get; set; }
public string MsgCouldNotRemove { get; set; }

//Main Form - ListView Details:
public string ClhFileName;
public string ClhProcessId;
public string ClhFileName { get; set; }
public string ClhProcessId { get; set; }

//Main Form - Status bar:
public string LblVersion;
public string LblVersion { get; set; }

//Main Form - ContextMenu Items:
public string CmiDetails;
public string CmiOpenInVirusTotal;
public string CmiDetails { get; set; }
public string CmiOpenInVirusTotal { get; set; }

//Main Form - ContextMenu Details:
public string CmiKill;
public string CmiOpenFileLocation;
public string CmiProperties;
public string CmiKill { get; set; }
public string CmiOpenFileLocation { get; set; }
public string CmiProperties { get; set; }

public string MsgSuccessfullyKilled;
public string MsgSuccessfullyKilled { get; set; }

//Main Form - NotifyIcon:
public string CmiHideShow;
public string CmiHideShow { get; set; }

//Main Form - Messages:
public string MsgAdministrator;
public string MsgDownloadNewVersion;
public string MsgLatestVersionAlreadyInstalled;
public string MsgAdministrator { get; set; }
public string MsgDownloadNewVersion { get; set; }
public string MsgLatestVersionAlreadyInstalled { get; set; }

#endregion

#region About_Form

//About Form - About:
public string TxtAboutCreated;
public string TxtAboutImages;
public string TxtAboutTheme;
public string TxtAboutCopyright;
public string TxtAboutTranslation;
public string TxtAboutCreated { get; set; }
public string TxtAboutImages { get; set; }
public string TxtAboutTheme { get; set; }
public string TxtAboutCopyright { get; set; }
public string TxtAboutTranslation { get; set; }

#endregion

#region Settings_Form

//Settings Form - General Tab:
public string LblGeneral;
public string ChbAutoUpdate;
public string ChbShowNotifyIcon;
public string ChbStartMinimized;
public string ChbShowAdminWarning;
public string LblGeneral { get; set; }
public string ChbAutoUpdate { get; set; }
public string ChbShowNotifyIcon { get; set; }
public string ChbStartMinimized { get; set; }
public string ChbShowAdminWarning { get; set; }

//Settings Form - Appearance Tab:
public string LblAppearance;
public string LblThemeStyle;
public string LblRememberFormSize;
public string LblShowDetails;
public string LblLanguage;
public string LblAppearance { get; set; }
public string LblThemeStyle { get; set; }
public string LblRememberFormSize { get; set; }
public string LblShowDetails { get; set; }
public string LblLanguage { get; set; }

//Settings Form - Advanced Tab:
public string LblAdvanced;
public string LblAutoRunDeadLock;
public string LblWindowsExplorerIntegration;
public string LblOwnership;
public string LblAdvanced { get; set; }
public string LblAutoRunDeadLock { get; set; }
public string LblWindowsExplorerIntegration { get; set; }
public string LblOwnership { get; set; }

//Settings Form - Buttons:
public string BtnReset;
public string BtnSave;
public string BtnReset { get; set; }
public string BtnSave { get; set; }

//Settings Form - ToggleButtons
public string TbtnOn;
public string TbtnOff;
public string TbtnOn { get; set; }
public string TbtnOff { get; set; }

#endregion

#region NativeMethods

public string MsgCouldNotRestart;
public string MsgCouldNotRegister;
public string MsgCouldNotList;
public string MsgCouldNotListResult;
public string MsgCouldNotRestart { get; set; }
public string MsgCouldNotRegister { get; set; }
public string MsgCouldNotList { get; set; }
public string MsgCouldNotListResult { get; set; }

#endregion

#region ProcessLocker

public string MsgAccessDenied;
public string MsgAccessDenied { get; set; }

#endregion
}
Expand Down
103 changes: 30 additions & 73 deletions DeadLock/Classes/LanguageManager.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -11,18 +11,17 @@ namespace DeadLock.Classes
/// </summary>
internal class LanguageManager
{
#region Variables

private Language _currentLanguage;

#endregion
/// <summary>
/// Gets the current language.
/// </summary>
internal Language CurrentLanguage { get; private set; }

/// <summary>
/// Generate a new LanguageManager.
/// </summary>
internal LanguageManager()
{
_currentLanguage = new Language();
CurrentLanguage = new Language();
}

/// <summary>
Expand All @@ -33,11 +32,9 @@ internal void LoadLanguage(string path)
{
try
{
XmlSerializer serializer = new XmlSerializer(_currentLanguage.GetType());
using (StreamReader reader = new StreamReader(path))
{
_currentLanguage = (Language)serializer.Deserialize(reader);
}
XmlSerializer serializer = new XmlSerializer(CurrentLanguage.GetType());
using StreamReader reader = new StreamReader(path);
CurrentLanguage = (Language)serializer.Deserialize(reader);
}
catch (Exception ex)
{
Expand All @@ -52,69 +49,29 @@ internal void LoadLanguage(string path)
/// <param name="index">The index of the language that should be loaded.</param>
internal void LoadLanguage(int index)
{
XmlSerializer serializer = new XmlSerializer(_currentLanguage.GetType());
using (MemoryStream stream = new MemoryStream())
XmlSerializer serializer = new XmlSerializer(CurrentLanguage.GetType());
using MemoryStream stream = new MemoryStream();
using StreamWriter writer = new StreamWriter(stream);
string res = index switch
{
using (StreamWriter writer = new StreamWriter(stream))
{
string res;
switch (index)
{
case 0:
res = Properties.Resources.nl;
break;
case 2:
res = Properties.Resources.fr;
break;
case 3:
res = Properties.Resources.ger;
break;
case 4:
res = Properties.Resources.ita;
break;
case 5:
res = Properties.Resources.kor;
break;
case 6:
res = Properties.Resources.pl;
break;
case 7:
res = Properties.Resources.rus;
break;
case 8:
res = Properties.Resources.sr;
break;
case 9:
res = Properties.Resources.esp;
break;
case 10:
res = Properties.Resources.swe;
break;
case 11:
res = Properties.Resources.tr;
break;
case 12:
res = Properties.Resources.bg;
break;
default:
res = Properties.Resources.eng;
break;
}
writer.Write(res);
writer.Flush();
stream.Position = 0;
_currentLanguage = (Language)serializer.Deserialize(stream);
}
}
}

/// <summary>
/// Get the current language.
/// </summary>
/// <returns>The current language.</returns>
internal Language GetLanguage()
{
return _currentLanguage;
0 => Properties.Resources.nl,
2 => Properties.Resources.fr,
3 => Properties.Resources.ger,
4 => Properties.Resources.ita,
5 => Properties.Resources.kor,
6 => Properties.Resources.pl,
7 => Properties.Resources.rus,
8 => Properties.Resources.sr,
9 => Properties.Resources.esp,
10 => Properties.Resources.swe,
11 => Properties.Resources.tr,
12 => Properties.Resources.bg,
_ => Properties.Resources.eng
};
writer.Write(res);
writer.Flush();
stream.Position = 0;
CurrentLanguage = (Language)serializer.Deserialize(stream);
}
}
}
Loading