klasy i metody w c++ :: 4programmers.net

Rejestracja:około 6 lat
Ostatnio:prawie 5 lat
Postów:25

0

Witam,
mam problem z zadaniem :

chce zacząć wypełniać metody, ale wyskakuję dużo błędów, więc proszę o pomoc w naprawieniu metod.

Kopiuj

class Set
{
char tab[];


public:
    Set (char tab[]);
    int size();
    bool contains(char znak) const;
    char insert(char znak);
    char remove(char znak);


};



char Set::insert (char znak)
{


}




char Set::remove (char znak)
{


}

char Set::size()
{
return sizeof(tab);
}
char Set::contains (char znak)

{

}





#include <iostream>


int main() {
Set set = Set().insert('a').insert('b').remove('b').remove('c');

std::cout << set.size() << std::endl << std::boolalpha;
std::cout << set.contains('a') << " " << set.contains('b') << " "  << set.contains('c') << " " << set.contains('d') << std::endl; }

błedy wyskakujące :
||=== Build file: "no target" in "no project" (compiler: unknown) ===|
:\10.2.11 Set.cpp|34|error: prototype for 'char Set::size()' does not match any in class 'Set'|
\KLASY\10.2.11 Set.cpp|9|error: candidate is: int Set::size()|
\KLASY\10.2.11 Set.cpp|38|error: prototype for 'char Set::contains(char)' does not match any in class 'Set'|
\\KLASY\10.2.11 Set.cpp|10|error: candidate is: bool Set::contains(char) const|
\10.2.11 Set.cpp||In function 'int main()':|
KLASY\10.2.11 Set.cpp|52|error: no matching function for call to 'Set::Set()'|
KLASY\10.2.11 Set.cpp|8|note: candidate: Set::Set(char*)|
\KLASY\10.2.11 Set.cpp|8|note: candidate expects 1 argument, 0 provided|
KLASY\10.2.11 Set.cpp|2|note: candidate: constexpr Set::Set(const Set&)|
KLASY\10.2.11 Set.cpp|2|note: candidate expects 1 argument, 0 provided|
\KLASY\10.2.11 Set.cpp|2|note: candidate: constexpr Set::Set(Set&&)|
KLASY\10.2.11 Set.cpp|2|note: candidate expects 1 argument, 0 provided|
||=== Build failed: 5 error(s), 0 warning(s) (0 minute(s), 0 second(s)) ===|

screenshot-20191227154306.png (89 KB) - ściągnięć: 119

konstruktory

Spearhead

2019-12-27 14:48

Spearhead2019-12-27 14:48

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

0

Czytaj te komunikaty z błędami

Masz w deklaracji klasy int size(); ale implementujesz char Set::size() { /* ... */ }
Wywołujesz Set() a nie masz konstruktora domyślnego

edytowany 2x, ostatnio: Spearhead 2019-12-27 14:49

heygo1

2019-12-27 15:01

heygo12019-12-27 15:01

Rejestracja:około 6 lat
Ostatnio:prawie 5 lat
Postów:25

0

Spearhead napisał(a):

Czytaj te komunikaty z błędami

Masz w deklaracji klasy int size(); ale implementujesz char Set::size() { /* ... */ }

Wywołujesz Set() a nie masz konstruktora domyślnego

taki konstruktor zrobiłem

Kopiuj

Set::Set(char tab[]):tab(){}

i poprawiłem metode size:

Kopiuj

int Set::size()
{
return sizeof(tab);
}

mam problem z Setem jeszcze :
C:\KLASY\10.2.11 Set.cpp|52|error: no matching function for call to 'Set::Set()'|

cały kod:

Kopiuj


class Set
{
char tab[];


public:
    Set (char tab[]);
    int size();
    bool contains(char znak) const;
    char insert(char znak);
    char remove(char znak);


};

Set::Set(char tab[]):tab(){}

char Set::insert (char znak)
{


}




char Set::remove (char znak)
{


}

int Set::size()
{
return sizeof(tab);
}
bool Set::contains(char znak) const

{

}





#include <iostream>


int main() {
Set set = Set().insert('a').insert('b').remove('b').remove('c');

std::cout << set.size() << std::endl << std::boolalpha;
std::cout << set.contains('a') << " " << set.contains('b') << " "  << set.contains('c') << " " << set.contains('d') << std::endl; }

edytowany 2x, ostatnio: heygo1 2019-12-27 15:05

Spearhead

2019-12-27 15:05

Spearhead2019-12-27 15:05

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

0

No to ci pisałem, że nie masz konstruktora domyślnego. Ten

Kopiuj

Set::Set(char tab[]):tab(){}

Nie jest domyślny, bo trzeba mu przekazać argument. A ty wywołujesz go bez niego.

heygo1

2019-12-27 15:25

heygo12019-12-27 15:25

Rejestracja:około 6 lat
Ostatnio:prawie 5 lat
Postów:25

0

Spearhead napisał(a):

No to ci pisałem, że nie masz konstruktora domyślnego. Ten
Kopiuj
Set::Set(char tab[]):tab(){}
Nie jest domyślny, bo trzeba mu przekazać argument. A ty wywołujesz go bez niego.

Kopiuj

Set::Set()
{
tab[]= ' ';
liczba=0;
znak = ' ';
}

o takie coś chodzi? Nie rozumiem za bardzo zamysłu
Set set = Set().insert('a').insert('b').remove('b').remove('c');

Czemu nazwa klasy jest tu funkcją która wywołuje metode. Jak to zaimplementować w kodzie

cały kod :

Kopiuj

class Set
{
char tab[];
int liczba;
char znak;

public:
    Set();
    int size(int liczba);
    bool contains(char znak) const;
    char insert(char znak);
    char remove(char znak);


};

Set::Set()
{
tab[]= ' ';
liczba=0;
znak = ' ';
}

char Set::insert (char znak)
{


}




char Set::remove (char znak)
{


}

int Set::size(int liczba)
{
return sizeof(tab);
}
bool Set::contains(char znak) const

{

}





#include <iostream>


int main() {
Set set = Set().insert('a').insert('b').remove('b').remove('c');

std::cout << set.size() << std::endl << std::boolalpha;
std::cout << set.contains('a') << " " << set.contains('b') << " "  << set.contains('c') << " " << set.contains('d') << std::endl; }

edytowany 1x, ostatnio: heygo1 2019-12-27 15:25

Delor 2019-12-27 16:55

https://en.wikipedia.org/wiki/Method_chaining Odpowiednik: Set set = Set(); set.insert('a'); set.insert('b'); ...

tajny_agent

2019-12-27 16:24

tajny_agent2019-12-27 16:24

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

1

Pozostaje Ci jedynie zaimplementować

Kopiuj

class Set
{
public:
  Nowoutworzony zbiór jest pusty.
  Set();
  // Stała bezargumentowa metoda `size` zwracająca liczbę elementów zbioru
  int size() const;  
  // Stałą metodę `contains`, która przyjmuje znak i zwraca prawdę jeśli należy on do zbioru albo fałsz w przeciwnym razie
  bool contains(char znak) const;
  // Metodę `insert`, która przyjmuje znak, dodaje go do zbioru i zwraca referencję obiektu, na rzecz którego została wywołana
  Set& insert(char znak); 
  // Metodę remove, która przyjmuje znak, usuwa go ze zbioru i zwraca referencję obiektu, na rzecz którego została wywołana
  Set& remove(char znak); 

private:
  char* data;
  int length;
};

heygo1 2019-12-27 16:42

dziękuje, posiłuję się z tym, mam teraz mocny punkt zaczepienia.

heygo1

2019-12-27 21:45

heygo12019-12-27 21:45

Rejestracja:około 6 lat
Ostatnio:prawie 5 lat
Postów:25

0

Nie mogę poradzić sobie z błędem :
C:\10.2.11 Set.cpp|46|error: assignment of member 'Set::length' in read-only object|
a dziwne, bo na takiej samej zasadzie przecież działa zmienna licz...

Kopiuj

class Set
{
public:

  Set();
  // Stała bezargumentowa metoda `size` zwracająca liczbę elementów zbioru
  int size() const;
  // Stałą metodę `contains`, która przyjmuje znak i zwraca prawdę jeśli należy on do zbioru albo fałsz w przeciwnym razie
  bool contains(char znak) const;
  // Metodę `insert`, która przyjmuje znak, dodaje go do zbioru i zwraca referencję obiektu, na rzecz którego została wywołana
  Set& insert(char znak);
  // Metodę remove, która przyjmuje znak, usuwa go ze zbioru i zwraca referencję obiektu, na rzecz którego została wywołana
  Set& remove(char znak);

private:
  char data[];
  int length;
  int licz;
};

Set::Set(): licz(0), length(0){}


 Set& Set::insert (char znak)
{

data[licz]=znak;
licz++;

}




Set& Set::remove (char znak)
{

for (int i=0; i<=licz; i++)

    if (data[i]== znak) {data[i]=' '; licz--;}
}

int Set::size() const
{
 length = sizeof(data)/sizeof(int);


}
bool Set::contains(char znak) const

{
for (int i=0; i<=length; i++)
    {

      if (znak==data[i]) return 1;

      else return 0;

    }
}





#include <iostream>


int main() {
Set set = Set().insert('a').insert('b').remove('b').remove('c');

std::cout << set.size() << std::endl << std::boolalpha;
std::cout << set.contains('a') << " " << set.contains('b') << " "  << set.contains('c') << " " << set.contains('d') << std::endl; }

edytowany 2x, ostatnio: heygo1 2019-12-27 22:13

Hodor

2019-12-27 22:22

Hodor2019-12-27 22:22

Rejestracja:ponad 7 lat
Ostatnio:dzień
Lokalizacja:Warszawa
Postów:327

1

Wywal ten const qualifier int size() const;. Wewnątrz const member funkcji nie można modyfikować memberów.

Bartłomiej Golenko 2019-12-27 22:39

NIE, usuwanie const to zły pomysł. Pole length nie musi być polem klasy (a jeśli jest - to nie musi być wyliczane w funkcji size() tylko aktualizowane przy dodawaniu/usuwaniu elementu).