Calculer le Pourcentage d'Augmentation et de Diminution

Je suis probable de penser cela, mais je suis à la recherche de quelque chose d'un peu plus élégant que ce que j'ai actuellement.

J'ai cette méthode est appelée à CalculatePercentageTrend, qui prend en long "précédent", "actuel" de la valeur. Tel qu'il est actuellement :

public static string CalculatePercentageTrend(long previousValue, long currentValue)
{
    var trend = "0.00%";

    //calculate percentage increase
    if (previousValue < currentValue)
    {
        if (previousValue != 0)
        {
            var difference = previousValue - currentValue;
            var pctDecrease = (double)(difference / previousValue) * 100;
            trend = pctDecrease.ToString("P");
        }
        else
        {
            trend = currentValue.ToString("P");
        }
     }
    //calculate percentage decrease
    else if (currentValue < previousValue)
    {
        if (previousValue != 0)
        {
            var difference = currentValue - previousValue;
            var pctIncrease = (double)(difference / previousValue) * 100;
            trend = pctIncrease.ToString("P");
        }
        else
        {
            trend = currentValue.ToString("P");
        }
    }
    return trend;
}

C'est très répétitif, et qu'il a quelques lacunes. Les tendances négatives ne sont pas correctement pris en compte, comme ils l'ont toujours une 0.00% variation - ce qui serait formidable si je pouvais obtenir un pourcentage négatif SI, en fait, la valeur précédente est supérieur à la valeur actuelle.

Aussi, je suis toute manipulation possible de 0 avant de le diviser, mais je me demandais si il y a une meilleure approche.

Ma Question :

Comment puis-je obtenir négatif pourcentages pour calculer correctement, et comment puis-je améliorer ce code?

Vous devez vous débarrasser de votre premier si d'autre si. Il suffit de vérifier si une variable est de 0. Alors n' (précédent - actuel) / précédente * 100. Ce sera le retour de positif si ce n'est une augmentation et négatif s'il s'agit d'une diminution.
Au moins, le changement (double)(difference / previousValue) * 100 à ((double) difference / previousValue) * 100;
Important de comprendre que vous ne devez PAS multiplier ce résultat par 100 si vous avez l'intention de les utiliser dans un mode autre que d'afficher un plus "amical" de la valeur à l'utilisateur. 31% == .31, 31% != 31. 31 == 3100%.

OriginalL'auteur X3074861X | 2014-11-25

  1. 15

    Vous vous trouvez sur la pensée.

    double CalculateChange(long previous, long current)
{
    if (previous == 0)
        throw new InvalidOperationException();

    var change = current - previous;
    return (double)change / previous;
}

    Pour afficher ces données à un utilisateur dans une ambiance conviviale pourcentage de format, vous pouvez utiliser:

    string DoubleToPercentageString(double d)
{
    return "%" + (Math.Round(d, 2) * 100).ToString();
}
    vous devez multiplier le résultat par 100 pour obtenir le nombre en pourcentage
    C'est strictement un affichage de préoccupation. Si vous multipliez ce résultat par 100, puis essayez de l'utiliser pour fonctionner sur n'importe quoi, vous n'obtiendrez pas le résultat souhaité. 50% == .5
    Oui, mais sa méthode indique la réponse sera retourné comme un pourcentage n'est pas un nombre décimal.
    Encore une fois, 50% est égal à 5, pas 50.
    1 mètre == 100 cm, mais si une fonction a été nommé GetMeters je m'attends à ce qu'il revienne 1 pas 100

    OriginalL'auteur Michael

  2. 2

    Je pense que cela permettra de résoudre votre problème, vous n'avez besoin que le calcul de la différence et du rapport à l'égard de la valeur précédente.

    public static string CalculatePercentageTrend(long previous, long current)
{
    return ((current - previous) / (double)previous).ToString("p");
}

    OriginalL'auteur Hamid Pourjam

  3. 1

    J'ai utilisé la fonction suivante avec une précision de 100%.

        decimal CalculateChange(decimal previous, decimal current)
    {
        if (previous == 0)
            return 0;

        if (current == 0)
            return -100;

        var change = ((current - previous) / previous) * 100;

        return change;
    }

    OriginalL'auteur Sikandar Amla