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

@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;

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

Новые блоги и статьи Все статьи Все блоги /
SDL3 для Web (WebAssembly): Реализация движения на Box2D v3 - трение и коллизии с повёрнутыми стенами 8Observer8 20.02.2026 Содержание блога Box2D позволяет легко создать главного героя, который не проходит сквозь стены и перемещается с заданным трением о препятствия, которые можно располагать под углом, как верхнее. . .	Конвертировать закладки radiotray-ng в m3u-плейлист damix 19.02.2026 Это можно сделать скриптом для PowerShell. Использование . \СonvertRadiotrayToM3U. ps1 <path_to_bookmarks. json> Рядом с файлом bookmarks. json появится файл bookmarks. m3u с результатом. # Check if. . .	Семь CDC на одном интерфейсе: 5 U[S]ARTов, 1 CAN и 1 SSI Eddy_Em 18.02.2026 Постепенно допиливаю свою "многоинтерфейсную плату". Выглядит вот так: https:/ / www. cyberforum. ru/ blog_attachment. php?attachmentid=11617&stc=1&d=1771445347 Основана на STM32F303RBT6. На борту пять. . .	Камера Toupcam IUA500KMA Eddy_Em 12.02.2026 Т. к. у всяких "хикроботов" слишком уж мелкий пиксель, для подсмотра в ESPriF они вообще плохо годятся: уже 14 величину можно рассмотреть еле-еле лишь на экспозициях под 3 секунды (а то и больше),. . .
И ясному Солнцу zbw 12.02.2026 И ясному Солнцу, и светлой Луне. В мире покоя нет и люди не могут жить в тишине. А жить им немного лет.	«Знание-Сила» zbw 12.02.2026 «Знание-Сила» «Время-Деньги» «Деньги -Пуля»	SDL3 для Web (WebAssembly): Подключение Box2D v3, физика и отрисовка коллайдеров 8Observer8 12.02.2026 Содержание блога Box2D - это библиотека для 2D физики для анимаций и игр. С её помощью можно определять были ли коллизии между конкретными объектами и вызывать обработчики событий столкновения. . . .	SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL_LoadPNG (без SDL3_image) 8Observer8 11.02.2026 Содержание блога Библиотека SDL3 содержит встроенные инструменты для базовой работы с изображениями - без использования библиотеки SDL3_image. Пошагово создадим проект для загрузки изображения. . .

@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