Numar prim

[MoNNsterFTW]

Salut,sunt pe clasa a 9a si am inceput c++ in scoala,am facut pseudocod si schema logica pe semestrul I,m-am mai interesat si am incercat sa fac un program despre aflarea unui numar prin,insa orice numar as introduce imi scrie ca numarul este prim... Am gasit 2 probleme: 1. cica: cannot open output file .......\Program 2.exe Permission denied,inainte sa pun si cout << s << p,a mers,dar a 2. problema este ca orice numar introduceam zicea ca e prim...

Enuntul problemei este: Se ciste un numar natural n. Sa se calculeze si sa se afiseze suma si produsul cifrelor sale impare. Sa se verifice daca aceasta ste numar prim.

#include<iostream>
#include<cmath>

using namespace std;

int main()
{
bool x;
int n, c, a ,s ,i ,p;
cin >> n;
s = 0;
p =1;
while (n!=0)
{

c=n%10;
if (c%2!=0)
{
s = s+n;
p = p*n;
cout << "Suma este: " << s << "Produsul este: " << p;
}
n=n/10;
}
x=1;
i=2;
while (i<=sqrt(s) && x==1)
{
if (s%i==0)
{
x=0;
}
else
{
i=i+1;
}
}
if (x=1)
{
cout << "Numarul este prim";
}
else
{
cout << "Numarul nu este prim";
}

return 0;
}

[mayavin]

In primul while, dupa if, trebuia sa fie

Code:

s=s+c; p=p*c;

pentru ca tu aduni/faci produsul cu cifra nu cu numarul initial n (cum ai scris tu) - conform cerintei.

Code:

cout << "Suma este: " << s << "Produsul este: " << p;

Partea asta ar trebui sa o pui dupa while, nu in el, pentru a-ti afisa corect suma/produsul (adica dupa ce termina de calculat).

Vad ca tu faci verificarea de numar prim doar pentru s, nu si pentru p. Iar dupa ce faci verificarea ai scris if(x=1), ar trebui sa fie if(x==1).

Ai o variabila 'a' declarata pe care nu o folosesti (deci poti sa o stergi).

Ai niste paranteze inutile. Daca dupa o conditie urmeaza o singura instructiune, atunci acoladele nu sunt necesare.

In legatura cu problema "cannot open output file... etc", s-ar putea ca programul sa-ti mai ruleze in fundal. Asigura-te ca programul nu ruleaza. Eventual inchizi si redeschizi codeblocks, sau ce IDE folosesti tu.

Un alt lucru pe care il observ este faptul ca exista si caz particulat pentru care algoritmul tau de verificare numar prim nu functioneaza. Daca sa zicem numarul e s=1. Atunci programul iti va afisa ca este numar prim, pentru ca nici nu intra in while si trece direct la if(x==1) unde rezultatul este adevarat, pentru ca tu ai declarat x=1 inainte de while. Trebuie tratat acest caz. Un mod ar fi de a pune un

Code:

if(s<2)    x=0;

inainte de verificarea pentru numar prim.

Ti-as mai atrage atentia si asupra indentarii. Este foarte important cand scrii cod, sa fie indentat pentru a fi usor de citit si inteles atat de tine cat si de altii.

Nu am rulat programul, cam astea sunt lucurile pe care le vad la prima vedere la 5 dimineata.
Bafta.

[MoNNsterFTW]

Mersi frumos a mers,stiu ca am facut totul aiurea,dar asa am gandit eu ca ar fi fost bine,cred ca faza asta cu == e la fel ca si in pseudocod = si ia valoarea(<--).

[mayavin]

In interiorul unei conditii intotdeauna trebuie == (verifici egalitatea), iar la atribuiri doar un singur =.