663. Sort 1 [PP0506A]

Dopiero zaczynam nauke kodowania, więc mój kod jest bardzo prymitywny, za co przepraszam. Biorę się za zadania powyżej mojego poziomu, ponieważ wierzę że to rozwija praktyczne zrozumienie kodowania. Także bez zbędnego przedłużania, będę bardzo wdzięczny za zerknięcie na mój kod i wyjaśnienie mi, dlaczego C++ uważa że próbuję zrobić w funkcji wskaźniki, mimo że nigdzie mu nic takiego nie sugeruję:

http://ideone.com/0OJKzi17

Błąd przy próbie kompilacji:

error: cannot convert 'float*' to 'float' in return
if (n==0) return odl_mix;

Mile widziane także opinie dot komunikatu:

error: invalid operands of types 'int [liczby]' and 'int [liczby]' to binary 'operator*'
odl[i]= sqrt((x*x)+(y*y));

Mam wrażenie że wynika to z podobnego błędu, ale nie jestem pewny.

Z góry dzięki za wszelką pomoc.

Uważaj, bo równie dobrze może to rozwijać praktyczną umiejętność bezrozumnego kopiowania cudzego kodu Jeżeli jednak będziesz ostrożny - dobry pomysł.

Masz niezbyt estetyczny kod. Najbardziej razi mnie klamrowanie w mainie - pierwszy for jest bez wcięć.

Masz funkcję float (na SPOJu używaj raczej double, ale mniejsza teraz o to) więc powinieneś zwracać zmienną tego samego typu, np. 3.14. Tymczasem zwracasz return odl_mix, a odl_mix jest tablicą zmiennych, a nie zmienną! Prawidłowy zapis to np. odl_mix[3] czyli zwracasz konkretną liczbę z całej tablicy.

Intuicja Cię nie zawodzi - błąd jest analogiczny. W związku z tym Tobie zostawiam jego poprawienie

PS
Wszedłem na forum zobaczyć co ciekawego się tutaj dzieje odkąd zniknąłem, ale świat studiów jeszcze przez pewien czas wymusza moją nieobecność więc ewentualne dalsze pytania kieruj do innych. No i powodzenia

PS2
Nie do końca rozumiem skąd biorą się Twoje trudności.

    if (x[i]<0) x[i]=x[i]*(-1);
    if (y[i]<0) y[i]=y[i]*(-1);
    odl[i]= sqrt((x*x)+(y*y));

Pierwsze dwa ify sugerują, że rozumiesz, iż x oraz y są tablicami. Z pewnością masz jakieś pojęcie o tablicach, czego dowodzi zapis int x[liczby-1]. Domyślam się, że odejmowanie jedności związane jest z numeracją od zera - wielki plus, że o tym pamiętasz. Mimo to pierwiastkując niejako zapominasz, że x i y to tablice i zamiast napisać x[i] oraz y[i] napisałeś x i y...

Bardzo dziękuję za szczegółową odpowiedź,

Nic nie kopiuje, póki nie zrozumiem jak i dlaczego działa, ale dzięki za ostrzeżenie!

To prawda, brak mi jeszcze ogłady. Póki co nie wiem co wysuwać, a co przycinać. Mam nadzieję że to przyjdzie z czasem

Tylko że ta funkcja ma właśnie zwrócić tablicę. Tylko nie wiem jak to zapisać. Zawsze mogę zrobić pętle for dla wszystkich elementów tablicy i zwracać je pojedynczo, ale zgaduję że jest jakiś szybszy i łatwiejszy sposób, żeby funkcja zwracała tablicę?

W takim razie tym bardziej dzięki za poświęcony czas

To akurat błąd pozostały z poprzedniej iteracji, zanim sobie przypomniałem że muszę coutować dane wejściowe, a nie tylko odległość od 0,0, Już poprawiłem, dzięki.

Także teraz pozostaje tylko pytanie jak mogę zapisać funkcję, która ma zwracać tablicę, a nie pojedynczego floata?

Dla przykładowych testów z zadania program działa prawidłowo.

AC

Mimo to spoj wywala błąd. Nie jestem w stanie sam go znaleźć.

Edit: wystarczyło zmienić tryb danych na double

Policz na kalkulatorze te odległości, wyświetl sobie twoją tablicę 'wyniki' i porównaj te liczby. Popróbuj też z innymi testami.

Twój program nie działa nawet dla przykładowego wejścia. https://ideone.com/AKkXce11

Przed proszeniem o pomoc na forum stwórz program, co do którego istnieje chociaż szansa, że jest poprawny, a nie program, który wywala się już na teście podanym w treści zadania.

W moim kompilatorze wyniki, które uzyskuje przy powyższym kodzie są poprawne, nie wiem dlaczego na ideone są inne.

to pomyśl - bo nikt za ciebie myśleć nie będzie, co najwyżej może ci zwrócić uwagę na coś, co przeoczyłeś, lub podać przykład dla którego program nie działa

Dzięki za pomoc, wszystko działa. Tak, korzystałem z kursu pana Z, jak również z kursu zamieszczonego na cpp0x. Czy to zły wybór? Nie pisałem żadnych komentarzy pod zadaniami, jest to także moja pierwsza aktywność na tym forum.

Bardzo zły. Uczy samych brzydkich technik. Kup jakiś poważniejszy kurs lub książkę.

PS.
Odniosłem się do kursu "pana Z", a nie cpp0x.

I bez tego się obejdzie. Dokumentacja, intensywne googlowanie, realizowanie własnych pomysłów itd. Sporo już źródeł i metod podano na forum

Może i tak, ale w dobrej książce masz wszystko skondensowane. Mi to przynajmniej odpowiada. Początkujący też często nie wiedzą co googlować, bo nie wiedzą, że "takie coś" istnieje.

Witam. Mam taki kod u mnie działa a nie akceptuje . Ktoś może mi podpowiedzieć co może być powodem?

code

Sorki troche popilem. Juz wiem co jest nie tak.

Witam, mógłbym ktoś zerknąć na ten błąd? Ciągle otrzymuje błąd wykonania.
ideone9

Błąd wykonania jest spowodowany tym, że zgodnie z treścią i przykładem, między testami pojawia się pusta linia. Musisz spróbować ją ignorować. Nie jestem ekspertem od pythona ale coś takiego:

try:
    clear = input()
except:
    clear = 0

przed n -= 1 powinno dać radę.

Spróbuj też usunąć to spację na początku Twojego wyjścia, bo nie wiem jak sędzia na to zareaguje.

Zadziała, ale nie musi być w bloku try - pusta linia nie zgłasza wyjątku
Wystarczy po prostu:
clear = input()
lub nawet nie przypisywać zwracanej danej do żadnej konkretnej zmiennej - w Pythonie służy do tego podkreślnik:
_ = input()
lub w ogólnie nie przypisywać:
input()

Próbowałem bez wyjątku, ale nie wiem czy w testach znajduje się na końcu pusta linia na końcu pliku. Jeśli nie, dzieje się to: https://ideone.com/DZKS7R9