Kaip užrašyti faktorialą?

[rickusdov]

Sveiki, pradėjau mokytis c++ ir užduotis manęs prašo parašyti programą, kuri skaičiuotų skaičiaus faktorialą. Tai kaip reikia užrašyti kad skaičiuotų faktorialą?

[Tautedorus]

Kodėl gi nepabandžius paguglinti?

Redaguota Sausio 23, 2018, nario Tautedorus

[#Ernie]

1/22/2018 23:01, 'Ričkus parašė:

Sveiki, pradėjau mokytis c++ ir užduotis manęs prašo parašyti programą, kuri skaičiuotų skaičiaus faktorialą. Tai kaip reikia užrašyti kad skaičiuotų faktorialą?

// Factorial
#include "stdafx.h"
#include <iostream>

int main()
{
    // Kuriami kintamieji
    int a; 
    long long factorialas = 1; 

    // Ivedimas is ekrano
    std::cin >> a;
    // surandame skaiciaus a factoriala
    for (int i = 1; i <= a; i++)
    {
        factorialas = factorialas * i;
    }
    // isvedimas i ekrana
    std::cout << "Skaiciaus " << a << " factorialas = " << factorialas << std::endl;
    system("Pause");
    return 0;
}

Redaguota Vasario 16, 2018, nario Tautedorus

[Tautedorus]

2/4/2018 14:11, #Ernie parašė:

// Factorial
#include "stdafx.h"
#include <iostream>

int main()
{
    // Kuriami kintamieji
    int a; 
    long long factorialas = 1; 

    // Ivedimas is ekrano
    std::cin >> a;
    // surandame skaiciaus a factoriala
    for (int i = 1; i <= a; i++)
    {
        factorialas = factorialas * i;
    }
    // isvedimas i ekrana
    std::cout << "Skaiciaus " << a << " factorialas = " << factorialas << std::endl;
    system("Pause");
    return 0;
}

Pirmiausia, nenaudok system() funkcijos, it's a bad practice. Jei nori pristabdyti konsolinę programą, naudok std::cin.get(). Taip pat pirmojo kintamojo deklaracijoje gal dar reikėtų naudoti ne int tipą, o undigned int, nes pagal apibrėžimą faktorialas taikomas sveikiesiems neneigiamiems skaičiams, bet tai nėra labai svarbu. Be to, kintamojo pavadinimas yra blogas, nes apie kintamąjį nieko nepasako. Galiausiai, venk std::endl. Šioje situacijoje jo tikrai nereikia, užtenka pridėti '\n' simbolį. Ką daro std::endl, tai prideda '\n' simbolį ir dar kviečia std::flush(), kas mažina efektyvumą.