Program do obsługi liczb zespolonych – code review

Rejestracja:ponad 7 lat
Ostatnio:ponad 6 lat
Postów:72

Mam napisać program do obsługi liczb zespolonych. Jestem raczej na poziomie podstawowym, więc pewnie dodawanie, odejmowanie i moduł wystarczą, jak myślicie? :D
Ale coś mi kod nie chce się kompilować. Mógłby ktoś zerknąć i powiedzieć o co chodzi. Jeśli macie uwagi do kodu i jego chaotyczności lub są jakieś nie potrzebne elementy, to piszcie. ;)
Na razie nic nie dawałam w funkcji main, bo chyba i tak najpierw musi mi zadziałać kod. Z góry dzięki ;)

Kopiuj

#include <iostream>
#include<cmath>

using namespace std;

class Zespolona{
    int rzeczywista;
    int zespolona;
public:
    Zespolona::Zespolona();
    Zespolona::Zespolona(int a, int b);
    Zespolona::~Zespolona;
    int Zespolona::getRzeczywista();
    Zespolona::setRzeczywista(int a);
    Zespolona operator+(const zespolona1 & const zespolona2);
    Zespolona operator-(const zespolona1 & const zespolona2);
    double Zespolona::modul(const zespolona1);

};
Zespolona::Zespolona(){
    rzeczywista = 1;
    zespolona = 1;
}
Zespolona::Zespolona(int a, int b){
    rzeczywista = a;
    zespolona = b;
}
Zespolona::~Zespolona(){
    cout<<"Koniec"<<endl;
}
int Zespolona::getRzeczywista(){
    return rzeczywista;
}
Zespolona::setRzeczywista(int a){
    rzeczywista = a;
}
Zespolona operator+(const zespolona1 & const zespolona2)
{
    Zespolona zesp1;
    zesp1.rzeczywista = zespolona1.rzeczywista+zespolona2.rzeczywista;
    zesp1.zespolona = zespolona1.zespolona + zespolona2.zespolona;
    return zesp1;
}
Zespolona operator+(const zespolona1 & const zespolona2)
{
    Zespolona zesp1;
    zesp1.rzeczywista = zespolona1.rzeczywista+zespolona2.rzeczywista;
    zesp1.zespolona = zespolona1.zespolona + zespolona2.zespolona;
    return zesp1;
}
Zespolona operator-(const zespolona1 & const zespolona2)
{
    Zespolona zesp1;
    zesp1.rzeczywista = zespolona1.rzeczywista-zespolona2.rzeczywista;
    zesp1.zespolona = zespolona1.zespolona - zespolona2.zespolona;
    return zesp1;
}
double Zespolona::modul(const zespolona1){
    mod = sqrt(zespolona1.rzeczywista^2+zespolona1.zespolona^2);
    return mod;
}
int main()
{
    return 0;
}

edytowany 3x, ostatnio: maxmiks 2018-04-09 21:34

flowCRANE 2018-04-09 21:32

Tytułowanie wątków, Tagowanie wątków, Formatowanie treści – zapoznaj się z tymi artykułami.

GutekSan

2018-04-09 21:33

GutekSan2018-04-09 21:33

Rejestracja:ponad 8 lat
Ostatnio:około 19 godzin
Postów:1265

maxmiks napisał(a):

Mam napisać program do obsługi liczb zespolonych. Jestem raczej na poziomie podstawowym, więc pewnie dodawanie, odejmowanie i moduł wystarczą, jak myślicie? :D
Ale coś mi kod nie chce się uruchamiać. Mógłby ktoś zerknąć i powiedzieć o co chodzi. Jeśli macie uwagi do kodu i jego chaotyczności lub są jakieś nie potrzebne elementy, to piszcie. ;)
Na razie nic nie dawałam w funkcji main, bo chyba i tak najpierw musi mi zadziałać kod. Z góry dzięki ;)

Skoro nic nie ma w main to nic nie zadziała.

Chyba, że chodzi Ci o to, że kod się nie kompiluje, a nie że się nie uruchamia.

maxmiks 2018-04-09 21:34

Dokładnie o to chodzi ;) Sorka

GutekSan

2018-04-09 21:39

GutekSan2018-04-09 21:39

Rejestracja:ponad 8 lat
Ostatnio:około 19 godzin
Postów:1265

W definicji klasy używasz int zamiast double do określenia Re i Im.
W definicji operatorów pomijasz w ogóle typ, masz jedynie modyfikator const
Nie mieszaj pojęć: liczba zespolona składa się z części rzeczywistej i urojonej a nie zespolonej.
setRzeczywista powinna zwracac void

Sparrow-hawk

2018-04-10 07:46

Sparrow-hawk2018-04-10 07:46

Rejestracja:prawie 13 lat
Ostatnio:3 miesiące
Lokalizacja:Katowice
Postów:189

Moje uwagi do pytania o code review:

Kod nie kompiluje się;
Brak testów (nie wymagam pisania testów jednostkowych, ale pusty main?);

Moje uwagi do kodu:

Często zapominasz o typach zmiennych i funkcji;
Nazewnictwo: Konstruktor przyjmuje argumenty a & b. Czy liczby zespolone mają pola a & b?
Funkcja modul:

Jak już chcesz przekazywać liczbę zespoloną do tej funkcji, to lepiej przez referencję, ale chyba lepiej jak wykorzystać pola klasy do obliczenia modułu liczby zespolonej?
Jakiego typu jest mod?
Od kiedy do C++ wprowadzono operator ^ potęgowania?

Pola klasy inicjalizuj za pomocą listy inicjalizacyjnych w konstruktorach:

Kopiuj

Complex::Complex(double re, double im) : re_(re), im_(im)
{
}

Operatory arytmetyczne, to dramat, błędnie przekazywane argumenty, brak typów. Operator może wyglądać np. tak:

Kopiuj

friend Complex std::operator+(const Complex& c1, const complex& c2)
{
  return Complex(c1.re_ + c2.re_, c1.im_ + c2.im_);
}
6. Przydałby się jeszcze konstruktor jednoargumentowy;

pingwindyktator

2018-04-10 20:30

pingwindyktator2018-04-10 20:30

Rejestracja:ponad 12 lat
Ostatnio:19 dni
Lokalizacja:Kraków
Postów:1055

operator+ jak i operator- powinieneś zaimplementować jako nie-member funkcje, coby supportować pełną symetrie.
setRzeczywista(int a); a gdzie return type?

maxmiks

2018-04-10 20:34

maxmiks2018-04-10 20:34

Rejestracja:ponad 7 lat
Ostatnio:ponad 6 lat
Postów:72

pingwindyktator napisał(a):

operator+ jak i operator- powinieneś zaimplementować jako nie-member funkcje, coby supportować pełną symetrie.

setRzeczywista(int a); a gdzie return type?

W jaki sposób mam zaimplementować operatory jako nie-member funkcje?

pingwindyktator

2018-04-10 20:36

pingwindyktator2018-04-10 20:36

Rejestracja:ponad 12 lat
Ostatnio:19 dni
Lokalizacja:Kraków
Postów:1055

Przecież kolega @Sparrow-hawk dobrze Ci napisał. Proponuję zastosować się do uwag wyżej, a potem pytać dalej.

maxmiks

2018-04-10 20:39

maxmiks2018-04-10 20:39

Rejestracja:ponad 7 lat
Ostatnio:ponad 6 lat
Postów:72

Bo w takich deklaracjach operatorów nie może być więcej niż 1 argument... zaczynam ogarniać :)

maxmiks

2018-04-10 20:39

maxmiks2018-04-10 20:39

Rejestracja:ponad 7 lat
Ostatnio:ponad 6 lat
Postów:72

A mogę zapisać je jako funkcje zaprzyjaźnione?

pingwindyktator

2018-04-10 20:40

pingwindyktator2018-04-10 20:40

Rejestracja:ponad 12 lat
Ostatnio:19 dni
Lokalizacja:Kraków
Postów:1055

Możesz zaimplementować to jako member funkcje, wtedy jednym z argumentów jest this, drugim jest ten przyjmowany jako argument funkcji. Ale to nie jest dobry pomysł i nie chcesz tak robić.
Możesz zrobić z tych operatorów funkcje zaprzyjaźnione, ale znowu - to nie jest dobry pomysł. Polegaj na publicznym interfejsie, nie na szczegółach implementacji. Oszczędzi Ci to refactoringu w przyszłości.

edytowany 1x, ostatnio: pingwindyktator 2018-04-10 20:41

maxmiks

2018-04-10 21:04

maxmiks2018-04-10 21:04

Rejestracja:ponad 7 lat
Ostatnio:ponad 6 lat
Postów:72

Program już się uruchamia, tylko dla każdej liczy moduł wynosi 0. Gdzie jest błąd?

Kopiuj

#include <iostream>
#include<cmath>

using namespace std;

class Zespolona{
    double rzeczywista;
    double urojona;
    double mod = sqrt(rzeczywista*rzeczywista+urojona*urojona);
public:
    Zespolona();
    Zespolona(int a, int b);
    ~Zespolona();
    double getRzeczywista();
    void setRzeczywista(int a);
    double getUrojona();
    void setUrojona(int b);
    double getModul();
    friend Zespolona operator+(const Zespolona & zespolona1, const Zespolona & zespolona2);
    friend Zespolona operator-(const Zespolona & zespolona1, const Zespolona & zespolona2);
};
Zespolona::Zespolona(){
    rzeczywista = 1;
    urojona = 1;
}
Zespolona::Zespolona(int a, int b){
    rzeczywista = a;
    urojona = b;
}
Zespolona::~Zespolona(){
    cout<<"Koniec"<<endl;
}
double Zespolona::getRzeczywista(){
    return rzeczywista;
}
double Zespolona::getUrojona(){
    return urojona;
}
void Zespolona::setRzeczywista(int a){
    rzeczywista = a;
}
void Zespolona::setUrojona(int b){
    urojona = b;
}
double Zespolona::getModul(){
    return mod;
}

Zespolona operator+(const Zespolona & zespolona1, const Zespolona & zespolona2)
{
    Zespolona zesp1;
    zesp1.rzeczywista = zespolona1.rzeczywista+zespolona2.rzeczywista;
    zesp1.urojona = zespolona1.urojona + zespolona2.urojona;
    return zesp1;
}
Zespolona operator-(const Zespolona & zespolona1, const Zespolona & zespolona2)
{
    Zespolona zesp1;
    zesp1.rzeczywista = zespolona1.rzeczywista-zespolona2.rzeczywista;
    zesp1.urojona = zespolona1.urojona - zespolona2.urojona;
    return zesp1;
}

int main()
{
    Zespolona z1(15, 9);
    Zespolona z2(6, 8);
    Zespolona z3, z4;
    z3 = z1+z2;
    cout<<z3.getRzeczywista()<<"+"<<z3.getUrojona()<<"i"<<endl;
    z4 = z1-z2;
    cout<<z4.getRzeczywista()<<"+"<<z4.getUrojona()<<"i"<<endl;
    cout<<"Modul liczby z2: "<<z2.getModul()<<endl;
    return 0;
}

pingwindyktator

2018-04-10 21:09

pingwindyktator2018-04-10 21:09

Rejestracja:ponad 12 lat
Ostatnio:19 dni
Lokalizacja:Kraków
Postów:1055

double mod = sqrt(rzeczywista*rzeczywista+urojona*urojona); inicjalizujesz mod w złym miejscu. Powinieneś robić to wtedy, kiedy znasz już liczby rzeczywista i urojona, tj:

Zespolona::Zespolona()
Zespolona::Zespolona(int a, int b)
Zespolona::setRzeczywista(int a)
Zespolona::setUrojona(int b)

W operatorach + i -- bardzo nieładnie to robisz: próbujesz ręcznie ustawić pola rzeczywista i urojona, nie ustawiasz jednak wartości mod. Zastosuj się do mojej uwagi o prywatnych funkcjach a rozwiązesz ten problem.

edytowany 1x, ostatnio: pingwindyktator 2018-04-10 21:10

maxmiks 2018-04-10 21:13

Chodzi Ci o definicje funkcji zaprzyjaźnionych?

pingwindyktator 2018-04-10 21:14

Chodzi o to, że Twoje operatory są zaprzyjaźnione z klasą. Nie powinny być. friend w 99% przypadków jest złym pomysłem.

maxmiks

2018-04-10 21:21

maxmiks2018-04-10 21:21

Rejestracja:ponad 7 lat
Ostatnio:ponad 6 lat
Postów:72

Czyli lepiej po prostu nie używać funkcji zaprzyjaźnionych? I mam jeszcze pytanie, tylko się nie denerwuj... Jak nie masz ochoty, to nie odpowiadaj :D
Czyli mam zastosować prywatne funkcje, czyli np. ```
double Zespolona::Suma()

Kopiuj


Ogólnie to mój kod już działa i dzięki wielkie za pomoc, ale chcę wiedzieć na przyszłość co robić lepiej i  jak to na prawdę powinno wyglądać. Zdaję sobie sprawę, że mój kod jest dosyć chaotyczny

edytowany 1x, ostatnio: maxmiks 2018-04-10 21:23

pingwindyktator

2018-04-10 21:26

pingwindyktator2018-04-10 21:26

Rejestracja:ponad 12 lat
Ostatnio:19 dni
Lokalizacja:Kraków
Postów:1055

Nie powinieneś używać friend funkcji - tylko tyle.
Takie operatory powinny wyglądać tak:

Kopiuj

Zespolona operator+(const Zespolona & zespolona1, const Zespolona & zespolona2)
{
    Zespolona zesp1(zespolona1.rzeczywista+zespolona2.rzeczywista, zespolona1.urojona + zespolona2.urojona);
    return zesp1;
}

Fajnie, że działa, ale w kodzie jest jeszcze co najmniej kilka błędów - zarówno "błędów w sztuce" jak i zwykłych bugów.

maxmiks 2018-04-10 21:36

A taka funkcja może przyjmować dwa argumenty? Bo jak próbuję kompilować, to wyskakuje informacja, że powinien być maksymalnie 1 argument

pingwindyktator 2018-04-10 21:37

Deklaracja takiej funkcji powinna wyglądać tak: Zespolona operator+(const Zespolona & zespolona1, const Zespolona & zespolona2);

MarekR22

2018-04-11 05:44

MarekR222018-04-11 05:44

Moderator C/C++

Rejestracja:około 17 lat
Ostatnio:4 minuty

http://www.cplusplus.com/reference/complex/complex/

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)

Program do obsługi liczb zespolonych – code review

maxmiks napisał(a):

pingwindyktator napisał(a):

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