Não posso falar sobre a lógica correta porque não sei o que deve fazer, mas há um erro básico do código em si. É possível ter um erro de referência nula, dado a característica da condição ser short-circuit, isto pode ser facilmente resolvido com a inversão da condição:
protected void ImprimeValores(Ilist<int> values) => if (values != null && values.Count > 0) foreach(int v in values) WriteLine(string.Concat("value:", v));
Pra falar a verdade acho desnecessário verificar se values
é maior que zero, a não ser que depois pretenda fazer algo diferente, neste código, é redundância. Também acho completamente desnecessário usar o método de concatenação (o compilador fará isto por você). Não está errado, tem gente que gosta.
Nem falei do erro de sintaxe, este o compilador já mostra.
Veja funcionando no ideone. E no .NET Fiddle. Também coloquei no GitHub para referência futura.
Também é possível mudar este if
por um contrato e evitar a verificação em tempo de execução em muitos casos, mas isto já é outro assunto.