NullPoinerException Linq max array

Asked Sep 17 '17 at 19:57

Active Sep 17 '17 at 21:37

Viewed 40 times

I Have a method that finds max and average from array of objects and returns it, but im getting NullPointerException at b.Brzina and a.Brzina and Im not sure how to fix that.

string foo = metoda();
Console.WriteLine(foo);
Console.ReadKey();

static Auto[] niz = new Auto[3];

public static string metoda()
{
    var max = niz.Max(b => b.Brzina);
    double avg = niz.Average(a => a.Brzina);
    string autoo = "najbrzi auto je" + max.ToString() + "a prosecna brzina je" + avg.ToString();
    return autoo.ToString();
}

edited Sep 17 '17 at 21:37

Axel

3,331
11
35
58

asked Sep 17 '17 at 19:57

Ilija Pavlovic

What is `niz`? How do you populate it. There must be a null value in it that's why you are seeing this error. – Chetan Sep 17 '17 at 20:01
Nis iz already populated in void main – Ilija Pavlovic Sep 17 '17 at 20:45
Did you debug and check if it doesn't have any null item? What is the type of `Brzina` property? You need to debug and see if there is any item in the list is null or any it in the list with `Brzina` set to null. – Chetan Sep 17 '17 at 20:48
http://prntscr.com/gmcr7s It is null only at the point where it needs to calculate max – Ilija Pavlovic Sep 17 '17 at 21:00
Remove `Auto[] niz = new Auto[3];` from inside the main method. – Chetan Sep 17 '17 at 21:03
Still get the same error – Ilija Pavlovic Sep 17 '17 at 21:06
Can you debug and check if `niz` is not null and all the items of it not null? Updating question with main may god would be helpful. Did you follow the answer marked as duplicate by Steve? – Chetan Sep 17 '17 at 21:09
http://prntscr.com/gmcy4m everything is populated in niz – Ilija Pavlovic Sep 17 '17 at 21:16
You could just pass a reference into the function to avoid scope confusion: `public static string metoda(Auto[] niz)` and then call it from main by `string foo = metoda(niz);` – Keyur PATEL Sep 18 '17 at 02:01

NullPoinerException Linq max array

0 Answers0