Скорее всего, делать классы
view(1/2/3)
нет смысла, вполне можно обойтись одним. И
view(1/2/3)--active
тоже лишние. Единственное, что нужно - менять класс у обёртки каталога. Тогда код становится таким:
$('.view').click(function(){
var $el = $(this),
type = $el.data('view-type');
$('.view.view--active').removeClass('view--active');
$el.addClass('view--active');
$('.maicol-catalog').removeClass('catalog-view1 catalog-view2 catalog-view3');
$('.maicol-catalog').addClass('catalog-view' + type);
});
Ну и 1/2/3 нужно заменить на осмысленные слова: table/cards/feed или что там у вас по смыслу.