witam!
wrzucę tutaj cały swój kod łącznie z cout'ami (to dla klarowności kodu, sprawdzajac wrzuce go juz bez cout'ów). mam do was ogromną prośbę: pomóżcie znaleźć mi bład w tym kodzie. za nic w świecie nie moge go znaleźć, u mnie na PC wszystko działa i kompiluje poprawnie ale jak tylko wrzuce go do sprawdzenia to wyskakuje mi że jest błąd

include

using namespace std;
int ile,ciastka,t;
int x=0;
int y;
int main()
{
cout<<"podaj liczbe testow"< cin>>t;
for(int i=0; i {
cout<<"ilu obzartuchow jest na zlocie? ";
cin>>ile;
int czasy[ile];
cout<<"ile jest ciastek w jednym pudelku? ";
cin>>ciastka;
for(int i=0; i {
cout<<"podaj czas w sekundach w jakim "<< i+1 <<" obzartuch zjada ciastko: ";
cin>>czasy[i];
y=86400/czasy[i];
x+=y;
}
if ((x%ciastka)==0)
cout<<"potrzeba "<< x/ciastka <<"opakowan ciastek"< if ((x%ciastka)!=0)
cout<<"potrzeba "<< x/ciastka+1 <<" opakowan ciastek"< }
return 0;
}

z góry dziekuje za pomoc!

Wklej kod w tagu dostępnym pod ctrl + k, to będzie sformatowany, wtedy się popatrzy. Tak na szybko: w którym miejscu zerujesz x-a? Najlepiej przesuń go do FOR-a, zmienne globalne to dość problematyczny pomysł. Najlepiej t daj w main-ie a całą resztę w forze.

Tworzysz tablicę o rozmiarze nieznanym (n wczytujesz dopiero później) i się dziwisz że ci błędami rzuca? Wywal ją całkiem (bo do niczego jej nie wykorzystujesz), zrób sobie inta w 2-giej pętli for i na nim operuj. Wtedy dostaniesz AC.

edit: kod dawaj w taku dostępnym przez ctrl + k, i usuń go z forum jak już ci zadziała.

zamiast tego daj ilosc+= 86400 / czas[i-1]; i usuń kod z forum bo będzie AC

Swoją drogą po co ci ta tablica? starczyła by pojedyncza zmienna

Moim zdaniem powinieneś od nowa przemyśleć algorytm, bo obecny wydaje mi się błędny. Swoją drogą przenieś deklarację zmiennych z wyjątkiem testy do pętli for albo przynajmniej je zeruj co obrót pętli, bo obecnie zostają w nich wartości z poprzedniego co może mieć wpływ na wyniki końcowe. Inna sprawa że przydało by się nadać nim wartość początkową, bo obecnie jest losowa co też może mieć wpływ na wyniki.

Hej, rzuci ktoś mądry okiem na to? Generalnie działa, tylko mam taki problem, że gdzieś coś pomieszałem z pętlą, bo po wypisaniu na ekran wyniku z pierwszego testu wyniki z drugiego dodaje do tego pierwszego, a nie liczy ich od nowa. Proszę o wskazówki.

include

include

using namespace std;

int main()
{
float suma=0;
int t;//Podaj liczbê tesów do wykonania
int n;//ilu ob¿artuchów
int d=86400;//liczba sekund w dobie
float czas[100000];//czas zjadanie jednego ciastka
long m;//ile ciastek w jednym pudelku
float wynik;
cin>>t;
for(int i=1; i<=t; i++)
{
cin>>n>>m;

for(int k=1; k<=n; k++)
{
cin>>czas[k];
czas[k]=floor(d/czas[k]);
suma=suma+czas[k];
cout<}
wynik=suma/m;
cout<

}

return 0;

}

Hmm ja nie jestem orłem, nie sprawdzałem działania twojego kodu, bo jest z nim na pewno coś nie tak na końcu, ale jak użyjesz floor to dostaniesz wynik zaokrąglony w dół, a Ty potrzebujesz ewentualnie zaokrąglić w górę - ceil, zgodnie z treścią zadania. Dodatkowym problemem jest pewnie to, że nie czyścisz tablicy na koniec danego testu i prawdopodobnie pozostają w niej śmieci z poprzedniego testu. Możesz sprawdzić moje wskazówki, ale zaznaczam, jak już napisałem, że nie jestem orłem.

Daj kod w tagach dostępnych pod ctrl + k albo linkuj do ideone, obecnie jest pocięty i nie wiadomo co z nim jest nie tak. Nie widać np co i jak wypisujesz

#include <iostream>
using namespace std;
int main()
{
unsigned short test, ilu;
unsigned long long ciastek;
cin>>test;
for (int i=0;i<test; i++)
{
cin>> ilu >> ciastek;
unsigned short suma=0,sekund;
for (int i=0; i<ilu;i++)
{
cin>> sekund;
suma+=(86400/sekund);
}
suma/=ciastek;
if (suma%ciastek<=0,5)
suma++;
cout<< suma<< endl;
}
return 0;
}

Witam co z tym kodem jest nie tak u mnie na VS 2012 dobrze działa jak również na ideone.

https://ideone.com/lQgHtv48

Ależ oczywiście, że na ideone nie działa dobrze...

Zmieniłem if (suma%ciastek<=0,5) na if (suma%ciastek !=0) ale sędzia i tego rozwiązania nie przyjmuje może jakaś sugestia żebym wiedział co poprawić?

1. Strukturę programu - wcięcia w kodzie.
2. Typy zmiennych
3. Nazwy zmiennych
4. Na początek pisz prosto,
5. Nie używaj tej samej zmiennej do kilku rzeczy [oczywiście nie mam na myśli zmiennej i - ta jest ok. suma [lepiej sumaZjedzonychCiastek. Więc nie suma /= ciastek tylko: ilePudelek = sumaZjedzonychCiastek / ciastekWpudelku. A potem if (sumaZjedzonychCiastek % ciastekWpudelku != 0 ) ilePudelek++

Jeżeli będziesz pisał tak jak teraz, nie tylko Ty ale i nikt inny nie będzie widział, albo będzie trudno znaleźć błąd - czy już go teraz widzisz?

Witam,

czy liczba pudełek powinna być liczona dla każdego obżartucha osobno czy najpierw liczę sumaryczną liczbę ciastek a później zaokrąglam do liczby ciastek w pudełku?

Pozdrawiam

Obżartuchy dostają ciastka a nie całe pudełka, wiec zaokrąglasz do pudełek po ustaleniu ile ciastek łącznie zjedzą.

include

using namespace std;

int ile_razy;
int main()
{
cin>>ile_razy;
for(int b=0; b {
int N=0,M=0,t=0,suma_ciastek=0,suma_pudelek=0, t_ciastko=0,grubas=0; //N ilosc grubasow, M ilosc ciastek
t=60*60*24; //wynik w "s"

 cin>>N>>M;
 t_ciastko= new int[N];
 grubas= new int[N];
 for(int i=0; i<N; i++)
   {
      cin>>t_ciastko[i];
      grubas[i]=t/t_ciastko[i];            //ilosc ciastek jakie zje
      suma_ciastek+=grubas[i];
      suma_pudelek=suma_ciastek/M;



   }

   if(suma_ciastek%M!=0)
        suma_pudelek++;
  cout<<suma_pudelek;

}

return 0;

}

Problem wyglada tak. Działac dziala, zadneg bledu nie wywala, u mnie wyniki mam dobre jak sprawdzam, ale testu przejsc nie chce. Na poczatku myslalem ze moze chodzi o zaokraglanie,bo zawsze zaokragla w dol, no ale jednak nie.

A dokładnie, jak to dokładnie sprawdzasz?

delete[]
Nie zapominaj o tym. Deklarowanie zmiennych globalnych jest uważane za złą praktykę, a w tak małych programach nie ułatwia w żaden sposób pisania. Najlepiej deklarować zmienne w miejscu gdzie są potrzebne.
Po co ci te ułamki? Żeby zrozumieć co tam napisałeś musiałem przeformatować u siebie kod do jakiejś bardziej czytelnej formy.
Popracuj nad kodem. Nad czytaniem poleceń też (uczestników jest 2-3?).
Najlepiej usuń to wszystko, przeczytaj parę razy polecenie i napisz to od nowa