std::string *p = wa; 有什么不对

表达式 `std::string *p = wa;` 的潜在问题是它试图将指针 `p` 初始化为指向 `wa` 对象的地址，其中 `wa` 可能是一个 `std::string` 类型的变量，也可能不是。如果 `wa` 实际上是一个 `std::string` 类型的值，那么这会将 `p` 指向 `wa` 的内容；但如果 `wa` 是另一个类型（比如整数或者指针），那么这个赋值就会导致未初始化的指针引用，可能会引发运行时错误，因为尝试访问不确定的内存位置。

此外，这种写法也暗示着可能存在某种动态分配的情况，如果 `wa` 是动态分配的字符串，而没有使用 `new` 来分配，那么忘记手动释放内存可能导致内存泄漏。

正确的做法应该是先检查 `wa` 是否确实是一个 `std::string` 类型的指针，然后进行适当的指针初始化，并在必要时处理内存管理。例如：

if (wa && dynamic_cast<std::string*>(wa) != nullptr) {
    std::string* p = static_cast<std::string*>(wa);
    // 确保wa已经被正确分配
}

或者直接使用智能指针，如 `std::unique_ptr<std::string>`，避免这些问题：

std::unique_ptr<std::string> wa; // 或者 wa = std::make_unique<std::string>();
if (wa) {
    std::string* p = wa.get(); // 使用get()访问底层对象
}