Prosty program w C

[michu_roztocz 17]

Witam,

Mam do napisania zapewne prosty dla Was programik w C. Jest to zadanie na poniedziałek, a ja z programowaniem w C jako takim zacząłem... dzisiaj. Rok temu programowaliśmy przez miesiąc w pascalu, ale od tamtej pory wszystko zapomniałem. Tak więc mam do napisania program mający za zadanie "zliczyć elementy dodatnie i ujemne dwu lub trzycyfrowe, których liczba dziesiątek jest większa niż liczba jedności do uzyskania wartości >= koniec, ale nie więcej niż n składników". Pomijając fakt, że nie jestem pewien czy dobrze rozumiem temat nie mam bladego pojęcia co zapisać, by program odróżnił liczby których liczba dziesiątek jest większa od jedności i zaczął je zliczać. Oto co udało mi się zrobić przy pomocy google

#include <stdio.h>
#include <math.h>
main()
{
	int koniec, liczba, suma_cyfr, a, b, c, ilosc;
	printf("Podaj  liczbe 'koniec'\n");
	scanf("%d",&koniec);

	do
	{
		printf("Podaj  liczbe\n");
		scanf("%d",&liczba);
		if (!(liczba>9 && liczba<1000))
		{
			liczba = 100*a + 10*b + c;
			suma_cyfr = a + b + c;
			printf("Ilosc elementow: %d\n", suma_cyfr);
		}
		else
			printf("Podales liczbe inna niz dwucyfrowa/trzycyfrowa\n");
	}
	 while (liczba != koniec);
	 return(0);
}

Wiem że jest to dopiero połowa kodu, a to co jest działa źle. W zasadzie program funkcjonuje odwrotnie niż zamierzone: jak podam liczbę powyżej 9 to wyświetla komunikat o liczbie dwu/trzycyfrowej, dlatego nie ma warunku dla ujemnych. Nie wiem dlaczego jest jak jest. Byłbym niesamowicie wdzięczny za uzupełnienie/poprawienie kodu z jakimś komentarzem, ale wystarczą wskazówki. Z góry wielkie dzięki!

[eNgine 0]

Przepraszam bardzo, czy to jest cały kod ? Przecież to się kupy nie trzyma - nigdzie nie inicjujesz a,b,c a na ich podstawie liczysz sumę ?

 

Zacznij od początku, przede wszystkim:

 

int main(void) {


return 0;
}

 

Parę wskazówek, pomyśl jak wydobyć cyfrę jedności i dziesiątek z liczby, podpowiem, że istnieje takie działanie jak dzielenie modulo (w C pod postacią %).

 

np. 14 % 10 = 4. Kombinuj dalej.

 

Edit.

 

jak podam liczbę powyżej 9 to wyświetla komunikat o liczbie dwu/trzycyfrowej

 

No przecież taki napisałeś warunek, jeżeli liczba nie jest z przedzialu (9;1000) to obliczenia bez sensu, w innym przypadku wyświetl komunikat.

[michu_roztocz 17]

Tak, racja. Obliczenia bez sensu bo nie wiedziałem jak to wykonać. Myślę dalej, a właściwie piszę od nowa, dzięki.

[michu_roztocz 17]

Żeby choć trochę się zrehabilitować

#include <stdio.h>
#include <conio.h>
int main()
{
  int koniec,liczba,z=0,m=0,i=1;

  printf("Program jest w fazie poczatkowej, prosze sie nie smiac :) \n");
  printf("\nWpisz koncowa liczbe \n");
  scanf("%d",&koniec);
  liczba=0;
  while (liczba!=koniec)
	 {
		   printf("Wpisz liczbe dwucyfrowa lub trzycyfrowa \n");
		   scanf("%d",&liczba);
		   if ((liczba>=10 && liczba<=999) || (liczba<=-10 && liczba>=-999))
			   {
					m=i+m;
					z=liczba+z;
				}
		   else 
		   printf("Podales liczbe inna niz dwucyfrowa/trzycyfrowa \n");
	 }
	 printf("Suma wszystkich liczb: %d\n",z);
	 printf("Liczba skladnikow: %d\n",m);
	 system("pause");


}

Sumowanie działa, teraz muszę jakoś zrobić żeby liczył to co trzeba.

[eNgine 0]

Żeby choć trochę się zrehabilitować

...

Sumowanie działa, teraz muszę jakoś zrobić żeby liczył to co trzeba.

 

Brakuje Ci return 0; mimo że zdefiniowałeś, że funkcja main() zwraca typ integer.

Zamiast pisać z=z+liczba możesz użyć operatora "+=", czyli "z+=liczba;" robi dokładnie to samo, czyli powiększa z o liczbę.

 

Jeżeli chcesz policzyć ilość składników (sumowań, więc i wywołań pętli wystarczy Ci jeden licznik, w twoim przypadku samo "m" bez tego "i") skorzystaj z powiększania licznika o wartość "1".

Zrobi to operator "++" (w twoim przypadku zamiast "i=1; m=m+i;" napisz po prostu "m++;" ).

[michu_roztocz 17]

Wydaje mi się, że skończyłem. Program działa jak należy. Są jakieś uwagi co do kodu?

#include <stdio.h>
#include <conio.h>
int main()
{
int koniec,liczba,liczbaj,liczbad,temp,m=0,n;

printf("Program zlicza elementy dodatnie i ujemne twu/trzycyfrowe ktorych liczba dziesiatek jest wieksza niz licza jednosci do uzyskania wartosci 'koniec', ale nie wiecej niz n skladnikow \n");
printf("\nWpisz koncowa liczbe: ");
scanf("%d",&koniec);
printf("\nTeraz wpisz n: ");
scanf("%d",&n);
liczba=0;
while (liczba!=koniec && n!=m)
{
	printf("Wpisz liczbe dwucyfrowa lub trzycyfrowa: ");
	scanf("%d",&liczba);
	liczbaj=liczba%10;
	temp=liczba;
	temp/=10;
	liczbad=temp%10;
	liczbad=fabs(liczbad);
	liczbaj=fabs(liczbaj);
	if ((liczba>=10 && liczba<=999) || (liczba<=-10 && liczba>=-999))
	{
		if (liczbad>liczbaj)
		{
			m++;
			printf("liczba jednosci %d\n",liczbaj);
			printf("liczba dziesiatek %d\n\n",liczbad);
		}
		else 
		printf("Liczba jednosci wieksza od dziesiatek, liczba zostanie pominieta\n\n");
	}
	else
	printf("Podales liczbe inna niz dwucyfrowa/trzycyfrowa\n\n");
}
printf("Liczba elementow: %d\n",m);
system("pause");
return 0;
}

[Bono[UG] 1920]

Dwie uwagi na szybko:

-popraw literówki w wyświetlanym tekście

-wstępne obliczenia w pętli, te zaraz po wczytaniu, przenieś za if. Po co ma się liczyć jak nie wiadomo czy się przyda. Np. ktoś wpisze 2345 i po wczytaniu program coś liczy, a dopiero potem sprawdza czy liczba jest poprawna.

[michu_roztocz 17]

Ok, ostatecznie i definitywnie wysyłam program w takiej formie

#include <stdio.h>
#include <conio.h>
int main()
{
int koniec,liczba,liczbaj,liczbad,m=0,n;
printf("Program zlicza elementy dodatnie i ujemne dwu/trzycyfrowe ktorych liczba dziesiatek jest wieksza niz licza jednosci do uzyskania wartosci 'koniec', ale nie wiecej niz 'n' skladnikow \n");
printf("\nWpisz liczbe 'koniec': ");
scanf("%d",&koniec);
printf("\nWpisz liczbe 'n': ");
scanf("%d",&n);
liczba=0;
while (liczba!=koniec && n!=m)
{
	printf("Wpisz liczbe dwucyfrowa lub trzycyfrowa: ");
	scanf("%d",&liczba);
	if ((liczba>=10 && liczba<=999) || (liczba<=-10 && liczba>=-999))
	{
		int temp;
		liczbaj=liczba%10;
		temp=liczba;
		temp/=10;
		liczbad=temp%10;
		liczbad=fabs(liczbad);
		liczbaj=fabs(liczbaj);
		if (liczbad>liczbaj)
		{
			m++;
			printf("liczba jednosci %d\n",liczbaj);
			printf("liczba dziesiatek %d\n\n",liczbad);
		}
		else 
		printf("Liczba jednosci wieksza lub rowna liczbie dziesiatek, liczba zostanie pominieta\n\n");
	}
	else
	printf("Podales liczbe inna niz dwucyfrowa/trzycyfrowa\n\n");
}
printf("Liczba elementow: %d\n\n",m);
system("pause");
return 0;
}

Dzięki za wskazówki

[Bono[UG] 1920]

Znowu dwie wskazówki:

-nadal masz literówkę - pierwszy printf "liczba dziesiątek jest większa niż licza jedności"

-"\n" lepiej wstawiać na koniec printfa. Jak jest na początku może czasami wypisywać inaczej niż chciałbyś. Kiedyś miałem z tym problem: http://forum.pclab.pl/t25993.html