Loops: Problem5 - Calculate - Софтуерен университет

val.minkov 0 Точки

Loops: Problem5 - Calculate

Всичко работи добре на пръв поглед, но ми дава грешни резултати...
static void Main(string[] args)
{
int x, n;
int result = 1;
n = int.Parse(Console.ReadLine());
x = int.Parse(Console.ReadLine());
double s=1;
int[] factorials = new int[15]; // съхранявам факториелите на числата от 1 до 16
for (int i = 0; i < n; i++)
{
for (int j = 1; j < i; j++) // с върешния цикъл откривам факториелите
{
result = result * j;
}
factorials[i] = result; // с външния ги запазвам в масива
}
for (int m = 0; m <= n; m++) // в този цикъл извършвам изчисленията
{
s = s + (factorials[m] / Math.Pow(x, m));
}
string sum = s.ToString();
String.Format("{0:0.00000}", sum); //форматирам сумата да се извежда до 5 знака след запетаята
Console.WriteLine(sum);
}

Тагове:

10/12/2015 16:40:05 Programming Basics

Filkolev 4482 Точки

Факториел над 25-30 препълва всички примитивни типове данни, включително unsigned long. Най-добре е да се ползва BigInteger за подобни сметки. В твоя случай не е задължително, но ограничаваш програмата до малки стойности (не знам в условието какво е казано, ако има подобно ограничение няма проблем).

На s викаш ToString(), след това викаш string.Format(), но резултата от метода не го записваш никъде. Друг проблем е, че подаваш стринг и се опитваш да го форматираш като число. Стандартно подаваш числото на Console.WriteLine() и вътре с плейсхолдъра задаваш и форматирането: Console.WriteLine("{0:F5}", sum)

Преименувай си някои от променливите (s, i, j, m), не е ОК да е нужно човек да чете коментарите, за да разбере какво точно се случва. Също така коментари на български - не.

10/12/2015 16:58:50

val.minkov 0 Точки

Благодаря!

P.S.: Коментарите ги добавях при качването тук, във форума.

10/12/2015 17:04:19 10/12/2015 17:15:11

Filkolev 4482 Точки

Пробвай да пазиш факториелите в масив от decimal и направи принтирането направо в Console.WriteLine(). Ако сметките са верни би трябвало да изкара резултатите, които се очакват.

10/12/2015 17:10:41

val.minkov 0 Точки

Добре, обаче ми даде гршка с кастването при Math.Pow(x,m);

10/12/2015 18:31:18

Покажи всички коментари

ppbaev 157 Точки

Здрасти. След като запазиш поредна стойност на факториел в масива, трябва да ресетваш променливата result пак на 1 и тогава работи донякъде кода.

След factorials[i] = result; добавяш result = 1;

10/12/2015 17:07:57

gabi.ivanova 370 Точки

Не схващам идеята масива да е с фиксирана големина, ако става въпрос за loops от programming basics?

S = 1 + 1!/x + 2!/x^2 + … + n!/x^n. трябва да се изчисли факториел на числата от 1 до n, ако искаш масив може да го направиш с размер [n] или без масив като вкараш всичко в един цикъл

 for (int i = 1; i <= n; i++)
    {
       nFactoriel *= i;
       double xPow = Math.Pow(x, i);
       sum += (xFactoriel / xPow);
    }

10/12/2015 18:11:43