Должен ли я всегда вызывать vector clear() в конце функции?

Если мы посмотрим на запись cppreference.com для std::vector::~vector говорит:

Разрушает контейнер. Вызываются деструкторы элементов и освобождается используемая память. Обратите внимание, что если элементы являются указателями, то объекты, на которые они указывают, не уничтожаются.

так что нет, вам не нужно вызывать clear.

Если мы хотим перейти к проекту стандарта, мы должны просмотреть раздел 23.2.1 Общие требования к контейнерам параграф 4, в котором говорится:

В таблицах 96 и 97 X обозначает класс-контейнер, содержащий объекты типа T, a и b обозначают значения типа X, [...]

а затем посмотрите на Table 96 — Container requirements, который имеет следующую запись выражения:

(&a)->~X()  

и следующее примечание:

примечание: деструктор применяется к каждому элементу a; вся память освобождается.

Обновить

Это RAII в действии, как говорит Бьерн Страуструп в Почему в C++ нет конструкции "finally"?:

Потому что C++ поддерживает альтернативу, которая почти всегда лучше: метод «получение ресурсов — это инициализация» (TC++PL3, раздел 14.4). Основная идея состоит в том, чтобы представить ресурс локальным объектом, чтобы деструктор локального объекта освободил ресурс. Таким образом, программист не может забыть освободить ресурс.

В этом нет абсолютно никакой необходимости. std::vector и все остальные контейнеры автоматически уничтожают свои элементы, когда они сами должны были быть уничтожены. Это означает, что их деструкторы несут ответственность за это действие. Так что не надо.

Прелесть этого в том, что контейнеры естественным образом защищены от исключений^[1]:

void someFunc(void) {

    std::vector<std::string> contentVector;

    // here are some operations on the vector

    throw std::runtime_error("I just want to throw!");

    contentVector.clear();
}

Будет ли линия contentVector.clear(); называться? Нет. Но вы по-прежнему в безопасности, потому что гарантированно будет вызван деструктор contentVector.

Из vector^[2]:

Разрушает контейнер. Вызываются деструкторы элементов и освобождается используемая память. Обратите внимание, что если элементы являются указателями, то объекты, на которые они указывают, не уничтожаются.

^[1] _{Однако вам все равно нужно сделать ваши элементы безопасными для исключений (чтобы они правильно освобождали свои ресурсы всякий раз, когда вызываются их деструкторы).}

^[2] _{См. комментарии ниже, в которых содержатся некоторые мысли о документах SGI STL}.

Нет необходимости, он будет очищен автоматически, как только выйдет за рамки, т.е. деструктор уничтожит контейнерный объект.

Вы можете не использовать функцию .clear(), потому что деструктор вектора запускается один раз contentVector выходит за рамки в «}».

Это освобождает память, в которой хранятся данные вектора.

Поскольку я не думаю, что кто-то еще упоминал об этом, но если ваш вектор был

std::vector<std::string*> vector;

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

for (auto i : vector) {
  delete i;
}