La scanf del lato del triangolo ha quel simbolo "<" che nn centra nulla....poi io metterei void main, dato che nn restituisce alcun valore!

poi invece di tutti quegli if che incasinano il programma, un bello:
codice:
...
switch(scelta)
{
case '1': /*codice riferito al triangolo*/; break;
case '2': /*codice riferito al quadrato*/; break;
}