Co do programu to wydaje mi się, że wszystko jest w porządku.. ale wyskakuje mi błąd: illegal syscall. Nie wiem co jest nie tak, pomoże ktoś?

C code

1 2 3 4 56 7 8 9 1011 12 13 14 1516 17 18 19 2021 22 23 24 2526 27 28 29 3031 32 33 34 3536 37 38 39 4041 42

#include <cstdio>   using namespace std;   int main(){     int a;     scanf("%d",&a);     bool* T=new bool[a+1];     int h, p, w;        for (h=2; h<=a; h++)         {                 T[h] = 1;         }          for (p=2; p*p<=a; p++)         {                 if (T[p]==1)                 {                         for (w=p*p; w<=a; w=w+p)                        {                                 T[w] = 0;                         }                 }         }        int n=4;       int c=0;       while (n <= a)          {                if (a % n ==0)                {                        if(T[n]==false) c++;                 }                 n++;           }       printf("%d",c);    return 0; }

U mnie kod się kompiluje bez problemu. Jednak masz w kodzie kilka błędów.

1) Zaalokowaną pamięć trzeba zwalniać, czyli jak się pojawia new musi się też pojawić delete inaczej jest wyciek. Tak na boku nie trzeba tej tablicy dynamicznie tworzyć takie coś wystarczy :

Cpp code
1	bool T[a+1];

I możesz zapomnieć o new oraz delete.

2) Masz pewna nie konsekwencje i z tego może wynikać twój błąd. Kod wygląda jakby miał być w C a posiada pewne elementy C++ generalnie to nie jest błąd bo w projektach c++ można mieć elementy C, jednak jak jest upierdliwy kompilator to powie ci żebyś stuknął się w głowę np. za pomocą illegal syscal. Chodzi o to :

- using namespace std - w C nie ma czegoś takiego
- w C nie ma typu bool zastępuje go int.

PS. spróbuj zamiast cstdio użyć stdio.h

__________________
Prorokiem nikt nie jest, im więcej informacji podasz, tym bardziej prawdopodobne, że uzyskasz szybką pomoc.

@PlayRoll

Co do punktu pierwszego, to VLA nie zostało usunięte bez powodu z nowego c++11. Zresztą długi czas było tematem dyskusji. Akurat jeśli masz użyć wartości, która nie jest const jako wielkość tablicy, należy raczej użyć*heap, nie stack. Taki sposób inicjalizowania tablicy jest często źródłem stack corruption.

Sytuacja "z życia wzięta", to kiedy RapidXML próbuje ładować pliki XML właśnie na stos...brrr...był to jeden z trudniejszych bugów do wyłapania. Dlaczego? Bo tracisz call stack, stos jest zwalony, debugger staje w jakimś losowym miejscu i dochodzenie, co się stało potrafi zająć dużo czasu. Po prostu nie ma punktu zaczepienia dla debuggera w takiej sytuacji.

Innymi słowy, jeśli używamy wartości stałych deklarując tablice, to w czasie kompilacji kompilator może krzyczeć, że trochę*przesadzamy jeśli chodzi o stos. No i to był właśnie powód usunięcia VLA jako rozszerzenia. Za dużo z tym problemów, szczególnie w niewprawnych rękach.

dokładnie to samo miałem napisać: programiści dbajcie o swoje stosy bo jak nie wy to nikt nie zadba!

PS dobrze cię tutaj widzieć, ciekaw jestem czy gdzieś jeszcze się udzielasz jako ekspert.

@jsr184 zgadzam się, to był głupi pomysł.

__________________
Prorokiem nikt nie jest, im więcej informacji podasz, tym bardziej prawdopodobne, że uzyskasz szybką pomoc.