Факториал числа вычисляется неверно

@bgm124 · Регистрация: 28.01.2012

Студворк — интернет-сервис помощи студентам

Почему-то показывает всегда нуль.Что-то не так или очень быстро это делается?

C#
1
2
3
4
5
6
7
8
9
10
11
   do
                {
                    Console.Write("Введите неотрицательное число: ");
                    match = Regex.Match(s = Console.ReadLine(), @"^[+]?[1-9]\d*$");
                } while (!match.Success);
                a = uint.Parse(s);
 
                sWatch.Start();
                Console.WriteLine("Факториал числа: " + MyMath.FactFor(a));
                sWatch.Stop();
                Console.WriteLine("Затраченное время: " + sWatch.ElapsedMilliseconds);

@kolorotur · 26.02.2012, 16:15

Выкладывайте код метода MyMath.FactFor - ошибка там.

Ну и избавьтесь от регекса - он не нужен.

@bgm124 · 26.02.2012, 17:25 **[ТС]**

Работает корректно

C#
1
2
3
4
5
6
 static public ulong FactFor(ulong a)
    {
        ulong i = 1;
        while (a != 0) i *= (a--);
        return i;
    }

Добавлено через 5 минут
Regex нужен для проверки правильногот ввода

@kolorotur · 26.02.2012, 17:30

Что-то я запутался.

Тема называется: "Факториал числа вычисляется неверно".
Вы говорите: "Работает корректно".
В первом сообщении у вас написано: "Что-то не так или очень быстро это делается?".

В чем проблема-то? В том, что факториал, якобы, вычисляется не верно, или в том, что затраченное время показывается как 0мс?
Во втором случае выводите свойство Elapsed - там и микро- и наносекунды покажет.

@bgm124 · 26.02.2012, 17:48 **[ТС]**

Ну я делаю вот так

C#
1
2
3
4
5
6
7
8
9
10
11
12
 case '2':
                do
                {
                    Console.Write("Введите неотрицательно число не превышающее 20: ");
                    match = Regex.Match(s=Console.ReadLine(), @"^[+-]?([\d]|[1-9]\d*)$");
                } while (!match.Success ||! ulong.TryParse(s, out a) || a > 20);
                sWatch.Start();
                Console.WriteLine("Факториал числа: " + MyMath.FactRec(a));
 
                  sWatch.Stop();
            Console.WriteLine("Время: " + sWatch.ElapsedMilliseconds.ToString());
                break;

Может оно очень маленькое??

Добавлено через 6 минут
А может так быть ,что время затраченное на вычисление факториала одного числа, отличается немного??

@kolorotur · 26.02.2012, 18:03

C#
1
Console.WriteLine("Время: " + sWatch.Elapsed);

@bgm124 · 26.02.2012, 18:26 **[ТС]**

Я делал так, как вы говорите.Делал так как на msdn.Но время всегда немного разное.Вот полный код программы, попробуйте выполнить код время всегда будет разным.
Программы вообще должна делать:
-факториал рекурсивно
-факториал в цикле
-сумма по формуле
-сумма в цикле
-решать уравнение
-переводить часы в минуты
при этом должны быть учтены любые ошибки (неверный ввод и т.п.)

По возможности просмотрите общий вид программы на наличие ошибок.

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
using System;
using System.Text.RegularExpressions;
using System.Diagnostics;
using System.Threading;
 
class Equation{
    int a, b, c;
    double x1, x2;
    bool any, have = true;   //Индикаторы решений
 
    public Equation(){
        Match match;
        string[] values;
 
        do
        {
            Console.Write("Введите коэффициенты уравнения (a,b,c):");
            match = Regex.Match(Console.ReadLine(), @"^[+-]?([\d]|[1-9]\d*),[+-]?([\d]|[1-9]\d*),[+-]?([\d]|[1-9]\d*)$");
            values = match.Groups[0].Value.Split(',');
        } while (!match.Success || !int.TryParse(values[0], out a) ||
             !int.TryParse(values[1], out b) || !int.TryParse(values[2], out c));
   }
 
    public bool Sgn(int a, int b)
    {
        if ((a > 0 && b > 0) || (a < 0 && b < 0)) return true;
        else return false;
    }
 
    public void Calculate()
    {
        if (a == 0 && b == 0 && c == 0)
            any = true;
 
        if (a == 0 && b == 0 && c != 0)
            have = false;
 
        if (a == 0 && b != 0 && c == 0)
            x1 = x2 = 0;
 
        if (a == 0 && b != 0 && c != 0)
            x1 = x2 = -(double)c / (double)b;
 
        if (a != 0 && b == 0 && c == 0)
            x1 = x2 = 0;
 
        if (a != 0 && b == 0 && c != 0)
            if (Sgn(a, c)) have = false;
            else x1 = x2 = -(double)c / (double)a;
 
        if (a != 0 && b != 0 && c == 0){
            x1 = 0;
            x2 = -(double)b / (double)a;
        }
 
        if (a != 0 && b != 0 && c != 0){
            double d = Math.Pow(b, 2) - 4 * a * c;
 
            if (d < 0) have = false;
            if (d >= 0){
                x1 = (-b + Math.Sqrt(d)) / (2 * a);
                x2 = (-b - Math.Sqrt(d)) / (2 * a);
            }
        }
    }
 
    public void showX()
    {
        if (any) Console.WriteLine("Решений бесконечно много.");
        if (!have) Console.WriteLine("Нет решений.");
        if (!any && have) Console.WriteLine("x1 = {0:f1}\nx2 = {1:f1}", x1, x2);
    }
}
 
class MyMath
{
    static public sbyte SignVal(long a) {
        if (a < 0) return -1;
        else return 1;
    }
   
    static public ulong FactRec(ulong a){
        if (a == 0) return 1;
        else return a * FactRec(a - 1);
    }
 
    static public ulong FactFor(ulong a){
        ulong i = 1;
        
        while (a != 0) i *= (a--);
        return i;
    }
 
    static public long Sum(long a){
        return Math.Abs(a) * (Math.Abs(a) + 1) / 2 *SignVal(a);
    }
 
    static public long SumFor(long a){
        long sum, i;
 
        for (sum = 0, i = 1; i <= Math.Abs(a); i++)
            sum += i;
        return sum*SignVal(a);
    }
}
 
class Clock{
    uint hours, min;
    
    public Clock(){
        Match match;
        string[] values;
        
        do {
                 Console.Write("Введите время (чч:мм): ");
                 match = Regex.Match(Console.ReadLine(), @"^[+-]?([\d]|[1-9]\d*);[+-]?([\d]|[1-9]\d*)$");
                 values = match.Groups[0].Value.Split (':');
         }while (!match.Success || !uint.TryParse(values[0],out hours)||
                !uint.TryParse(values[1],out min)|| min >60);
   }
 
    public void ShowClockMin(){
        Console.WriteLine("Время в минутах: " + (hours * 60 + min));
    }
}
 
class TryEx1{
    public static void Main(){
        char ch;
        Stopwatch sWatch = new Stopwatch();
       
        do
        {
            Console.WriteLine("Menu:\n" +
               " 1.Квадратное уравнение.\n" +
               " 2.Факториал рекурсивно.\n" +
               " 3.Факториал в цикле.\n" +
               " 4.Частичная сумма ряда (формула).\n" +
               " 5.Частичная сумма ряда в цикле.\n" +
               " 6.Перевести время в минуты\n" +
               " 0.Exit.\n");
           
        }while (!char.TryParse(Console.ReadLine(), out ch) || ch < '0' || ch > '6');
 
        if (ch == '0') Environment.Exit(0);
 
        ulong a;
        long sum;
        Match match;
        string s;
        TimeSpan ts;
 
        switch (ch)
        {
            case '1':
                Equation et = new Equation();
                
                et.Calculate();
                et.showX();
                break;
            
            case '2':
                do
                {
                    Console.Write("Введите неотрицательно число не превышающее 20: ");
                    match = Regex.Match(s=Console.ReadLine(), @"^[+-]?([\d]|[1-9]\d*)$");
                } while (!match.Success ||! ulong.TryParse(s, out a) || a > 20);
                
                sWatch.Start();
                a = MyMath.FactRec(a);
                sWatch.Stop();
                ts = sWatch.Elapsed;
 
                Console.WriteLine("Факториал числа: " + a);
                Console.WriteLine("RunTime " + ts.TotalMilliseconds);
              break;
           
            case '3':
                do
                {
                    Console.Write("Введите неотрицательно число не превышающее 20: ");
                    match = Regex.Match(s = Console.ReadLine(), @"^[+-]?([\d]|[1-9]\d*)$");
                } while (!match.Success ||! ulong.TryParse(s, out a) || a > 20);
 
                sWatch.Start();
                a = MyMath.FactFor(a);
                sWatch.Stop();
                ts = sWatch.Elapsed;
 
                Console.WriteLine("Факториал числа: " + MyMath.FactRec(a));
                Console.WriteLine("RunTime " + ts.TotalMilliseconds);
                break;
 
            case '4':
                do
                {
                    Console.Write("Введите число: ");
                    match = Regex.Match(s = Console.ReadLine(), @"^[+-]?([\d]|[1-9]\d*)$");
                } while (!match.Success ||! long.TryParse(s, out sum));
                
                
                sWatch.Start();
                sum=MyMath.Sum(sum);
                
                sWatch.Stop();
                ts = sWatch.Elapsed;
 
                Console.WriteLine("Частичная сумма ряда: " + sum);
                Console.WriteLine("RunTime " + ts.TotalMilliseconds);
                break;
                             
            case '5':
                do
                {
                    Console.Write("Введите число: ");
                    match = Regex.Match(s = Console.ReadLine(), @"^[+-]?([\d]|[1-9]\d*)$");
                } while (!match.Success ||! long.TryParse(s, out sum));
 
                sWatch.Start();
                sum=MyMath.SumFor(sum);
                
                sWatch.Stop();
                ts = sWatch.Elapsed;
 
                Console.WriteLine("Частичная сумма ряда: " + sum);
                Console.WriteLine("RunTime " + ts.TotalMilliseconds);
                break;
            case '6':
                Clock clock = new Clock();
                clock.ShowClockMin();
                break;
        }
    }
}

Добавлено через 1 минуту
И показывать конечно затраченное время

@kolorotur · 26.02.2012, 18:37

Так а что смущает-то? То, что время все время разное? Это нормально, в системе не одно ваше приложение работает. В этот раз удалось по-больше время ЦП цапнуть, а в следующий не вышло.

@buntar · 05.04.2012, 16:56

все гениальное просто!
вот вычисление факториала:

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
namespace ConsoleApplication2
{
    class Program
    {
        static void Main(string[] args)
        {
            Console.Write("Введите число для вычсления факториала = ");
            int num = Convert.ToInt32(Console.ReadLine());
            Fact(num);
            
            Console.ReadLine();
        }
 
        public static void Fact (int n)
        {
            int res=1;
            if (n<=1) res=1; 
            
            else 
            {
                for(int i=n; i>1; i--)
                    res*=i;
            }
            Console.WriteLine("Факториал \"{1}\" = {0}",res, n);
        }
    }
}

@OmgMrL · 07.04.2012, 15:37

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
 public int Fact1(int n)
        {
            int res;
            if (n == 1) return 1;
            res = Fact1(n - 1) * n;
            return res; 
        }
        public int Fact2(int n)
        {
            int t, res;
            res = 1;
            for (t = 1; t <= n; t++) res *= t;
            return res;
        }

Вот нашел два метода решения.

@kolorotur · 07.04.2012, 15:43

Для любителей LINQ:

C#
1
2
3
4
int Factorial(int n)
{
   return Enumerable.Range(1, n).Aggregate(1, (x, y) => x * y);
}

Не по теме:

Сообщение от OmgMrL

C#
1
2
int t
for (t = 1; t <= n; t++) res *= t;

Всегда был интересен смысл объявления переменной цикла за пределами этого самого цикла.

Новые блоги и статьи Все статьи Все блоги /
Отчёт о затраченных материалах за определенный период с макетом печатной формы Maks 21.04.2026 Отчёт из решения ниже размещён в конфигурации КА2. Задача: разработка отчёта по затраченным материалам за определённый период, с возможностью вывода печатной формы отчёта с шапкой и подвалом. В. . .	Отчёт о спецтехнике находящейся в ремонте Maks 20.04.2026 Отчёт из решения ниже размещен в конфигурации КА2. Задача: отобразить спецтехнику, которая на данный момент находится в ремонте. Есть нетиповой документ "Заявка на ремонт спецтехники" который. . .	Памятка для бота и "визитка" для читателей "Semantic Universe Layer (Слой семантической вселенной)" Hrethgir 19.04.2026 Сгенерировано для краткого описания по случаю сборки и компиляции скелета серверного приложения. И пусть после этого скажут, что статьи сгенерированные AI - туфта и не интересно. И это не реклама -. . .	Запрет удаления строк ТЧ документа при определённом условии Maks 19.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа "Аккумуляторы", разработанного в конфигурации КА2. У данного документа есть ТЧ, в которой в зависимости от прав доступа. . .
Модель заражения группы наркоманов alhaos 17.04.2026 Условия задачи сформулированы тут Суть: - Группа наркоманов из 10 человек. - Только один инфицирован ВИЧ. - Колются одной иглой. - Колются раз в день. - Колются последовательно через. . .	Мысли в слух. Про "навсегда". kumehtar 16.04.2026 Подумалось тут, что наверное очень глупо использовать во всяких своих установках понятие "навсегда". Это очень сильное понятие, и я только начинаю понимать край его смысла, не смотря на то что давно. . .	My Business CRM MaGz GoLd 16.04.2026 Всем привет, недавно возникла потребность создать CRM, для личных нужд. Собственно программа предоставляет из себя базу данных клиентов, в которой можно фиксировать звонки, стадии сделки, а также. . .	Знаешь почему 90% людей редко бывают счастливыми? kumehtar 14.04.2026 Потому что они ждут. Ждут выходных, ждут отпуска, ждут удачного момента. . . а удачный момент так и не приходит.

@kolorotur 17823 / 12973 / 3382 Регистрация: 17.09.2011 Сообщений: 21,261
	26.02.2012, 16:15
	Выкладывайте код метода MyMath.FactFor - ошибка там. Ну и избавьтесь от регекса - он не нужен. 0

@kolorotur 17823 / 12973 / 3382 Регистрация: 17.09.2011 Сообщений: 21,261
	26.02.2012, 17:30
	Что-то я запутался. Тема называется: "Факториал числа вычисляется неверно". Вы говорите: "Работает корректно". В первом сообщении у вас написано: "Что-то не так или очень быстро это делается?". В чем проблема-то? В том, что факториал, якобы, вычисляется не верно, или в том, что затраченное время показывается как 0мс? Во втором случае выводите свойство Elapsed - там и микро- и наносекунды покажет. 1

@kolorotur 17823 / 12973 / 3382 Регистрация: 17.09.2011 Сообщений: 21,261
	26.02.2012, 18:37
	Так а что смущает-то? То, что время все время разное? Это нормально, в системе не одно ваше приложение работает. В этот раз удалось по-больше время ЦП цапнуть, а в следующий не вышло. 1