javascript - how to stop array.filter() function - Stack Overflow

admin管理员组

文章数量:1345895

Am using custom search filter

HtML

 <input type="text" pInputText class="ui-widget ui-text" [(ngModel)]
 ="gloablFilterValue" (ngModelChange)="splitCustomFilter()" placeholder="Find" />

I am using ngModelChange() event on search box

---

globalSearch(realData, searchText, columns) {
        searchText = searchText.toLowerCase();
        return realData.filter(function (o) {
            return columns.some(function (k) {
                return o[k].toString().toLowerCase().indexOf(searchText) !== -1;
            });
        });
    }

    splitCustomFilter() {
    let columns =  
   ['PartNoCompleteWheel', 'DescriptionCompleteWheel', 'PartNoTyre', 'DescriptionTyre', 'PartNoRim', 'DescriptionRim','DeletedDateFromKDPStr', 'DateFromKDPStr', 'Status'];
   this.tyreAndRimList = this.globalSearch(this.tyreAndRimList, this.gloablFilterValue, columns); 
    }

Am using custom search filter

HtML

 <input type="text" pInputText class="ui-widget ui-text" [(ngModel)]
 ="gloablFilterValue" (ngModelChange)="splitCustomFilter()" placeholder="Find" />

I am using ngModelChange() event on search box

---

globalSearch(realData, searchText, columns) {
        searchText = searchText.toLowerCase();
        return realData.filter(function (o) {
            return columns.some(function (k) {
                return o[k].toString().toLowerCase().indexOf(searchText) !== -1;
            });
        });
    }

    splitCustomFilter() {
    let columns =  
   ['PartNoCompleteWheel', 'DescriptionCompleteWheel', 'PartNoTyre', 'DescriptionTyre', 'PartNoRim', 'DescriptionRim','DeletedDateFromKDPStr', 'DateFromKDPStr', 'Status'];
   this.tyreAndRimList = this.globalSearch(this.tyreAndRimList, this.gloablFilterValue, columns); 
    }

The this.tyreAndRimList list of values for the columns which is mentioned in a column variable.

Problem

The filter is working good! But the main problem is filter performance is very poor while the record count is huge(more than 100 rows per every column)

When

The filter is working good if am entering a single character (like a). But when I was typing the character continuously the browser is hanging. the reason is the filter has been firing every typing on the filter box(because of am using ngModelChange()// onchange() event)

What I want

I want to stop filtering if the user typing continuously on the search box. Once the user has stop the typing then only I need to start filtering.

What I did

I have tried to handle this by using setTimeout(). But it just wait the filter call for a second. It is working if the user entered just 2 or 3 character's continuously. But if the user typing more than 7 or 8 or above character's, it continues to hang the browser. because of many filter callbacks are processing on the same time.

 setTimeout(() => //code of filtering ,1000);

Question

How to stop filtering while user continuously entering value and start the filtering once the user has been stop the typing?

I am working in angular-2 and typescript. But this question is not related with only for angularjs or angular or JavaScript or typescript because of I want an idea not a solution. So I'll add those all tags for this question. Don't remove it. Thanks

• javascript
• arrays
• angular
• typescript

Share Improve this question Follow edited Jan 18, 2018 at 10:31 Ramesh Rajendran asked Dec 11, 2017 at 15:05 Ramesh RajendranRamesh Rajendran 38.7k4949 gold badges159159 silver badges239239 bronze badges 8

• 9 You can't stop a filter before it's done. You can, on the other hand, use a for loop that you can break. Other possibility : debounce your filter – Jeremy Thille Commented Dec 11, 2017 at 15:09
• 5 you need to debounce the event. – Aᴍɪʀ Commented Dec 11, 2017 at 15:10
• So look at debounce and also if filtered, there is no need to filter the entire list when they typed one more character, just filter the filtered set. – epascarello Commented Dec 11, 2017 at 15:11
• Of course debounce, but on top of that you could probably optimise the code some more. For example you could(assuming the text is static) make a single concatenated string during init of all the cells per row. And then only check the concatenated string per row. That way you can already skip looping all columns+strtolower in your filter. – René Commented Dec 11, 2017 at 15:21
• 1 Try to take a look at stackoverflow./a/36849347/356380 – Whisher Commented Dec 11, 2017 at 15:26

 |  Show 3 more ments

4 Answers 4

Sorted by: Reset to default 2

Debounce the function. See how underscore does it here: Function Debouncing with Underscore.js.

You would then generate a debounced version of your function like this:

var globalSearchDebounced = _.debounce(globalSearch, 100, false);

It will only call after the user has stopped typing for at least one second.

It's not possible to interrupt the Array.filter method. Based on what you need you could handle this like this:

let timerId = null

function handleChange() {
  if(timerId) {
    clearTimeout(timerId)
  }

  timerId = setTimeout(() => /* Array.filter(...) */, 1000)
}

Explanation

Have a variable which will contain the timerId returned from the setTimeout function. Every time the model get changed the handleChange function will be called (in this example). The function checks if the variable which contains the timerId is set and contains a timerId, when the variable contains the timerId the clearTimeout function will be called to clear the previous timeout after that the handleChange creates a new timeout and assigns the timerId (returned from the setTimeout function) to the variable.

• Documentation for setTimeout
• Documentation for clearTimeout

Without underscore, and without a Timeout (that will trigger the whole Angular lifecycle by the way), you can use an Observable with the async pipe and a debounce.

In your global search function :

return Observable.of(/* filter here and return the filtered array */).debounceTime(1000)

In your list (that has to be somewhere I guess)

<list-item *ngFor="let x of myFilteredResults | async">...</list-item>

I have plete it by using Subject to debounceTime.

private subject = new Subject<string>()
ngOnInit() {
        this.subject.debounceTime(300).subscribe(inputText => {
        this.gloablFilterValue = inputText;
        this.splitCustomFilter();   // filter method  
        });    
    }

Now when I change the value in this.gloablFilterValue object by using change event. It just waiting for the end of event pletion.

本文标签： javascripthow to stop arrayfilter() functionStack Overflow