3

我要做的就是var place正确声明，所以一旦我进入foreach循环它仍然在范围内。我假设我需要在iffor 语句之前声明它connections。这是一个正确的假设吗？如果是，我该如何声明？谢谢！

using (var db = new DataClasses1DataContext())
        {
            if (connections == "Connections")
            {
                var place = (from v in db.pdx_aparts
                             where v.Latitude != null && v.Region == region && v.WD_Connect >= 1
                             select new
                             {
                                 locName = v.Apartment_complex.Trim().Replace(@"""", ""),
                                 latitude = v.Latitude,
                                 longitude = v.Longitude
                             }).Distinct().ToArray();
            }
            else
            {
                var place = (from v in db.pdx_aparts
                             where v.Latitude != null && v.Region == region &&    ((v.WD_Connect == null) || (v.WD_Connect == 0))
                             select new
                             {
                                 locName = v.Apartment_complex.Trim().Replace(@"""", ""),
                                 latitude = v.Latitude,
                                 longitude = v.Longitude
                             }).Distinct().ToArray();

            }

            foreach (var result in place)
            ....
4

3 回答 3

7

您可以创建一个包含单个条目的数组，该条目的值稍后会被忽略：

// Note: names *and types* must match the ones you use later on.
var place = new[] { new { locName = "", latitude = 0.0, longitude = 0.0 } };
if (connections = "Connections")
{
    // Note: not a variable declaration
    place = ...;
}
else
{
    place = ...;
}

这是有效的，因为每次使用具有相同名称和类型的属性的匿名类型，以相同的顺序，都将使用相同的具体类型。

我认为最好让代码只在它需要的部分有所不同：

var query = v.db.pdx_aparts.Where(v => v.Latitude != null && v.Region == region);
query = connections == "Connections"
    ? query.Where(v => v.WD_Connect >= 1)
    : query.Where(v => v.WD_Connect == null || v.WD_Connect == 0);
var places = query.Select(v =>  new
                          {
                              locName = v.Apartment_complex
                                         .Trim()
                                         .Replace("\"", ""),
                              latitude = v.Latitude,
                              longitude = v.Longitude
                          })
                  .Distinct()
                  .ToArray();

在这里更容易看出唯一取决于connections值的部分是查询中处理WD_Connect.

于 2013-01-15T23:20:35.350 回答
2

您可以将 转换if为?:.

var place = connections == "Connections" ? monsterQuery1 : monsterQuery2;

我认为这不是一个好的解决方案，因为您的查询太大（不可读）。

如果您引入一个您使用的命名类而不是匿名类型，那会好得多。R# 在“灯泡菜单”重构中为您做到这一点。

于 2013-01-15T23:18:20.637 回答
1

您可以只使用 1 查询，因为它们几乎相同，只需在 where 子句中添加额外的条件

var place = (from v in db.pdx_aparts
             where v.Latitude != null && v.Region == region 
             &&  connections == "Connections" 
             ? v.WD_Connect >= 1 
             : ((v.WD_Connect == null) || (v.WD_Connect == 0))
             select new
             {
                 locName = v.Apartment_complex.Trim().Replace(@"""", ""),
                 latitude = v.Latitude,
                 longitude = v.Longitude
             }).Distinct().ToArray();


 foreach (var result in place)
 ....
于 2013-01-15T23:23:56.677 回答