fundamental javascript question

Question

I declared a global:

var myClient;

I have a function:

//Contact is a record object that has toString() which prints the name.
function getClient() {
   myClient = new Object();
   debug(input.contact); // This prints properly
   myClient.contact = input.contact;
   debug(myClient.contact); // This prints properly
}

I have another function that is trying to use the same:

function dispatchClient() {
    debug(myClient.contact);
}

And the result I see is undefined. Is something wrong here? (Ignoring the design aspect)

If that is wrong, then how can I pass the state of global between functions? If that is not wrong, then hmm, I may need to dig deeper!

You haven't shown us how you are calling the functions, and you haven't shown us how input is defined. — Quentin
– Quentin, Commented Apr 29, 2011 at 18:30
javascript is case sensitive i.e. myclient and myClient are different names. — Dan D.
– Dan D., Commented Apr 29, 2011 at 18:30
Are you calling getClient() before you call dispatchClient()? This works fine: jsfiddle.net/f7xbK. — mellamokb
– mellamokb, Commented Apr 29, 2011 at 18:31

Imran · Accepted Answer · 2011-04-29 18:30:56Z

2

dispatchClient is probably being called before getClient gets called. At that point, myClient would be still undefined.

answered Apr 29, 2011 at 18:30

Imran

91.7k26 gold badges101 silver badges132 bronze badges

Sign up to request clarification or add additional context in comments.

6 Comments

Kiran Over a year ago

If 'input' is destroyed after 'getClient' function ends, would myClient still be valid?

Imran Over a year ago

If 'input.name' is String or Number, then yes

Kiran Over a year ago

Thanks, what if it is an object that has sub properties?

mVChr Over a year ago

Yes, because you're setting the name prop, which I assume is a string and not an object. If you were assigning the whole input object to myClient, or if input.name is an object then it would be a problem because objects are passed by reference.

sirlark Over a year ago

Agreed, can't think of a reason this wouldn't work unless dispatchClient is called first. If these functions are attached as event handlers then it's entirely possible they are being called in the wrong order. If this is the case, then do a check for a null myClient value in dispatch, and if it is null, call getClient to set it up before doing anything else

|

Headshota · Accepted Answer · 2011-04-29 18:32:52Z

0

myclient and myClient are two different variables.

but code must still work. unless you call dispatchClient before getClient function.

answered Apr 29, 2011 at 18:32

Headshota

21.4k13 gold badges63 silver badges82 bronze badges

1 Comment

Kiran Over a year ago

Sorry, not me. Up voted to correct against my spelling mistake

Vismari · Accepted Answer · 2011-04-29 18:35:56Z

0

If you are calling getClient() before dispatchClient() *myClient* variable would be undefined.

Other possiblity is the name(case sensitive), in dispatchClient() you are using my*C*lient that is diferent than the global variable my*c*lient.

If possible(depend on your code), create a instance at the beginning:

var myclient = new Object();

answered Apr 29, 2011 at 18:35

Vismari

7453 silver badges12 bronze badges

Comments

ShaneBlake · Accepted Answer · 2011-04-29 18:36:29Z

0

Your var is myclient while your using the object later as myClient. Javascript is case sensitive, so these arn't the same variable.

answered Apr 29, 2011 at 18:36

ShaneBlake

11.1k2 gold badges28 silver badges43 bronze badges

Collectives™ on Stack Overflow

fundamental javascript question

4 Answers 4

6 Comments

1 Comment

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

6 Comments

1 Comment

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related