Expression has changed after it was checked

Question

Why is the component in this simple plunk    Component     selector   my-app     template    lt div gt I m   message    lt  div gt       export class App     message string    loading         ngAfterViewInit         this updateMessage           updateMessage        this message    all done loading             throwing      EXCEPTION  Expression  I m   message    in App 0 5  has changed after it was checked  Previous value   I m loading       Current value   I m all done loading      in  I m   message    in App 0 5    when all I m doing is updating a simple binding when my view is initiated

User · Answer

I couldnt comment on  Biranchi s post since I dont have enough reputation  but it fixed the problem for me   One thing to note  If adding changeDetection  ChangeDetectionStrategy OnPush on the component didn t work  and its a child component  dumb component  try adding it to the parent also    This fixed the bug  but I wonder what are the side effects of this

User · Answer

This error is coming because existing value is getting updated immediately after getting initialized  So if you will update new value after existing value is rendered in DOM  Then it will work fine Like mentioned in this article Angular Debugging  Expression has changed after it was checked   for example you can use  ngOnInit         setTimeout      gt            code for your new value               or   ngAfterViewInit       this paginator page        pipe            startWith null             delay 0             tap      gt  this dataSource loadLessons               subscribe        As you can see i have not mentioned time in setTimeout method  As it is browser provided API  not a JavaScript API  So this will run seperately in browser stack and will wait till call stack items are finished    How browser API envokes concept is explained by Philip Roberts in one of Youtube video What the hack is event loop

User · Answer

It throw an error because your code get updated when ngAfterViewInit   is called  Mean your initial value got changed when ngAfterViewInit take place  If you call that in ngAfterContentInit   then it will not throw an error   ngAfterContentInit         this updateMessage

User · Answer

ngAfterViewChecked   worked for me   import   Component  ChangeDetectorRef   from   angular core     import ChangeDetectorRef  constructor private cdr  ChangeDetectorRef      ngAfterViewChecked         your code to update the model    this cdr detectChanges

User · Answer

I got similar error while working with datatable  What happens is when you use  ngFor inside another  ngFor datatable throw this error as it interepts angular change cycle  SO instead of using datatable inside datatable use one regular table or replace mf data with the array name  This works fine

User · Answer

I think a most simple solution would be as follows    Make one implementation of assigning a value to some variable i e  via function or setter  Create a class variable  static working  boolean  in the class where this function exists and every time you call the function  simply make it true whichever you like  Within the function  if the value of working is true  then simply return right away without doing anything  else  perform the task you want  Make sure to change this variable to false once the task is completed i e  at the end of the line of codes or within the subscribe method when you are done assigning values

User · Answer

For this I have tried above answers many does not work in latest version of Angular  6 or later   I am using Material control that required changes after first binding done       export class AbcClass implements OnInit  AfterContentChecked          constructor private ref  ChangeDetectorRef             ngOnInit                   your tasks                   ngAfterContentChecked                 this ref detectChanges                      Adding my answer so  this helps some solve specific issue

User · Answer

I switched from AfterViewInit to AfterContentChecked and It worked for me  Here is the process  Add dependency in your constructor  constructor  private cdr  ChangeDetectorRef      and call your login in implemented method code here   ngAfterContentChecked          this cdr detectChanges         call or add here your code

User · Answer

In my case  it happened with a p-radioButton  The problem was that I was using the name attribute  which wasn t needed  alongside the formControlName attribute like this   lt p-radioButton formControlName  quot isApplicant quot  name  quot isapplicant quot  value  quot T quot  label  quot Yes quot  gt  lt  p-radioButton gt   lt p-radioButton formControlName  quot isApplicant quot  name  quot isapplicant quot  value  quot T quot  label  quot No quot  gt  lt  p-radioButton gt   I also had the initial value  quot T quot  bound to the isApplicant form control like this  isApplicant    quot T quot    I fixed the problem by removing the name attributes in the radio buttons  Also  because the 2 radio buttons have the same value  T  which is wrong in my case  simply changing one of then to another value  say F  also fixed the issue

User · Answer

Can t you use ngOnInit because you just changing the member variable message   If you want to access a reference to a child component  ViewChild ChildComponent   you indeed need to wait for it with ngAfterViewInit   A dirty fix is to call the updateMessage   in the next event loop with e g  setTimeout   ngAfterViewInit       setTimeout      gt        this updateMessage         1

User · Answer

As stated by drewmoore  the proper solution in this case is to manually trigger change detection for the current component  This is done using the detectChanges   method of the ChangeDetectorRef object  imported from angular2 core   or its markForCheck   method  which also makes any parent components update  Relevant example   import   Component  ChangeDetectorRef  AfterViewInit   from  angular2 core    Component     selector   my-app     template    lt div gt I m   message    lt  div gt       export class App implements AfterViewInit     message  string    loading         constructor private cdr  ChangeDetectorRef        ngAfterViewInit         this message    all done loading         this cdr detectChanges             Here are also Plunkers demonstrating the ngOnInit  setTimeout  and enableProdMode approaches just in case

User · Answer

First  note that this exception will only be thrown when you re running your app in dev mode  which is the case by default as of beta-0   If you call enableProdMode   when bootstrapping the app  it won t get thrown  see updated plunk     Second  don t do that because this exception is being thrown for good reason  In short  when in dev mode  every round of change detection is followed immediately by a second round that verifies no bindings have changed since the end of the first  as this would indicate that changes are being caused by change detection itself   In your plunk  the binding   message   is changed by your call to setMessage    which happens in the ngAfterViewInit hook  which occurs as a part of the initial change detection turn  That in itself isn t problematic though - the problem is that setMessage   changes the binding but does not trigger a new round of change detection  meaning that this change won t be detected until some future round of change detection is triggered somewhere else    The takeaway  Anything that changes a binding needs to trigger a round of change detection when it does    Update in response to all the requests for an example of how to do that   Tycho s solution works  as do the three methods in the answer  MarkRajcok pointed out  But frankly  they all feel ugly and wrong to me  like the sort of hacks we got used to leaning on in ng1     To be sure  there are occasional circumstances where these hacks are appropriate  but if you re using them on anything more than a very occasional basis  it s a sign that you re fighting the framework rather than fully embracing its reactive nature    IMHO  a more idiomatic   Angular2 way  of approaching this is something along the lines of   plunk    Component     selector   my-app     template    lt div gt I m   message   async    lt  div gt      export class App     message Subject lt string gt    new BehaviorSubject  loading          ngAfterViewInit         this message next  all done loading

User · Answer

You just have to update your message in the right lifecycle hook  in this case is ngAfterContentChecked instead of ngAfterViewInit  because in ngAfterViewInit a check for the variable message has been started but is not yet ended   see   https   angular io docs ts latest guide lifecycle-hooks html   afterview  so the code will be just   import   Component   from  angular2 core    Component     selector   my-app     template    lt div gt I m   message    lt  div gt       export class App     message  string    loading         ngAfterContentChecked          this message    all done loading                   see the working demo on Plunker

User · Answer

I fixed this by adding ChangeDetectionStrategy from angular core   import    Component  ChangeDetectionStrategy   from   angular core    Component     changeDetection  ChangeDetectionStrategy OnPush    selector   page1     templateUrl   page1 html

User · Answer

You can also put your call to updateMessage   in the ngOnInt  -Method  at least it works for me  ngOnInit         this updateMessage        In RC1 this does not trigger the exception

User · Answer

You can also create a timer using the rxjs Observable timer function  and then update the message in your subscription                                          Observable timer 1  subscribe     gt  this updateMessage

User · Answer

The article Everything you need to know about the ExpressionChangedAfterItHasBeenCheckedError error explains the behavior in great details   The problem with you setup is that ngAfterViewInit lifecycle hook is executed after change detection processed DOM updates  And you re effectively changing the property that is used in the template in this hook which means that DOM needs to be re-rendered     ngAfterViewInit         this message    all done loading         needs to be rendered the DOM       and this will require another change detection cycle and Angular by design only runs one digest cycle   You basically have two alternatives how to fix it    update the property asynchronously either using setTimeout  Promise then or asynchronous observable referenced in the template perform the property update in a hook before the DOM update - ngOnInit  ngDoCheck  ngAfterContentInit  ngAfterContentChecked

[typescript] Expression ___ has changed after it was checked

Examples related to typescript

Examples related to angular