1 No entiendo por que hacen una funcion "total" que despues usan en un caso si y en otro no, cuando sirve para los dos, simplemente dandole un iva y el otro
2 Me parece que es demasiado simple lo que hace esa funcion como para crearla, pero bueno, eso ya es tema de gustos
3 Necesitas la variable global iva? de no ser asi te diria que la metas en el main
4 usas un double(int) para guardar el numero 18, cuando con un unsigned char alcanza y sobra (para el uso que le estas dando). O, si en realidad no lo necesitas, podes en el if en vez de poner iva, poner 18 y ahorrarte una variable