Как перенести математическую формулу в программу?

Question

[kos_dev]

Здравствуйте, есть формула


Верно ли я ее реализую в коде или ошибаюсь?

Код

for(int i = 0; i < foundLettersFrequency.size(); i++)
{
    double letterFrequency = foundLettersFrequency[i];
    double stringLength = 10; // Длина строки находится иначе. это для упрощения
    // Сама формула, на каждой итерации цикла - Верно ли?
    double IC = letterFrequency * (letterFrequency - 1) / stringLength * (stringLength - 1);
}

Comments

[Ринат Ильясов]

Вроде как верно)

Answer 1

[Rsa97]

Ошибаетесь. Деление и умножение - равноправные левоассоциативные операции.
И суммирования в вашем цикле нет.

Comments

[kos_dev]

А так?

double epsilon = 0;
for(int i = 0; i < foundLettersFrequency.size(); i++)
{
    double letterFrequency = foundLettersFrequency[i];
    double stringLength = 10; // Длина строки находится иначе. это для упрощения
    // Сама формула, на каждой итерации цикла - Верно ли?
    double IC = (letterFrequency * (letterFrequency - 1)) / (stringLength * (stringLength - 1));
    epsilon += IC; 
}

[Rsa97]

kos_dev, Так правильно. Можно ещё

double IC = letterFrequency * (letterFrequency - 1) / stringLength / (stringLength - 1);

А если stringLength внутри цикла не меняется, то можно заранее посчитать знаменатель или вообще вынести его за цикл.

Answer 2

[hint000]

1. объявление переменных внутри цикла режет глаз, фу.
2. IC не инициализирована нулём перед циклом, в результате переменная может иметь рандомное значение.
3. IC += ... вам же нужна сумма, а где у вас суммирование?
4.

double stringLength = 10; // Длина строки находится иначе. это для упрощения

вещественная длина строки??

Comments

[kos_dev]

Я обязательно приведу все в нормальный вариант, это просто набросок, чтобы показать примерно как реализовано.

Так соответствует формуле?

double epsilon = 0;
int stringLength = 10; // Длина строки находится иначе. это для упрощения
for(int i = 0; i < foundLettersFrequency.size(); i++)
{
    double letterFrequency = foundLettersFrequency[i];
    // Сама формула, на каждой итерации цикла - Верно ли?
    double IC = 0;
    IC = (letterFrequency * (letterFrequency - 1)) /  (stringLength * (stringLength - 1));
    epsilon += IC; 
}

[hint000]

kos_dev, не требуется ещё одна переменная.

double IC = 0;
int stringLength = 10; // Длина строки находится иначе. это для упрощения
for(int i = 0; i < foundLettersFrequency.size(); i++)
{
    double letterFrequency = foundLettersFrequency[i];
    // Сама формула, на каждой итерации цикла - Верно ли?
    IC += (letterFrequency * (letterFrequency - 1)) /  (stringLength * (stringLength - 1));
}