[powodzenia]

#programowanie #java

uwaga amator here
zrobiłem 25% kursu javy u Tima Buchalki i zrobiłem sobie mały czelen, myślałem, że rozjebe w moment ale nie idzie mi, więc mam pytanka

co chciałem zrobić: program który będzie mi mówił za ile dni mam w aucie przegląd, za ile kilometrów wymiana świec, serwis podtlenku lpg itd.

moje wyobrażenie było takie, że zrobię oddzielną klasę dla każdej ze sprawdzanych rzeczy, czyli mam teraz klasę wymianaŚwiec, serwisLPG, następnyPrzegląd, a wszystkie z tych klas są "extends" klasy diagnoza - coś czuję że już tutaj gdzieś z------m przy samym założeniu xD
w każdej klasie porobiłem metody, które liczą mi to co chcę i to działa OK

w diagnoza konstruktor który każe podać tylko przebieg (inne dane potrzebne do kalkulacji jak np. data ostatniego przeglądu lub przebieg przy ostatnim serwisie są wpisane na sztywno w klasach i będą modyfikowane setterami)
poza tym w konstruktorze diagnoza chciałem po prostu wołać metody z poszczególnych klas, np.

w klasie diagnoza:

public class diagnoza {

private int przebieg;

gasCheck gaz = new gasCheck(przebieg);
swieceCheck swiece = new swieceCheck(przebieg);

public diagnoza(int przebieg) {
this.przebieg = przebieg;
gaz.showLpgCheck(przebieg);
swiece.swieceCheck(przebieg);

}

następne w main chciałem tylko robić coś w tym stylu:

diagnoza diagnoza = new diagnoza(104000);

i liczyłem na to, że wyprintuje mi wszystkie checki z wszystkich klas, ale dostaję poniższy error

Exception in thread "main" java.lang.StackOverflowError

no i c--j no i cześć.

da się to w ogóle czytać?
da się mnie zrozumieć?
co robe źle?

[Defined]

@powodzenia: ewidentny błąd języka

[Myzreal]

@powodzenia: Jeśli GasCheck dziedziczy po Diagnoza i w konstruktorze Diagnoza tworzysz nowy GasCheck to wjebujesz się w nieskończony łańcuch tworzenia, który powoduje przepełnienie stosu (StackOverflow)

Przebiega to tak:
1. Wywołujesz konstruktor klasy Diagnoza
2. W jego konstruktorze tworzysz nową klasę GasCheck
3. Klasa GasCheck dziedziczy po Diagnoza, więc wywołuje też konstruktor Diagnoza
4. Patrz punkt 1

[e__e]

@powodzenia masz gdzieś cały ten kod? Jak wygląda klasa gasCheck?
Na początek wyrób sobie nawyki nazw klas z dużej litery i zdecyduj się jednak na angielski w całości.

[powodzenia]

@e__e: w sumie to mam wszystko po angielsku, ale nie wiem po c--j zadałem sobie trud przetłumaczenia na polski dla wykop.pl, zaraz wkleję

[e__e]

@powodzenia @Myzreal ma rację. Obliczenia gazu rób w klasie dla gazu.

Jak chcesz zrobić sobie wypisanie wszystkiego to może zacznij od testów jednostkowych. Sam program powinien mieć osobną klasę do zabawy tym. Jakąś główną uruchomieniową. I tam w metodzie main twórz obiekty i wywołuj sobie metody. Ale lepiej to w testach robić. Od razu masz pokrycie.

[powodzenia]

@powodzenia:

package com.company;

public class lpgCheck extends diagnosis

[Myzreal]

@powodzenia: Type ale już ci wyjaśniłem w czym problem.

@e__e: On klasy z małej litery nazywa a chcesz żeby unity ogarnął :P

[e__e]

@Myzreal na pewno sprawdzi co to i się nauczy. Powinni mówić o tym na początku i od początku TDD uczyć.

[powodzenia]

@Myzreal: widzę, że wyjaśniłeś, ale e__e prosił o kod
od dziś klasy nazywam z wielkiej litery (╯︵╰,)

[Hrrr_tfu]

@Myzreal: smiechlem z w---------o informatyka

[powodzenia]

@Myzreal: @e__e: udało mi się zrobić co założyłem, ale jest jeszcze coś
ponieważ mam teraz na sztywno wpisane wartości kiedy była ostatnia wymiana świec i ostatni serwis gazu, chciałem je pozmieniać setterami będąc w klasie Diagnosis

na linii 13

[e__e]

@powodzenia masz taką metodę w DiagnosisCandles? Jest publiczna? Pobiera inta?

[UnitrA]

@powodzenia: btw, co ile te świece wymieniasz? Bo ja wymieniałem przy lpg co rok, ale stwierdziłem że nadgorliwy jestem, skoro mam przebiegi po 15k

[powodzenia]

@e__e: mam, publiczna i pobiera int
@UnitrA: co 30k

[e__e]

@powodzenia czekaj, pomyliłem. Te same pytania ale o setLastGasCheckAt i klasę LPG

[powodzenia]

@e__e: nie wiem o co chodzi, gdzie jest różnica?
udało mi się przywołać te metody w klasie main:

[cochese]

@powodzenia: Chrystusie Nazareński, weźże zmień te "Candles" na "SparkPlugs", bo ktoś pęknie ze śmiechu i będziesz go miał na sumieniu (╯°□°）╯︵ ┻━┻
Chyba, że celowo robisz #angielskiztuskiem , wtedy spoko (╭☞σ ͜ʖσ)╭☞

[powodzenia]

@cochese: możemy powiedzieć że angielski z tuskiem, ale prawda jest taka, że wpisałem w google tłumacz "świece samochodowe" (mogłem wpisać zapłonowe xD). tłumaczenie wydawało mi się na tyle nieprawdopodobne, że wpisałem w google grafika car candles, dostałem w c--j wyników na których były świece zapłonowe, no i dałem się przekonać że jednak tak to się nazywa, choć mi mocno nie pasowało no i siup

[Diplo]

@powodzenia: swiece to nie candles

Spark plugs