В приложении ASP.NET обнаружен недоступный код

Question

Я получаю сообщение «Обнаружен недоступный код» в Visual Studio 2010 для метода поиска в приложении ASP.NET. Это метод:

 public ActionResult SearchIndex(string artist, string albumGenre, string searchString)
    {
        var GenreList = new List<string>();

        var GenreQuery = from d in storeDB.Albums orderby d.Genre.Name select          d.Genre.Name;
        GenreList.AddRange(GenreQuery.Distinct());
        ViewBag.albumGenre = new SelectList(GenreList);

        var ArtistList = new List<string>();

        var ArtistQuery = from a in storeDB.Artists orderby a.Name select a.Name;
        ArtistList.AddRange(ArtistQuery.Distinct());
        ViewBag.artist = new SelectList(ArtistList);

        var albums = from m in storeDB.Albums select m;

        if (string.IsNullOrEmpty(artist))
        {
            return View(albums);
        }
        else
        {
            return View(albums.Where(f => f.Artist.Name == artist));
        }

        if (!String.IsNullOrEmpty(searchString))
        {
            return View(albums.Where(s => s.Title.Contains(searchString)));
        }

        if (string.IsNullOrEmpty(albumGenre))
        {
            return View(albums);
        }
        else
        {
            return View(albums.Where(x => x.Genre.Name == albumGenre));
        }
  }

Для этого утверждения я получаю сообщение:

       if (!String.IsNullOrEmpty(searchString))
        {
            return View(albums.Where(s => s.Title.Contains(searchString)));
        }

Где я иду не так?

Answer 1

, потому что в предыдущем блоке if-else вы обязательно вернетесь и откажетесь от своего метода, либо из if, либо из блока else.

так что ваш следующий код никогда не будет выполнен ни в коем случае.

Answer 2

Предыдущий if имеет оператор return в обеих своих ветвях, поэтому ваша функция всегда будет возвращаться до достижения этой второй if.

Answer 3

Вот исправленный код

public ActionResult SearchIndex(string artist, string albumGenre, string searchString)
  {
      var GenreList = new List<string>();

      var GenreQuery = from d in storeDB.Albums orderby d.Genre.Name select          d.Genre.Name;
      GenreList.AddRange(GenreQuery.Distinct());
      ViewBag.albumGenre = new SelectList(GenreList);

      var ArtistList = new List<string>();

      var ArtistQuery = from a in storeDB.Artists orderby a.Name select a.Name;
      ArtistList.AddRange(ArtistQuery.Distinct());
      ViewBag.artist = new SelectList(ArtistList);

      var albums = from m in storeDB.Albums select m;

      if (!string.IsNullOrEmpty(artist))
      {
          return View(albums.Where(f => f.Artist.Name == artist));
      }
      else if (!String.IsNullOrEmpty(searchString))
      {
          return View(albums.Where(s => s.Title.Contains(searchString)));
      }
      else if (!string.IsNullOrEmpty(albumGenre))
      {
          return View(albums.Where(x => x.Genre.Name == albumGenre));
      }
      else
      {
          return View(albums);
      }

}