C'è un modo migliore per farlo?Query Linq: questa matrice contiene questa stringa?
public bool IsServiceRunning(string serviceName)
{
string[] services = client.AllServices();
return (from s in services
where s.Equals(serviceName, StringComparison.InvariantCultureIgnoreCase)
select s).Count() > 0;
}
L'insensibilità del case nel confronto è importante.
utilizzare il metodo di estensione Any LINQ:
public bool IsServiceRunning(string serviceName)
{
string[] services = client.AllServices();
return services.Any(s =>
s.Equals(serviceName, StringComparison.InvariantCultureIgnoreCase));
}
In questo modo, non appena viene trovata una corrispondenza, l'esecuzione si fermerà.
Prova:
return services.Any(s =>
s.Equals(serviceName, StringComparison.InvariantCultureIgnoreCase));
Grandi menti, grandi menti. Giusto fino all'interruzione della linea. – recursive
@recursive: sì, ho visto che la barra di scorrimento orizzontale e il resto sono cronologia :) –
Un'alternativa non LINQ:
public bool IsServiceRunning(string serviceName)
{
string[] services = client.AllServices();
return Array.Exists(services,
s => s.Equals(serviceName, StringComparison.InvariantCultureIgnoreCase));
}
ne dici di questo?
public bool IsServiceRunning(string serviceName)
{
string[] services = client.AllServices();
foreach(string service in services)
{
if(service.Equals(serviceName, StringComparison.OrdinalIgnoreCase))
{
return true;
}
}
return false;
}
Davvero, è così semplice, ora tornare al lavoro risolvere reali problemi. ;)
Ya ya buon punto, ma mi interessa anche imparare le funzionalità della lingua :). – noctonura
Come è più chiaro di un semplice ciclo foreach è oltre me. –
Penso che le soluzioni proposte siano ugualmente chiare per un ciclo foreach. – recursive
Io no, ma a ciascuno credo. –