search

markparticle / WebServer

C++ Linux WebServer服务器
Apache License 2.0
3.58k stars 733 forks source link

关于RAII使用的疑惑 #35

Open modao233 opened 3 years ago

modao233 commented 3 years ago

你好，我看了你的项目，其中说到，数据库连接池使用了RAII，但是在httprequest.cpp实现的233行你为什么还是手动释放sql。

xiaoxixideyu commented 3 years ago

疑惑+1，这里只是用了一个临时的请求，请求完RAII对象就直接释放了，相应的连接就算被用着也直接进入就绪队列里面了。个人认为应该声明一个RAII局部变量，让程序去释放连接，不要主动去释放连接，不然RAII就没有意义了。

lovelyzzc commented 3 years ago

同意楼上

XueshiQiao commented 3 years ago

可以提个 PR

Withouthead commented 2 years ago

在181行，只是用了一个临时变量，让sql指针指向一个连接，然后这个临时变量就析构了，也就是这个连接又推回了mysql连接池中...这个写法我觉得有点问题

InternetofThingsprogrammer commented 2 years ago

你好，我看了你的项目，其中说到，数据库连接池使用了RAII，但是在httprequest.cpp实现的233行你为什么还是手动释放sql。

解决了?没有233行

CzYoL6 commented 2 years ago

image 这个匿名对象构造过后马上就析构了