Fangen wir mal mit den zwei offensichtlichsten Dingen an.
1. wenn du in einer if-Abfrage eine auf 0 initialisierte Variable die im main nur hochgezählt wird auf kleiner 0 vergleichst, wird die Anweisung im true Zweig wohl nie ausgeführt.
2. wo rufst du denn deine ausgabe-Funktion auf?

Solche Sachen, wie fehlende Klammern erkennt man am Besten, wenn man den Code ordentlich einrückt. Das wirkt bei dir etwas unsortiert.

Beinahe hätte ich noch was wichtiges vergessen. Anweisungsblöcke müssen auch immer in Klammern gepackt werden, wenn sie aus mehr als einer Anweisung bestehen.

Es ist ein riesen Unterschied ob du
Code:
if (i>1) i--;
a=i;
oder
Code:
if (i>1)
{
  i--;
  a=i;
}
schreibst. Im ersten Fall wird a=i; unabhängig von i immer ausgeführt. Im zweiten Fall nur wenn i>1 erfüllt ist.

sast