Amigo, acredito que se performance for uma preocupação sua, seria mais interessante - e sucinto - utilizar um Regex para esta busca.
A resposta do @Pedro Paulo é interessante, porém utiliza vários processos que podem ser otimizados com o uso do Regex.
static List<string> BuscarPalavras(string consulta, string regex = @"(?<!\w)@\w+")
{
var resultado = new List<string>();
foreach (Match match in Regex.Matches(consulta, regex))
resultado.Add(match.Value);
return resultado;
}
Cada if
e for
removidos são complexidade a menos na aplicação.
Dessa forma a função tem duas entradas, sua consulta e o padrão do regex que já vem com valor default se tornando um parametro opcional, mas que pode ser modificado para buscar outros tipos de palavras. E um valor de saída apenas, no caso a lista de palavras.
Se tiver dificuldade com Regex, você pode jogar o valor que está no código no Regex101 que ele vai te explicar o que ele ta fazendo com mais detalhes, e você pode até modificá-lo.
Espero ter ajudado. :)