Point et Ligne de classe en c++?

Vous ne devriez pas utiliser des pointeurs à tous dans votre code. Utiliser des objets réels. Les pointeurs sont effectivement utilisé très rarement en C++.

class Point
{
    private:
        int x, y;
    public:
        Point() : x(0), y(0) {}
        Point(int x, int y) : x(x), y(y) {}
}

class Line
{
    private:
        Point p1;
        Point p2;
    public:
        Line(const Point & p1, const Point & p2 ) : p1(p1), p2(p2) {}

        void setPoints( const Point & ap1, const Point & ap2)
        {
            p1 = ap1;
            p2 = ap2;
        }
}

OriginalL'auteur

De savoir si ou non le Point de membres de votre ligne de la classe sont des pointeurs ou pas crée un type très différent de la classe. En utilisant les pointeurs entraînera un classique CoGo approche de style, qui peuvent être considérés comme des points comme des clous dans une planche, et les lignes qui sont des bandes de caoutchouc de la connexion de ces clous. La modification d'un point, c'est comme le déplacement d'un clou, tous les associés de la ligne fonctionne automatiquement suit, ce qui est souhaitable dans certaines applications.

Utilisation littérale points signifie que les lignes sont toutes indépendantes l'une de l'autre, ce qui est approprié pour les autres types de demande.

C'est une critique de la conception à la décision pour vos classes à ce stade.

Edit: Comme indiqué dans d'autres posts et les commentaires ci-dessous, en utilisant de simples pointeurs pour réaliser l'association entre plusieurs lignes et de points présente également un grave problème potentiel. Plus précisément, si un point est supprimé ou déplacé dans la mémoire, tous les liens se référant à ce point doit être mis à jour. Dans la pratique, cela a tendance à être surmontés par l'utilisation unique Id de point à point, plutôt que de simples pointeurs. Cela permet aussi à l'CoGo structures pour être facilement sérialisé /sauvé. Ainsi, notre classe point aurait un membre statique pour obtenir un point basée sur l'ID, et notre ligne de classe aurait deux Id de points plutôt que des pointeurs.

OriginalL'auteur SmacL

Il n'est pas nécessaire d'utiliser des pointeurs dans votre Ligne de la classe.

Également, la ligne suivante est incorrecte:

Line(Point &p1, Point &p2) : p1(p1), p2(p2) {}

Pourquoi? L'affectation d'un Point & (p1) à une Point * (Ligne::p1), ce qui est illégal. Vous voulez des pointeurs.

Votre classe Point n'a aucun moyen de modifier ses données. Pas trop utile...

Une Ligne de classe pour me ressemblerait à quelque chose comme ceci:

class Line
{
    private:
        Point p1, p2;

    public:
        Line()
        {
        }

        Line(Point start, Point end) :
            p1(start), p2(end)
        {
        }

        const Point &startPoint() const
        {
            return p1;
        }

        Point &startPoint()
        {
            return p1;
        }

        const Point &endPoint() const
        {
            return p2;
        }

        Point &endPoint()
        {
            return p2;
        }
};

Vous pouvez maintenant modifier votre ligne comme ceci:

Line line;
line.startPoint() = Point(4, 2);
line.endPoint() = Point(6, 9);

OriginalL'auteur strager

Je vois que peu de valeur au Point de pointeurs (autres que pour l'ironie de la valeur). Votre Point est 8 octets sur un système 32 bits (2 int). Un pointeur prend 4 octets. vous êtes en train d'enregistrer 4 octets.

Que l'exactitude de votre Ligne de constructeur de références, mais vous êtes en les assignant à des pointeurs. Qui ne devrait même pas compiler. Vous êtes également fait la même chose dans les points de consigne. Il serait préférable de simplement faire les deux points d'objets réels et de la copie de leurs valeurs.

OriginalL'auteur Erik Funkenbusch

Le compilateur va donner des erreurs sur ce code:

1. Dans la liste d'initialisation du constructeur de lignes, vous affectez Point de référence p1 membre de classe p1, qui est un pointeur vers le Point. Pour obtenir cette compilation, vous devez utiliser la Ligne(Point &p1, Point &p2) : p1(&p1), p2(&p2).
2. Le même problème se produit dans les points de réglage de la méthode. Que shoudl être changé à cette question->p1 = &p1, etc.
3. Problème mineur: après la clôture } de la classe, mettre dans un point-virgule (;)

Je pense que conclut la syntaxe.

Il y a un autre problème avec le code:

P1 et p2 les membres de la classe de Ligne sont des pointeurs sur des Point d'instances. Qui crée ces instances et qui va les supprimer quand ils ne sont plus nécessaires? Si vous concevez votre classe comme elle l'est maintenant, le code qui crée une instance de Ligne de passe de deux instances du constructeur. Si ces instances sont supprimés avant la Ligne, la Ligne est à gauche avec deux pendants de pointeurs (les pointeurs qui ne font plus référence à une instance valide).

Aussi, les deux instances qui font maintenant partie de la Ligne peut être modifiée à partir du code en dehors de la Ligne de la classe. Cela peut aboutir à des situations indésirables.

Dans cette situation, je voudrais déclarer p1 et p2 de membres que le Point (au lieu d'un pointeur). De cette façon, le Point d'instances qui sont transmis au constructeur sont copiés sur les membres de la classe. Aussi longtemps que la Ligne existe, le Point membres. Ils ne peuvent être modifiées par la ligne de la classe elle-même.

OriginalL'auteur Renze de Waal

Utiliser des objets dans votre Ligne de la classe. La propriété des points n'est pas clair et vous risque de se retrouver avec balançant des pointeurs ou des fuites de mémoire.

Votre constructeur par défaut est un problème que vous aurez rarement voulez créer un Point de coordonnées (0,0). Vous êtes mieux de réglage de la valeur par défaut valeurs x,y à quelque chose comme exemple maxint et en affirmant que toute utilisation du Point de ne pas disposer d'exemple maxint comme l'un de ses valeurs. Avoir un is_valid() la fonction de sorte que les clients peuvent tester. Votre Ligne de la classe peut également avoir une condition que les deux points ne sont pas valides. Le payer de ne pas permettre un bon point pour avoir un exemple maxint valeur, c'est que vous pouvez détecter si les Points n'ont pas été initialisé correctement, et vous aurez zap certains sinon difficile de trouver des bugs dans la classe d'utilisation.

OriginalL'auteur lilburne