[SOLUCIONADO] Duda con números negativos

[Avya]

Hola!
Alguno me prodria aclarar la duda que tengo. El programa funciona correctamente excepto que los valores negativos no los considera enteros. Les dejo el código y espero sepan orientarme.

Código code:

/*
Leer un vector de 30 elementos y efectuar e imprimir las siguientes sumas:
 
"   Los valores enteros.
"   Los valores decimales.
"   Dentro de los enteros, sumar:
a)  los positivos
b)  los negativos
 
*/
 
#include <iostream>
#include <cstdlib>
 
using namespace std;
 
int main()
{
    int sE = 0;
    float sD = 0;
    int sP = 0;
    int sN = 0;
    float v[10];
    
    for (int i = 1; i <= 10; i++)
    {
        cout << "Ingrese valores para el vector: ";
        cin >> v[i];
        
        if (v[i] - int(v[i]) == 0)
        {
            sE += v[i];
            if (v[i] > 0)
            {
                sP += v[i];
            }
            else
            {
                sN += v[i];
            }
        }
        else
        {
            sD += v[i];
        }
    }
    cout << "La suma de los enteros es: " << sE << endl;
    cout << "La suma de los positivos es: " << sP << endl;
    cout << "La suma de los negativos es: " << sN << endl;
    cout << "La suma de los decimales es: " << sD << endl;
    
    system ("pause");
    return 0;
}
 

[xKuZz]

¿Por qué no? Yo lo ejecuto y me funciona correctamente. ¿Puedes decir un ejemplo de datos para los que no funcione?

[dehm]

Se supone que cuando sumas positivos o negativos es independientemente de si son enteros o decimales, pero sólo consideras los enteros, luego los decimales nunca van a añadirse a las variables sP y sN.

Si no he entendido mal lo que se supone que has de averiguar, el programa falla tanto con números decimales positivos como negativos....
Añado unos cout a tu programa para que entiendas lo que quiero decir:

Código C++:

if (v[i] - (int)v[i] == 0)
            {
                sE += v[i];
                if (v[i] > 0)
                {
                    sP += v[i];
                    cout<<"Solo añado los enteros positivos pero no los decimales"<<endl;
                }
                else
                {
                    sN += v[i];
                    cout<<"Solo añado los enteros negativos pero no los decimales"<<endl;
                }
            }
            else
            {
                sD += v[i];
            }
 

[dehm]

Por aclarar algo mas, este (creo) sería el pseudocódigo:

Código C++:

if (entero)//(v[i] - (int)v[i] == 0
sE+=v[i] 
else
sD+=v[i]
if (positivo) //(v[i] >= 0)
sP=v[i]
else
sN=v[i]
 

Eso y cambiar los tipos de sP y sN a float

Si no es esto, es que no he entendido el criterio, así que disculpa por el posible error.

[Malenko]

La suma de enteros la va acumulando en sE y, posterioremente, hace las sumas en sP o sN si son positivos o negativos.

Por otro lado, declara el vector así v[10] eso quiere decir que tiene 10 posiciones pero en C/C++ las posiciones de los vectores empiezan por 0, es decir que el vector va de v[0] a v[9], con lo que la última vuelta del for está sobreescribiendo una zona de memoria.

Tampoco estaría de más poner nombres de variables más descriptivos, por ejemplo en lugar de "sE" poner "sumaValoresEnteros".