1

Buenos días.

Quisiera saber por qué no me funciona correctamente éste pequeño programita. Llevo casi 1 hora intentando encontrar el error, pero no hay manera. El programa me compila y se me ejecuta pero a la hora de introducir por ejemplo: 1 2 3 0, me dice que hay almenos un número negativo.

De antemano, muchas gracias.

Código:

#include <stdio.h>
        int main (void) {
            int n, cont, cont1, cont2;

            printf ("Escribe una serie de números acabada en 0: "); 
            scanf ("%d", &n);

            cont = 0;

            while ((n != 0) && (n > 0)){
                if (n > 0) {
                    cont1 ++;
                }
                else {
                    cont2 ++;
                }
                scanf ("%d", &n);
                cont ++;
            }       

            if (cont == cont1){
                    printf ("Ésta es una serie positiva.\n");
                }

            else {
                printf ("Hay almenos un número negativo. Vuelve a introducir la serie.\n");
            }

            return 0;
        }
9
  • Primer de todo, gracias por responder. Mi intención era hacer que "cont" contara todos los números de la serie, y luego comparar: Si todos los números de la serie positivos == cont, quiere decir que la serie es positiva. "Y ¿no te parece que esta condición podría simplificarse un poco: (n != 0) && (n > 0)" no se me ocurre nada más simplificable la verdad. el 21 ene. 2018 a las 10:24
  • Eso de dejar las variables sin inicializar ... ¿ Es por algún motivo que se me escapa ? No se, porque busques comenzar con valores aleatorios que den resultados imprevisibles o algo así ;-)
    – Trauma
    el 21 ene. 2018 a las 10:40
  • Joder jaja, se me había escapado xD. Gracias tío ya funciona! el 21 ene. 2018 a las 10:43
  • 1
    Es una buena práctica inicializar los contadores a cero al mismo tiempo que han sido declarados: cont = 0, cont1 = 0, cont2 = 0;.
    – Xam
    el 21 ene. 2018 a las 22:38
  • @Trauma ¿y la respuesta donde está? si no la publicas dificilmente la pregunta va a aparecer como solucionada...
    – eferion
    el 24 ene. 2018 a las 23:02

1 respuesta 1

0

En C debes darle un valor inicial a todas las variables locales antes de usarlas (como las variables n, cont, cont1, cont2 dentro de tu método main), de lo contrario tendrán un valor indeterminado y cualquier operación realizada con estas variables tendrá un comportamiento indefinido

En tu código inicializas sólo la variable n al valor dado por el usuario y cont a 0, pero olvidas darle un valor a las variables cont1 y cont2. Modifica esta línea de código

    int n, cont, cont1, cont2;

por

    int n, cont, cont1 = 0, cont2 = 0;

Tu Respuesta

Al pulsar en “Publica tu respuesta”, muestras tu consentimiento a nuestros términos de servicio, política de privacidad y política de cookies

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.