Oct 2015
83 / 133
Jun 2019
Jan 2024
thyrz_92
Jun '19
1
korkirw

Dzięki za odpowiedź

  1. Masz rację, jednakże chciałem załączyć oba kody - może ktoś bardziej doświadczony ode mnie by coś wyłapał, coś co by wskazywało czemu jeden kod działa a drugi nie.

  2. Zgodzę się przy rozwleczeniu, przy C++ trochę ''na ślepo" dawałem się prowadzić za rączkę przez Visual Studio który zachęcał do wypełniania konstruktorów, getterów i setterów poza klamrami klasy (głównie to tak rozwlekło kod). Ale jak SPOJ przyjmuje to już tak zostawiłem. Jak mi się będzie chciało to wrzucę tam bardziej kompaktowy kod. Także są settery niepotrzebne przy kodzie w C++ (przeciążony konstruktor już je ma, wystarczyłoby zamiast nowaX = x używać czegoś w stylu this.x = x jak zrobiłem w kodzie Java - jak się da).

Tak czy inaczej dzięki za sugestie.

EDIT: Postanowiłem kod skrócić już teraz, skrócił się do 100 linii.

korkirw
Jun '19

Jeśli jesteś zainteresowany to na priv mogę dać Ci mój kod z C++. Może podsunie Ci jakiś pomysł a w najgorszym wypadku będziesz miał dwa kody do wyboru. Co do Javy nie będę Ci dawał rad bo się na tym nie znam.

rapast
Jun '19

Ogarnąłem Twoje rozwiązanie w Javie. Powodem niedziałania było to, że tworzyłeś 2 obiekty Scanner
(jeden w funkcji main, a drugi w fiilList). Wystarczy, że w main utworzysz obiekt Scanner i przekażesz go jako argument do metody fillList.

thyrz_92
Jun '19

Dzięki wielkie, głupie błędy są moją domeną. Usuwam kody z posta.

d0man199
Jun '19
1

Ktoś ma jakiś pomysł dlaczego wywala Błędna odpowiedź?
(Kod ukryty :slight_smile: )
Jestem “świeżak” w tym, więc jeśli ktoś ma dodatkowe wskazówki, co do kodu to będę wdzięczny

korkirw
Jun '19
1

Twój program źle liczy dla chociażby takich danych
A 1 18
B 3 17
Błąd jest banalny i łatwy do naprawienia. Jeśli ta podpowiedź Cię nie naprowadzi, to podam dalsze wskazówki :slight_smile:
A zresztą…
Podam słowo klucz - METRYKA

d0man199
Jun '19

Co prawda, to nie było tylko to co pisałeś (inty jeszcze :)) ale tak, zrozumiałem, poprawilem. Zaliczone. Dziękuje.

9 months later
narbej
Mar '20

Zerknąłem, no i trochę błędów tam jest. :wink:

.
.
.
.
.

===================================
Jeżeli dostajesz WA [wrong answer - błdna odpowiedź], to natychmiast powinieneś, ze skupienie i zrozumieniem jeszcze raz zajrzeć do treści zadania:

nazwa x y
gdzie nazwa jest unikalnym dla każdego testu ciągiem co najwyżej 10 liter alfabetu łacińskiego,

10 [słownie dziesięć] != 1 [słownie jeden]

PS
dokleiłeś siędo wątku, ale warto także do niego zajrzeć.

pawoj20
Mar '20
adrmysl

W funkcji sortowanie_babelkowe jest parametr int n natomiast wewnatrz funkcji jest deklaracja char n
Moze to wprowadzac zamieszanie.
Jeszcze obliczanie odleglosci punktu.
moze moj test jest troche zlosliwy i moze sedzia nie ma tego typu testu.

wejscie
1
3
A 0 0
C 1 1
B 0 -1
wyjscie
A 0 0
B 0 -1
C 1 1

math.h jest biblioteka typowa dla zwyklego C

adrmysl
Mar '20
narbej

jednak jest napisane “co najwyzej” czyli mysle ze nie powinno byc problemu z tym, ze nazwa ma mniej niz 10 liter

adrmysl
Mar '20
pawoj20

masz racje, nazwa n mogla robic zamieszanie, jednak po zmianie nazwy,by uniknac powtorek dalej jest bledna odpowiedz.

math.h jednak z tego co widze moze byc stosowana rowniez co C++ gdzie ulatwia obliczanie poteg i pierwiastkow

korkirw
Mar '20

Niestety jest problem z tym, że dopuszczasz nazwy tylko jednoliterowe. Co Twój program zrobi, gdy w teście będzie nazwa dwu, trzy,… literowa. Gdybyś dopuścił nazwy 100 literowe, to rzeczywiście z punktu widzenia logiki rozwiązanie nie przeszkadzałoby to. Poszedłeś w złą stronę.
C++ dopuszcza bibliotekę math.h ale dlaczego nie korzystać z biblioteki dedykowanej wprost do C++ czyli cmath? Masz zamiar w przyszłości swój kod przerabiać na C?

adrmysl
Mar '20

dobrze, jednak po zmianie rodzaju zmiennej ze znakow czyli char na string SPOJ nadal uwaza ze jest bledna odpowiedz, wiec nie w tym problem. Mysle ze skoro i jedna i druga biblioteka spelnia swoje zadanie to nie ma co sie zaglebiac co dalej bede z tym robil. Wazniejsza kwestia dla mnie jest rozwiazanie zagadki dlaczego kod nie przechodzi, co nie dziala poprawnie

pawoj20
Mar '20

tu powinna byc liczba mnoga: zmiennych.

Jednak prawidlowa obsluga przyklad jest obowiazkowy do zaliczenia

wejscie
1
3
A 0 0
C 1 1
B 0 -1
wyjscie
A 0 0
B 0 -1
C 1 1

adrmysl
Mar '20

mhm… czyli czepiamy sie bezsensownych rzeczy a rozwiazania dalej brak, dzieki za pomoc :slight_smile:

korkirw
Mar '20

Gdzie zamieściłeś poprawiony (chociaż wciąż niedziałający) kod po sugestiach @narbej i @pawoj20? Dlaczego się spodziewasz, że ktoś będzie chciał pomagać, jeśli nic mu nie ułatwisz? Co z wykonaniem testu podanego przez @pawoj20?

pawoj20
Mar '20

trzeba poprawic w wielu miejscach. Czy poprawiles w wielu miejscach?

moj przyklad nie jest bezsensowny bo pokazuje bardzo wazna rzecz dotyczaca obliczania odleglosci.

uwaga dotyczaca biblioteki moze i jest najmniej sensowna, ale gdyby sie jej pozbyc i nie wstawiac innej to powyzszy problem rozwiazalby sie.
PS.
Na Twoim kodzie dostalem AC jako uzytkownik Kucyk (konto uzywam do cudzych kodow, eksperymentow itp. nie chce miec balaganu)

18 days later
tomciolev
Apr '20
2

Witajcie, napisałem kod do tego zadania, gdy wrzucam do sędziego pokazuje, że jest błędna odpowiedź. Wszystkie “trudne” przypadki podane we wcześniejszych komentarzach do tego postu wkleiłem i odpowiedzi wchodzą dobre Czy jest ktoś wstanie dać jakąś wskazówkę co tutaj jest źle?
Problem rozwiązany