Type script get max array

Question

I have an array with one column n_fnc and I would like to find the max value . I tried with this but I don"t get anything.

 let first = this.fncs.map(item => item.n_fnc);
   console.log("First",first);
    x= Math.max(...first);

[ max aray

fnc service

getlastid(response,fncs:Fnc[]):void{
      let fnc:Fnc;
      response.forEach(element => {
        
        fnc =new Fnc();
        fnc.n_fnc=element.n_fnc;
        fncs.push(fnc);

    });

Fnc component.ts

this.fncs=[];
this.fncservice.obs.subscribe((response)=>this.fncservice.getlastid(response,this.fncs));
     console.log("A",this.fncs);
    var max= Math.max.apply(Math, this.fncs.map((m) => m.n_fnc));
    console.log("Max",max);

to get a max of "simple elements": developer.mozilla.org/es/docs/Web/JavaScript/Referencia/… — Eliseo
– Eliseo, Commented Dec 24, 2020 at 9:12
This has nothing to do with the finding max. Just for the sake of performance, I recommend using regular for loop to find the max if possible. It seems like you just create another list from original and then reiterate to get the max. The list seems can be quite big imo. Looks like you can get the number relatively easy. Don't be obsessed with those fancy shortcut unless you want to reuse the max list somewhere else. — Yunhai
– Yunhai, Commented Dec 24, 2020 at 9:22

Nikhil Patil · Accepted Answer · 2020-12-24 09:44:28Z

1

You can use apply function to do that -

var fncs = [{
 n_fnc: 1
},{
 n_fnc: 499
},{
 n_fnc: 99
},{
 n_fnc: 10
}];

var max = Math.max.apply(Math, fncs.map((m) => m.n_fnc));
console.log(max);

The code block in fnc.component.ts should be like -

this.fncs=[];
this.fncservice.obs.subscribe((response)=> {        
    this.fncservice.getlastid(response,this.fncs);
    console.log("A",this.fncs);
    var max= Math.max.apply(Math, this.fncs.map((m) => m.n_fnc));
    console.log("Max",max);
});

edited Dec 24, 2020 at 9:44

answered Dec 24, 2020 at 9:02

Nikhil Patil

2,5331 gold badge9 silver badges20 bronze badges

Sign up to request clarification or add additional context in comments.

6 Comments

isawid Over a year ago

I have fncs table as table of object . this.fncs=[]; this.fncservice.obs.subscribe((response)=>this.fncservice.getlastid(response,this.fncs)); var max= Math.max.apply(Math, this.fncs.map((m) => m.n_fnc)); I tried this but she return Infinity in max value

Nikhil Patil Over a year ago

Can you post structure of your array?

Nikhil Patil Over a year ago

@isawid, you are calling Math.max.apply outside of subscribe. It should be inside of subscribe block.

isawid Over a year ago

How I would do it ,please?

Nikhil Patil Over a year ago

@isawid, I have updated my answer with fnc.component.ts snippet.

|

Ravi Ashara · Accepted Answer · 2020-12-24 09:30:03Z

0

You can use apply function to do that -

let fncs = [{
 n_fnc: 1
},{
 n_fnc: 499
},{
 n_fnc: 99
},{
 n_fnc: 10
}];

let mytemp = fncs.sort((a, b) =>  { return b.n_fnc > a.n_fnc ? 1 : -1; });

console.log(mytemp[0]);

edited Dec 24, 2020 at 9:30

answered Dec 24, 2020 at 9:22

Ravi Ashara

1,2169 silver badges17 bronze badges

1 Comment

isawid Over a year ago

error in function(a,b) ... Type 'boolean' is not assignable to type 'number'.

Collectives™ on Stack Overflow

Type script get max array

2 Answers 2

6 Comments

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

6 Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related