Repeticion

[alvarito_gool]

Hola, Tengo que hacer un programa, al cual se le ingrese una lista de numeros con repeticiones, y guardar en un nuevo vector los mismos numeros pero sin repeticiones, hice esto pero no entiendo por que se desordenan los numeros Ademas no se si el programa puede fallar para un caso especifico.
Si alguien me puede hacer un analisis agradeceria, y explicarme si el desorden no tiene nada que ver, y el programa no falla.

Código:

#include<iostream.h>
int main()
{short n, i, j,t, rep;
cout<<"Digite la cantidad de numeros \n";
cin>>n;
short V[n];
for(i=0;i<=n-1;i++)
{cout<<"Digite el elemento de la posicion "<<i+1<<"\n";
cin>>V[i];}
cout<<"La lista de numeros si repetir es\n";
for(i=0;i<=n-1;i++)
{t=0;
for(j=i+1;j<=n-1;j++)
if(V[i]==V[j])
t++;
if(t==0)
cout<<V[i]<<"\t";} 
system("pause");
}

Muchas gracias.

[cristian_qc]

Primero: <iostream.h> debe ser <iostream>.
Segundo: La función Main debe retornar un valor entero: return 0.
Tercero: No especificaste el espacio de nombres. En este caso es std: using namespace std;
Cuarto: system("pause"); Solo funciona en Windows. Si querés que la pantalla no se cierre automáticamente podés usar cin.get();
Quinto: Las variables deberían ser más específicas. Siempre veo que usan i, j, n, v, es mucho más entendible usar palabras completas. Yo generalmente a las variables auxiliares les pongo letras comenzando desde la "a".
Sexto: Los números se muestran desordenados porque lo que hace el último ciclo es verificar si el elemento actual se va a repetir más adelante, si es así no lo muestra, entonces solo muestra las últimas repeticiones y no las primeras.
Séptimo: Si bien a veces compila, lo siguiente es erróneo:

Código C++:

cin >> n;
short V[ n ];
 

Lo ideal es hacerlo con memoria dinámica, pero también podés declarar un vector con un número máximo de elementos, y no permitir que se ingresen más que esa cantidad.
Último: Mejoré un poco el código:

Código C++:

#include<iostream>
using namespace std;
int main(){
    short cantidad; // Cantidad de elementos
    
    // Ingresar cantidad de elementos
    cout << "Digite la cantidad de numeros: "; cin >> cantidad;
    
    /* Esto aunque a veces compile es completamente ERRONEO. La forma
        correcta es con memoria dinamica o declarando un vector con un
        número máximo de elementos. */
    // short vector[ cantidad ];
    
    short *vector = new short[ cantidad ]; // vector dinámico
    
    // Ingresar números
    for( short a = 0; a < cantidad; a++ ){
        cout << "Digite el elemento de la posicion " << a+1 << ": ";
        cin >> vector[ a ];
    }
    
    // Mostrar números
    cout << "La lista de numeros sin repetir es: " << endl;
    for( short a = 0; a < cantidad; a++ ){
        bool repetido = false;
        
        // Recorrer todos los elementos ANTERIORES al actual.
        for( short b = 0; b < a; b++ ){
            if( vector[ a ] == vector[ b ] ) repetido = true;
        }
        // Se muestra el elemento solo si no está repetido
        if( !repetido ) cout << vector[ a ] << "\t";
    }
    delete[] vector; // Liberar memoria
    cout << endl; // Último fin de linea
    cin.get();
    return 0;
}
 

Saludos!.

[alvarito_gool]

Muchas gracias cristian_cq me has ayudado mucho.

[cristian_qc]

Por nada ! Saludos !