Discussion :

[dhoorens]

Bonjour, ca fait deux jours que je me tire les cheveux sur ce problème..
Voici le code:

Fichier "Curve3.h"

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
 
#pragma once
 
#include "Config.h"
#include "Vector.h"
 
class Curve
{
public:
	class Point
	{
	private:
		double x_;
		double y_;
	public:
		Point () : x_(0.0),	y_(0.0){}
		Point (const double x, const double y) :x_(x), y_(y){}
		void x(const double x) {x_ = x;}     ///////ICI//////
		void y(const double y) {y_ = y;}    ///////ET ICI ///////
	};
 
	Curve (){};
	Curve (const vector<double>& vx, const vector<double>& vy)
	{
		if (vx.size() == vy.size())
		{
			for (int i =1; i < vx.size();i++)
			{
				v_[i].x(vx[i]);
				v_[i].y(vy[i]);
			}
		}
	}
private:
	vector<Point> v_;
};

et voici le main

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
 
#pragma once
#include "Config.h"
#include "Vector.h"
#include "Curve3.h"
void main()
{	
	int i;
	vector<double> x2(5);
	vector<double> y2(5);
	for (i =0; i < x2.size(); i++)
	{ 
		x2[i] = 200 + 10 + i;
		y2[i] = 200 + 20 + i;
	}
	Curve c2(x2,y2);
};

Je travaille sous winxp + visual studio 2003

Lorsque je compile, il me donne une erreur sur Curve:: Point:: x() et Curve:: Point:: y()
Je ne comprends vraiment pas. Il n'y a rien de spécial quand meme???
Si qqun sait m'aider, ce serait vraiment sympa...
Tks
dh

[Laurent Gomila]

Tu aurais pu donner le message d'erreur...

Je dirais que le compilo râle parce que le paramètre et la fonction portent tous deux le même nom.

[dhoorens]

oups pardon.. En fait l'erreur est dans le titre:
"Access violation writing location 0x00000010"

J'ai déjà essayé avec setx et sety mais rien n'y fait,j'ai le meme message..
J'avoue que je ne comprends rien..
Rassure moi , mon code n'est quand meme pas si fou que ca??

[Laurent Gomila]

C'est une erreur d'exécution, pas de compilation.

Tu ne dimensionnes pas v_ avant d'accéder à ses éléments (dans le constructeur de Curve), utilise push_back ou dimensionne le avec le constructeur adéquat.

[dhoorens]

Effectivement
il faut coder ceci

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
	Curve (const vector<double>& vx, const vector<double>& vy)
	{
		if (vx.size() == vy.size())
		{
                                       v_.resize(vx.size());			                          for (int i =1; i < vx.size();i++)
			{
				v_[i].x(vx[i]);
				v_[i].y(vy[i]);
			}
		}
	}

et ca marche

Merci

[Laurent Gomila]

Pour pinailler, ceci serait un poil meilleur :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
Curve (const vector<double>& vx, const vector<double>& vy)
{
    if (vx.size() == vy.size())
    {
        for (int i = 1; i < vx.size(); i++)
        {
            v_.push_back(Point(vx[i], vy[i]));
        }
    }
}

Au fait, pourquoi ta boucle démarre à l'indice 1 et non 0 ?

[dhoorens]

pcq il s'agit d'une faute... elle commence au 0 :-)

Merci beaucoup pour le pinaillage, je vais l'utiliser