На сколько целесообразно изменять состояние в shouldComponentUpdate?

1.Нарушает логику использования фреймворка - что само по себе очень плохо, также может привести к бесконечной рекурсии(нужно смотреть исходники).
2.Не нужно пытаться всевозможные переменные, которые используется при рендеринге пихать в стейт. Просто отрефактори свой код следующим образом: вместо непосредственного обращения кthis.state.val1вызывай что-то в духеthis.getVal1()
что может быть реализованно вот так:

class MyComp extends React.Component {
   ...
  getVal1() {
    return this.state.val2 == 1;
  }
}