Изменение архитектуры

Author: OneCodeUnit

MainForm.cs

@@ -8,7 +8,7 @@ public partial class MainForm : Form
         float USD_Steam; // Рублей за доллар (Steam)
         float KZT_Qiwi; // Рублей за тенге (Qiwi)
         float USD_Qiwi; // Тенге за доллар (Qiwi)
-        float USD_Web; // Тенге за доллар (Webmoney)
+        float USD_Web; // Рублей за доллар (Webmoney)
         float RUB_Input; // Рублей до конвертации
         float RUB_Output; // Рублей после конвертации
 
@@ -25,10 +25,10 @@ public MainForm()
 
             if (USD_Steam != 0 && USD_Qiwi != 0 && KZT_Qiwi != 0 && USD_Web != 0)
             {
-                LabelUSD.Text = "Рублей за доллар - " + Math.Round(USD_Steam, 4).ToString(CultureInfo.InvariantCulture);
-                LabelUSDWeb.Text = "Рублей за доллар - " + Math.Round(USD_Web, 4).ToString(CultureInfo.InvariantCulture) + " (P2P)";
-                LabelKZT.Text = "Рублей за тенге     - " + Math.Round(KZT_Qiwi, 3).ToString(CultureInfo.InvariantCulture);
-                LabelKZT_USD.Text = "Тенге за доллар    - " + Math.Round(USD_Qiwi, 2).ToString(CultureInfo.InvariantCulture);
+                LabelUSD.Text = $"Рублей за доллар - {Math.Round(USD_Steam, 4)}";
+                LabelUSDWeb.Text = $"Рублей за доллар - {Math.Round(USD_Web, 4)} (P2P)";
+                LabelKZT.Text = $"Рублей за тенге     - {Math.Round(KZT_Qiwi, 3)}";
+                LabelKZT_USD.Text = $"Тенге за доллар    - {Math.Round(USD_Qiwi, 2)}";
 
                 PictureBoxUSD.Image = Properties.Resources.warn_c;
                 PictureBoxUSDWeb.Image = Properties.Resources.warn_c;
@@ -48,77 +48,77 @@ private void ButtonGet_Click(object sender, EventArgs e)
             ButtonGetRates.Text = "Обновление";
             ButtonGetRates.Enabled = false;
             ButtonReverse.Enabled = false;
-            PictureBoxKZT.Image = Properties.Resources.wait_c;
-            PictureBoxUSD.Image = Properties.Resources.wait_c;
-            PictureBoxUSDWeb.Image = Properties.Resources.wait_c;
-            PictureBoxKZTUSD.Image = Properties.Resources.wait_c;
             TextBoxInput.Enabled = false;
             TextBoxOutput.Enabled = false;
 
-            // 1 - доллары, 5 - рубли (https://partner.steamgames.com/doc/store/pricing/currencies)
-            SteamJson jsonUSD = SteamHttpClient.GetSteamJson(1);
-            SteamJson jsonRUB = SteamHttpClient.GetSteamJson(5);
-
-            string rawUSD = jsonUSD.lowest_price[1..^4];
-            string rawRUB = jsonRUB.lowest_price[0..^5];
-
-            if (jsonUSD.success == false || jsonRUB.success == false)
+            PictureBoxUSD.Image = Properties.Resources.wait_c;
+            SteamJson steam = new();
+            // Рублей за доллар по курсу стима
+            float usdSteam = steam.GetRate();
+            if (usdSteam == 0)
             {
                 PictureBoxUSD.Image = Properties.Resources.no_c;
             }
             else
             {
-                float steamUSD = Convert.ToSingle(rawUSD, new CultureInfo("en-US"));
-                float steamRUB = Convert.ToSingle(rawRUB, new CultureInfo("ru-RU"));
-                USD_Steam = steamRUB / steamUSD;
-
+                USD_Steam = usdSteam;
                 Properties.Settings.Default["USD_Steam"] = USD_Steam;
-                LabelUSD.Text = "Рублей за доллар - " + Math.Round(USD_Steam, 4).ToString(CultureInfo.InvariantCulture);
+                LabelUSD.Text = $"Рублей за доллар - {Math.Round(USD_Steam, 4)}";
                 PictureBoxUSD.Image = Properties.Resources.yes_c;
-                TextBoxInput.Enabled = true;
-                TextBoxOutput.Enabled = true;
             }
 
-            // 398 - тенге, 643 - рубли, 840 - доллары, 978 - евро, 156 - юани
-            QiwiJson jsonQiwi = SteamHttpClient.GetQiwiJson();
-
-            PictureBoxKZT.Image = Properties.Resources.no_c;
-            PictureBoxKZTUSD.Image = Properties.Resources.no_c;
-            foreach (var rateIter in jsonQiwi.result)
+            PictureBoxKZT.Image = Properties.Resources.wait_c;
+            PictureBoxKZTUSD.Image = Properties.Resources.wait_c;
+            QiwiJson qiwi = new();
+            qiwi = qiwi.GetData();
+            // Рублей за тенге по курсу qiwi
+            float kzt = qiwi.GetRate(("643", "398"));
+            if (kzt == 0)
             {
-                if (rateIter.from == "643" && rateIter.to == "398")
-                {
-                    KZT_Qiwi = rateIter.rate;
-
-                    Properties.Settings.Default["KZT_Qiwi"] = KZT_Qiwi;
-                    LabelKZT.Text = "Рублей за тенге     - " + Math.Round(KZT_Qiwi, 3).ToString(CultureInfo.InvariantCulture);
-                    PictureBoxKZT.Image = Properties.Resources.yes_c;
-                    TextBoxInput.Enabled = true;
-                    TextBoxOutput.Enabled = true;
-                }
-                if (rateIter.from == "398" && rateIter.to == "840")
-                {
-                    USD_Qiwi = rateIter.rate;
-
-                    Properties.Settings.Default["USD_Qiwi"] = USD_Qiwi;
-                    LabelKZT_USD.Text = "Тенге за доллар    - " + Math.Round(USD_Qiwi, 2).ToString(CultureInfo.InvariantCulture);
-                    PictureBoxKZTUSD.Image = Properties.Resources.yes_c;
-                    TextBoxInput.Enabled = true;
-                    TextBoxOutput.Enabled = true;
-                }
+                PictureBoxKZT.Image = Properties.Resources.no_c;
+            }
+            else
+            {
+                KZT_Qiwi = kzt;
+                Properties.Settings.Default["KZT_Qiwi"] = KZT_Qiwi;
+                LabelKZT.Text = $"Рублей за тенге     - {Math.Round(KZT_Qiwi, 3)}";
+                PictureBoxKZT.Image = Properties.Resources.yes_c;
+            }
+            // Тенге за доллар по курсу qiwi
+            float usdQiwi = qiwi.GetRate(("398", "840"));
+            if (usdQiwi == 0)
+            {
+                PictureBoxKZTUSD.Image = Properties.Resources.no_c;
+            }
+            else
+            {
+                USD_Qiwi = usdQiwi;
+                Properties.Settings.Default["USD_Qiwi"] = USD_Qiwi;
+                LabelKZT_USD.Text = $"Тенге за доллар    - {Math.Round(USD_Qiwi, 2)}";
+                PictureBoxKZTUSD.Image = Properties.Resources.yes_c;
             }
 
-            PictureBoxUSDWeb.Image = Properties.Resources.no_c;
-            USD_Web = SteamHttpClient.GetWebMoneyRate();
-            if (USD_Web != -1)
+            PictureBoxUSDWeb.Image = Properties.Resources.wait_c;
+            WMJson webmoney = new();
+            // Рублей за доллар по курсу вебмани
+            webmoney = webmoney.GetData();
+            float usdWeb = webmoney.GetRate();
+            if (usdWeb == 0 || usdWeb == -1)
+            {
+                PictureBoxUSDWeb.Image = Properties.Resources.no_c;
+            }
+            else
             {
+                USD_Web = usdWeb;
                 Properties.Settings.Default["USD_Web"] = USD_Web;
-                LabelUSDWeb.Text = "Рублей за доллар - " + Math.Round(USD_Web, 4).ToString(CultureInfo.InvariantCulture) + " (P2P)";
+                LabelUSDWeb.Text = $"Рублей за доллар - {Math.Round(USD_Web, 4)} (P2P)";
                 PictureBoxUSDWeb.Image = Properties.Resources.yes_c;
             }
 
             Properties.Settings.Default.Save();
             CalcPercent();
+            TextBoxInput.Enabled = true;
+            TextBoxOutput.Enabled = true;
             ButtonGetRates.Enabled = true;
             ButtonReverse.Enabled = true;
             ButtonGetRates.Text = "Обновить";
@@ -276,7 +276,7 @@ private void ButtonTargetSystem_Click(object sender, EventArgs e)
                 TextBoxInputKZT.Visible = true;
                 LabelKZTInput.Visible = true;
             }
-            string temp = string.Empty;
+            string temp;
             if (Reverse == false)
             {
                 temp = TextBoxInput.Text;

QiwiJson.cs

@@ -7,10 +7,12 @@
 + Runtime - только запуск приложений.
 + Desktop Runtime - только запуск настольных приложений для Windows.
 
-Для Windows 7 и 8.1 подойдёт только .Net 6, а для Windows 10 (версия 1607+) и 11 (версия 22000+) можно установить .Net 7.
+Для Windows 7 и 8.1 подойдёт только .Net 6, а для Windows 10 (версия 1607+) и 11 (версия 22000+) можно установить .Net 7. Начиная с версии 3.0 программы поддерживается только .Net 7 и Windows 10+
 
 [Скачать программу](https://github.com/OneCodeUnit/SteamCurrency/releases/latest)
 
+[Скачать старую версию для .Net 6 и Windows 7+](https://github.com/OneCodeUnit/SteamCurrency/releases/tag/v2.1.0)
+
 ## Работа программы
 Программа через Rest-запросы узнаёт у Steam, WebMoney (окольными путями) и Qiwi (через API) курсы валют и производит вычисления суммы, которая поступит, и суммы, которую "съест" конвертация. Из-за способа получения курсов у Steam, присутствует погрешность на уровне 1 рубля.
 
@@ -21,7 +23,7 @@
 + Курс стима считается через сравнение цен на торговой площадке у одного и того же товара. Возможно, есть более правильный способ, так как цены на торговой площадке округлены до двух знаков после запятой, что приводит к погрешностям при вычислениях.
 + Подсказка к вводимой сумме отображается с округлением до двух знаков после запятой, что соответствует точности поля ввода на сайте Qiwi.
 + Qiwi конвертирует тенге в доллары по курсу, который примерно на 4% выше того, о котором он сообщает. Я не знаю, почему так. Вариант с дополнительной валютой не подходит. Однако программа учитывает этот процент при вычислениях.
-+ Курс WebMoney берется на основе самого выгодного P2P-предложения, которое предполагает определенный объем сделки. Предложение для вашего объема может иметь менее выгодный курс.
++ Курс WebMoney берется на основе самого выгодного P2P-предложения, которое предполагает определенный объем сделки. Предложение для вашего объема может иметь менее выгодный курс. Программа это не учитывает.
 + WebMoney имеет комиссию 4% на пополнение Steam. Программа это учитывает.
 + Существует вероятность, что программа может не получить или получить неправильные курсы. Подождите некоторое время и попробуйте снова.
 + Не стоит проверять курсы слишком часто (речь о десятках раз в час), так как это может привести к временному бану по IP.

README.md

@@ -2,7 +2,7 @@
 
   <PropertyGroup>
     <OutputType>WinExe</OutputType>
-    <TargetFramework>net6.0-windows</TargetFramework>
+    <TargetFramework>net7.0-windows7.0</TargetFramework>
     <Nullable>enable</Nullable>
     <UseWindowsForms>true</UseWindowsForms>
     <ImplicitUsings>enable</ImplicitUsings>
@@ -11,18 +11,19 @@
     <AnalysisLevel>latest-recommended</AnalysisLevel>
     <ApplicationIcon>Flag.ico</ApplicationIcon>
     <PackageLicenseFile>LICENSE.txt</PackageLicenseFile>
-    <AssemblyVersion>2.1.0</AssemblyVersion>
-    <FileVersion>2.1.0</FileVersion>
-    <Version>$(VersionPrefix)2.1.0</Version>
+    <AssemblyVersion>3.0.0</AssemblyVersion>
+    <FileVersion>3.0.0</FileVersion>
+    <Version>$(VersionPrefix)3.0.0</Version>
     <RepositoryType>git</RepositoryType>
     <RepositoryUrl>https://github.com/OneCodeUnit/SteamCurrency</RepositoryUrl>
     <NeutralLanguage>ru</NeutralLanguage>
-    <Title>Steam Cyrrency Calculator</Title>
+    <Title>Steam Currency Calculator</Title>
     <Authors>OliveWizard</Authors>
     <Description>Калькулятор пополнения Steam через Qiwi</Description>
     <PackageReadmeFile>README.md</PackageReadmeFile>
-    <PackageTags>Steam; Currency; Calculator; Currency rate; Qiwi</PackageTags>
-    <PackageId>SteamCyrrencyCalculator</PackageId>
+    <PackageTags>Steam; Currency; Calculator; Currency rate; Qiwi; WebMoney</PackageTags>
+    <PackageId>SteamCurrencyCalculator</PackageId>
+    <SupportedOSPlatformVersion>7.0</SupportedOSPlatformVersion>
   </PropertyGroup>
 
   <PropertyGroup Condition="'$(Configuration)|$(Platform)'=='Debug|x64'">
@@ -42,11 +43,13 @@
   </PropertyGroup>
 
   <ItemGroup>
-    <Content Include="Flag.ico" />
+    <Compile Remove="SteamCurrencyLib\**" />
+    <EmbeddedResource Remove="SteamCurrencyLib\**" />
+    <None Remove="SteamCurrencyLib\**" />
   </ItemGroup>
 
   <ItemGroup>
-    <PackageReference Include="Newtonsoft.Json" Version="13.0.3" />
+    <Content Include="Flag.ico" />
   </ItemGroup>
 
   <ItemGroup>
@@ -88,4 +91,8 @@
     <Folder Include="Resources\" />
   </ItemGroup>
 
+  <ItemGroup>
+    <ProjectReference Include="SteamCurrencyLib\SteamCurrencyLib.csproj" />
+  </ItemGroup>
+
 </Project>

SteamCurrency.csproj

@@ -5,6 +5,8 @@ VisualStudioVersion = 17.4.33403.182
 MinimumVisualStudioVersion = 10.0.40219.1
 Project("{9A19103F-16F7-4668-BE54-9A1E7A4F7556}") = "SteamCurrency", "SteamCurrency.csproj", "{9B2D65AA-8397-4746-8D99-C980A526C774}"
 EndProject
+Project("{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}") = "SteamCurrencyLib", "SteamCurrencyLib\SteamCurrencyLib.csproj", "{B084AA91-3AC0-484D-9E70-A3F26E97B12A}"
+EndProject
 Global
 	GlobalSection(SolutionConfigurationPlatforms) = preSolution
 		Debug|Any CPU = Debug|Any CPU
@@ -15,6 +17,10 @@ Global
 		{9B2D65AA-8397-4746-8D99-C980A526C774}.Debug|Any CPU.Build.0 = Debug|Any CPU
 		{9B2D65AA-8397-4746-8D99-C980A526C774}.Release|Any CPU.ActiveCfg = Release|Any CPU
 		{9B2D65AA-8397-4746-8D99-C980A526C774}.Release|Any CPU.Build.0 = Release|Any CPU
+		{B084AA91-3AC0-484D-9E70-A3F26E97B12A}.Debug|Any CPU.ActiveCfg = Debug|Any CPU
+		{B084AA91-3AC0-484D-9E70-A3F26E97B12A}.Debug|Any CPU.Build.0 = Debug|Any CPU
+		{B084AA91-3AC0-484D-9E70-A3F26E97B12A}.Release|Any CPU.ActiveCfg = Release|Any CPU
+		{B084AA91-3AC0-484D-9E70-A3F26E97B12A}.Release|Any CPU.Build.0 = Release|Any CPU
 	EndGlobalSection
 	GlobalSection(SolutionProperties) = preSolution
 		HideSolutionNode = FALSE

SteamCurrency.sln

@@ -0,0 +1,56 @@
+using Newtonsoft.Json;
+using System.Collections.Generic;
+using System.Net.Http;
+
+namespace SteamCurrencyLib
+{
+#pragma warning disable IDE1006, CS8618
+    public class QiwiJsonResult
+    {
+        public string set { get; set; }
+        public string from { get; set; }
+        public string to { get; set; }
+        public float rate { get; set; }
+    }
+
+    public class QiwiJson
+    {
+        public List<QiwiJsonResult> result { get; set; }
+
+        public QiwiJson()
+        {
+            result = new List<QiwiJsonResult>();
+        }
+
+        public QiwiJson GetData()
+        {
+            HttpResponseMessage response;
+            try
+            {
+                response = SCHttpClient.Client.GetAsync("https://edge.qiwi.com/sinap/crossRates").Result;
+            }
+            catch
+            {
+                return new QiwiJson();
+            }
+            string text = response.Content.ReadAsStringAsync().Result;
+            QiwiJson json = JsonConvert.DeserializeObject<QiwiJson>(text);
+            return json;
+        }
+
+        // 398 - тенге, 643 - рубли, 840 - доллары, 978 - евро, 156 - юани
+        public float GetRate((string, string) pair)
+        {
+            float rate = 0;
+            foreach (var rateIter in result)
+            {
+                if (rateIter.from == pair.Item1 && rateIter.to == pair.Item2)
+                {
+                    rate = rateIter.rate;
+                }
+            }
+            return rate;
+        }
+    }
+#pragma warning restore IDE1006, CS8618
+}

SteamCurrencyLib/Flag.ico

@@ -0,0 +1,13 @@
+using System.Net.Http;
+
+namespace SteamCurrencyLib
+{
+    internal class SCHttpClient
+    {
+        internal static readonly HttpClient Client = new HttpClient();
+        static SCHttpClient()
+        {
+            Client.DefaultRequestHeaders.UserAgent.ParseAdd("Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:110.0) Gecko/20100101 Firefox/110.0");
+        }
+    }
+}