Pomoć u programiranju (ispravljanje koda)

#include<iostream>

#include<vector>

using namespace std;

void ispis(int &n,int &fmin,int &fmax)

{

vector<int>faktor;

if(n==1) throw "xxx";

if(n<0) n=(-1)*n;

for(int i(2);i<n;i++)

{cout<<"1";

while (n>1)

{

if (n%i==0)

{

faktor.push_back(i);

n/=i;

cout<<"2";

}

i++;

}

}

   fmax=faktor[0];

   fmin=faktor[0];

   for(int i(0); i<faktor.size(); i++){

   if(fmax<faktor[i]) fmax=faktor[i];

     if(fmin>faktor[i]) fmin=faktor[i];

     cout<<"3";

   }

     if (faktor.size()==0)

     cout<<"Uneseni broj nema prostih faktora"<<endl;

}

int main()

{

   int n,fmin,fmax;

   try {

   cout<<"Ova funkcija ispisuje najveci i najmanji prosti faktor unesenog broja."<<endl;

   cout<<"Unesite broj: n= ";

   cin>>n;

   ispis(n,fmin,fmax);

   if(fmin==fmax)

   cout<<"Uneseni broj ima samo jedan prosti faktor koji mu je i minimum i maksimum: "<<fmin<<endl;

   else

   cout<<"Za uneseni broj najmanji prosti faktor je "<<fmin<<",a najveci je "<<fmax<<endl;

   }catch(...)

   {

     cout<<"GRESKA! Uneseni broj ne podlijeze trazenoj analizi"<<endl;}

   return 0;

}

Korektno mi treba da radi za sve brojeve.Ne vidim  neku gresku ali ne radi za neke brojeve pa ako mozete pomoci :-)

Evo jednog kodića da se igraš :D

#include<iostream>
#include<set>
#include<cmath>

using namespace std;

bool prost(const int& n)
{
     for(int i=2;i<sqrt(n);++i)
       if (n%i==0) return false;
    
     return true;
}

void prosti_faktori(int& n, int& mini, int& maxi)
{
     set<int> faktori;
     set<int>::iterator prvi,zadnji;
    
     for (int i=2;i<=n;++i)
       if ( n%i==0 && prost(i) ) faktori.insert(i);
      
     prvi=faktori.begin();
     zadnji=faktori.end();
     zadnji--; 
     mini=*prvi;
     maxi=*zadnji;
}

void prosti_faktori_bez_seta(int& n, int& mini, int& maxi)
{  
     for (int i=2;i<=n;++i)
       if ( n%i==0 && prost(i) ) { mini=i; break; }
      
     for (int i=n;i>=2;--i)
       if ( n%i==0 && prost(i) ) { maxi=i; break; }
}


int main()
{
    int n,maxi,mini;
   
    do
    {
        cout<<"Ucitaj n: ";
        cin>>n;
       
    }while(n<=1);
   
    prosti_faktori_bez_seta(n,mini,maxi);
   
    if (mini==maxi) cout<<"Broj "<<n<<" je prost!"<<endl;
     else cout<<"Najmanji prosti faktor broja "<<n<<" je "<<mini<<", a najveci "<<maxi<<endl;
 
    //system("pause");
    return 0;   
}

A možeš i pomoću vectora...

 void prosti_faktori(int& n, int& mini, int& maxi)
{
     vector<int> faktori;
    
     for (int i=2;i<=n;++i)
       if ( n%i==0 && prost(i) ) faktori.push_back(i);
     
     mini=faktori[0];
     maxi=faktori[faktori.size()-1];
}

Pa dao sam ti 3 verzije rješenja, sa setom, vektorom i bez ijednog spremnika. Na tebi je samo da se odlučiš koja verzija ti je najljepša.

Moje isprike, zeznuo sam funkciju prost, treba ići  i<=sqrt(n), a ne samo i<sqrt(n).

Drugo je ok :D

Možeš mi slobodno reć "ti", nisam ja tak star :D

max = matrica[0][0]; - ovdje ti je bila greška
for (i=0;i<n;i++)
{
       for (j=0;j<m;j++)
      {
           if (matrica[i][j]>max)
          {
                max=matrica[i][j];

          }

       }

}


Inače, ovaj kod ti ne može raditi , ako želiš ovako deklarirati matricu int matrica[n][m]; moraš m i n navesti kao const int vrijednosti u programu

ili to ili dinamička alokacija preko malloc-a.

(to je što se tiče traženja max elementa u matrici)

za ovu drugu matricu, sredi malo kod da bude pregledan i objasni što hoćeš postići s njom.

Pretpostavljam da hoćeš dobiti novu matricu koja ima jedan stupac manje nego stara. U tom slučaju ti ne treba nikakvo tramakivanje sa pomoćnom varijablom c, dovoljno je ovo:

int nova_matrica[n][m-1];
for (i=0;i<n;i++)
{
    for (j=0;j<(m-1);j++)
    {
       nova_matrica[i][j] = matrica[i][j];
    }
}