Pierwszy program w C / Program przestał działać

Question

Witam jest to mój pierwszy post proszę o pomoc.

Napisałem pierwszy program samodzielnie co by potrenować pętle i przy kompilacji jest ok ale jak uruchamiam program to wyskakuje kom *. program przestał działać HELP

#include <stdio.h>
#include <math.h>
main ()
{
	enum dzialania { SUM=1, ROZ, ILOR, ILOCZ};
	
int n;
int a;
int b;
int suma = a+b;
int roznica = a-b;
int mnozenie = a*b;
float dzielenie =a/b;
printf("wprowadz a\n");
scanf ("%d", &a);
printf("wprowadz b\n");
scanf ("%d", &b);
printf("wybierz rodzaj działania\n");
printf("1 - dodawanie\n",suma);
printf("2 - odejmowanie\n",roznica);
printf("3 - mnozenie\n",mnozenie);
printf("4- dzielenie\n",dzielenie);
scanf ("%d", &n);
switch (n)
{
	case SUM: printf("Wynik dodawania %d",suma) ;
	break;
	case ROZ: printf("Wynik odejmowania %d",roznica) ;
	break;
	case ILOR: if(a==0)
	{
	printf("wynik mnożenia przez zero jest zawsze 0\n");
}
 else
	{ 
	if (b!=0)
	printf("wynik mnozenia %d",mnozenie);
	else
	printf("nie mozna mnozyc\n");
}
	break;
	case ILOCZ: 	if(a==0)
	{
	printf("nie dziel przez zero\n");
}
 else
	{ 
	if (b!=0)
	printf("wynik dzielenia %.1f",dzielenie);
	else
	printf("nie mozna dzielic\n");
}
	break; 	
}
getchar();
  return 0;

}

 

Answer 1

Witam,

przełóż zmienne

int suma = a+b;
int roznica = a-b;
int mnozenie = a*b;
float dzielenie =a/b;

Zauważ, że zmienne a i b są niezdefiniowane(mają losowe wartości). Dopiero niżej podajesz im wartości przez scanf(). Usuń jeszcze z printf() zmiennej gdyż nie odwołujesz się do wartości tych zmiennych.

#include <stdio.h>
#include <math.h>
main ()
{
    enum dzialania { SUM=1, ROZ, ILOR, ILOCZ};

int n;
int a;
int b;

printf("wprowadz a\n");
scanf ("%d", &a);
printf("wprowadz b\n");
scanf ("%d", &b);
printf("wybierz rodzaj działania\n");
printf("1 - dodawanie\n");
printf("2 - odejmowanie\n");
printf("3 - mnozenie\n");
printf("4- dzielenie\n");
scanf ("%d", &n);
int suma = a+b;
int roznica = a-b;
int mnozenie = a*b;
float dzielenie =a/b;
switch (n)
{
    case SUM: printf("Wynik dodawania %d",suma) ;
    break;
    case ROZ: printf("Wynik odejmowania %d",roznica) ;
    break;
    case ILOR: if(a==0)
    {
    printf("wynik mnożenia przez zero jest zawsze 0\n");
}
 else
    {
    if (b!=0)
    printf("wynik mnozenia %d",mnozenie);
    else
    printf("nie mozna mnozyc\n");
}
    break;
    case ILOCZ:     if(a==0)
    {
    printf("nie dziel przez zero\n");
}
 else
    {
    if (b!=0)
    printf("wynik dzielenia %.1f",dzielenie);
    else
    printf("nie mozna dzielic\n");
}
    break;
}
getchar();
  return 0;

}

 

Comments

O właśnie, nie zauważyłem tego string format, tutaj może lecieć crash.

Btw wartości nie są losowe, tylko śmieciowe. Słowo "losowe" sugeruje ze wartość jest losowania przy jej tworzeniu, podczas gdy to śmieć, który został na stosie. Można nawet przewidzieć wartość śmiecia w obrębie jednego programu

---

Semantycznie masz racje. Losowe wartości to bardziej te zwrócone przez funkcje rand(). Można użyć słowo śmieciowe bądź niezidentyfikowane zależy od gustu.

---

Pomogło przełożenie zmiennych oraz usuniecie działań w pritf

Answer 2

Pracujesz na śmieciowych danych, tzn najpierw tworzysz zmienne i liczysz jakieś śmieci, a dopiero później przypisujesz im prawidłowe wartości.

Kod jest niepoprawny, tj nie masz gwarancji ze np przy starcie w b nie ma zera, co przy dzieleniu powoduje crash