Dziękuje za odpowiedź.
Przed napisaniem wiadomości przeczytałem post dla początkujących i starałem się postępować według tego co było tam napisane:
-
starałem się wyszukać rozwiązanie na forum zanim napisałem wiadomość ale nie znalazłem
-
nie założyłem nowego wątku tylko napisałem wiadomość już w istniejącym, dotyczącym tego problemu
-
wkleiłem kod według instrukcji (wkleiłem kod skopiowany z strony ideonem.com1 za pomocą ikonki kopiuj ponieważ, gdy wklejałem link to w mojej wiadomości nie pojawiał się on ani kod tylko taki obrazek:
-
nie obraziłem nikogo, byłem kulturalny i pisałem poprawnie
Jeżeli zrobiłem coś źle to nie jestem świadomy co zrobiłem źle i nadal będę to robił chyba że mnie uświadomisz albo wyrzućcie mnie po prostu z tego forum.
Jeżeli chodzi o mój kod to:
"…Co robi tablica t…" - przechowuje czasy pojedynczo zjedzonych ciastek.
"…i skąd się wziął jej rozmiar…" - rozmiar nadałem jej 3 ponieważ do wykonania programu podane są 2 testy w których są maksymalnie 3 czasy.
Rozumiem że sędzia testuje więcej więcej niż 3 czasy dlatego kod mój nie jest poprawny dlatego zmieniłem go tak (zastosowałem dynamiczne alokowanie pamięci) aby był możliwe stosowanie więcej niż 3 czasów. Nazwę zmiennej t zmieniłem też na nazwę czas. Niestety dostaje informacje “Błędna odpowiedź” pomimo, że wyniki testów w Code::Blocks są poprawne.
Podejrzewam, że napisałaś:
"skoro jesteś “mocno początkujący” to może przeczytaj post dla początkujących ?.."
dlatego, że źle wkleiłem kod więc wklejam nowy kod jako tekst bezpośrednio z Code::Blocks:
#include
using namespace std;
int ile,ip2,n;
float ip,m,ip3;
int main()
{
cin>>ile;
for(int i=1; i<=ile; i++)
{
cin>>n>>m;
int suma=0;
float *czas;
czas = new float [n];
for (int i=0; i<n; i++)
{
cin>>czas[i];
}
for (int i=0; i<n; i++)
{
suma+=(86400/czas[i]);
}
ip=suma/m;
ip2=ip;
ip3=ip-ip2;
if (ip3!=0)
{
ip2+=1;
}
cout<<ip2<<endl;
delete []czas;
}
return 0;
}
Proszę o pomoc.
w poście dla początkujących o ile dobrze pamiętam pisało by dla udostępniania kodu używać odpowiednich tagów by był w całości a nie pokawałkowany, a jeszcze lepiej LINK do ideone z kodem i testami… Bo teraz by sprawdzić twój kod muszę kopiować 3 kawałki i domyślać sie bibliotek…
np: https://ideone.com/qSIXxm3
zastanów sie jak to zrobić na intach… Ten kod mi przeszedł jak zmieniłam na inty oraz dodałam jedną linijkę do sprawdzenia pełnych pudełek
zamiast tego
ip=suma/m;
ip2=ip;
ip3=ip-ip2;
cout<<"wczt2"<<endl;
cout<<"wczt1"<<endl;
Oto przyczyna. Na wyjściu ma się znależć dokładnie to, czego chce autor. Wypisanie nawet jednej litery a w jakimkolwiek miejscu (albo np. “TAK” zamiast “tak”) spowoduje błędną odpowiedź. Pewną swobodę masz jedynie, jeśli chcesz dodać więcej białych znaków niż jest wymagane (albo zamiast spacji wypisywać ‘\n’ i vice versa), ale i to nie we wszystkich zadaniach (jak w treści napisane jest, że włączony jest sędzia dokładny, wyjście musi być IDEALNIE zgodne z tym, czego chce autor. Brak jednej spacji albo nadmiarowy ‘\n’ na końcu spowoduje błędną odpowiedź).
A, no i jak usuniesz powyższe wypisywanie to okaże się, że całe wyjście twojego programu jest sklejone bez żadnych białych znaków. Dodaj w odpowiednim miejscu wypisywanie spacji albo newline i powinno śmigać.
Cześć,
błędy:
1: [linia:5] Podczas definicji zmiennych poza funkcją, zmienne mają ustaloną wartość na początku. W przypadku, gdy nie ma inicjalizacji, domyślnie przyjmują wartość 0, czyli taki zapis:
int ile, N, M, *czas=new int[N], suma=0, paczki;
powoduje, że tworzonych jest 6 zmiennych, one się tworzą po kolei, na samym początku programu, jeszcze przed tym, jak zostanie uruchomiona funkcja main, czyli:
ile ma wartosc 0
N ma wartość 0
M ma wartość 0
czas jest wskaźnikiem i po inicjalizacji wskazuje na obszar pamięci zaalokowany dla N elementów, czyli dla 0 elementów.
suma po inicjalizacji ma wartośc 0
paczki ma wartość 0
…
Dopiero po tym jest uruchamiana funkcja main, która to tak na prawdę jest w stanie wczytać wartośc N i innych danych.
2: [linia:13] Gdy tworzysz tablicę dla N elementów, to indeksy w tablicy zaczynają się od 0, czyli od 0 do N-1.
Czy w tej pętli czasami i z j sie nie pokiełbasiły?
3: W jednym odpaleniu programu może być wykonwanych wiele zestawów testowych, co oznacza, że N może się zmieniać pomiędzy testami.
4: Gdy alokujesz dane za pomocą new to musisz zadbać o to, aby na końcu zwolnić pamięć:
...
czas = new int [N];
// kod uzywający tablicę czas
delete [] czas;
...
Próbuj dalej
Te linie są ciekawe:
13: czas=new int[N];
16: cin>>czas[i];
18: suma+=86400/czas[i];
W lini 13 alokujesz obszar, który może pomieścić N liczb,
czyli można przyjąc umownie, że w tablicy czas jest dostęp do komórek od 0 do N-1.
W liniach 16 i 18 używasz komórki o indeksie i, pytanie, czy i jest w tym zakresie od 0 do N-1 ?
Z lini 10 kodu wynika, że i jest w zakresie od 0 do ile:
10: for(int i=0;i<=ile;i++)
Co w przypadku, gdy ile jest większe lub równe N ?
Dobrze sobie przetestować program w ideone3, to znaczy, przed wybraniem “Uruchom” wkleić testowe dane wejściowe do pola “enter input (stdin)”. Czy wynik działania programu jest na pewno poprawny?