Numbers app #14
Numbers app #14
Conversation
|
Ditt användarnamn motsvarar mina känslor när jag jobbar med |
| var inputNumber = Console.ReadLine(); | ||
|
|
||
| var inputStringArray = inputNumber.Split(',', StringSplitOptions.RemoveEmptyEntries);//new [] för FrameWork | ||
| var numbers = new double[inputStringArray.Length]; |
There was a problem hiding this comment.
Prova att använda en nullbar datatyp här för att hålla reda på om Double.TryParse lyckats eller inte. Om man inte gör det påverkas snittet.
There was a problem hiding this comment.
| //while() | ||
| for (int i = 0; i < inputStringArray.Length; i++) | ||
| { | ||
| Console.WriteLine("Värdet är:".PadRight(14) + inputStringArray[i]); |
There was a problem hiding this comment.
Att använda metodaccess direkt följande literaler på detta sättet är något vi anser påverkar läsbarheten i kod.
Vidare så körs operationen PadRight multipla gånger iom att den är placerad i loopen utan att värdet förändras under körning.
Skriv istället:
var label = "Värdet är:";
var paddedLabel = label.PadRight(14);
for (int i = 0; i < inputStringArray.Length; i++)
{
Console.WriteLine(paddedLabel + inputStringArray[i]);
|
|
||
| //Printing out evaluated values | ||
| Console.WriteLine(line); Console.WriteLine(); | ||
| Console.WriteLine($"Högsta värdet: {numbersArray.Max()}");//Max value |
There was a problem hiding this comment.
Här kallas numbersArray.Max() igen, även fast det redan görs på rad 63.
There was a problem hiding this comment.
Vidare så tar numbersArray.Min() och numbersArray.Min() inte hänsyn till om det uppstått fel i parseing ovan.
Dvs att om värdena anders,12,34,5 anges kommer Min() att returnera 0.
I wrote this number app as a part of a exercise for school. This is my proposed solution for the stated problem.