Apua tarvitaan

Siinäpä koulutehtävä. Pyysitkö opettajalta lupaa kopiointiin tänne vai toimitko luvatta?

Mitä järkeä sinun on ohjelmointia opetella, jos et osaa aivan yksinkertaisimpia tehtäviä tehdä? Et opi mitään, jos täällä vain kyselet neuvoa. Sen sijaan voisit vaikkapa paperilla miettiä, mitä ohjelma tarvitsee toimiakseen, ja miten se toimii.

Noh, tässä summafunktio pseudokoodina. Loput funktiot osaat varmaan tehdä tämän esimerkin perusteella.

void summa() {
int luku1, luku2, summa;
tulosta("Syötä 1. luku");
kysyLuku(luku1);
tulosta("Syötä 2. luku");
kysyLuku(luku2);

summa = luku1 luku2;
tulosta(summa);
}

Fiksu koodari olisi toteuttanut kaikki laskutoimitukset yhdellä funktiolla;
float laske(int toimitus, float luku1, float luku2) {
if (toimitus == 1) {
return luku1 luku2;
} else if (toimitus == 2) {
return luku1 - luku2;
} else if (toimitus == 3) {
return luku1 * luku2;
} else {
tulosta("Tuntematon laskutoimitus");
return 0;
}
}

Neuvoa saa toki kysyä, mutta pitää ensin itse yrittää. Toivottavasti nämä kaksi eivät ole olleet ainoat tehtäväsi.

Alla on se, mitä olen saanut koodattua ja lopuksi oikea ratkaisu.
Missä menee vikaan, kun ohjelma pyytää syöttämään ensimmäistä lukua, vaikka ohjelma pitäisi lopettaa negatiivisella luvulla?

1:kahden luvun summa
2:kahden luvun erotus
3:kahden luvun tulo

Tässä tämä nykyinen koodi.
#include

void summa(void);
void erotus(void);
void tulo(void);

int main(void)
{
int valinta, luku1, luku2;
do{

printf("1:kahden luvun summa\n2:kahden luvun erotus\n3:kahden luvun tulo\n