Ну, моя критика состояла бы в том, что вы делаете слишком много встроенных вещей. map
дает вам новый массив, так что я предполагаю, что это нормально (setState
подтвердит изменение, если обертка - это другая ссылка), но порядок операций в вашей последней строке, вероятно, вызывает проблемы.
.sort((a,b) => (a.completed === b.completed)? 0 : a? -1 : 1)
Я бы попытался поместить это в блоки фигурных скобок, и вы можете проверить a.completed
. Я думаю, что главная проблема в том, что он оценивает вашу (a,b) => (a.completed === b.completed)
как правдивую вещь