Aby ktoś coś kolwierk zobaczył powinieneś to ułatwić. Pisząc tylko dla siebie i kompilatora możesz cały kod napisać nawet w jednej linii [jeżeli się da] lub kilku. Prosząc o pomoc postaraj się jeszcze staranniej, dokładniej i konsekwentniej formatować swój kod. W tej chwili tak nie jest.
Na teraz jedyne uwagi to:
-
short a,b=0,c,licznik=0;
scanf("%hi",&a);
while(b!=c)
c jest nieinitializowane, więc może przyjąć dowolną wartość. Co jeżeli będzie już na początku równe 0?
- Używanie typu short jest rzadko uzasadnione [np duże tablice stosunkowo małych liczb]. Najczęściej najbezpieczniej i najwygodniej jest zawsze używać [tutaj też] tylko typu int.
- Popraw formatowanie, może wtedy z przyjemnością “poczytam” twój kod.