Pomoże ktoś co tutaj jest źle? Tu będzie chyba jakiś mały błąd, ale znaleźć nie mogę. Proszę o pomoc.
int a,b,i;
int main()
{
for(i=1;i<3;i++)
{
cin>>a;
b=pow(a,2);
for(;b>1;b--)
{
if(a%b==0)
{
cout<<"\n TAK"< }
else
{
cout<<"\n NIE"< }
}
}return 0;
}
Źle jest w zasadzie "wszystko", usuń i zacznij od początku. Pamiętaj też że liczba danych wejściowych nie wynosi dokładnie 3 za każdym razem, jest różna, i podawana jako pierwsza liczba na wejściu
test
3
11
27
99
Daje 3x TAK, co jest naturalnie bzdurą (powinno być 3 x NIE. Zresztą poczytaj o "sicie Eratostenesa", będziesz miał dużo lepszy czas. A już na pewno wywal elsea pod którym jest TAK, potwierdzenie masz dawać dopiero jak wszystkie będą miały resztę z dzielnie różną od zera a nie tylko 2
Masz w swoim kodzie jeszcze sporo błędów, więc jak będziesz miała problemy, z samodzielnym poprawieniem, wystarczy że się uśmiechniesz 
, to może Ci wyśle poprawiony twój kod na pw.
Błędy, jakie znalazłem:
Ta instrukcja, nie jest konieczna:
[bbone=cpp,2593]if (test<100000)[/bbone]
a dodatkowo tu jest błędna. zAMIEŃ na:
[bbone=cpp,2594]if (test <= 100000)[/bbone]
analogicznie
[bbone=cpp,2595] if (liczba <= 10000)[/bbone]
Ta instrukcja:
[bbone=cpp,2596] else if(liczba==2||3)[/bbone]
powinna być zapisana:
[bbone=cpp,2597] else if(liczba == 2 || liczba == 3)[/bbone]
ale będzie u Ciebi działać, bo warunek zawsze jest true, i tak naprawdę u Ciebie, zadziała też samo else [inne warunki są już rozpatrzone wcześniej:
[bbone=cpp,2598] else
{
cout << "TAK\n";
}
[/bbone]
No i najważniejsze, musisz poprawić główną pętlę samego sprawdzania.
Witam.
Chciałbym poprosić o wskazówkę, gdzie popełniłem błąd.
#include <iostream>
#include <vector>
#include <iterator>
#include <algorithm>
static const auto max_test = 100000;
void print(std::vector<int> _v)
{
for (auto&& x : _v)
{
auto flag = true;
if (x < 2)
{
std::cout << "NIE\n";
flag = false;
continue;
}
for (int i = 2; i*i <= x; i++)
{
if (x%i == 0)
{
std::cout << "NIE\n";
flag = false;
continue;
}
}
if(flag)
std::cout << "TAK\n";
}
}
int main()
{
int x;
std::cin >> x;
std::vector<int> v;
if (x < max_test)
{
int y;
for (int i = 0; i < x; ++i)
{
std::cin >> y;
if(1 <= x && x <= 10000)
v.push_back(y);
}
print(v);
}
return 0;
}Z góry dziękuję za podpowiedź.
Ok @krystian dzieki.
male moje niedopatrzenie w tym programie powodowalo ze w szczegolnym przypadku ten program nie dzialal poprawnie.
Po malej korekcji program zostal zaliczony:P
Mam problem z powyższym kodem mógłby ktoś rzucić okiem i dać jakąś uwagę na co zwrócić uwagę czy cały jest zły ? Wyskakuje błąd przekroczenia czasu połączenia 
unsigned short inputCout = 0;to zbyt niska pojemność jak na 100000 testów, daj zwykłego inta i usuń kod z forum bo będzie AC.
ps za zgodność testów opisem w zadaniu odpowiada ich autor, nie musisz ich sprawdzać.
1) Program powinien wypisywać TAK i NIE, "sędziuje" program porównujący pliki więc wielkość liter jest ważna
2) Może być 100000 testów, podczas gdy ty masz tablicę tylko na 10000.
3) Za zgodność wejścia z specyfikacją w zadaniu odpowiada twórca testów, nie musisz tego sprawdzać. Ze względu na błąd z punktu drugiego to była jedna z przyczyn nie zaliczenia.
Po poprawieniu punktów 1 i 2 dostałem AC, więc usuń kod z forum
Witam mam prośbę o pomoc lub udzielenie wskazówki nad poprawą programu. Program działa poprawnie, sędzia go zaakceptową, jednak chciałbym w nim poprawić to żeby pozbyć się breaków oraz zmienić pętlę nieskończoną.
kod programu:
#include <iostream>
using namespace std;
int main()
{
int testy, n, pomocnicza;
cin >> testy;
for(int i=0; i<testy; i++)
{
cin >> n;
pomocnicza = 2;
for( ; ; )
{
if (pomocnicza == n)
{
cout << "TAK" << endl; break;
}
else if(0 == n%pomocnicza)
{
cout << "NIE" << endl; break;
}
else if(n<2)
{
cout << "NIE" << endl; break;
}
pomocnicza++;
}
}
return 0;
}Po pierwsze nie dajemy tu działających kodów, po drugie jak chcesz go znacznie przyśpieszyć, to zwróć się z pomocą do pana Eratostenesa
Witam,
Chciałbym się dowiedzieć dlaczego mój kod nie przechodzi. Gdzie robię błąd?
Kod wygoda w tak.
Dzieki sig
Testowaniem zajmuje się maszyna, więc wielkość liter ma znaczenie. Jak daszif(pierwsza(a))
printf("TAK\n");
else
printf("NIE\n");
To ci zaliczy ten program. Jak już to zrobi nie zapomnij usunąć kodu z forum.
