Skip to main content
se añadieron 1529 caracteres en el cuerpo
Origen Enlace
PaperBirdMaster
  • 48.1k
  • 6
  • 45
  • 86

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!.

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]);
    }
}

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!.

Origen Enlace
PaperBirdMaster
  • 48.1k
  • 6
  • 45
  • 86

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]);
    }
}