const

  • Thread starter Thread starter shapper
  • Start date Start date
S

shapper

Hello,

I have a class, TweetService, with many methods which all use a
Twiiter object:

public class TwitterService : ITwitterService {

private const Twitter twitter = new Twitter("myusername",
"mypassword");

public void Create(Tweet tweet) {
TwitterStatus status = twitter.Status.Update
(tweet.Text);
} // Create

// Other methods
}

Is it correct to define it as a const?
Or should I just define it as follows:

private Twitter twitter;

And then initialized it on the TwitterService constructor:

public TwitterService() {
twitter = new Twitter("myusername", "mypassword");
} // TwitterService

Or some other way?

I just want twitter to be available to all methods and not having to
define it all the time and having the username and password every
where.

Thanks,
Miguel
 
shapper said:
Hello,

I have a class, TweetService, with many methods which all use a
Twiiter object:

public class TwitterService : ITwitterService {

private const Twitter twitter = new Twitter("myusername",
"mypassword");

public void Create(Tweet tweet) {
TwitterStatus status = twitter.Status.Update
(tweet.Text);
} // Create

// Other methods
}

Is it correct to define it as a const?
Or should I just define it as follows:

private Twitter twitter;

And then initialized it on the TwitterService constructor:

public TwitterService() {
twitter = new Twitter("myusername", "mypassword");
} // TwitterService

Or some other way?

I just want twitter to be available to all methods and not having to
define it all the time and having the username and password every
where.

Thanks,
Miguel
Click to expand...

I believe you must use the second option, as const cannot be used on
references. Didn't you get an error trying the first method?

Mike
 
Hello,

I have a class, TweetService, with many methods which all use a
Twiiter object:

public class TwitterService : ITwitterService {

private const Twitter twitter = new Twitter("myusername",
"mypassword");

// Other methods
}

Is it correct to define it as a const?
Click to expand...

Try compiling it and see.
Or should I just define it as follows:

private Twitter twitter;
Click to expand...

Do you want a different value for the variable for each instance of
TwitterService? If so, the above is exactly right. If not, you should
make the value "static".
And then initialized it on the TwitterService constructor:

public TwitterService() {
twitter = new Twitter("myusername", "mypassword");
} // TwitterService
Click to expand...

If it's an instance member, you can initialize it using the initializer
syntax you showed originally.

Pete
 
shapper said:
Hello,

I have a class, TweetService, with many methods which all use a
Twiiter object:

public class TwitterService : ITwitterService {

private const Twitter twitter = new Twitter("myusername",
"mypassword");

public void Create(Tweet tweet) {
TwitterStatus status = twitter.Status.Update
(tweet.Text);
} // Create

// Other methods
}

Is it correct to define it as a const?
Click to expand...

No. C# "const" is pretty different from what const means in other
languages. I think what you really want is "static readonly".
 
Back
Top