• Najnowsze pytania
  • Bez odpowiedzi
  • Zadaj pytanie
  • Kategorie
  • Tagi
  • Zdobyte punkty
  • Ekipa ninja
  • IRC
  • FAQ
  • Regulamin
  • Książki warte uwagi

Program przeliczający minuty i sekundy na stopnie

+1 głos
678 wizyt
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

+1 głos
odpowiedź 26 lipca 2016 przez Ehlert Ekspert (204,990 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,550 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 (204,990 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,960 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 (204,990 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.

+1 głos
odpowiedź 26 lipca 2016 przez TheFeniks Gaduła (4,710 p.)
Polecam klawisz "Enter" ;)

Podobne pytania

+2 głosów
2 odpowiedzi 7,880 wizyt
0 głosów
2 odpowiedzi 508 wizyt
0 głosów
2 odpowiedzi 556 wizyt
pytanie zadane 23 października 2016 w C i C++ przez MadMax Początkujący (260 p.)
Porady nie od parady
Komentarze do pytań nie służą do odpowiadania, od tego jest wydzielona sekcja odpowiedzi. Funkcją komentarzy jest natomiast możliwość uzyskania dodatkowych informacji na temat samego posta.Komentarze

85,082 zapytań

133,883 odpowiedzi

296,819 komentarzy

56,216 pasjonatów

Motyw:

Akcja Pajacyk

Pajacyk od wielu lat dożywia dzieci. Pomóż klikając w zielony brzuszek na stronie. Dziękujemy! ♡

Oto dwie polecane książki warte uwagi. Pełną listę znajdziesz tutaj.

...