C++的初级问题，送分

有函数
double Test(int p)
{
   double ret;   if (p == 1)
     ret = 1;
   else
     ret = 2;   return ret;
}这样写有没有隐患？
听朋友说最好不要以函数的局部变量做为返回，因为一旦函数运行完成后，局部变量就失效了，最好以指针参数的方式返回，不知道对不对，还请指教