Los últimos cuatro "Ingrese valor: "
se imprimen en la función de cambiarSigno
:
void cambiarSigno(int vect[],int longitud)
{
for (int i = 0; i < longitud; i++)
{
cout << "Ingrese valor: " << endl;
vect[i] = (-1 * vect[i]);
}
}
Seguramente querías hacer esto:
void cambiarSigno(int vect[],int longitud)
{
for (int i = 0; i < longitud; i++)
{
vect[i] = (-1 * vect[i]);
}
}
Otras cosas a tener en cuenta.
- Evita usar formaciones estáticas, usa contenedores.
- Evita el uso de variables globales.
- Pasa tus colecciones de datos por referencia para ahorrarte copias.
- Usa los bucles de rango, son menos propensos a errores.
Con estos consejos, tu código podría parecerse a:
#include <iostream>
#include <vector>
using namespace std;
using enteros = vector<int>;
enteros datos()
{
cout << "Ingrese longitud del vector: " << endl;
int longitud;
cin >> longitud;
enteros resultado(longitud, 0);
for (auto &valor : resultado)
{
cout << "Ingrese valor: ";
cin >> valor;
}
return resultado;
}
void cambiarSigno(enteros &valores)
{
for (auto &valor : valores)
{
valor = -valor;
}
}
void mostrarVector(enteros &valores)
{
for (auto &valor : valores)
{
cout << valor << " ";
}
}
int main()
{
auto vect = datos();
cambiarSigno(vect);
mostrarVector(vect);
return 0;
}
Puedes ver el código funcionando en Try it online!.