Program działa, tylko nwm gdzie jest błąd, ma za zadanie wypisać liczby pierwsze z pliku do pliku.

Question

Program ma wypisać z jednego pliku liczby, następnie policzyć, które są liczbami pierwszymi i zapisać w pliku txt.

#include <iostream>
#include <fstream>

using namespace std;

int main()
{
    fstream plik;
    plik.open("dane_6.txt", ios::in);
    int dane[2000], i=0, a, d=2,  n;

    if(plik.good()==false)
    {
        cout << "Plik jest nie dostepny!";
    }

    else
    {
        while (!plik.eof())
        {
            plik >> dane[i];
            i++;
        }
        while (a && d*d<=dane[i])
        {
            if (dane[i]%d==0) a=false;
            else d++;
        }
        if (a>0) cout << "Liczby pierwsze: " << a << endl;
    }

    plik.close();

    plik.open("wynik_6_1.txt", ios::out);

    for(int j=0;j<i-1;j++)
    {
        plik << dane[j] << endl;
    }

    plik.close();
    return 0;
}

 

Comments

@BedrisK

Mam takie pytania...

Co Ci szkodzi napisać co dokładnie nie działa? Co Ci szkodzi napisać jak wygląda twój plik tekstowy?

Uwierz, nie każdy jest na tyle ofiarny żeby skopiować twój kod i przeanalizować go, żeby określić poprawny kształt pliku wejściowego i zastanowić się co działa niezgodnie z twoimi oczekiwaniami.

Dlatego tak długo czekasz na odpowiedź. Polecam dać więcej informacji, a szybciej ktoś Ci pomoże        

 

---

bo to nie jest autor programu, więc nie potrafi CI powiedzieć co nie działa- tylko "nie działa"

---

już znalazłem odpowiedź, następnym razem będę pamiętał, aby dodać plik i szczegółowo wyjaśnić co nie działa

---

@BedrisK, po co Ci ta tablica? Czytasz liczbę, sprawdzasz ją i zapisujesz, jeśli trzeba.

while (!plik.eof())
        {
            plik >> dane[i];
            i++;
        }

Ten eof w pętli to jakieś przekleństwo ;/

while (plik >> dane[i]) { ++i; }

 

---

tablica jest mi potrzebna aby z pliku dać dane do i, aby nimi operować, bo jest zakres liczb

Answer 1

IMO bardziej intuicyjnie w użyciu jest zastosowanie strumieni ofstream, ifstream oraz lambdy jako filtru.

#include <iostream>
#include <fstream>
#include <iterator>
#include <functional>
#include <list>

using namespace std;

list<int> operator>>( ifstream&& input , function<bool(int)> lambda )
{
    list<int> data { istream_iterator<int>{input} , {} };
    data.remove_if( lambda );
    return data;
}

ofstream& operator>>( const list<int>& data , ofstream&& output )
{
    for( const auto& number : data ) output << number << endl;
    return output;
}

int main()
{
    auto filter = []( int candidate )
                  {
                       if( (candidate & 1) == 0 ) return (candidate == 2) ? false : true;
                       for( auto i = 3; (i * i) <= candidate; i += 2) if( (candidate % i) == 0 ) return true;
                       return candidate == 1;
                  };

    ifstream{ "input.txt" } >> filter >> ofstream{ "output.txt" };
}