DOMANDA Ancora problema con array c++

[MPG]

Crea un array contenente valori casuali compresi tra 1 e 9 ed effettua 5 scambi casuali.
Fatto cosi' ma non viene..
Aiuto!!!

#include <cstdlib>
#include <ctime>
#include <cmath>
#include <iostream>
using namespace std;

int main()
{
  srand(time(NULL));
  int dim;
  cout<<"Inserisci dim: "<<endl;
  cin>>dim;
  cout<<endl;
int A[dim];
cout<<A[dim-1]<<endl;
for(int i=0;i<dim;i++)
{
A[i]=rand()%10+1;
cout<<A[i]<<endl;
}
cout<<endl;
for(int i=0;i<4;i++)
{
int k=rand()%dim;
int h=rand()%dim;
int temp;
A[k]=temp;
A[k]=A[h];
A[h]=temp;
}
for(int i=0;i<dim;i++)
{
cout<<A[i]<<endl;
}

    return 0;
}

 

[rodhellas]

Crea un array contenente valori casuali compresi tra 1 e 9 ed effettua 5 scambi casuali.
Fatto cosi' ma non viene..
Aiuto!!!

#include <cstdlib>
#include <ctime>
#include <cmath>
#include <iostream>
using namespace std;

int main()
{
  srand(time(NULL));
  int dim;
  cout<<"Inserisci dim: "<<endl;
  cin>>dim;
  cout<<endl;
int A[dim];
cout<<A[dim-1]<<endl;
for(int i=0;i<dim;i++)
{
A[i]=rand()%10+1;
cout<<A[i]<<endl;
}
cout<<endl;
for(int i=0;i<4;i++)
{
int k=rand()%dim;
int h=rand()%dim;
int temp;
A[k]=temp;
A[k]=A[h];
A[h]=temp;
}
for(int i=0;i<dim;i++)
{
cout<<A[i]<<endl;
}

    return 0;
}

Inverti A[k]=temp;

 

[DispatchCode]

Dovresti anche cambiare la condizione:

for(int i=0;i<4;i++)

con questo non hai 5 scambi ma 4.

 

[MPG]

Inverti A[k]=temp;

Non capisco...

 

[rodhellas]

Non capisco...

Non stai assegnando a temp il valore di A[k] ma il contrario...

 

[MPG]

Saro' rimbambito oggi ma ancor non capisco ........

 

[rodhellas]

A[k]=temp la variabile di sinistra (a[k]) prende il valore della variabile di destra (temp). Ma a temp quando viene assegnato un valore?

 

[MPG]

Ma se faccio temp= A[k] ho provato nn viene lo stesso.... e anche gli scambi non sono 5..

 

[DispatchCode]

Ma se faccio temp= A[k] ho provato nn viene lo stesso.... e anche gli scambi non sono 5..

A questo ti ho risposto io...

Inviato da ONEPLUS A5000 tramite App ufficiale di Tom\'s Hardware Italia Forum

 

[rodhellas]

Ma se faccio temp= A[k] ho provato nn viene lo stesso.... e anche gli scambi non sono 5..

Gli scambi devi aumentare nel ciclo for come detto da DispatchCode. Comunque da me funziona, prova a stampare ad ogni cambio

 

[MPG]

Fatto cosi', se provo a mettere es 6 come dim viene cosi:

4199419
7
4
3
6
6
9

9
7
4
3
6
6

#include <cstdlib>
#include <ctime>
#include <cmath>
#include <iostream>
using namespace std;

int main()
{
  srand(time(NULL));
  int dim;
  cout<<"Inserisci dim: "<<endl;
  cin>>dim;
  cout<<endl;
int A[dim];
cout<<A[dim-1]<<endl;
for(int i=0;i<dim;i++)
{
A[i]=rand()%10+1;
cout<<A[i]<<endl;
}
cout<<endl;
for(int i=0;i<5;i++)
{
int k=rand()%dim;
int h=rand()%dim;
int temp;
temp=A[k];
A[k]=A[h];
A[h]=temp;
}
for(int i=0;i<dim;i++)
{
cout<<A[i]<<endl;
}

    return 0;
}

 

[rodhellas]

Fatto cosi', se provo a mettere es 6 come dim viene cosi:

4199419
7
4
3
6
6
9

9
7
4
3
6
6

#include <cstdlib>
#include <ctime>
#include <cmath>
#include <iostream>
using namespace std;

int main()
{
  srand(time(NULL));
  int dim;
  cout<<"Inserisci dim: "<<endl;
  cin>>dim;
  cout<<endl;
int A[dim];
cout<<A[dim-1]<<endl;
for(int i=0;i<dim;i++)
{
A[i]=rand()%10+1;
cout<<A[i]<<endl;
}
cout<<endl;
for(int i=0;i<5;i++)
{
int k=rand()%dim;
int h=rand()%dim;
int temp;
temp=A[k];
A[k]=A[h];
A[h]=temp;
}
for(int i=0;i<dim;i++)
{
cout<<A[i]<<endl;
}

    return 0;
}

Guarda che funziona. L'unico miglioramento sarebbe quello di imporre che k != h, sennò non ha senso fare lo scambio

 

[MPG]

Ma quel numero 4199419 che viene? Se metto dim 6 vinee perennemente questo numero prima, Io uso codeblocks mi fai vedere il tuo screenshot, a me questo appare!

 

[MPG]

E' questo che rompe
cout<<A[dim-1]<<endl;

E' in piu'...