[maniek910]

A więc chciałem napisać program który rozkłada mi na czynniki pierwsze wszystkie liczby w tablicy niestety program zamienia pierwszą i się zatrzymuję :

Kod:

#include <iostream> using namespace std; int const n=4; int main() {     int liczba,p=2;     int tab[n];     for(int i=0;i<n;i++)cin >> tab[i];     for(int i=0;i<n;i++){     while(tab[i]!=1){                      if(tab[i]%p==0){                                      cout << p << endl;                                      tab[i] = tab[i] / p;                                      }                                      else                                      p++;                                      }     }     system("PAUSE");     return EXIT_SUCCESS; }

jak by tu zaradzić żeby rozkładało też pozostało ?

[ExeQtoR]

"P" masz deklarowane globalnie - zły nawyk;-)))
wiec na bank po drugim przejściu pętli FOR będzie inne niż dla i=0 ))
algorytmu nie znam i nie kojarzę xD ale skoro te same operacje dla i=1 mają być to parametr P też powinien wynosić 2 a nie wiadomo do końca co tam siedzi po pierwszej interakcji pętli FOR

wiec na koniec pętli FOR już po tym jak pętla WHILE się skończy powinieneś ustawić p=2;
albo
zrezygnować z deklaracji globalnych i zadeklarować zmienną P zaraz na początku FOR

for(....)
{
int p=2;
....
}

[maniek910]

Ty to jesteśakiego jednak mózg a ja pała xD że takiego czegoś nie zauważyłem

Dla potomnych może komuś się kiedyś do czegoś przyda

Kod:

/*Program rozkłada liczby z tablicy na czynniki pierwsze , np dla 20 to 2,2,5 => 2*2*5 =20 wszystkie czynniki to liczby pierwsze ,rozkład następuje wedle algorytmu : 56|2 28|2 14|2  7|7  1|  */ #include <iostream> using namespace std; int const n=4; int main() {     int liczba;     int tab[n];     for(int i=0;i<n;i++)cin >> tab[i];     for(int i=0;i<n;i++){         int p=2; // zmienna lokalna dla pętli         cout <<"liczba " << tab[i] << endl;     while(tab[i]!=1){                      if(tab[i]%p==0){                                      cout << p << endl;                                      tab[i] = tab[i] / p;                                      }                                      else                                      p++;                                      }     }     system("PAUSE"); }

[ExeQtoR]

Kod:

... int liczba; ...

masz sensowne wytłumaczenie potrzeby inicjalizowania zmiennej typu int o nazwie "liczba" ? ))

[maniek910]

Kod:

int liczba

sensowne nie xD zmienna została po tym jak program pisałem dla 1 liczby i zapomniałem usunąć . Tak program kąpiluje się elegancko luknij

[ExeQtoR]

hmmm... dziwnie....

a masz tam na końcu kodu:

Kod:

return 0;

albo:

Kod:

return EXIT_SUCCESS;

bo według mnie jeżeli tego nie masz to przeczy założeniu iż jeżeli funkcja ma coś zwrócić w tym wypadku funkcja główna o nazwie "main" zwraca typ liczbowy INT to kompilator nie powinien dopuścić aby ta wartość była nie ustalona, po zakończeniu tejże funkcji, chyba że ja czegoś nie wiem

[maniek910]

Powiem Ci że zazwyczaj piszę "return EXIT_SUCCESS;" ale bez zazwyczaj działa ... kiedy nie działa ? Jak dla mnie kwestia kompilatora ,zarówno w DevC++(bardzo stare standardy) jak i codeblock(trochę nowsze standardy) i gdy piszę coś w g++ na ubuntu (najnowsze standardy kąpilacji) też działa a czasem nie dziwne prawda ale tak wynika z wielokrotnych prób

[ExeQtoR]

ale przyzwyczaj się do pisania tego czasami może Ci się program dziwnie wykrzaczyć i nie będziesz wiedział dlaczego )))