Program przeliczający minuty i sekundy na stopnie

pytanie zadane 26 lipca 2016 w C i C++ przez dziubi21 Początkujący (310 p.)
edycja 26 lipca 2016 przez dziubi21

Witam, proszę o ocenę kodu programu(który poprosi użytkownika o podanie długości geograficznej w stopniach, minutach i sekundach, wyliczający następnie długość geograficzną w formacie dziesiętnym). Czy dobrze wygląda ten kod?

#include<iostream>

using namespace std;

int main()
{
	double stopnie;
	double minuty;
	double sekundy;
	cout << "Podaj długość w stopniach, minutach i sekundach: ";
	cout << "\nNajpierw podaj stopnie: ";
	cin >> stopnie;
	cout << "Nastepnie podaj minuty stopnia łuku: ";
	cin >> minuty;
	cout << "Na koniec podaj sekundy: ";
	cin >> sekundy;
	const double sekundy_na_minuty = sekundy / 60;
	const double minuty_na_dziesietny = (minuty+sekundy_na_minuty) / 60;
	cout << stopnie << " stopni, " <<minuty <<" minut, " << sekundy << " sekund = ";
	cout << stopnie+minuty_na_dziesietny ;
}

2 odpowiedzi

odpowiedź 26 lipca 2016 przez Ehlert Ekspert (212,670 p.)

const musi być znany w momencie kompilowania. Najprawdopodobniej przeszło na g++
main jest intem więc wypadało żeby coś zwracał chciaż to bardzo drażliwy temat, przynajmniej na tym forum :D
Poza tym nie ma co oceniać. Możesz to opakować w jakiś obiektowy wrapper i będzie ładnie. Jeśli się uczysz to nie trać czasu na takie programy. Pędź dalej :D

komentarz 26 lipca 2016 przez Szykem2 Nałogowiec (29,510 p.)

1. "main jest intem" - z tym muszę się nie zgodzić. Main jest funkcją( a w zasadzie w takiej formie w jakiej to zapisałeś jest wskaźnikiem na funkcję), a nie zwykłą zmienną typu int.

2. Kompilator powinien wygenerować komendę return 0; w mainie jeżeli takowej brakuje, aczkolwiek dobrze jest to napisać.

3. Co do oceniania to jest to tak prosty kod że jakości kodu rzeczywiście nie ma jak oceniać ewentualnie dodać ze trzy puste linie i rozdzielić dodawanie spacjami, ale to kwestie subiektywne.

1	komentarz 26 lipca 2016 przez Ehlert Ekspert (212,670 p.) Dzięki. Do tej pory myślałem że main to zmienna. Wszystko teraz mi się rozjaśniło.

komentarz 27 lipca 2016 przez smh Obywatel (1,940 p.)

const musi być znany w momencie kompilowania

Nieprawda. constexpr już tak.

wypadało żeby coś zwracał

Zwraca. Poprzez domniemanie, na koniec main zostanie zwrócone EXIT_SUCCESS.

komentarz 27 lipca 2016 przez Ehlert Ekspert (212,670 p.)

Co do const zwracam honor. Ale porządek w kodzie jest dobrą rzeczą i jeden return nikogo nie zbawi. Lepiej unikać konstruktorów i innych cudów domniemanych.