我正在尝试从函数返回一个值。函数WcfProvider.MetalsPrices
可能会抛出异常。我想避免它。
public IEnumerable<PriceOfMetal> GetPrice(int id, DateTime time)
{
bool condition = false;
DateTime timenew = time.AddDays(-1);
var allPrice = from c in db.PriceOfMetal
select c;
foreach (var i in allPrice)
{
if (i.Date.Date == timenew.Date && i.ListOfMetaL_Id==id)
{
condition = true;
}
}
try
{
if (condition == false)
{
var price = WcfProvider.MetalsPrices(id, time, time).Tables[0].AsEnumerable()
.Select(
a =>
new PriceOfMetal()
{
Date = a.Field<DateTime>("Date"),
ListOfMetaL_Id = a.Field<int>("MetalId"),
Value = a.Field<System.Double>("Price")
})
.ToList().Single();
db.PriceOfMetal.Add(price);
db.SaveChanges();
}
}
finally
{
var all = from c in db.PriceOfMetal select c;
return all;
}
我想最后返回块的值。可能吗?我收到了错误。
答案 0 :(得分:4)
您可能需要这样的模式
try
{
return here
}
catch(Exception ex)
{
// Catch any error
// re throw if you choose,
// or you can return if you choose
return here
}
finally
{
// allways do whats here
}
您可能希望阅读此处的几个页面:try-catch-finally (C# Reference)
只是为了更多地构建这个,想象一下,如果我们可以在finally块中返回
你可能会遇到一些令人讨厌的代码,如下所示,这会让人感到困惑
try
{
return 10;
}
catch (Exception e)
{
return 20;
}
finally
{
return 30;
}
编译器会返回什么?
答案 1 :(得分:4)
如果内部发生异常,您必须决定您的函数是正常还是异常返回。
如果异常(您的来电者将看到异常):
try {
// do stuff
return answer;
}
finally {
// cleanup stuff
}
如果正常,您需要处理异常:
try {
// do stuff
}
catch {
// recover stuff
}
// cleanup stuff
return answer;
您永远不能在return
块中放置finally
语句,因为finally
在存在未捕获的异常时运行,并且当您的函数由于未捕获的异常而(异常)结束时,没有回报价值。
答案 2 :(得分:0)
我很抱歉这样说,但你的问题很模糊,很难回答。你的代码看起来很复杂。无论如何它是假期。也许下面会帮助你。但不保证。
public IEnumerable<PriceOfMetal> GetPrice(int id, DateTime time)
{
DateTime timenew = time.AddDays(-1);
var allPrice = from c in db.PriceOfMetal
select c;
where c.Date.Date == timenew.Date
and c.ListOfMetal_Id == id
if (!allPrice.Any())
{
try
{
var price = WcfProvider.MetalsPrices(id, time, time).Tables[0].AsEnumerable()
.Select(a =>new PriceOfMetal
{
Date = a.Field<DateTime>("Date"),
ListOfMetaL_Id = a.Field<int>("MetalId"),
Value = a.Field<System.Double>("Price")
})
.ToList().Single();
db.PriceOfMetal.Add(price);
db.SaveChanges();
}
catch
{
// Eating exceptions like this is really poor. You should improve the design.
}
}
return db.PriceOfMetal;
}