2

El objetivo es pedir 5 numeros y cada vez que se pide uno, se muestra ese mismo. Es fácil, sin embargo no sale el resultado:

#include <stdio.h>

int main(){
  int valor;

    for(int i = 1; i <= 5; i++){
      printf("Valor %d: ", i);
      scanf("%s", &valor);
      printf("%d\n", valor);
    }
    return 0;
  }

Lo que imprime por pantalla es:

Valor 1: 3
2883635 //deberia imprimir 3
Valor 2: 5
2883637 //deberia imprimir 5
Valor 3: 7
2883639 //deberia imprimir 7
Valor 4: 1
2883633 //deberia imprimir 1
Valor 5: 9
2883641 //deberia imprimir 9

¿Por que no imprime esos numeros?

7
  • "%s" ¿El compilador te advirtió que esto está mal? ideone.com/tJaBQR el 18 jun. 2020 a las 7:59
  • @n.'pronouns'm. no me lo ha advertido. Me acabo de instalar el entorno de Atom y aun no se muy bien como funciona. Sabrías como puedo poner para que me advierta de esto?
    – sonlas10
    el 18 jun. 2020 a las 8:05
  • ¿Qué compilador usas? Todos los compiladores modernos advierten de tales errores. Quizás solo necesites descargar la versión actual. el 18 jun. 2020 a las 8:12
  • instalé el paquete de gpp-compiler @n.'pronouns'm.
    – sonlas10
    el 18 jun. 2020 a las 8:13
  • gpp-compiler no es un compilador. Es solo un paquete de ayuda; utiliza un compilador real como mingw. el 18 jun. 2020 a las 8:17

2 respuestas 2

3

Cambia el valor en el scanf %s por %d que recoges un int como has declarado previamente. %s es para formato string.

De:

scanf("%s", &valor);

A:

scanf("%d", &valor);

Quedaría el código:

#include <stdio.h>

int main(){
  int valor;

    for(int i = 1; i <= 5; i++){
      printf("Valor %d: ", i);
      scanf("%d", &valor);
      printf("%d\n", valor);
    }
    return 0;
  }
2

Estás formateando de manera equivocada la variable en el scanf(), el operador %s sirve para formatera una cadena de carateres. Si desea escanear un valor númerico, lo adecuado es usar %i o bien %d

También es recomendable, ordenar mejor tu código, indenta y no pegues tanto las lineas.

scanf("%d",&valor);

Espero haberte ayudado, saludos.

4
  • el main() puede ser perfectamente un void NO, es falso. el 18 jun. 2020 a las 8:13
  • dime por qué es falso?
    – user166844
    el 18 jun. 2020 a las 8:19
  • Porque el estándar de C dice eso. ideone.com/Rlt4uY el 18 jun. 2020 a las 8:28
  • 1
    @gabrielcoronel El estándar (Página 24, 5.1.2.2.1 Program startup) afirma que la forma como se debe definir el main es así: int main(void) {...}. Sin embargo, tampoco está mal definir el main de esta manera: void main(){ ...}, simplemente dependerá si el compilador lo permite o no (esto el estándar también lo menciona). Por cuestión de portabilidad, lo recomendable es hacer lo que dice el estándar..
    – MrDave1999
    el 18 jun. 2020 a las 14:01

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

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