Jun 2016
131 / 277
Jan 2018
May 2023
mariusz193
Dec '17

i po raz kolejny muszę pisać:

  1. nie zakładamy nowego wątku, gdy już istnieje wątek poświęcony temu zadania - to zadanie ma wiele wątków, dzięki niefrasobliwości takich jak ty
  2. jeżeli jest wątek, czytamy - z dużym prawdopodobieństwem problem został dawno temu omówiony
  3. wskazane jest przekazywanie kodu poprze ideone.com2 - tam też można przetestować rozwiązanie
  4. oczywiście przed napisaniem na forum warto program przetestować przynajmniej dla danych podanych w zadaniu - twój program daje dla tych danych niepoprawny wynik
1 month later
tsq1956
Jan '18

Nie mam pojęcia już czemu… jakieś podpowiedzi nie liczę na rozwiązanie lecz na dobry kierunek…

– kod usunięto –

tarpauwatratar
Jan '18

Wkleiłeś kod w taki sposób że trudno go analizować, oczywiście post jest w złym miejscu i niezgodnie z regulaminem, ale wg mnie nieprawidłowo definiujesz tablicę dynamiczną.

tsq1956
Jan '18

Faktycznie wrzucilem deklarowanie do maina… dodalem jeszcze do warunku ze a>0 i poszlo . Dziekuje tarpauwatratar:)

orlowskiw
Jan '18

Wydaje mi się, że kod działa, jednak nie jest akceptowany. Gdzie leży przyczyna?

`#include

using namespace std;

int main()
{
int n, d;
cin>>d;
for(int j=1; j<=d; j++)
{
while(cin >> n)
{

    if(n==1){
        
        break;
    }
    bool pierwsza = true;
    for(int i=2; i*i<=n; i++)
    {
        if(n%i==0){ 
            pierwsza = false;
        }
    }

if(pierwsza==true){
    cout << "TAK" << endl;
}
else{
    cout << "NIE" << endl;
}
}
}
return 0;

}`

radektcz
Jan '18

Albo źle przekopiowałeś albo na samym początku nie masz “#include” - nie dodałeś biblioteki chociażby w celu użycia cin/cout. Używaj ideone. Na pierwszy rzut oka widać, że twoja pętla jest źle wykonana. Sprawdź dodatkowo co się stanie jak wprowadzisz 0 albo 1 jako liczbę do sprawdzenia.

orlowskiw
Jan '18

Bardzo dziekuje, po dodaniu biblioteki i cout “NIE” dla wartosci <=1 kod zaakceptowany:)

wiktort1
Feb '18
esbe

Możesz przyspieszyć program (jeśli używasz cin i cout, a nie scanf i printf) dodając na początku funkcji main() dwie linijki kodu, a mianowicie: cin.tie(NULL);ios_base::sync_with_stdio(0);
Mi się udało w ten sposób zejść z 0.16s do 0.01 :wink:

spacehunter
Feb '18
1

Jestem tu nowy i niestety napotkałem problem przy pierwszym zadaniu. Niestety nie wiem czemu program mi nie działa. Jeśli ktoś mógł by mi podpowiedzieć co robię nie tak bym był wdzięczny

mariusz193
Feb '18

to może po prostu popatrz na wynik testu podany w zadaniu i na wynik twojego programu
albo jeszcze raz przeczytaj dokładnie treść zadania

spacehunter
Feb '18

z przeprowadzonych testów za każdym razem wyrzuca mi prawidłową odpowiedź dlatego nie wiem czemu nie chce mi przejść

redysz
Feb '18

Nie wyrzuca. W zasadzie nie masz ani jednej prawidłowej odpowiedzi. Przypatrz się uważnie - co masz wyświetlić, a co wyświetlasz.

spacehunter
Feb '18
1

Coś się chyba źle skopiowało. Wszystkie odp są prawidłowe więc wklejam rezultat. Może jest jakiś problem z wklejaniem.

tarpauwatratar
Feb '18

Czy jeżeli będę trzecią osobą, która powie Ci, że Twój output jest różny od outputu w treści zadania, poprawisz oczywisty błąd?

narbej
Feb '18

cZWARTY, tzn czwarty, a może cZwArTy albo CzWaRtY albo Czwarty a jeżeli Ci nie [z]robi różnicy to może nawet CZWARTY! :wink:

spacehunter
Feb '18

Z treści przecież wynika że jeśli liczba jest pierwsza to wyjście jest Tak a jeśli nie jest pierwsza to wyjście Nie. Można jaśniej a później pośmieje się z wami.

narbej
Feb '18

Dla każdej liczby słowo TAK …

Czxy TAK == Tak czy NIE czy Nie czy nie?

PS
sorry, za ; - ), ale nie był źłośliwy :wink:

spacehunter
Feb '18
1

Ech język polski mnie przytłacza :slight_smile: poprawiłem i nadal mi nie przechodzi może jeszcze jedna podpowiedź

narbej
Feb '18

Ale co?
Dalej masz “Nie” i “Tak” zamiast “NIE” i “TAK”.

Użyj wszędzie: “NIE” i “TAK”

a nie: “Nie” i “Tak”