C# constructor problem

[zikje]

Hello,

I've got a really weird problem.

public Map(float zoomFactor, int panStep, float maxWidth, float minWidth,
Document svgDocument, string svgId)
{
this.zoomFactor = zoomFactor;
this.panStep = panStep;
this.maxWidth = maxWidth;
this.minWidth = minWidth;
this.svgDocument = svgDocument;
this.svgId = svgId;
}

and

this.m_map = new
Map(0.6f,15,1477f,50f,m_esvgControl.GetDocument(), "svgmap");
None of the values I fill in, are taken over by the instance of Map. I don't
get this. This is like the same for every programming language, and as far as
I can tell from books and examples, also counts for C#. What am I doing wrong?

[Ollie Riches]

This is because the parameter list for the constrcutor method have EXACTLY
the same names as the Map class variable names. In other words you are not
doing an assignment of the constructor parameter to the class variable but
the assigning the class variable to the class variable :)

Somthing like this should 'work'
public Map(float zf, int ps, float maxw, float minw, Document svgdoc, string
si)
{
this.zoomFactor = zf;
this.panStep = ps;
this.maxWidth = maxw;
this.minWidth = minw;
this.svgDocument = svgdoc;
this.svgId = si;
}
HTH

Ollie Riches


"zikje" <zi***@discussions.microsoft.com> wrote in message
news:B8**********************************@microsof t.com...

Hello,

I've got a really weird problem.

public Map(float zoomFactor, int panStep, float maxWidth, float minWidth,
Document svgDocument, string svgId)
{
this.zoomFactor = zoomFactor;
this.panStep = panStep;
this.maxWidth = maxWidth;
this.minWidth = minWidth;
this.svgDocument = svgDocument;
this.svgId = svgId;
}

and

this.m_map = new
Map(0.6f,15,1477f,50f,m_esvgControl.GetDocument(), "svgmap");
None of the values I fill in, are taken over by the instance of Map. I
don't
get this. This is like the same for every programming language, and as far
as
I can tell from books and examples, also counts for C#. What am I doing
wrong?

[Peter Kirk]

"Ollie Riches" <ol**********@phoneanalyser.net> skrev i en meddelelse
news:Om**************@TK2MSFTNGP11.phx.gbl...

This is because the parameter list for the constrcutor method have EXACTLY
the same names as the Map class variable names. In other words you are not
doing an assignment of the constructor parameter to the class variable but
the assigning the class variable to the class variable :)

I don't think that is the problem. I do it almost all the time:

public MyClass(int a, string b) {
this.a = a;
this.b = b;
}

should work fine (note the use of "this").

Can't say I can see what the OP's problem is though....

[Joanna Carter [TeamB]]

"zikje" <zi***@discussions.microsoft.com> a écrit dans le message de news:
B8**********************************@microsoft.com...

| public Map(float zoomFactor, int panStep, float maxWidth, float minWidth,
| Document svgDocument, string svgId)
| {
| this.zoomFactor = zoomFactor;
| this.panStep = panStep;
| this.maxWidth = maxWidth;
| this.minWidth = minWidth;
| this.svgDocument = svgDocument;
| this.svgId = svgId;
| }
|
| and
|
| this.m_map = new
| Map(0.6f,15,1477f,50f,m_esvgControl.GetDocument(), "svgmap");
| None of the values I fill in, are taken over by the instance of Map. I
don't
| get this. This is like the same for every programming language, and as far
as
| I can tell from books and examples, also counts for C#. What am I doing
wrong?

Well, I just copied your code into a new module, added private fields and
public properties to read the values and it compiled and ran as expected;
both in debug mode and at runtime.

Could you show more code of how you are using the class ?

Joanna

--
Joanna Carter [TeamB]
Consultant Software Engineer

[Oliver Sturm]

Ollie Riches wrote:

This is because the parameter list for the constrcutor method have EXACTLY
the same names as the Map class variable names. In other words you are not
doing an assignment of the constructor parameter to the class variable but
the assigning the class variable to the class variable :)

Somthing like this should 'work'
public Map(float zf, int ps, float maxw, float minw, Document svgdoc,
string si)
{
this.zoomFactor = zf;
this.panStep = ps;
this.maxWidth = maxw;
this.minWidth = minw;
this.svgDocument = svgdoc;
this.svgId = si;
}

This is not true. I use the same syntax (as the OP) all the time -
prefixing the field instances of the names with "this." should make it
work correctly.

I don't see anything wrong with the code as shown. The problem has to be
somewhere else, like in the definition of the fields. Could we see more of
the code of that Map class?
Oliver Sturm
--
Expert programming and consulting services available
See http://www.sturmnet.org (try /blog as well)

[Ollie Riches]

I stand corrected :)

"Oliver Sturm" <ol****@sturmnet.org> wrote in message
news:xn****************@msnews.microsoft.com...

Ollie Riches wrote:

This is because the parameter list for the constrcutor method have EXACTLY
the same names as the Map class variable names. In other words you are not
doing an assignment of the constructor parameter to the class variable but
the assigning the class variable to the class variable :)

Somthing like this should 'work'
public Map(float zf, int ps, float maxw, float minw, Document svgdoc,
string si)
{
this.zoomFactor = zf;
this.panStep = ps;
this.maxWidth = maxw;
this.minWidth = minw;
this.svgDocument = svgdoc;
this.svgId = si;
}

This is not true. I use the same syntax (as the OP) all the time -
prefixing the field instances of the names with "this." should make it
work correctly.

I don't see anything wrong with the code as shown. The problem has to be
somewhere else, like in the definition of the fields. Could we see more of
the code of that Map class?
Oliver Sturm
--
Expert programming and consulting services available
See http://www.sturmnet.org (try /blog as well)

[Matt]

zikje wrote:

Hello,

I've got a really weird problem.

public Map(float zoomFactor, int panStep, float maxWidth, float minWidth,
Document svgDocument, string svgId)
{
this.zoomFactor = zoomFactor;
this.panStep = panStep;
this.maxWidth = maxWidth;
this.minWidth = minWidth;
this.svgDocument = svgDocument;
this.svgId = svgId;
}

and

this.m_map = new
Map(0.6f,15,1477f,50f,m_esvgControl.GetDocument(), "svgmap");
This looks fine.
None of the values I fill in, are taken over by the instance of Map. I don't
get this. This is like the same for every programming language, and as far as
I can tell from books and examples, also counts for C#. What am I doing wrong?

I'm guessing that you are reassigning the m_map variable after you do
this, and that's resetting all of the values. Can you show the rest of
the code? There's nothing wrong with your constructor or code as shown.

Matt

[Bruce Wood]

One very important thing that you don't show is whether Map is a class
or a struct. If it's a struct, you may be doing something after the
original instantiation that is causing your problem.

Could you post one of Jon Skeet's famous "short but complete programs"
to demonstrate what you're seeing? See Jon's page here:

http://www.yoda.arachsys.com/csharp/complete.html

for instructions. :)