Sprawdz prosty kod

Witam,
Jestem tu nowy oraz początkującą osobą uczącą się programowania. Mam prośbę czy może ktoś sprawdzić prosty kod do zamiany cali na cm. Program działa ale czy na pewno jest dobrze napisany czy nie wymaga jakiś udoskonaleń.

#include <iostream>

using namespace std;
float liczbaCali,cm,cal;

int main()
{
    cal=2.54;

    cout<< "***** Autor: xxxxxx xxxxxxx *****\n";
    cout<< "**** Data prod.:18.01.2016r. ****\n";
    
    cout<<"\nPodaj dlugosc wyrazona w Calach: ";
    cin>> liczbaCali;
    cm = (liczbaCali*cal);

    cout<< "\nDlugosc cali zamieniona na centymetry: "<<cm<<endl;
    return 0;
}

Nie używaj zmiennych globalnych jeśli nie musisz (nie musisz). Deklaruj je jak najbliżej miejsca użycia, w jak najmniejszym zakresie. cal jest dość mało deskryptywną nazwą, a samo w sobie powinno być stałą (najlepiej constexpr, jeśli kompilator pozwala). Poza tym wygląda ok, ale kod jest szalenie krótki ;)

Nie używaj zmiennych globalnych. Za to używaj const tam, gdzie const powinno być.
Sugerowałbym raczej double, float jest głupi, ma słabą precyzje i nie jest w stanie reprezentować wszystkich liczb. Mówię tu o prostych, krótkich liczbach. Zaraz sobie przypomne.
EDIT: o, patrzcie
http://melpon.org/wandbox/permlink/FqGJ7Z6kBb2Hxxyt

Szalenie krótki ale to chyba dobrze? Dziękuje za cenne uwagi

To ja tez się o coś zapytam

#include <iostream>

using namespace std;

void generuj(int tab[], int n)
{
	for (int i = 0; i < n; i++)
	{
		cin >> tab[i];
		
	}
}

void wyswietl(int tab[], int n)
{
	for (int i = 0; i < n; i++)
	{
		cout << tab[i] << "  ";
	}
}
int main()
{
	const int N = 25;
	int tab[N], n;
	cout << "Podaj liczbe elementow do generowania: " << endl;
	cin >> n;
	generuj(tab, n);
	wyswietl(tab, n);

	getchar();
}

Dlaczego to nie śmiga?

Ale jak nie śmiga?
http://melpon.org/wandbox/permlink/EO0NIIVNC2IWghVl

U mnie w Visual Studio wykonuje się część kodu, czyli cout << "Podaj liczbe elementow do generowania". Potem gdy wpisuje jakąś liczbę, nic się nie dzieje.