Никита Михайлов

.comment-form-rating .stars:first-of-type {
  display: none;
}

Так уберёшь первый элемент

Работа с DOM само по себе дорогостоящее занятие. Любые повторяющиеся события внутри DOM могут приводить к проблеме с производительностью.

В чём смысл каждый раз устанавливать стили для .ucity при условии, что scrollTop не добрался до нужного значения? Эти стили можно задать изначально в CSS. Уже одним изменением меньше. Всё остальное уберите в отдельные классы. Проверяйте, есть ли такой класс при scrollTop() < ucity_prop.pos.top и если что — удаляйте класс

Кстати можно ещё дополнительно посмотреть в сторону https://developer.mozilla.org/ru/docs/Web/CSS/will...

Ищите по основной структуре каталогов. Она должна быть такая

Циклы, объекты, массивы будут нужны везде. Чем дальше вы будете углубляться в логику приложений, тем чаще это всё будет нужно.

Нет ничего страшного в том, чтобы перейти на фреймворк. Возьмите любой из популярных и тренируйтесь. Подтянете сам язык и разберётесь со сложными вещами. Поймёте назначение разных типов данных и научитесь ими управлять (в том числе и с помощью циклов).