指向函数参数传入的std::vector成员的最佳方式及相关疑问

嗨，Eduard！很高兴你第一次在Stack发帖，不用太担心规范问题，有问题随时调整就好 😊

我来逐一解答你的疑问：

1. 用指针指向vector元素是否属于不良实践？ #

这不算绝对的不良实践，但存在需要警惕的风险：

• 只要objects_1在你使用指针的期间没有发生内存重新分配（比如调用push_back/insert导致容量不足触发扩容），这些指针就是完全有效的，可以安全修改指向的元素。
• 但如果vector发生扩容，内部元素会被移动到新的内存地址，此时之前保存的指针就会变成野指针，访问它会导致未定义行为。

所以如果能保证这段时间内vector的大小不会触发扩容（比如提前调用reserve预留足够空间，或者这段逻辑里不修改vector的长度），这种做法是完全可行的。

2. 是否需要手动delete这些指针？ #

绝对不要手动delete！
这些指针指向的是vector容器管理的元素，vector会负责这些元素的内存生命周期：当vector被销毁时，它会自动调用元素的析构函数并释放对应的内存。你手动delete的话，相当于试图释放不属于你分配的内存，会直接导致未定义行为（程序崩溃、内存错乱等）。

3. 有没有更好的实现方式？ #

根据你的场景，有几种替代方案可以参考：

• 使用迭代器代替指针：迭代器的语义更符合STL的使用习惯，和指针一样，只要vector不扩容，迭代器就有效。用法和指针类似，比如auto it = std::find(objects_1.begin(), objects_1.end(), target);，之后通过*it修改元素。
• 给元素添加唯一标识符：如果无法避免vector扩容，或者需要长期的逻辑关联，可以给myClass添加一个唯一的ID字段（比如int id）。然后维护一个std::unordered_map<int, myClass*>，把ID和元素指针映射起来。这样即使vector扩容，只要ID不变，就能通过map快速找到对应的元素。注意要同步维护这个map（比如元素被删除时也要从map中移除）。
• 如果允许调整元素存储逻辑：如果后续有机会调整容器类型，也可以考虑使用std::vector<std::unique_ptr<myClass>>，这样元素的内存地址不会因为vector扩容而变化（因为vector存储的是指针，扩容时移动的是指针而非元素本身）。不过你提到函数签名不能改，这个方案可能暂时不适用。

另外你提到的std::shared_ptr，确实它更适合动态分配内存的场景，你的情况是vector存储的是实体对象，用shared_ptr反而会增加不必要的复杂度，所以不推荐。

内容的提问来源于stack exchange，提问作者Eduard Barnoviciu