Harjoitustyö valmis

HT/TiedotOsingoista/TiedotOsingoista.cs

@@ -31,7 +31,7 @@ public class TiedotOsingoista
     /// </summary>
     /// <param name="url">Haluttu URL-osoite</param>
     /// <returns>Halutun url-osoitteen HTML-datan merkkijonona</returns>
-    public static string HaeNetista(string url)
+    private static string HaeNetista(string url)
     {
         HttpClient client = new HttpClient();
         string sisalto = client.GetStringAsync(url).GetAwaiter().GetResult();
@@ -42,17 +42,17 @@ public class TiedotOsingoista
     /// <summary>
     /// Aliohjelma kysyy käyttäjältä Helsingin pörssissä listatun yhtiön nimeä ja selvittää tietoja sen osingoista.
     /// </summary>
-    public static void Kysy()
+    private static void Kysy()
     {
+        int oikeaAlkioidenMaara = 15;
         while (true)
         {
             Console.Write("Anna yrityksen nimi (Vähintään 2 kirjainta. Enter lopettaa) >");
             string syote = Console.ReadLine();
             if (syote == "" || syote == null) return;
             if (syote.Length < 2) continue;
-            string[] yhtiot = new string[5];
-            yhtiot = EtsiTiedot(syote);
-            if (yhtiot.Length != 15)
+            string[] yhtiot = EtsiTiedot(syote);
+            if (yhtiot.Length != oikeaAlkioidenMaara)
             {
                 Console.WriteLine("Antamallasi nimellä ei löydy yritystä.");
                 Console.WriteLine();
@@ -142,7 +142,7 @@ public class TiedotOsingoista
     /// Aliohjelma tulostaa parametrina viedyn pörssiyhtiön osinkotiedot.
     /// </summary>
     /// <param name="firma">Pörssiyhtiö, jonka osinkotiedot tulostetaan</param>
-    public static void Tulosta(string[] firma)
+    private static void Tulosta(string[] firma)
     {
 
         Console.WriteLine("Yritys         Osinko    Osinkotuotto-%    Irtoamispäivä     Osta viimeistään");

HT/TiedotOsingoistaTest/TiedotOsingoistaTest.cs

@@ -9,26 +9,26 @@ using static TiedotOsingoista;
 	public  class TestTiedotOsingoista
 	{
 		[Test]
-		public  void TestEtsiTiedot76()
+		public  void TestEtsiTiedot77()
 		{
 			string syote = "upm";
 			string[] firma = new string[5];
 			firma = EtsiTiedot(syote);
-			Assert.AreEqual( "UPM-Kymmene", firma[1] , "in method EtsiTiedot, line 80");
-			Assert.AreEqual( "1.50 €", firma[4] , "in method EtsiTiedot, line 81");
-			Assert.AreEqual( "5.11 %", firma[7] , "in method EtsiTiedot, line 82");
-			Assert.AreEqual( 15, firma.Length , "in method EtsiTiedot, line 83");
+			Assert.AreEqual( "UPM-Kymmene", firma[1] , "in method EtsiTiedot, line 81");
+			Assert.AreEqual( "1.50 €", firma[4] , "in method EtsiTiedot, line 82");
+			Assert.AreEqual( "5.11 %", firma[7] , "in method EtsiTiedot, line 83");
+			Assert.AreEqual( 15, firma.Length , "in method EtsiTiedot, line 84");
 			syote = "lkdfldjjflasldfjldfl";
 			firma = new string[5];
 			firma = EtsiTiedot(syote);
-			Assert.AreEqual( 5, firma.Length , "in method EtsiTiedot, line 87");
+			Assert.AreEqual( 5, firma.Length , "in method EtsiTiedot, line 88");
 			syote = "nordea";
 			firma = new string[5];
 			firma = EtsiTiedot(syote);
-			Assert.AreEqual( "Nordea Bank", firma[1] , "in method EtsiTiedot, line 91");
-			Assert.AreEqual( "24.3.2023", firma[10] , "in method EtsiTiedot, line 92");
-			Assert.AreEqual( "23.3.2023", firma[13] , "in method EtsiTiedot, line 93");
-			Assert.AreEqual( 15, firma.Length , "in method EtsiTiedot, line 94");
+			Assert.AreEqual( "Nordea Bank", firma[1] , "in method EtsiTiedot, line 92");
+			Assert.AreEqual( "24.3.2023", firma[10] , "in method EtsiTiedot, line 93");
+			Assert.AreEqual( "23.3.2023", firma[13] , "in method EtsiTiedot, line 94");
+			Assert.AreEqual( 15, firma.Length , "in method EtsiTiedot, line 95");
 		}
 	}