Quelle due funzioni sono scritte correttamente. Ci sono però altri problemi, primo fra tutti il fatto che nel main hai un ciclo infinito:
- Codice:
while (true)
{
cout << "Seleziona il mese [1;12]: ";
cin >> m;
cout << averagedT(m) << endl;
}
Inoltre in converToKelvin hai messo una virgola invece che un punto. Se formatti con un formattatore automatico diventa immediatamente visibile (ma mi aspetterei almeno un warning dal compilatore).
Usare così tanto le variabili globali è comunque visto come una cattiva prassi nel C++. Similmente dovresti sempre mettere le parentesi negli if e nei for: sembra inutile ma può evitarti numerosi errori. Altra cosa che dovresti fare è aggiungere dei controlli a ciò che ricevi come input.