Vou considerar que o enunciado é exatamente este. Você não está fazendo o que o enunciado pede. Note que ele tem duas partes:
- "leia um vetor de 50 elementos" (você está fazendo muito mais que isso no loop que deveria ser bem simples;
- "imprima o vetor desconsiderando os valores repetidos" (você está imprimindo tudo).
Pode parecer bobagem, mas faz diferença. Programação é assim, tem que ficar atento a todos os detalhes. Aí neste segundo loop é que deve verificar se um valor já existe.
Resolvendo isto, automaticamente não terá o problema do loop ficar empacado.
Sem modificar muito e sem otimizar da melhor forma possível (mas otimizei da forma mais óbvia) fiz um teste rápido que resolve da maneira como foi proposto. Não cheguei fazer nenhum teste extensivo. Também não modifiquei muito seu código ou melhorei a formatação da saída, acho que isto você sabe se virar.
#include <iostream>
using namespace std;
#define MAX_ITENS 50 //número de itens a serem lidos se quiser modificar para testes
int main() {
int vet[MAX_ITENS];
bool repetido;
for(int i = 0; i < MAX_ITENS; i++) {
cout << ("Numero: ");
cin >> vet[i];
}
for(int i = 0; i < MAX_ITENS; i++) {
repetido = false;
for(int j = 0; j < i; j++) { //verifica se existem iguais apenas entre os anteriores
if (vet[j] == vet[i]) {
repetido = true;
break; //tem repetido, não precisa continuar com a procura
}
}
if (!repetido) {
cout << vet[i] << endl;
}
}
}
Veja funcionando no ideone. E no repl.it. Também coloquei no GitHub para referência futura.