Différence entre une variable locale et une variable appelée à partir d'une méthode? C#

Question

Je voudrais savoir ce qui est plus rapide. Aide moi.

J'ai une variable déclarée dans une méthode comme ceci:

public static Regex FindNumber() 
{ return new Regex(@"\d+", RegexOptions.IgnoreCase | RegexOptions.Compiled); } 

Comme vous pouvez le voir renvoie une expression régulière.

J'ai aussi une autre méthode qui ressemble à ceci:

private static string TestOne(string RawData) 
{ 
    Regex rgxFindNumber = FindNumber(); 
    Regex rgxFindDays = FindDays(); 
    for (int i = 0; i < mc.Count; i++) 
    { 
     int days = Convert.ToInt32(rgxFindNumber.Match(rgxFindDays.Match(mc[i].Value).Value).Value); 
    } 
    return RawData; 
} 

maintenant est la méthode testone va être plus rapide ou est testtwo?

 private static string TestTwo(string RawData) 
{ 
    for (int i = 0; i < mc.Count; i++) 
    { 
     int days = Convert.ToInt32(FindNumber().Match(FindDays().Match(mc[i].Value).Value).Value); 
    } 
    return RawData; 
} 

maintenant im curieux parce que testone peut obtenir appelé beaucoup aweful dans mon code et je voudrais donc savoir ce qui serait mieux à mettre en œuvre.

Merci les gars.

** Edit: ** Le code que j'utilise a une classe extrêmement grande. C'est un analyseur de texte pour un jeu de stratégie basé sur le texte. J'essaie de refactoriser un peu et c'est ce que je me demande ici. Si je crée une variable privée pour le Regex, ne serait-il pas exécuté chaque fois que la classe est accédée? C'est ma question pour vous.

Answer 1

TestOne sera plus rapide que TestTwo, parce que vous n'êtes pas créer une nouvelle expression régulière pour chaque itération de la boucle.

Cela a deux avantages:

• Le temps utilisé pour analyser et construire les objets pour la regex est effectuée qu'une seule fois, au lieu de mc.Count fois
• moins de pression sur la collecte des ordures depuis moins d'objets sont construits.

Cependant, je voudrais aller un peu plus loin. Si vous renvoyez toujours cette même expression régulière et que vous êtes préoccupé par la vitesse, je mettrais cet objet regex en cache dans un champ statique.

Par exemple, vous pourriez considérer ceci:

private static Regex _FindNumber; 
public static Regex FindNumber() 
{ 
    if (_FindNumber == null) 
     _FindNumber = new Regex(@"\d+", RegexOptions.IgnoreCase | RegexOptions.Compiled); 
    return _FindNumber; 
} 

Cela créerait un seul objet, au total, et le garder autour.

Cependant, et voici ma vraie réponse.

Pour savoir lequel sera le plus rapide, vous devrez mesurer votre code, éventuellement avec ma variante lancée, puis décider. Ne décidez jamais des optimisations sans données concrètes, vous pourriez finir par passer du temps à réécrire du code, ce qui peut introduire de nouveaux bugs, qui nécessiteront des corrections, sur lesquelles vous passerez plus de temps, seulement 1% de performance supplémentaire.

Les grandes optimisations sont faites de manière algorithmique, comme changer le type d'algorithme de tri, et seulement après, si nécessaire, vous passez à des optimisations locales comme le réglage de boucle. Cela dit, j'éviterais au moins de construire l'objet dans la boucle, c'est du bon sens.

Answer 2

Je crois que TestOne sera plus rapide parce que dans TestTwo vous créez un nouvel objet Regex chaque fois que vous bouclez. Si FindDays est implémenté de la même manière que FindNumber, ce sera encore pire car vous allez créer deux objets.

Answer 3

Techniquement, TestOne sera plus rapide, car TestTwo ajoute un cadre de pile en appelant FindNumber().

Je ne sais pas combien de différence cela fera, je doute que ce soit autant. Votre méthode est statique, donc c'est juste la création de l'objet, ce qui devrait être assez rapide.

Ma question est pourquoi utilisez-vous un appel de fonction pour retourner la même chaîne encore et encore? Pourquoi ne déclarez-vous pas simplement une vraie variable?

Comme,

private static Regex _findNumber = new Regex(@"\d+", RegexOptions.IgnoreCase | RegexOptions.Compiled);