Sprawdz prosty kod :: 4programmers.net

Rejestracja:ponad 9 lat
Ostatnio:około 3 lata
Postów:19

0

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ń.

Kopiuj

#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;
}

C++

edytowany 1x, ostatnio: awium 2016-01-18 18:32

~~karolinaa~~ 2016-01-18 18:50

to ma coś wspólnego z tymi manifestacjami KOD - komitet ochrony demokracji?

awium 2016-01-18 19:53

hehe nie pomyślałem nawet w ten sposób

kq

2016-01-18 18:34

kq2016-01-18 18:34

Moderator C/C++

Rejestracja:prawie 12 lat
Ostatnio:dzień
Lokalizacja:Szczecin

4

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 ;)

edytowany 1x, ostatnio: kq 2016-01-18 18:35

pingwindyktator

2016-01-18 18:38

pingwindyktator2016-01-18 18:38

Rejestracja:ponad 12 lat
Ostatnio:około 2 miesiące
Lokalizacja:Kraków
Postów:1055

4

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

edytowany 1x, ostatnio: pingwindyktator 2016-01-18 18:44

Erpeg 2016-01-19 17:18

Dzięki za info, ciekawy przykład.

Azarien 2016-01-20 15:07

float ma dokładność około 7 cyfr (z przecinkiem w dowolnym miejscu), a to są 8-cyfrowe liczby.

pingwindyktator 2016-01-20 16:51

@Azarien jasne, to jest oczywiste dla kogoś, kto zna background reprezentacji liczby float. Ale może być bardzo mylące dla początkującego, skoro http://melpon.org/wandbox/permlink/VvhksvlwVX4T0xkd

awium

2016-01-18 18:39

awium2016-01-18 18:39

Rejestracja:ponad 9 lat
Ostatnio:około 3 lata
Postów:19

0

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

kq 2016-01-18 18:40

Po prostu nie bardzo jest gdzie się przyczepić. Jak na wykonywane operacje jest adekwatnej długości.

_13th_Dragon 2016-01-18 18:57

Tak, tylko jakieś 50% więcej od niezbędnego rozmiaru w wierszach.

awium 2016-01-18 19:34

co masz na mysli?

_13th_Dragon 2016-01-18 19:50

Ze masz 19 wierszy a można to samo napisać w 13 wierszy 13*150%=19.5, zaokrąglenia wg C/C++ :D

mariusz326

2016-01-20 14:53

mariusz3262016-01-20 14:53

Rejestracja:ponad 10 lat
Ostatnio:ponad 8 lat
Postów:23

0

To ja tez się o coś zapytam

Kopiuj

#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?

kq

2016-01-20 14:55

kq2016-01-20 14:55

Moderator C/C++

Rejestracja:prawie 12 lat
Ostatnio:dzień
Lokalizacja:Szczecin

1

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

mariusz326

2016-01-20 15:30

mariusz3262016-01-20 15:30

Rejestracja:ponad 10 lat
Ostatnio:ponad 8 lat
Postów:23

0

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.