From 2d213063f0bf1aa8c0bc07897b525bea104d6762 Mon Sep 17 00:00:00 2001 From: MaikyM <51415805+MaikyM@users.noreply.github.com> Date: Fri, 12 Jul 2019 15:51:31 -0600 Subject: [PATCH] Optimized aa5c83c26a9df333a8deb6dd1550b33ceff5d3bb, a lot of code was unnecessary u.u --- FModel/Forms/SearchFiles.cs | 82 +++++-------------------------------- 1 file changed, 10 insertions(+), 72 deletions(-) diff --git a/FModel/Forms/SearchFiles.cs b/FModel/Forms/SearchFiles.cs index ef8798ec..e002bbb9 100644 --- a/FModel/Forms/SearchFiles.cs +++ b/FModel/Forms/SearchFiles.cs @@ -166,52 +166,21 @@ namespace FModel.Forms { for (int i = 0; i < _myInfos.Count; i++) { + bool checkSearch = false; string[] searchTextMultiple = textBox1.Text.Trim().Split(' '); if (searchTextMultiple.Length > 1) { - bool checkSearch = false; for (int s = 0; s < searchTextMultiple.Length; ++s) { checkSearch = Utilities.CaseInsensitiveContains(_myInfos[i].FileName, searchTextMultiple[s]); if (!checkSearch) break; } - - if (checkSearch) - { - if (_myInfos[i].FileName.Contains(".uasset") || _myInfos[i].FileName.Contains(".uexp") || _myInfos[i].FileName.Contains(".ubulk")) - { - if (!_myFilteredInfosDict.ContainsKey(_myInfos[i].FileName.Substring(0, _myInfos[i].FileName.LastIndexOf(".", StringComparison.Ordinal)))) - { - _myFilteredInfosDict.Add(_myInfos[i].FileName.Substring(0, _myInfos[i].FileName.LastIndexOf(".", StringComparison.Ordinal)), ThePak.CurrentUsedPak); - - _fileName = _myInfos[i].FileName.Substring(0, _myInfos[i].FileName.LastIndexOf(".", StringComparison.Ordinal)); - _myFilteredInfos.Add(new FileInfoFilter - { - FileName = _fileName, - PakFile = ThePak.CurrentUsedPak, - }); - } - } - else - { - if (!_myFilteredInfosDict.ContainsKey(_myInfos[i].FileName)) - { - _myFilteredInfosDict.Add(_myInfos[i].FileName, ThePak.CurrentUsedPak); - - _fileName = _myInfos[i].FileName; - _myFilteredInfos.Add(new FileInfoFilter - { - FileName = _fileName, - PakFile = ThePak.CurrentUsedPak, - }); - } - } - - ShowItemsVirtualFiltered(_myFilteredInfos); - } } - else if (Utilities.CaseInsensitiveContains(_myInfos[i].FileName, textBox1.Text)) + else + checkSearch = Utilities.CaseInsensitiveContains(_myInfos[i].FileName, textBox1.Text); + + if (checkSearch) { if (_myInfos[i].FileName.Contains(".uasset") || _myInfos[i].FileName.Contains(".uexp") || _myInfos[i].FileName.Contains(".ubulk")) { @@ -257,52 +226,21 @@ namespace FModel.Forms { for (int i = 0; i < _myInfos.Count; i++) { + bool checkSearch = false; string[] searchTextMultiple = textBox1.Text.Trim().Split(' '); if (searchTextMultiple.Length > 1) { - bool checkSearch = false; for (int s = 0; s < searchTextMultiple.Length; ++s) { checkSearch = Utilities.CaseInsensitiveContains(_myInfos[i].FileName, searchTextMultiple[s]); if (!checkSearch) break; } - - if (checkSearch) - { - if (_myInfos[i].FileName.Contains(".uasset") || _myInfos[i].FileName.Contains(".uexp") || _myInfos[i].FileName.Contains(".ubulk")) - { - if (!_myFilteredInfosDict.ContainsKey(_myInfos[i].FileName.Substring(0, _myInfos[i].FileName.LastIndexOf(".", StringComparison.Ordinal)))) - { - _myFilteredInfosDict.Add(_myInfos[i].FileName.Substring(0, _myInfos[i].FileName.LastIndexOf(".", StringComparison.Ordinal)), ThePak.AllpaksDictionary[Path.GetFileNameWithoutExtension(_myInfos[i].FileName)]); - - _fileName = _myInfos[i].FileName.Substring(0, _myInfos[i].FileName.LastIndexOf(".", StringComparison.Ordinal)); - _myFilteredInfos.Add(new FileInfoFilter - { - FileName = _fileName, - PakFile = ThePak.AllpaksDictionary[Path.GetFileNameWithoutExtension(_myInfos[i].FileName)], - }); - } - } - else - { - if (!_myFilteredInfosDict.ContainsKey(_myInfos[i].FileName)) - { - _myFilteredInfosDict.Add(_myInfos[i].FileName, ThePak.AllpaksDictionary[Path.GetFileName(_myInfos[i].FileName)]); - - _fileName = _myInfos[i].FileName; - _myFilteredInfos.Add(new FileInfoFilter - { - FileName = _fileName, - PakFile = ThePak.AllpaksDictionary[Path.GetFileName(_myInfos[i].FileName)], - }); - } - } - - ShowItemsVirtualFiltered(_myFilteredInfos); - } } - else if (Utilities.CaseInsensitiveContains(_myInfos[i].FileName, textBox1.Text)) + else + checkSearch = Utilities.CaseInsensitiveContains(_myInfos[i].FileName, textBox1.Text); + + if (checkSearch) { if (_myInfos[i].FileName.Contains(".uasset") || _myInfos[i].FileName.Contains(".uexp") || _myInfos[i].FileName.Contains(".ubulk")) {