Zaczynam przygode z programowaniem i mam problem z tym zadaniem

Question

1.4.3 Napisz program wczytujący ze standardowego wejścia trzy dodatnie
liczby całkowite n, m i k, i wypisujący w kolejnych wierszach wszystkie
wielokrotności n większe od m i mniejsze od k

Taka jest tresc zadanka

#include <iostream>
#include <cstdlib>
#include <conio.h>
using namespace std;
int m,n,k;
int main ()

{
     cout << "Podaj liczbe ktora bedzie wypisywana" << endl;
     cin >>n;
    cout << "Podaj liczbe ktora bedzie gorna granica" << endl;
     cin >>k;
    cout << "Podaj liczbe ktora bedzie dolna granica" << endl;
     cin >>m;

     for(int i=0;i<=k;i++)
     {
         int mnoznik=i+2;
        if((mnoznik<=k)&&(mnoznik>=m))
            cout <<n*mnoznik<<endl;
        else
           getch() ;
     }

  return(0);
}

A oto mój kod i teraz mam pytanie czy dobrrą drogą ide i gdzie jest błąd

z góry dziekuje ;)

Comments

Kod wstawiamy w blok code (ostatni, w menu edytora). 

Nazwa tematu powinna sugerować o co chodzi.

 

C++ dawno temu się uczyłem, ale wydaje mi się, że if tak jak w innych językach ma strukturę:

if (warunek) {
 coś tam wykonaj;
}else{
coś tam wykonaj;
}

 

---

Tak z ciekawości co robi w tym kodzie #include <conio.h>

---

Wpisz w google i będziesz miał pełno o tym. :)

---

Jak jest jedna instrukcja to klamry pomijać można, zasada nie tylko w C++. Conio.h nie jest częścią C++.

---

Mi chodzi o to co w tym konkretnym kodzie robi conio.h

dobra już doczytałem że w Windowsach zamyka się konsola bez getch

Tylko że zamiast tego getch można użyć getchar i nie trzeba includować głupiego conio.

---

Powoduje błąd, jeżeli chodzi ci o coś innego to sprecyzuj proszę.

---

Chyba chodzi mu o to, że się zastanawia po co autor użył tego conio.h i czemu ma służyć w kodzie? Jak dla mnie to jest zbędna w ogóle ta linijka. Skasowałem całego elsa'a i jego zawartość i działa tak jak trzeba.

---

Jest to po to, aby móc korzystać z funkcji getch(), ponieważ znajduje się ona właśnie w bibliotece conio.h

---

No dobra. Ale mi chodziło o to po co jest to getch? Jakie zastosowanie ma w kodzie i jaka jest przewaga między kodem z tą linijka a między kodem bez niej.

Bo z moich obserwacji wynika, że z tą linjka program nie działa poprawnie, a bez niej już tak..

---

Bo w zależności od IDE albo zobaczymy wynik, albo konsola się otworzy i zamknie w ułamku sekundy. Zainstaluj sobie visual studio i tam wypróbuj ten program w trybie debugowania(domyślnie f5). Bez tej linijki program ci się skompiluje ale nie będziesz w stanie zobaczyć okienka konsoli, zaś dzięki takim trickom jak getch(), która oczekuje na pobranie znaku możemy sobie spokojnie popatrzeć na wynik działania naszego programu.

Answer 1

Nie do końca dobrą drogą idziesz.

"wypisujący w kolejnych wierszach wszystkiewielokrotności n większe od m i mniejsze od k"

Twój program wypisuje wielokrotności n co prawda większe od n ale już nie mniejsze od k tylko po prostu tyle razy ile K wynosi.

Po pierwsze zastanowiłbym się na warunakami IF'a:

if((mnoznik<=k)&&(mnoznik>=m))

Sprawdzasz czy mnożnik jest mniejszy od k...

A wypisujesz mnoznik*n.

cout <<n*mnoznik<<endl;

Jeśli K=22, n=10, m=0 to powinno się wykonać 2 razy, nie? A u Ciebie

int mnoznik=i+2;

będzie się inkrementował 20 razy za każdym razem będzie mniejszy od K, więc się wypisze 20 razy. Dlatego pomyśl nad tym warunkiem - co chcesz sprawdzać - jaka była treść zadania.

Jeszcze propo tego:

int mnoznik=i+2;

Czemu chcesz od 2 zaczynać? n*2 = 2n, a czemu ma nie być pierwszej wielokrotności?

Po co ta linijka w ogóle się znajduje w kodzie? w jakim celu?

getch() ;

I ten else?

EDIT!: Dobra, to że mi bez tego działa poprawnie to nie znaczy, że komuś nie potrzebna. Z tym też działa, także odwołuje to co napisałem.