More refactoring & cleanup

Author: Decimation

SmartImage.Lib/Engines/BaseSearchEngine.cs

@@ -1,191 +1,75 @@
-global using R1 = SmartImage.Lib.Resources;
+// Author: Deci | Project: SmartImage.Lib | Name: BaseSearchEngine.cs
+// Date: 2024/06/06 @ 14:06:00
+
+global using R1 = SmartImage.Lib.Resources;
 global using Url = Flurl.Url;
-using System.Collections.Concurrent;
 using System.Diagnostics;
-using System.Drawing;
-using System.Text.Json;
-using System.Resources;
-using Novus.Utilities;
-using SmartImage.Lib.Results;
-using AngleSharp.Dom;
-using AngleSharp.Html.Parser;
 using Flurl.Http;
-using Flurl.Http.Configuration;
 using Kantan.Diagnostics;
 using Kantan.Net.Utilities;
-using Kantan.Net.Web;
-using Microsoft.Extensions.Http.Logging;
 using Microsoft.Extensions.Logging;
 using SmartImage.Lib.Engines.Impl.Search;
 using SmartImage.Lib.Engines.Impl.Search.Other;
-using SmartImage.Lib.Images;
-using SmartImage.Lib.Utilities;
-using SmartImage.Lib.Results.Data;
-using SmartImage.Lib.Diagnostics;
+using SmartImage.Lib.Results;
 using SmartImage.Lib.Utilities.Diagnostics;
 
 namespace SmartImage.Lib.Engines;
 #nullable enable
 public abstract class BaseSearchEngine : IDisposable, IEquatable<BaseSearchEngine>
 {
 
-	protected static FlurlClient Client { get; }
-
-	/// <summary>
-	/// The corresponding <see cref="SearchEngineOptions"/> of this engine
-	/// </summary>
-	public abstract SearchEngineOptions EngineOption { get; }
-
-	/// <summary>
-	/// Name of this engine
-	/// </summary>
-	public virtual string Name => EngineOption.ToString();
-
-	public virtual Url BaseUrl { get; }
-
-	public bool IsAdvanced { get; protected init; }
-
-	public TimeSpan Timeout { get; set; } = TimeSpan.FromSeconds(15);
-
-	public string? EndpointUrl { get; }
-
-	protected long? MaxSize { get; set; }
-
-	protected virtual string[] ErrorBodyMessages { get; } = [];
-
-	protected BaseSearchEngine(string baseUrl, string? endpoint = null)
-	{
-		BaseUrl     = baseUrl;
-		IsAdvanced  = true;
-		EndpointUrl = endpoint;
-		MaxSize     = null;
-	}
-
-	protected static readonly ILogger Logger = AppSupport.Factory.CreateLogger(nameof(BaseSearchEngine));
-	
-	/*protected IFlurlRequest Build(IFlurlRequest request)
-	{
-		return request.WithTimeout(Timeout);
-	}*/
-
-
 	static BaseSearchEngine()
 	{
-		/*var handler = new LoggingHttpMessageHandler(Logger)
-		{
-			InnerHandler = new HttpLoggingHandler(Logger)
-			{
-				InnerHandler = new HttpClientHandler()
-			}
-		};
-
-		Client = new FlurlClient(new HttpClient(handler))
-		{
-			Settings =
-			{
-				Redirects =
-				{
-					Enabled                    = true,
-					AllowSecureToInsecure      = true,
-					ForwardAuthorizationHeader = true,
-					MaxAutoRedirects           = 20,
-				},
-			}
-		};*/
-		
-
 		Client = (FlurlClient) FlurlHttp.Clients.GetOrAdd(nameof(BaseSearchEngine), null, builder =>
 		{
 			builder.Headers.AddOrReplace("User-Agent", HttpUtilities.UserAgent);
-			
+
 			// builder.Settings.JsonSerializer = new DefaultJsonSerializer();
 
 			builder.Settings.AllowedHttpStatusRange = "*";
-			builder.OnError(f=>
+
+			builder.OnError(f =>
 			{
+				Debugger.Break();
 				Logger.LogError(f.Exception, $"from {f.Request}");
-				return;
 			});
 
 			builder.AddMiddleware(() => new HttpLoggingHandler(Logger));
 
-		});;
-	}
-
-	public override string ToString()
-	{
-		return $"{Name}: {BaseUrl} {Timeout}";
-	}
-
-	public virtual async Task<SearchResult> GetResultAsync(SearchQuery query, CancellationToken token = default)
-	{
-		var b = await VerifyQueryAsync(query);
-
-		/*
-		if (!b) {
-			// throw new SmartImageException($"{query}");
-			Debug.WriteLine($"{query} : Verification error", LogCategories.C_ERROR);
-		}
-		*/
-
-		var srs = b ? SearchResultStatus.None : SearchResultStatus.IllegalInput;
-
-		var res = new SearchResult(this)
-		{
-			RawUrl       = GetRawUrl(query),
-			ErrorMessage = null,
-			Status       = srs
-		};
-
-		lock (res.Results) {
-			res.Results.Add(res.GetRawResultItem());
-		}
-
-		Debug.WriteLine($"{Name} | {query} - {res.Status}", LogCategories.C_INFO);
-
-		return res;
+		});
+		;
 	}
 
-	protected virtual Url GetRawUrl(SearchQuery query)
+	protected BaseSearchEngine(string baseUrl, string? endpoint = null)
 	{
-		//
-		Url u = ((BaseUrl + query.Upload));
-
-		return u;
+		BaseUrl     = baseUrl;
+		EndpointUrl = endpoint;
+		MaxSize     = null;
 	}
 
-	public virtual ValueTask<bool> VerifyQueryAsync(SearchQuery q)
-	{
-		/*if (q.Upload is not { }) {
-			return false;
-		}*/
-
-		bool b = true;
+	protected static readonly ILogger Logger = AppSupport.Factory.CreateLogger(nameof(BaseSearchEngine));
 
-		if (MaxSize.HasValue) {
-			b = q.Source.Size <= MaxSize;
-		}
+	/// <summary>
+	///     The corresponding <see cref="SearchEngineOptions" /> of this engine
+	/// </summary>
+	public abstract SearchEngineOptions EngineOption { get; }
 
-		/*if (MaxSize == NA_SIZE || q.Size == NA_SIZE) {
-			b = true;
-		}
+	/// <summary>
+	///     Name of this engine
+	/// </summary>
+	public virtual string Name => EngineOption.ToString();
 
-		else {
-			b = q.Size <= MaxSize;
-		}*/
+	public virtual Url BaseUrl { get; }
 
-		return ValueTask.FromResult(b);
-	}
+	public TimeSpan Timeout { get; set; } = TimeSpan.FromSeconds(15);
 
+	public string? EndpointUrl { get; }
 
-	// TODO: move config application to ctors?
+	protected long? MaxSize { get; set; }
 
-	public abstract void Dispose();
+	protected virtual string[] ErrorBodyMessages { get; } = [];
 
-	/*
-	public static readonly BaseSearchEngine[] All =
-		ReflectionHelper.CreateAllInAssembly<BaseSearchEngine>(InheritanceProperties.Subclass).ToArray();
-		*/
+	protected static FlurlClient Client { get; }
 
 	public static IEnumerable<BaseSearchEngine> GetSelectedEngines(SearchEngineOptions options)
 	{
@@ -242,21 +126,87 @@ public static IEnumerable<BaseSearchEngine> GetSelectedEngines(SearchEngineOptio
 
 	}
 
-	#region
+	public static bool operator ==(BaseSearchEngine? left, BaseSearchEngine? right)
+	{
+		return Equals(left, right);
+	}
+
+	public static bool operator !=(BaseSearchEngine? left, BaseSearchEngine? right)
+	{
+		return !Equals(left, right);
+	}
 
-	public bool Equals(BaseSearchEngine? other)
+	public virtual async Task<SearchResult> GetResultAsync(SearchQuery query, CancellationToken token = default)
 	{
-		if (ReferenceEquals(null, other)) return false;
-		if (ReferenceEquals(this, other)) return true;
+		var b = await VerifyQueryAsync(query);
 
-		return EngineOption == other.EngineOption;
+		/*
+		if (!b) {
+			// throw new SmartImageException($"{query}");
+			Debug.WriteLine($"{query} : Verification error", LogCategories.C_ERROR);
+		}
+		*/
+
+		var srs = b ? SearchResultStatus.None : SearchResultStatus.IllegalInput;
+
+		var res = new SearchResult(this)
+		{
+			RawUrl       = GetRawUrl(query),
+			ErrorMessage = null,
+			Status       = srs
+		};
+
+		lock (res.Results) {
+			res.Results.Add(res.GetRawResultItem());
+		}
+
+		Debug.WriteLine($"{Name} | {query} - {res.Status}", LogCategories.C_INFO);
+
+		return res;
+	}
+
+	protected virtual Url GetRawUrl(SearchQuery query)
+	{
+		//
+		Url u = ((BaseUrl + query.Upload));
+
+		return u;
+	}
+
+	public virtual ValueTask<bool> VerifyQueryAsync(SearchQuery q)
+	{
+		bool b = true;
+
+		if (MaxSize.HasValue) {
+			b = q.Source.Size <= MaxSize;
+		}
+
+		return ValueTask.FromResult(b);
+	}
+
+	public int GetHashCode(BaseSearchEngine obj)
+	{
+		return (int) obj.EngineOption;
+	}
+
+	public override string ToString()
+	{
+		return $"{Name}: {BaseUrl} {Timeout}";
 	}
 
 	public override bool Equals(object? obj)
 	{
-		if (ReferenceEquals(null, obj)) return false;
-		if (ReferenceEquals(this, obj)) return true;
-		if (obj.GetType() != this.GetType()) return false;
+		if (obj is null) {
+			return false;
+		}
+
+		if (ReferenceEquals(this, obj)) {
+			return true;
+		}
+
+		if (obj.GetType() != GetType()) {
+			return false;
+		}
 
 		return Equals((BaseSearchEngine) obj);
 	}
@@ -266,21 +216,32 @@ public override int GetHashCode()
 		return (int) EngineOption;
 	}
 
-	public static bool operator ==(BaseSearchEngine? left, BaseSearchEngine? right)
-	{
-		return Equals(left, right);
-	}
+	public abstract void Dispose();
 
-	public static bool operator !=(BaseSearchEngine? left, BaseSearchEngine? right)
+	public bool Equals(BaseSearchEngine? other)
 	{
-		return !Equals(left, right);
+		if (other is null) {
+			return false;
+		}
+
+		if (ReferenceEquals(this, other)) {
+			return true;
+		}
+
+		return EngineOption == other.EngineOption;
 	}
 
-	public int GetHashCode(BaseSearchEngine obj)
+	/*protected IFlurlRequest Build(IFlurlRequest request)
 	{
-		return (int) obj.EngineOption;
-	}
+		return request.WithTimeout(Timeout);
+	}*/
+
 
-	#endregion
+	// TODO: move config application to ctors?
+
+	/*
+	public static readonly BaseSearchEngine[] All =
+		ReflectionHelper.CreateAllInAssembly<BaseSearchEngine>(InheritanceProperties.Subclass).ToArray();
+		*/
 
 }