Patarkit su C# suskaičiuoto rezultato iš metodo gavimu

Sveiki, gavau užduotį iš objektinio programavimo, kad parašyti sistemą, kuri surastų ilgiausią lentą pagal ilgio matmenį (užduotis ilgesnė, bet šiuo metu noriu atlikti tik jos vieną dalį). Taigi metode parašiau tikrinimą, dėl kurio tinkamumo labiausiai ir abejoju. Gal koks C# asas galėtų žvilgtelti į kodą ir užvesti ant kelio, kad toliau galėčiau judėti tinkama linkme?

Kodas:

https://pastebin.com/UQHEk42v

Metodu pavadinimuose lietuvisku simboliu nenaudok ir siaip stenkis programas angliskai rasyt.

O del sprendimo siulyciau prikurt lentu daugiau nei viena, sumest i lista arba masyva ir parasyk metoda kuris eidamas per lentu lista ar masyva ims lentos ilgi su tuo paciu gautiIlgi metodu ir tikrins ar tai yra max.

l1.ImtiIlgį()

l2.ImtiIlgį()

l3.ImtiIlgį()

Pradžioje metodo gali sukišti į double kintamuosius, tai nereikės visur kartoti tų ilgų rašliavų if'uose ir pagerės skaitomumas. Pirmam if'e return nereikalingas, jeigu jau nusprendęs pabaigoje max grąžinti.

Metodu pavadinimuose lietuvisku simboliu nenaudok ir siaip stenkis programas angliskai rasyt.

 

O del sprendimo siulyciau prikurt lentu daugiau nei viena, sumest i lista arba masyva ir parasyk metoda kuris eidamas per lentu lista ar masyva ims lentos ilgi su tuo paciu gautiIlgi metodu ir tikrins ar tai yra max.

Metodo dar neleido nauodoti, tai kolkas nepavyks

per sudėtingai galvoji

užtenka:

public static double IlgiausiaLenta(Lenta l1, Lenta l2, Lenta l3)
{
var max = l1.ImtiIlgį();
var l2ilgis = l2.ImtiIlgį();
var l3ilgis = l3.ImtiIlgį();

if (l2ilgis > max)
	max = l2ilgis;
if (l3ilgis > max)
	max = l3ilgis;

return max;
}

patarimas:

kadangi naudoji C# tai vietoj to, kad turėtum getterio funkciją (šiuo atvėju ImtiIlgį) naudok properties, tai yra C# kalbos dalis tai ne taip kaip C++, nėra priežasties jos nenaudoti.

Taip pat kai naudoji ar getterį ar property, išsaugok reikšmę lokaliai. C# kompileris yra gan durnas ir tokių dalykų dažniausiai neoptimizuoja.

taip niekas neprogramuoja - IlgiausiaLenta(Lenta l1, Lenta l2, Lenta l3) funkcija visiškai nepraktiška, tavo programa gali dirbti tik su trim lentom? ką jeigu man reikės dviejų arba keturių lentų ilgius rasti, vėl atskiras funkcijas rašyti? Turi į funkciją paduoti duomenų masyvą ar listą ir jame ieškoti.

Čia pvz su LINQ, greičiausia ir tvarkingiausia jeigu lentų būtų iki kokio 100000 vnt, po to pradeda lėteti perforamance.

public static double IlgiausiaLenta(List<Lenta> lentos)
{
return lentos.Max(e=> e.ImtiIlgį());
}

Kadangi jums aktualus mokymosi procese pats algoritmas ir parodyti, kad jį moki tai paprastas maxso algoritmas liste turi būti, kuri manau pati pirma išmoko:

public static double IlgiausiaLenta(List<Lenta> lentos)
{
double maxValue = 0;
for(int i = 0; i < lentos.Count; i++)
{
  if(maxValue >= lentos[i].ImtiIlgį())
    maxValue = lentos[i].ImtiIlgį();
}
return maxValue;
}

Aš šį klausimą spręsčiau taip:

class Program
{
   	static void Main(string[] args)
   	{
       	Dictionary<string, double> AllBoards = new Dictionary<string, double>
       	{
           	{ "pirma lenta", 1.20 },
           	{ "antra lenta", 6 },
           	{ "trecia lenta", 5.455 },
           	{ "ketvirta lenta", 50 },
           	{ "penkta lenta", 0.20 }
       	};

       	string LongestBoard = FindLongestBoard(AllBoards);

       	Console.WriteLine("Ilgiausia lenta: " + LongestBoard);
       	Console.ReadKey();
   	}

   	public static string FindLongestBoard(Dictionary<string, double> boards)
   	{
       	string LongestBoard = "";
       	double Max = 0;

       	foreach (KeyValuePair<string, double> item in boards)
       	{
           	if (item.Value > Max)
           	{
               	Max = item.Value;
               	LongestBoard = item.Key;
           	}
       	}
       	return LongestBoard;
   	}
}

Ačiū visiems už atsakymus, bandysiu tvarkyti programą :)