Co jest złego w tym kodzie?

Question

Cześć! :)

Napisałam program z zadania: 

• Napisz program, który wczyta liczbę, a następnie wypisze ją na ekranie. Zabezpiecz przed wczytaniem nieprawidłowej liczby. Wykorzystaj do tego celu wiedzę z jednego z poprzednich rozdziałów. Sposób działania programu: 
  1. Podaj liczbę 
  2. Jeżeli błąd, wróć do kroku 1. 
  3. Wypisz liczbę, która została podana.

Oto kod: 

#include <iostream>
/*Program: zadanie domowe nr 1 z lekcji: Pętle do ... while. */
using namespace std;
int main()
{
	double liczba;
	cout<<"Podaj liczbe:"<<endl;
	cin>>liczba;
	if(cin.fail())
	do
	{
		double liczba;
		cout<<"Podaj liczbe:"<<endl;
		cin>>liczba;
	} while(cin.good());
	
	if(cin.good())
	cout<<"Podana liczba to:"<<liczba<<endl;

	return 0;
	system("pause");
}

Wszystko się ładnie kompiluje, tylko, że program źle działa. Po:

(przykład)

Podaj liczbę: t

Podaj liczbę:

Program się zamyka, a ja chciałabym, żeby robił pętlę jak w treści zadania. Pytanie jak wyżej. 

Dziękuję za pomoc. 

Answer 1

To jest przecież bez sensu.. po co dublować dane?

 

// wrzuciłem w kompilator, działa.

 

#include <iostream>

/*Program: zadanie domowe nr 1 z lekcji: Pętle do ... while. */

using namespace std;

int main()

{

    char liczba;

   do

{

 cout<<"Podaj liczbe:"<<endl;

    cin>>liczba;



    } while((liczba>47)&&(liczba>58));;

     

    cout<<"Podana liczba to:"<<liczba<<endl;


    return 0;

    system("pause");

}

 

Comments

Okej, rozumiem kod.

Tylko to nie do końca pewnie ten kod, który ma być w tym zadaniu, bo mam użyć pętli do ... while (a ty użyłeś while), no a biblioteka string w tym kursie (bo to zadanie z kursu) jest dalej.

Ale mimo wszystko dziękuję.

:)

---

Dobra tutaj masz już wszystko tak jak powinno być: 

 

#include <iostream>

/*Program: zadanie domowe nr 1 z lekcji: Pętle do ... while. */

using namespace std;

int main()
{
    int liczba;
    do
    {
        cout<<"Podaj liczbe:\n";
        cin.clear();
        cin.sync();
    }
    while (!(cin>>liczba));

    cout<<"Twoja liczba to: "<<liczba;
    
    return 0;
}

 

 

Nie użyłem System("Pause");

---

Ok, dziękuję :)

---

Nawet jeśli nie użyłem do-while to chyba nie problem przerobić skoro zasada działania jest identyczna z małą różnicą. Trochę samodzielności :)

---

Super.

Dopiero teraz miałam okazję, porównując kody, stwierdzić to, co napisałeś w komentarzu powyżej.

A forum jest od zadawania pytań, może nie raz prostych i głupich, ale nadal pytań :)

(Każdy przecież zaczynał)

Answer 2

Czy ten program musi wykorzystywać pętle do...while? Łatwiej byłoby użyć funkcji 'goto'

Comments

goto to obejście problemu- nie zalecane.

---

Ale po co korzystać z ASCII skoro możemy w dwóch linijkach zapętlić program za pomocą 'goto'?

---

A no po to,że goto to brzydka składnia.

---

Tak, musi.

Ale jeśli masz inną propozycję i czas, to napisz.

:)

---

Nieumiejętne posługiwanie się instrukcją goto może powodować trudne do wykrycia błędy typu zapętlenie programu w nieskończoność i jeżeli nie jest to uzasadnione zalecia się omijanie tej instrukcji :P goto można użyć np. do wyjścia odrazu z wielu zagnieżdzonych pętli ale budowanie pętli działających jak do .. while za pomocą goto mija się z celem i drastycznie powoduje, że kod staje się mniej czytelny :)

---

goto używa się tylko w asemblerze. Pobnieważ jest to brzydka funkcja któa podowuje   bałagan w kodzię i w wydajnośći sprzętowej. Dużo skoków i odkładania na stos kodu programu, skutkiem tego jest spadek wydajności i bardzo ciężie debugowanie programu.

Answer 3

skrócony zapis (dla tego wykorzystuję się pętle do_while)

 

#include <iostream>
/*Program: zadanie domowe nr 1 z lekcji: Pętle do ... while. */
using namespace std;
int main()
{
  double liczba;
    do
    {
       cin.clear();
	   cin.sync();
        cout<<"Podaj liczbe:"<<endl;
        cin>>liczba;
		
	} while(cin.fail());
     
    if(cin.good())
    cout<<"Podana liczba to:"<<liczba<<endl;

 
  system("pause");
   return 0;
}

 

Comments

Dziękuję. To mi się podoba.

:)

---

Ten ostatni if jest również raczej zbędny. do.. while będzie tak długo działał, aż "cin będzie good", więc nie ma potem potrzeby sprawdzać czy "cin jest goog" ifem. Po pierwszej pętli od razu cout i też powinno śmigać.

---

Faktynicze, skupiłem się na pętli :D

---

ja to zrobilem w ten sposob, dziala, ale nadal uwazam, ze cos zrobilem nie tak. Bylbym wdzieczny za pomoc i komentarze.

 

 

#include <iostream>
#include <limits>

using namespace std;

int main()
{
int liczba;
bool sukces;

if(liczba != sukces)
do
{
    cout << "Podaj liczbe: ";
    cin >> liczba;
    cin.clear();
    cin.ignore(numeric_limits < streamsize >::max(), '\n');
}while(liczba == sukces);
cout << "Podales liczbe: " << liczba <<endl;


    return 0;
}

 

Answer 4

Wydaję mi się że błedem jest tu że najpierw zapisujesz strumień do zmiennej a potem sprawdzasz strumień.

Więc jeśli wpiszesz znak np t
to najpierw spróbuje t zapisać jako double (char do double) i tu sie wysypuje.
Dopiero potem sprawdza If'a

Answer 5

W pętli nie musisz deklarować znowu zmiennej "double liczba" linia 12, ponieważ tworzy toz meinna lokalna inna niż w lini 6. ZMienna lokalna będzie usunieta po wyjściu z pętli, tak więc w lini 18 wypisujesz na ekran starą zmienną ("t"), anie tą poprawioną w pętli.

Answer 6

Sprawdź swój program bez lini 12

Comments

Sprawdzałam.

Nie działa.

Mam nadzieję, że to nie przez kompilator.

Answer 7

//Zaminiona linia 21 z 22 oraz dopiski o czyzzczeniu bufora i synchronizacji.
//(musisz wyczyścić bufor ze złej zmiennej , bo znak zostaje  w buforze).
//Oraz zmiana warunku pętli (pętla do_while wykonuje sie póki warunek jest prawdziwy)

#include <iostream>
/*Program: zadanie domowe nr 1 z lekcji: Pętle do ... while. */
using namespace std;
int main()
{
    double liczba;
    cout<<"Podaj liczbe:"<<endl;
    cin>>liczba;
    if(cin.fail())
    do
    {
       cin.clear();
	   cin.sync();
        cout<<"Podaj liczbe:"<<endl;
        cin>>liczba;
		
    } while(!cin.good());
     
    if(cin.good())
    cout<<"Podana liczba to:"<<liczba<<endl;
 
  system("pause");
   return 0;
}

 

Comments

Dziękuję!

:)

---

#include<iostream>
using namespace std;
int main()
{
    int liczba;
    do
    {
        cout << "Podaj liczbe - ";
        cin.clear(); cin.sync();
        cin >> liczba;
    } while (!liczba);
    cout << "Podales liczbe - " << liczba << endl;
    return 0;
}

Ja zrobilem w taki sposob i dziala mi na dev i borlandzie ale nie dziala niestety na VS2015.

Na VisualStudio2015 gdy wpisze litere - wyswietla w jej miejsce dlugi ciag liczb.

Niebardzo wiem jak to naprawic ale ogolnie na innych kompilatorach ten kod dziala prawidlowo.