Poprawna forma wykonania zadania?

Question

W toku nauczania wracam z kolejnym pytaniem:

Otóż wykonuje zadanie o treści:

Napisz krótki program, który poprosi o podanie wzrostu w centymetrach, a następnie przekonwertuj ten wzrost na stopy i cale (1 cal to 30,48 cm). Niech program za pomocą podkreślenia odróżnia rodzaj odpowiedzi. Użyj stałej symbolicznej const, aby zapisać współczynnik konwersji.

Pomijamy fakt, że podana jednostka jest błędna 1 stopa to 30,48  a cal to 1/12 stopy.

O to konkretne zadanie ktoś już pytał w 2017r. odpowiedź co prawda została udzielona, ale porównałem sobie całość z moim kodem, który jest znacznie krótszy i zacząłem się zastanawiać czy może coś zrobiłem nie tak. 

https://forum.pasja-informatyki.pl/239578/ocena-kodu-c

Wprowadziłem kod podany pod linkiem do kompilatora i wyrzuca on identyczne dane jak moje ale ilość tekstu jest znacznie mniejsza.

#include<iostream>

using namespace std;

main()

{

  cout << "Podaj swoj wzrost w centymetrach:"; 
  int centymetry;
  cin >> centymetry;

  const double stopa = 30.48;  // 1 stopa =12 cali
  const double cal = 30.48 / 12 ;


  cout << "Sprawdzam ile wynosi on w stopach i calach." << endl;
  cout << "Masz: " <<  centymetry / stopa << " stop, lub " << centymetry / cal << " cali."  << endl; 

}

Rozumiem, że zadanie ma mnie oswoić z używaniem stałej symbolicznej const. Brakuje mi w książce trochę tego, że autor nie podaje przykładowego rozwiązania, dzięki czemu miałbym pewność, że zadanie zostało wykonane zgodnie ze "sztuką" 

Dzięki za wszelkie uwagi. 

Comments

Rozumiem, że zadanie ma mnie oswoić z używaniem stałej symbolicznej const.

warto tu też wspomnieć o:

• Naming: Why should named constants be all uppercase

• What is the history for naming constants in all uppercase?

---

Najlepiej odstawić Pratę i szukać wiedzy w nowszych źródłach, chociażby tutaj. Od tego czasu wyszło trochę nowych standardów, a więc i nowych rzeczy, o których się w Pracie nie dowiesz. 

---

Whiskey_Taster zerknę na podesłany link.

 

 

Answer 1

Wzrost w formacie "anglosaskim" podajemy raczej jako np. 6 stóp i 3 cale, a nie 6,25 stopy lub 75 cali. Najpierw zatem dzielenie całkowite (wynik jest liczbą całkowitą) wzrostu przez stałą stopa, następnie dzielenie pozostałości przez stałą cale (tu chyba może być część dziesiętna, ale można też zaokrąglić wynik do liczby całkowitej). Lub prościej: liczysz liczbę cali, a potem dzielisz przez 12 i masz liczbę stóp, a reszta z dzielenia to cale.

Comments

#include<iostream>

using namespace std;

main()

{

  cout << "Podaj swoj wzrost w centymetrach:"; 
  int centymetry;
  cin >> centymetry;



  const double stopa = 30.48;  // 1 stopa =12 cali
  const double cal = 30.48 / 12 ; // 1 stopa = 30,48 cm

  int wynik_stopa = centymetry / stopa ; 
  int w_na_cal = centymetry / cal ;

  int wynik_cale = (wynik_stopa * 12);
  int wynik_cale2  = w_na_cal - (wynik_stopa * 12);

  cout << "Masz: " << wynik_stopa << " stop, oraz " << wynik_cale2  << " cali." << endl; ;
  
  



  

Odniosłem się do uwag, które mi naświetliłeś, z czego powstał powyższy kod, taka forma jest odpowiednią?

---

Obliczasz wynik_cale i nie "używasz" go. 

#include<iostream>
 
using namespace std;
 
int main() 
{ 
  cout << "Podaj swoj wzrost w centymetrach:"; 
  int centymetry;
  cin >> centymetry; 
 
  const double stopa = 30.48;  // 1 stopa =12 cali
  const double cal = 30.48 / 12 ; // 1 stopa = 30,48 cm
 
  int wynik_stopa = centymetry / stopa ; 
  int w_na_cal = centymetry / cal ;
 
  int wynik_cale = (wynik_stopa * 12);
  int wynik_cale2  = w_na_cal - (wynik_stopa * 12);
 
  cout << "Masz: " << wynik_stopa << " stop, oraz " << wynik_cale2  << " cali." << endl;

  return 0;
}

 

Twój kod zapisałbym np. tak

#include <iostream>
 
using namespace std;
 
int main()
{
  cout << "Podaj swoj wzrost w centymetrach: "; 
  int centymetry;
  cin >> centymetry;
 
  const double stopa = 30.48;
  const double   cal = stopa / 12;
 
  int wynik_stopa = centymetry / stopa; 
  
  cout << "Masz: " << wynik_stopa << " stop, oraz "
       << (int)((centymetry / cal) - (wynik_stopa * 12)) << " cali" << endl;
  
  return 0;
}

 

lub cale z większą "precyzją"

#include <iostream>
 
using namespace std;
 
int main()
{
  cout << "Podaj swoj wzrost w centymetrach: "; 
  int centymetry;
  cin >> centymetry;
 
  const double stopa = 30.48;
  const double   cal = stopa / 12;
 
  int wynik_stopa = centymetry / stopa; 
  
  cout << "Masz: " << wynik_stopa << " stop, oraz ";
  printf("%.1lf cali", ((centymetry / cal) - (wynik_stopa * 12)));
  
  return 0;
}

 

---

Na pewno zajrzę jeszcze do kodu w późniejszym czasie, żeby na świeżo rzucić okiem, czy mogę go "napisać" lepiej. Na ten moment wydaje mi się, że najważniejsze jest dojście do sedna zadania, używając do tego zagadnień z danego rozdziału.

Dziękuje wszystkim za wypowiedzi, które naprowadziły mnie na właściwą drogę odnośnie wykonania zadania.