Po pierwsze: nazwy funkcji nie są zbyt logiczne, nie oddają tego, co funkcja robi. Jeśli masz funkcję która pobiera rozmiar tablicy, czemu nie nazwiesz jej PobierzRozmiarTablicy?
Po drugie: miałeś napisać funkcję, która zapyta o pozycję i zwróci wartość elementu o podanej pozycji w tablicy. Mogłeś to zrobić w jednej funkcji.
Po trzecie: nie robi się takich nieskończonych pętli jak masz w tej chwili. Nigdzie nie masz napisane, że masz sprawdzać wartości w nieskończoność - wystarczy obsłużyć wyjątek jeśli pojawi, bez żadnych pętli.
Po czwarte: nie zabezpieczyłeś programu przed sytuacją, gdy użytkownik poda jako rozmiar wartość ujemną. Tworzenie nowej tablicy może też nie udać się z innych przyczyn, np. przekroczenie dopuszczonej pamięci.
0