Wczytywanie liczb pierwszych z pliku txt

Question

#include <iostream>
using namespace std;
int main ()
{
int a;
int podzielna;

cout << "podaj a: ";
cin >> a;

cout << 2 << endl;

for (int kolejna=3; kolejna<=a; kolejna++)
{
podzielna = 0;

for (int i = 2; i < kolejna; i++)
{
if (!(kolejna%i))
{
podzielna = 1;
break;
} 
}

if (!podzielna)
{
cout << kolejna << endl;
}
}
}

Jak zmienić ten kod żeby wczytywało z pliku liczby, zamiast podawać liczbę w programie

Answer 1

Witam,

w tym przypadku musisz tylko "podmienić" cin na pętlę, która obsługuje czytanie z pliku i stamtąd pobieramy właściwą liczbę i prowadzimy obliczenia, które już podałeś. Przykładowe rozwiązanie

#include <iostream>
#include <fstream>

int main()
{
    std::string line;
    std::ifstream myfile("numbers.txt");
    if(myfile.is_open())
    {
        while(myfile >> line)
        {
            //std::cout << line << " " << std::endl;//podglad pliku
            int a = std::stoi(line);
            //Twoj kod
            int podzielna;
            std::cout << "Nowa liczba" << std::endl; //poinformaowanie o nowej liczbie dla czytelności
            std::cout << 2 << std::endl;
            for (int kolejna=3; kolejna<=a; kolejna++)
            {
            podzielna = 0;

            for (int i = 2; i < kolejna; i++)
            {
            if (!(kolejna%i))
            {
            podzielna = 1;
            break;
            }
            }

            if (!podzielna)
            {
            std::cout << kolejna << std::endl;
            }
            }
            //nowa liczba


        }
        myfile.close();
    }


    return 0;
}

Można ten kod poprawić bo nie działa prawidłowo dla wartości mniejszych od 2 np dla jedynki. W pliku numbers.txt umieści liczby całkowite oddzielone spacją bądź nową linią.

1 2 3 4 5 6 7 8 9
10 22 23

Więcej informacji:

• http://www.math.us.edu.pl/pgladki/faq/node46.html
• http://www.cplusplus.com/doc/tutorial/files/

Comments

#include <iostream>
#include <fstream>
 
int main()
{
    std::string line;
    std::ifstream myfile("liczby.txt");
    if(myfile.is_open())
    {
        while(myfile >> line)
        {
            std::cout << line <<"C:\\Users\\u\\Desktop\\Ksyś pastal\\dev " << std::endl;
              if (myfile.is_open())
  {
    while ( getline (myfile,line) )
    {
//.................
    }
    myfile.close();
  }

  else cout << "Nie mozna otworzyc pliku.";
  
  
			int a = std::stoi(line);
            
            int podzielna;
            std::cout << "Nowa liczba" << std::endl; 
            std::cout << 2 << std::endl;
            for (int kolejna=3; kolejna<=a; kolejna++)
            {
            podzielna = 0;
 
            for (int i = 2; i < kolejna; i++)
            {
            if (!(kolejna%i))
            {
            podzielna = 1;
            break;
            }
            }
 
            if (!podzielna)
            {
            std::cout << kolejna << std::endl;
            }
            }
            //nowa liczba
 
 
        }
        myfile.close();
    }
 
 
    return 0;
}

Przypadkowo edytowałem ten komentarz, proszę wybaczyć mi za to - myślałem że pisze nowy. Dalej nie działa :(

---

Dobrze podałeś ścieżkę do pliku? Staraj się użyć ścieżki bezwzględnej do pliku np

C:\\Users\\Jon\\Desktop\\numbers.txt

Możesz dodać else, która wyświetli wiadomość jeśli nie udało się otworzyć pliku.

  if (myfile.is_open())
  {
    while ( getline (myfile,line) )
    {
//.................
    }
    myfile.close();
  }

  else cout << "Nie mozna otworzyc pliku."; 

To było w linku, który podałem w pierwszej odpowiedzi.

---

#include <iostream>
#include <fstream>
 
int main()
{
    std::string line;
    std::ifstream myfile("liczby.txt");
    if(myfile.is_open())
    {
        while(myfile >> line)
        {
            std::cout << line <<"C:\\Users\\u\\Desktop\\Ksyś pastal\\dev " << std::endl;
              if (myfile.is_open())
  {
    while ( getline (myfile,line) )
    {
//.................
    }
    myfile.close();
  }

  else cout << "Nie mozna otworzyc pliku.";
  
  
			int a = std::stoi(line);
            
            int podzielna;
            std::cout << "Nowa liczba" << std::endl; 
            std::cout << 2 << std::endl;
            for (int kolejna=3; kolejna<=a; kolejna++)
            {
            podzielna = 0;
 
            for (int i = 2; i < kolejna; i++)
            {
            if (!(kolejna%i))
            {
            podzielna = 1;
            break;
            }
            }
 
            if (!podzielna)
            {
            std::cout << kolejna << std::endl;
            }
            }
            //nowa liczba
 
 
        }
        myfile.close();
    }
 
 
    return 0;
}

Mam to zrobione w ten sposób, co jest źle bo nie rozumiem? A ps. co do jedynki to nauczyciel kazał mi żeby tam była... Więc to nie jest błąd, ale dziękuję za link.

(plik txt znajduje się w tym samym folderze na użytkowniku U na pulpicie w folderze z tą nazwą) 

---

Bardzo dużo rzeczy jest źle. Kod jest nielogiczny, błędny po prostu bez namysłu wkleiłeś kod. Użyj tego kody tylko zmień ścieżkę do Twojego pliku .txt z liczbami

#include <iostream>
#include <fstream>

int main()
{
    std::string line;
    std::ifstream myfile("numbers.txt");
    if(myfile.is_open())
    {
        while(myfile >> line)
        {
            //std::cout << line << " " << std::endl;//podglad pliku
            int a = std::stoi(line);
            //Twoj kod
            int podzielna;
            std::cout << "Nowa liczba" << std::endl; //poinformaowanie o nowej liczbie
            std::cout << 2 << std::endl;
            for (int kolejna=3; kolejna<=a; kolejna++)
            {
                podzielna = 0;

                for (int i = 2; i < kolejna; i++)
                {
                    if (!(kolejna%i))
                    {
                    podzielna = 1;
                    break;
                    }
                }

                if (!podzielna)
                {
                    std::cout << kolejna << std::endl;
                }
            }
            //nowa liczba


        }
        myfile.close();
    }
    else
    {
        std::cout << "Nie mozna otworzyc pliku.";
    }


    return 0;
}

 

---

Zrobiłem jak kazałeś, wyskakuje mi jeden błąd - 13    21  

 C:\Users\u\Desktop\Ksyś pastal\dev\BezNazwy3.cpp    [Error] 'stoi' is not a member of 'std'

- w 13 linijce kodu -       int a = std::stoi(line);

---

Masz dość stary kompilator nie posiadający standardu C++11

https://stackoverflow.com/questions/38034197/compile-error-stoi-is-not-a-member-of-std

Zamień

int a = std::stoi(line);

na

 int a = atoi(line.c_str());

 

---

13    38    C:\Users\u\Desktop\Ksyś pastal\dev\BezNazwy3.cpp    [Error] 'atoi' was not declared in this scope

Znów to samo, po zamianie, korzystam z Dev c++ który jest obowiązkowy w mojej szkole

---

Wpisuj sobie komentarze błędów kompilatora do wyszukiwarki. W zazwyczaj pierwszym odnośniku jest rozwiązanie problemu.

https://bytes.com/topic/c/answers/954092-whats-wrong-program

Dodaj w pierwszej linijce:

#include <cstdlib>

 

---

@Krzysztof Pasztaleni,   po co Ci to atoi, skoro możesz od razu czytać liczbę?

int value;

while (myfile >> value) {
    ...
}