35 / 58
Jul 2019

Pomagając innym, powinieneś poprzestać na swoich dcwóch pierwszych zdaniach. Podpowiadanie, poprzez pokazywanie działającego [AC] kodu nie jest dobrym pomysłem i nie jest tu na forum praktykowane.
Jednak Ty chciałbyś dodatkowo, przy okazji uzyskać opinie o swoim kodzie. Ponieważ uzyskałeś już ją od @mariusz193 i możliwe, że za chwilę ja napiszę parę słów, powinieneś usunąć link do pastebin.

Twój kod jest dokładnie tym o co chodiło w tym zadaniu, i masz prawo być z niego dumny.
Warto jednak wiedzieć o paru rzeczach, o których teraz pomyślałem

  1. Brak metryki zadania [autor, data itd] i komentarza. Komentarz powin ien zawierać bardzo krótki opis, co program robi, lub krótkie streszczenie kwintensencji treści zadania.
  2. Opis - wykaz zmiennych - w tym zadaniu tylko kilka, w większych więcej zmiennych i można się pogubić. Warto używać wtedy takich nazw jakie zostały użyte w treści zadania.
  3. while (1) ? A co gdy na wejściu nie pojawią się trzy, albo żadna 42? Wprawdzie autor daje gwarancje ale jednak może lepiej i bezpieczniej:
    while (cin >> n) ?
  4. Zamiast zmiennej int temp, ja użyłbym zmiennej bool.
  5. W programowaniu, lepiej odliczać w dół do zera, czyli tutaj zamiast
    int count = 0,
    a potem
    count++;
    lepiej
    int count = 3
    i
    –count
    i wtedy można while (coun)

PS
Ponieważ masz AC, to faktycznie łatwiej podesłać Ci kod, niż opisywać kolejne propozycje poprawek :wink:

AD 1
Co jak u siebie będziesz miał kod, a nie będziesz miał dostępu do SPOJ’a. Będziesz wiedział, że zadanie zaliczone - AC, ale co ono robi? Dodatkowo warto też mieć “u siebie” przykładowe testy + ewentualne swoje. Może więc warto w komentarzu umieszczać nawet całą treść zadania?.

Dzięki za wszystkie wskazówki. Ogólnie to jestem samoukiem i po prostu nie mam aż takiego doświadczenia w programowaniu, więc mój kod wygląda tak, a nie inaczej :stuck_out_tongue: Skomentowałem, bo chciałem udzielić podpowiedzi, co znacznie mogłoby zmienić postrzeganie zadania przez @porshe911 i jednocześnie uzyskać opinię o własnym kodzie. Wpis edytowałem i usunąłem link do kodu.

Już napisałem Ci, że twój kod jest bardzo dobry, ok i spoko. :wink: Napisałeś go starannie, dobrze sformatowałeś, użyłeś gdzie trzeba, wystarczająco białych znaków [spacji] i program robi to co ma robić. To, że @mariusz193 i ja podaliśmy kilka wskazówek, świadczy tylko, że w programowaniu wiele dróg prowadzi do celu i każdy koder/programista to samo zadanie zrobi inaczej albo zrobi na kilka sposobów. To, że jesteś samoukiem nie ma specjalnego znaczenia, liczy się pasja do programowania, do nauki i eksperymentów. Czym więcej będziesz “ćwiczył” tym więcej zdobedziesz doświadczenia, a samouczenie jest tu raczej plusem a nie minusem - nie jesteś tutaj jedynym samoukiem.:wink:
Co do kodu @porshe911 w porównaniu do twojego, to jak porównanie samochodu marki syrenka do samochodu porshe, z tym, że to twój kod jest tu tym lepszym ;-). Sam nie wiedziałem jak podpowiedzieć @porshe911’emu i naprowadzić go na dobrą drogę i twoja podpowiedź była i jest bardzo dobra. Co do zamieszczania na forum działającego kodu, to nawet “światłym” użytkownikom, którzy zalecają usuwanie, zdarza się nie przestrzeganie własnych zaleceń.

Co do zadania i kodu . Można zauważyć, że liczby w zadaniu są dwucyfrowe, więc zakres int, jest tu nadmiarowy, tak jak i do licznika count. Jednak int jest tu najwygodniejszy i zupełnie nie warto oszczędzać tych dwu czy czterech bajtów, bo jest to zupełnie bez sensu - chyba, że naprawdę tego potrzebujesz i wiesz co robisz… Można także wczytywać liczby jako stringi lub wczytywać je znak po znaku. Ale to zupełnie bez znaczenia a poprostu inny sposób i trochę inna droga. Warto to wiedzieć i ewentualnie poeksperymentować, na tak prostym zadaniu, bo nabyta wiedza zaprocentuje na trudniejszych i bardziej skomplikowanych.

1 year later

Witam. Nie rozumiem dlaczego dostaje TLE mimo iż kod przeszedł pozytywnie test na ideone.
Niżej link

a znasz przysłowie, że jedna jaskółka wiosny nie czyni ?

1
42
1
42
1
42
6 months later

Zrobilem omawiane zadanie poprzez warunki if w javie.
Na Ideone,com poprawnie rozwiązuje.
Na spoj daje błąd wykonania (NZEC) .

W czym sie myle. ???

class Main
{
public static void main (String[] args) throws java.lang.Exception
{

  Scanner sc = new Scanner(System.in);
  int a=0;
  boolean warunek=true; 

// System.out.println(“czekam -1”);
a=sc.nextInt(); System.out.println(a);
// System.out.print("_-1");

  while(warunek) {
     if (a==42) {
//        System.out.println("czekam 0");
        a=sc.nextInt(); System.out.println(a); 
   //     System.out.print("_0");
     }
     if (a!=42) {
      //  System.out.println("czekam 1");
        a=sc.nextInt();System.out.println(a);
      //  System.out.print("_1");
        if (a==42) {
         //  System.out.println("czekam 2");
           a=sc.nextInt();System.out.println(a);
          // System.out.print("_2");
           
           if (a!=42) {
             // System.out.println("czekam 3");
              a=sc.nextInt();System.out.println(a);
              //System.out.print("_3");
              if (a==42) {
                 //System.out.println("czekam 4");
                 a=sc.nextInt();System.out.println(a);
                // System.out.print("_4");
                 
                 if (a!=42) {
                   // System.out.println("czekam 5");
                    a=sc.nextInt();System.out.println(a);
                   // System.out.print("_5");
                    if (a==42) {
                       warunek=false;
                       
                       
                    }
                 }  
                 
                 
                 
              }
           }   
           
           
           
        }
     }
     
     
     
  }
  
  //System.out.println("sukces"); 

}
}

Re powyzszego - zrobilem je na dwoch zmiennych.
------------naiistotniejszy fragment--------------

a=sc.nextInt(); System.out.println(a);
while(true) {
b=sc.nextInt(); System.out.println(b);
if ( a!=42 && b==42 ) c++;
a=b;
if (c==3) return;
}


Ale dlaczego nie idzie na if ?

to nie są równoważne algorytmy

0
42
1
2
42
3
4
42
5
6
42

Witam. Próbowałem wykonać program za pomocą wektorów, jednak wyskakiwało przekroczenie limitu czasu. Zmieniłem koncept na użycie stringów, ale dzieje się to samo. Mógłby mi ktoś pomóc? Link: https://ideone.com/NI4bae3

   using namespace std;`
    int main()
    {
    string do_wyswietlenia="";
    string zapasowy;
    int stop;
    cin>>zapasowy;
    do_wyswietlenia+=zapasowy+"\n";
    while(stop!=3)
    {
BBB:
        stop=0;
AAA:
        if(zapasowy!="42")
        {
            cin>>zapasowy;
            do_wyswietlenia+=zapasowy+"\n";
            if(zapasowy=="42")
            {
                stop++;

                if(stop!=3)
                {
                    cin>>zapasowy;
                    do_wyswietlenia+=zapasowy+"\n";
                    goto AAA;
                }
            }
            else
                goto BBB;
        }



        if(stop!=3)
        {
            cin>>zapasowy;
            do_wyswietlenia+=zapasowy+"\n";
        }

    }
    cout<<do_wyswietlenia;



    return 0;
}

Czy powinienem zmienić cały koncept czy da się to jakoś naprawić?

Raczej zmienić koncept i wypisywać na bieżąco wczytane liczby, próbowałem tak przerobić ten kod ale wyskoczył mi błąd wykonania SIGXFSZ (co to jest?).
Przerób tak żeby nie używać instrukcji goto.

Spróbowałem zrobić to zrobić tak, ale nadal dostaje przekroczenie limitu czasu.
link1

using namespace std;
int n,licznik=0;
int main()
{
    bool przelacznik=false; //false-poprzednia 42 true-poprzednia !42
    while(licznik!=3)
    {
        cin>>n;
        cout<<n<<endl;
        if(n!=42)
            {przelacznik=true;
            cin>>n;
            cout<<n<<endl;
            if(n!=42){licznik=0;}}
        if(n==42&&przelacznik==true)
        {
            licznik++;
            przelacznik=false;
        }
        else if(n==42)
        {
            przelacznik=false;
            licznik=0;
        }
    }

    return 0;
}

in:
11
11
42
11
11
42
11
11
42

  • w zasadzie to chyba wa ale program się wykonuje i wykonuje i wychodzi tle.

Witam, udało mi się zrobić program, który po wprowadzeniu danych wejściowych poprawnie wyprowadza dane wyjściowe, jednak sędzia wyrzuca błąd “Przekroczono limit czasu”. Może ma ktoś pomysł jak skrócić czas działania mojego programu?
Mój kod: http://ideone.com/3BoEWy10

Patrz przykład powyżej i przeczytaj jeszcze raz treść zadania.

Faktycznie nie przechodziło tego testu. Niepotrzebnie w jednym momencie zerowałem zmienną x, dlatego program mógł wykonywać się w nieskończoność. Poprawiłem to, sprawdziłem, wydaje mi się, wszystkie przypadki (moim zdaniem wszystko działa tak jak powinno), jednak dalej wywala błąd “Przekroczono limit czasu”.
PS. W zadaniu dane wejściowe wyglądają tak [Po lewej], u mnie wyglądają tak [Po prawej]:
Wejście:
42 42
42 42
12 12
13 13
42 42
11 11
42 42
43 43
42 42
42
99
01
Nie wiem w jaki sposób autorowi zadania udało się dopisać kolejne trzy liczby, nie ustalając z góry ilości testów (w tym przypadku 12)
Podejrzewam, że program kończy się na 01, ponieważ nie jest to de facto liczba dwucyfrowa.
U mnie program kończy się na trzeciej “42” wpisanej po innej liczbie.
Mój leciutko poprawiony kod: http://ideone.com/IyJQ4D12

Poprawiłem program, ale nadal występuje przekroczenie czasu. Nie rozumiem też,czy te ‘42’ poprzedzone inną liczbą mają się powtórzyć w kolejności czy nie, bo wyżej dwie osoby pisały co innego.
link3

Gdy próbowałem zamienić cout na printf() i cin na scanf() wyskoczył mi błąd SIGXFSZ.

42 poprzedzone inną liczbą nie muszą być w kolejności, 42 może być poprzedzone wieloma innymi liczbami tj odstępy między nimi mogą być większe niż jedna liczba, czyli przykład:
11
11
42
11
11
42
11
11
42
11
12
powinien zakończyć się na trzeciej liczbie 42.

Co masz na myśli pisząc “mają się powtórzyć w kolejności”?
PS. Twój program nie przechodzi testu powyżej:
in:
11
11
42
11
11
42
11
11
42
Edit. Przeanalizowałem twój program i miałeś dokładnie taki sam błąd co ja, czyli wyzerowałeś daną “licznik” w jednym miejscu niepotrzebnie (lub dwóch, musisz sprawdzić) wystarczy to skasować.
Wklepałem poprawiony twój kod do spoja i sędzia go przyjął :smiley:

if ((n<10)||(n>99)) - warunek jest zbędny;
nie wiem co tu poprawić, jedynie uprościć, wystarczy zapamiętać poprzednio wczytywaną liczbę w jakiejś zmiennej tmp i dwa warunki.