大家来看看这段程序的问题，谢谢！

[common_man]

程序如下：

void executeAction (const NamedValue * nvPtr)
{
try {
if (NULL != nvPtr)
nvPtr ->executeActions();
}
catch (StandardException exception) {
throw(exception);
}
}

NamedValue * sendRequest( const NamedValue * nvPtr)
{
NamedValue * nvResponsePtr = NULL;
try {
nvResponsePtr = new NamedValue (*nvPtr);
if (NULL != nvResponsePtr)
executeAction(nvResponsePtr);
}
catch (Exception e) {
delete nvPtr;
throw(e);
}
return nvResponsePtr;
}

void addTask()
{
NamedValue * nvPtr = NULL;
NamedValue * nvResponsePtr = NULL;

nvPtr = new NamedValue ("CreateTaskManager", "1.0");
nvResponsePtr = sendRequest(nvPtr);
if (nvResponsePtr) {
nvResponsePtr->print();
delete nvResponsePtr;
}
return;
}

void main()
{
……
addTask();
}

请问这段代码有问题吗？谢谢

[ouyh12345]

把问题说清楚点，没人会运行的

[caocheng8230]

如上

[healer_kx]

是啊,想运行代码也不全啊...

[jixingzhong]

贴的这个部分，
没有看出有什么问题 ～～

[cunsh]

void executeAction (const NamedValue * nvPtr)
{
try {
if (NULL != nvPtr)
nvPtr ->executeActions();
}
catch (StandardException exception) { //应该用引用StandardException &
throw(exception); //重新抛出应该用 throw;
}
}

[cunsh]

NamedValue * sendRequest( const NamedValue * nvPtr)
========
这个函数也可能抛出异常. 调用这个函数的代码应该放到 try { } 中
能抛出异常的函数应该在后边写上异常规范呀

[common_man]

to ouyh12345() :

考试的时候是否可以直接在题目下面写上：“把问题说清楚点，没人会运行的”，就可以交卷子了，呵呵

[common_man]

to healer_kx(甘草 春去春回,花落谁家?心寄笔端,谁解我意?) ：
注意，运行是用来验证答案而不是获取答案的，呵呵

[common_man]

也许是我没说清楚，是考试题目，呵呵，海涵

[搬不搬砖]

void addTask()
{
NamedValue * nvPtr = NULL;
NamedValue * nvResponsePtr = NULL;

nvPtr = new NamedValue ("CreateTaskManager", "1.0"); //new可能异常
nvResponsePtr = sendRequest(nvPtr);
if (nvResponsePtr) {
nvResponsePtr->print();
delete nvResponsePtr;
}
return;
}

[搬不搬砖]

new是有可能抛出异常的,所以要catch

[qq_sugar]

NamedValue * sendRequest( const NamedValue * nvPtr)
{
NamedValue * nvResponsePtr = NULL;
try {
nvResponsePtr = new NamedValue (*nvPtr);
if (NULL != nvResponsePtr)
executeAction(nvResponsePtr);
}
catch (Exception e) {
delete nvPtr;
throw(e);
}
return nvResponsePtr;
}

该函数中如果executeAction出现异常，nvResponsePtr 对象是否应该释放
另外在异常中是否应该释放传入的参数 nvPtr 对象？

感觉异常中应该 delete nvResponsePtr ，而不是nvPtr

[helanshan]

delete nvResponsePtr;
----------------------------
这个很明显有问题,不知道你delete 的哪个对象..