Vamos a centrarnos en la pieza central de tu algoritmo:
cout << "Ingrese la letra: ";
cin >> letra;
for (int i = 0; i < 20; i++) {
if (letra != vector[i]) {
vector[i] = letra;
}
else {
cout << "Esta letra ya ha sido ingrasada " << endl;
}
}
1er error
Imagínate que es la primera letra que se introduce... el algoritmo la comparará con las 20 posiciones que tiene el array... ¿Y qué hay en esas posiciones del array? dado que no lo has inicializado, basura... así que una posibilidad podría ser:
df!f20y43%_SD435?a.{
Fíjate que en las últimas posiciones hay una a
:
df!f20y43%_SD435?a.{
^
Así que el algoritmo te dirá que la letra está repetida pese a que es la primera letra que se ingresa...
La solución a este problema es tan sencilla como incializar vector
antes de introducir una palabra:
std::fill(vector,vector+20,'0');
O también puedes inicializar el vector al declararlo:
char vector[TamVector] = {'\0'};
Aunque a mi personalmente no me hace mucha gracia tener ese 20
pululando por el código... si luego tengo que cambiar el tamaño del array me toca buscar ese 20
por todo el código para evitar errores. Sería mejor tener algo así:
const int TamVector = 20;
char vector[TamVector];
// ...
for( int i=0; i<TamVector; i++ )
// ...
2º error
y la vuelvo a ingresar deberia de aparecerme que ya ha sido ingresada pero no lo logro hacer...
Si te fijas, verás que vector
es una variable local de noRepetir
. Lo que hace noRepetir
es leer una única letra y buscarla en vector
, tras lo cual devuelve el control a main
. Cuando esto sucede el contenido de vector
se pierde, ya que la variable se destruye... Dado que la idea es que el array sea algo más persistente quizás debieras declararlo en main
:
const int TamVector = 20;
int main()
{
char vector[TamVector] = {'\0'};
noRepetir(vector, TamVector);
}
void noRepetir(char *vector, int tamVector)
{
// ...
}
Con esto conseguirás que la información que almacenes en vector
sea persistente entre llamadas a noRepetir
... pero el programa seguirá sin funcionar...
... y el motivo es la línea que destaco:
if (letra != vector[i]) {
vector[i] = letra; // <<---
}
¿Por qué es problemática esa línea?
Veamos... el usuario introduce una letra y el algoritmo la compara con las 20 posiciones de vector
. Si la letra no es igual a la almacenada en el vector... sobreescribe la letra almacenada en el vector con la letra que ha introducido el usuario.
Es decir, si asumimos que inicializas vector
y que es la primera letra que introduce el usuario, después de llamar a noRepetir
el vector quedará así (supongamos que el usuario introduce a
):
aaaaaaaaaaaaaaaaaaaa
Si después introduce una x
:
xxxxxxxxxxxxxxxxxxxx
Visto lo visto no parece una buena idea que el algoritmo machaque posiciones de vector
así a la ligera.
Lo que tienes que hacer es modificar el bucle para que itere desde 0 hasta que se encuentre una posición del vector no inicializada vector[i]=='\0'
y, si la letra introducida por el usuario no se encuentra en vector
, añadirla únicamente en la primera posición disponible. Algo así:
int i; // Ahora es necesario que `i` esté fuera del bucle
bool unica = true;
// Primero comprobamos que la letra no se ha introducido aun
for( i=0; i<tamVector && vector[i] != '\0' && unica; i++ )
unica = (vector[i] == letra);
// Y despues, si no esta repetida, la añadimos al array
if( unica )
{
if( i < tamVector ) // Importante para no escribir fuera del array
vector[i] = letra; // antes de esta linea vector[i]=='\0'
}
else
{
std::cout << "Esta letra ya ha sido ingrasada\n";
}
El programa, con las correspondientes correcciones:
#include <iostream>
void noRepetir(char * vector, int tamVector);
int main()
{
int const TamVector = 20;
char vector[TamVector] = {'\0'};
int opc = 0;
do {
noRepetir(vector,TamVector);
std::cout << "Ingresar otro dato? (1 = Si | 0 = No) ";
std::cin >> opc;
} while (opc == 1);
return EXIT_SUCCESS;
}
void noRepetir(char * vector, int tamVector)
{
char letra;
std::cout << "Ingrese la letra: ";
std::cin >> letra;
int i;
bool unica = true;
for( i=0; i<tamVector && vector[i] != '\0' && unica; i++ )
unica = (vector[i] != letra);
if( unica )
{
if( i < tamVector )
vector[i] = letra;
}
else
{
std::cout << "Esta letra ya ha sido ingrasada\n";
}
}