Zmienna wskaźnikowa w klasie

Question

Witajcie. Czy dobrze wykonałem poniższe polecenie?

Utwórz klasę zawierającą zmienną wskaźnikową bez konstruktora kopiującego.

#include "pch.h"
#include "Cukierki.h"
#include <cstdlib>

int main()
{
	Cukierki C1("zielony", 8);
	Cukierki C2;
	system("PAUSE");
}


============================================================

#pragma once
#include <iostream>
#include <string>


class Cukierki
{
private:
	std::string kolorCukierka;
	int *liczbaCukierkow;


public:
	Cukierki(std::string kolorCukierka = "czerwony", int liczbaCukierkow = 5);
	virtual ~Cukierki();
};


=========================================================



#include "pch.h"
#include "Cukierki.h"


Cukierki::Cukierki(std::string kolor, int liczba):
	kolorCukierka(kolor), liczbaCukierkow(&liczba)
{
	this->kolorCukierka = kolorCukierka;
	this->liczbaCukierkow = liczbaCukierkow;
	std::cout << std::endl << "Zadzialal konstruktor" << std::endl;
	std::cout << kolorCukierka << liczba << std::endl;
}


Cukierki::~Cukierki()
{

}

 

Answer 1

Nie. Niektóre kompilatory mogą znaleźć problemy za mnie, więc pozwolę sobię pomóc:

<source>:21:44: warning: initializing pointer member 'liczbaCukierkow' with the stack address of parameter 'liczba' [-Wdangling-field]
    kolorCukierka(kolor), liczbaCukierkow(&liczba)
                                           ^~~~~~

Przypisujesz do `liczbaCukierkow` wskaźnik na zmienną tymczasową. Nie ma to wiele sensu. (W zadaniu nie chodzi aby o to, by ta klasa trzymała jakąś zmienną zaalokowaną dynamicznie pod tym adresem?)

<source>:23:25: warning: assigning field to itself [-Wself-assign-field]
    this->kolorCukierka = kolorCukierka;
                        ^
<source>:24:27: warning: assigning field to itself [-Wself-assign-field]
    this->liczbaCukierkow = liczbaCukierkow;
                          ^

Przipisujesz pole do samego siebie. Nie wiem czemu. W końcu linię wyżej już ustawiłeś wartości zmiennych.