Point2d c++ :: 4programmers.net

chytant

2018-01-04 20:17

chytant2018-01-04 20:17

Hobbistycznie próbuje coś skrobaćw c++ i zabrałem się za tzw. Point2D. Jak ten kod usprawnić ? Po części wiem, że jest on napisany fatalnie ale o tyle o ile działa.
Za pomoc serdecznie dziękuje.

Kopiuj

#include <iostream>
#include <cmath>
 
using namespace std;
 
class Point2d
{
private:
       int x;
       int y;
public:
       int getX()
       {
           return x;
       }
       int getY()
       {
           return y;
       }
 
        Point2d(int a, int b)
       {
             x = a;
             y = b;
       }
};
 
double dod(Point2d p1, Point2d p2)
{
	return p1.getX()+p2.getX(),p1.getY()+p2.getY();
}

double odej(Point2d p1, Point2d p2)
{
	return p1.getX()-p2.getX(),p1.getY()-p2.getY();
}

double dzie(Point2d p1, Point2d p2)
{
	return p1.getX()/2,p1.getY()/2;
}
  
double mnoz(Point2d p1, Point2d p2)
{
	return p1.getX()*2,p1.getY()*2;
}

double wypisz(Point2d p1, Point2d p2)
{
	return p1.getX(), p1.getY();
}

double dst(Point2d p1, Point2d p2)
{
       return sqrt(pow((p2.getX()-p1.getX()),2)+pow((p2.getY()-p1.getY()),2));
}
 
 
 
 
 
int main()
{
    Point2d p1(10,10);
    Point2d p2(10,10);
    Point2d p3();
 
 	cout<<"p3 = p1 + p2  "<<dod(p1,p2)<<","<<dod(p1,p2)<<endl;
 	cout<<"p3 = p1 - p2  "<<odej(p1,p2)<<","<<odej(p1,p2)<<endl;
 	cout<<"p3 = p1/2     "<<dzie(p1,p2)<<","<<dzie(p1,p2)<<endl;
 	cout<<"p3 = p1*2     "<<mnoz(p1,p2)<<","<<mnoz(p1,p2)<<endl;
 	cout<<"Wypisz =      "<<wypisz(p1,p2)<<","<<wypisz(p1,p2)<<endl;
    cout<<"Distance      "<<dst(p1,p2)<<endl;
    return 0;
}

edytowany 1x, ostatnio: cerrato 2019-09-02 12:50

PanRiK

2018-01-06 10:56

PanRiK2018-01-06 10:56

Rejestracja:ponad 7 lat
Ostatnio:ponad rok
Lokalizacja:Szczecin
Postów:104

Poczytaj o przeciążeniu operatorów
Plus możesz poczytać jeszcze o this

edytowany 2x, ostatnio: PanRiK 2018-01-06 10:59

tajny_agent

2018-01-06 22:33

tajny_agent2018-01-06 22:33

Rejestracja:ponad 11 lat
Ostatnio:ponad rok
Postów:1340

A teraz przetestuj swój program z takimi danymi:

Kopiuj

Point2d p1( 5, 10 );
Point2d p2( 50, 80 );

chytant

2018-01-07 10:35

chytant2018-01-07 10:35

tajny_agent napisał(a):

A teraz przetestuj swój program z takimi danymi:
Kopiuj
Point2d p1( 5, 10 );
Point2d p2( 50, 80 );

Działa błędnie, wiem o tym, jak to rozwiązać ?

tajny_agent

2018-01-07 12:29

tajny_agent2018-01-07 12:29

Rejestracja:ponad 11 lat
Ostatnio:ponad rok
Postów:1340

Skoro dodajesz/odejmujesz/mnożysz/dzielisz dwa Point2d to zwracaną wartością również powinien być Point2d, a nie double.
Zamiast

Kopiuj

double dod(Point2d p1, Point2d p2)
{
    return p1.getX()+p2.getX(),p1.getY()+p2.getY();
}

Zrób:

Kopiuj

Point2d dod( const Point2d& p1, const Point2d& p2 )
{
  return Point2d( p1.getX() + p2.getX(), p1.getY() + p2.getY() );
}

No i takie coś return p1.getX(), p2.getX() nie działa tak jak myślisz. Return jeśli ma coś zwracać to tylko i wyłącznie jedną wartość.

chytant

2018-01-07 13:40

chytant2018-01-07 13:40

tajny_agent napisał(a):

Skoro dodajesz/odejmujesz/mnożysz/dzielisz dwa Point2d to zwracaną wartością również powinien być Point2d, a nie double.
Zamiast
Kopiuj
double dod(Point2d p1, Point2d p2)
{
    return p1.getX()+p2.getX(),p1.getY()+p2.getY();
}
Zrób:
Kopiuj
Point2d dod( const Point2d& p1, const Point2d& p2 )
{
  return Point2d( p1.getX() + p2.getX(), p1.getY() + p2.getY() );
}
No i takie coś return p1.getX(), p2.getX() nie działa tak jak myślisz. Return jeśli ma coś zwracać to tylko i wyłącznie jedną wartość.

Przy takim rozwiązaniu muszę dodać do:
int getX(), int getY() - const ?
Bo po zamienieniu kodu wartość obliczeń się nie wywołuje.

PanRiK

2018-01-07 14:48

PanRiK2018-01-07 14:48

Rejestracja:ponad 7 lat
Ostatnio:ponad rok
Lokalizacja:Szczecin
Postów:104

Kopiuj

    Point2D operator+ (Point2D &ob2){
        int x = this->x + ob2.x;
        int y = this->y + ob2.y;

        Point2D ob3 = Point2D(x,y);

        return ob3;

    }

Jest to bardziej czytelne.

Przeładować operator możesz tak dla każdego swojego działania.

Wywołujesz tak:

Kopiuj

   Point2D a = Point2D(1,2);
   Point2D b = Point2D(3,4);
   Point2D c = a+b;

   std::cout << c.x << " " << c.y;

Edit:

chytant napisał(a):

Przy takim rozwiązaniu muszę dodać do:
int getX(), int getY() - const ?
Bo po zamienieniu kodu wartość obliczeń się nie wywołuje.

@tajny_agent Chodziło o to, że przy argumentach funkcji dajesz słowo kluczowe "const" po to, aby przypadkiem tych argumentów nie modyfikować podczas dodawania. Takie zabezpieczenie.

edytowany 3x, ostatnio: PanRiK 2018-01-07 15:01

chytant

2018-01-07 15:04

chytant2018-01-07 15:04

PanRik totalnie nie zrozumiałem Twojego przekazu.
Zgodnie ze wskazówkami tajny_agent podmieniłem kawałek kodu, ale program nie działa, coś jest nie tak z wywołaniem, tylko co ?

Kopiuj

#include <iostream>
#include <cmath>

using namespace std;

class Point2d
{
private:
       int x;
       int y;
public:
       int getX() const
       {
           return x;
       }
       int getY() const
       {
           return y;
       }

        Point2d(int a, int b)
       {
             x = a;
             y = b;
       }
};

Point2d dod( const Point2d& p1, const Point2d& p2 )
{
  return Point2d( p1.getX() + p2.getX(), p1.getY() + p2.getY() );
}

int main()
{
    Point2d p1(10,10);
    Point2d p2(10,10);

    cout<<dod(p1,p2);
    return 0;
}

edytowany 1x, ostatnio: cerrato 2019-09-02 12:50

PanRiK

2018-01-07 15:11

PanRiK2018-01-07 15:11

Rejestracja:ponad 7 lat
Ostatnio:ponad rok
Lokalizacja:Szczecin
Postów:104

Kopiuj

cout<<dod(p1,p2);

Próbujesz wyświetlić obiekt klasy Point2D.

Przeczytaj o przeciążeniu operatorów, naprawdę ten sposób co ja zrobiłem jest bardzo przyjemny.

Żeby Ci zobrazować to co wcześniej chciałem Ci przekazać:

Kopiuj

#include <iostream>

class Point2D{

private:
    int x;
    int y;
public:

    int getX(){
        return this->x;
    }
    int getY(){
        return this->y;
    }

    Point2D(int x, int y){
        this->x = x;
        this->y = y;
    }

    Point2D operator+ (Point2D &ob2){
        int x = this->getX() + ob2.getX();
        int y = this->getY() + ob2.getY();

        Point2D ob3 = Point2D(x,y);
        return ob3;
    }
};

int main() {

   Point2D first = Point2D(1,2);
   Point2D secound = Point2D(3,4);
   Point2D afterSum = first + secound;

   std::cout << afterSum.getX() << " " << afterSum.getY();

    return 0;
}

I za pomocą przeciążenia operatorów, da się wyświetlić w taki sposób jak Ty chcesz ;)

edytowany 3x, ostatnio: PanRiK 2018-01-07 15:22

chytant

2018-01-08 17:56

chytant2018-01-08 17:56

PanRik - wielkie dziekuje, wszystko jasne :)

Krystian Zdziebło

2019-09-02 12:47

Krystian Zdziebło2019-09-02 12:47

Rejestracja:ponad 5 lat
Ostatnio:około 5 lat
Postów:1

Kopiuj

Point2D operator+ (Point2D &o){
        x += o.getX();
        y += o.getY();

        return *this;
}

lub

Kopiuj

Point2D operator+ (Point2D &o){return Point2D (x+o.gx(),y+o.gy(),z+o.gz());}

edytowany 3x, ostatnio: Krystian Zdziebło 2019-09-02 13:02

Zobacz pozostałe 5 komentarzy

Krystian Zdziebło 2019-09-02 14:43

Szybko krytykujecie, zamiast pomyśleć...

kq 2019-09-02 14:46

A co tu jest do myślenia, skoro łamiesz podstawowe zasady? Polecam lekturę: https://dsp.krzaq.cc/post/304/jak-przeladowywac-operatory-w-mojej-klasie/

Krystian Zdziebło 2019-09-02 14:49

Gdy dojdziesz do pewnego poziomu tez zaczniesz ;) Nie ma co podawać ludziom ryby, bo jej nie docenią. Jednak docenią, jak podasz wędkę. Do pewnych spraw sami muszą dojść, to dla nich bardziej rozwojowe niż tępe przepisywanie kodu.

kq 2019-09-02 15:28

W tej analogii podajesz im zatrutą rybę, a nie wędkę.

Krystian Zdziebło 2019-09-02 15:34

Za to uzyskałem, to co chciałem uzyskać ;) - Dziękuję.

Bartłomiej Golenko

2019-09-02 14:23

Bartłomiej Golenko2019-09-02 14:23

Rejestracja:prawie 6 lat
Ostatnio:około 15 godzin
Postów:289

W tym przypadku podejście z operatorami wydaje się być OK, ale zalecałbym ogólną ostrożność ;)
Nazwa funkcji często więcej mówi o jej działaniu niż symbol operatora. A "to przecież oczywiste" bywa oczywiste w różny sposób zależnie od osoby ;).

Mój ulubiony przykład:

Kopiuj

// zwykły "matematyczny" wektor w przestrzeni 2D - w zasadzie niczym nie różniący się od punktu z tego wątku ;)
Wektor2D w1 (1,1), w2 (2, 3);    
...

auto v1 = w1 + w2;
auto v2 = w1 * w2;

Pytania

jakiego typu będą v1 i v2 ?
co będą zawierać ?

edytowany 1x, ostatnio: Bartłomiej Golenko 2019-09-03 16:22

Krystian Zdziebło 2019-09-02 14:33

Wektor2D, 3,4; 2,3;

Bartłomiej Golenko 2019-09-02 14:41

Nie, to zła odpowiedź, wynikająca z tego co Tobie wydaje się oczywiste... Hint: ile znasz różnych rodzajów iloczynów dla wektorów ?

MarekR22 2019-09-04 14:04

ja znam 4 rodzaje iloczynów wektorów: skalarny, wektorowy, kartezjański, krakowian.

Bartłomiej Golenko 2019-09-04 15:26

Zgadza sie... A jeśli potraktujesz wektor jako przypadek szczególny macierzy to robi się jeszcze weselej. Dlatego ja wolę zapis w stylu auto v3 = inner_product(w1, w2);

Liczba odpowiedzi na stronę

Treść *

Treść

Podgląd

Kliknij, aby dodać treść...

Kliknij, aby dodać załącznik lub wklej ze schowka.

Instrukcja obsługi Markdown

Pomoc 1.18.8

Typografia

Edytor obsługuje składnie Markdown, w której pojedynczy akcent *kursywa* oraz _kursywa_ to pochylenie. Z kolei podwójny akcent **pogrubienie** oraz __pogrubienie__ to pogrubienie. Dodanie znaczników ~~strike~~ to przekreślenie.

Możesz dodać formatowanie komendami , , oraz .

Ponieważ dekoracja podkreślenia jest przeznaczona na linki, markdown nie zawiera specjalnej składni dla podkreślenia. Dlatego by dodać podkreślenie, użyj underline.

Komendy formatujące reagują na skróty klawiszowe: Ctrl+B, Ctrl+I, Ctrl+U oraz Ctrl+S.

Linki

By dodać link w edytorze użyj komendy lub użyj składni [title](link). URL umieszczony w linku lub nawet URL umieszczony bezpośrednio w tekście będzie aktywny i klikalny.

Jeżeli chcesz, możesz samodzielnie dodać link: <a href="link">title</a>.

Wewnętrzne odnośniki

Możesz umieścić odnośnik do wewnętrznej podstrony, używając następującej składni: [[Delphi/Kompendium]] lub [[Delphi/Kompendium|kliknij, aby przejść do kompendium]]. Odnośniki mogą prowadzić do Forum 4programmers.net lub np. do Kompendium.

Wspomnienia użytkowników

By wspomnieć użytkownika forum, wpisz w formularzu znak @. Zobaczysz okienko samouzupełniające nazwy użytkowników. Samouzupełnienie dobierze odpowiedni format wspomnienia, zależnie od tego czy w nazwie użytkownika znajduje się spacja.

Znaczniki HTML

Dozwolone jest używanie niektórych znaczników HTML: <a>, , , <kbd>, <del>, , <dfn>, <pre>, <blockquote>, <hr/>, ,  oraz <img/>.

Skróty klawiszowe

Dodaj kombinację klawiszy komendą notacji klawiszy lub skrótem klawiszowym Alt+K.

Reprezentuj kombinacje klawiszowe używając taga <kbd>. Oddziel od siebie klawisze znakiem plus, np <kbd>Alt+Tab</kbd>.

Indeks górny oraz dolny

Przykład: wpisując H2O i m2 otrzymasz: H₂O i m².

Składnia Tex

By precyzyjnie wyrazić działanie matematyczne, użyj składni Tex.

<tex>arcctg(x) = argtan(\frac{1}{x}) = arcsin(\frac{1}{\sqrt{1+x^2}})</tex>

Kod źródłowy

Krótkie fragmenty kodu

Wszelkie jednolinijkowe instrukcje języka programowania powinny być zawarte pomiędzy obróconymi apostrofami: `kod instrukcji` lub ``console.log(`string`);``.

Kod wielolinijkowy

Dodaj fragment kodu komendą . Fragmenty kodu zajmujące całą lub więcej linijek powinny być umieszczone w wielolinijkowym fragmencie kodu. Znaczniki ``` lub ~~~ umożliwiają kolorowanie różnych języków programowania. Możemy nadać nazwę języka programowania używając auto-uzupełnienia, kod został pokolorowany używając konkretnych ustawień kolorowania składni:

```javascript document.write('Hello World'); ```

Możesz zaznaczyć również już wklejony kod w edytorze, i użyć komendy by zamienić go w kod. Użyj kombinacji Ctrl+`, by dodać fragment kodu bez oznaczników języka.

Tabelki

Dodaj przykładową tabelkę używając komendy . Przykładowa tabelka składa się z dwóch kolumn, nagłówka i jednego wiersza.

Wygeneruj tabelkę na podstawie szablonu. Oddziel komórki separatorem ; lub |, a następnie zaznacz szablonu.

nazwisko;dziedzina;odkrycie
Pitagoras;mathematics;Pythagorean Theorem
Albert Einstein;physics;General Relativity
Marie Curie, Pierre Curie;chemistry;Radium, Polonium

Użyj komendy by zamienić zaznaczony szablon na tabelkę Markdown.

Lista uporządkowana i nieuporządkowana

Możliwe jest tworzenie listy numerowanych oraz wypunktowanych. Wystarczy, że pierwszym znakiem linii będzie * lub - dla listy nieuporządkowanej oraz 1. dla listy uporządkowanej.

Użyj komendy by dodać listę uporządkowaną.

1. Lista numerowana
2. Lista numerowana

Użyj komendy by dodać listę nieuporządkowaną.

* Lista wypunktowana
* Lista wypunktowana
** Lista wypunktowana (drugi poziom)

Składnia Markdown

Edytor obsługuje składnię Markdown, która składa się ze znaków specjalnych. Dostępne komendy, jak formatowanie , dodanie tabelki lub fragmentu kodu są w pewnym sensie świadome otaczającej jej składni, i postarają się unikać uszkodzenia jej.

Dla przykładu, używając tylko dostępnych komend, nie możemy dodać formatowania pogrubienia do kodu wielolinijkowego, albo dodać listy do tabelki - mogłoby to doprowadzić do uszkodzenia składni.

W pewnych odosobnionych przypadkach brak nowej linii przed elementami markdown również mógłby uszkodzić składnie, dlatego edytor dodaje brakujące nowe linie. Dla przykładu, dodanie formatowania pochylenia zaraz po tabelce, mogłoby zostać błędne zinterpretowane, więc edytor doda oddzielającą nową linię pomiędzy tabelką, a pochyleniem.

Skróty klawiszowe

Skróty formatujące, kiedy w edytorze znajduje się pojedynczy kursor, wstawiają sformatowany tekst przykładowy. Jeśli w edytorze znajduje się zaznaczenie (słowo, linijka, paragraf), wtedy zaznaczenie zostaje sformatowane.

Ctrl+B - dodaj pogrubienie lub pogrub zaznaczenie
Ctrl+I - dodaj pochylenie lub pochyl zaznaczenie
Ctrl+U - dodaj podkreślenie lub podkreśl zaznaczenie
Ctrl+S - dodaj przekreślenie lub przekreśl zaznaczenie

Notacja Klawiszy

Alt+K - dodaj notację klawiszy

Fragment kodu bez oznacznika

Alt+C - dodaj pusty fragment kodu

Skróty operujące na kodzie i linijkach:

Alt+L - zaznaczenie całej linii
Alt+, Alt+ - przeniesienie linijki w której znajduje się kursor w górę/dół.
Tab/⌘+] - dodaj wcięcie (wcięcie w prawo)
Shit+Tab/⌘+[ - usunięcie wcięcia (wycięcie w lewo)

Dodawanie postów:

Ctrl+Enter - dodaj post
⌘+Enter - dodaj post (MacOS)

Point2d c++

chytant

chytant

tajny_agent napisał(a):

chytant

tajny_agent napisał(a):

chytant napisał(a):

chytant

chytant

Smileys & People

Animals & Nature

Food & Drink

Travel & Places

Activities

Objects

Symbols

Typografia

Linki

Wewnętrzne odnośniki

Wspomnienia użytkowników

Znaczniki HTML

Skróty klawiszowe

Indeks górny oraz dolny

Składnia Tex

Kod źródłowy

Krótkie fragmenty kodu

Kod wielolinijkowy

Tabelki

Lista uporządkowana i nieuporządkowana

Składnia Markdown

Skróty klawiszowe