Re: OdbcDataReader calling IsDBNull changes the behaviour of Get methods - C# Discussion Boards

Re: OdbcDataReader calling IsDBNull changes the behaviour of Get methods

__John_3-Jan-12 5:22

3-Jan-12 5:22

Hi Dan,

Using the indexer, casting to an int always works and casting to a string always fails (invalid cast).

So the behavour is predictable and my problem 'kind of' goes away. So for now I will use the indexer and not use the Get???? methods.

@PIEBALDconsult: You mentioned indexers earlier but I was not sure what you were getting at. Turns out you were on to something. Thanks.

Re: OdbcDataReader calling IsDBNull changes the behaviour of Get methods

Dan Mos3-Jan-12 7:08

Dan Mos

3-Jan-12 7:08

Again you used nColNumber twice. You should have used nColNumber and nColName.
That was your problem. And yeah I don't use GetXXX either. But that wasn't what caused the exception.

You tried to get a string from an int DB value or a int form a varchar DB value courtesy
of using twice nColNumber.

All the best,

Dan

Re: OdbcDataReader calling IsDBNull changes the behaviour of Get methods

PIEBALDconsult3-Jan-12 4:00

PIEBALDconsult

3-Jan-12 4:00

Well, personally, I never use the Get methods anyway; I just use the indexers.

__John_ wrote:
odbcDataReader.IsDBNull(nColNumber);

Shouldn't you be testing the result?

Re: OdbcDataReader calling IsDBNull changes the behaviour of Get methods

Dan Mos3-Jan-12 4:31

Dan Mos

3-Jan-12 4:31

What is the query that gets executed on the DB while doing a cmd.ExecuteReader()?
Maybe there is something missing/wrong there.

All the best,

Dan

Re: OdbcDataReader calling IsDBNull changes the behaviour of Get methods

__John_3-Jan-12 4:38

__John_

3-Jan-12 4:38

Hi Moshu,

The command that I execute is as follows..

odbcSelectCommand.CommandText = @"SELECT name, iq FROM Table_People WHERE iq < 3 ORDER BY iq";

I get the 'name' and 'iq' but I am only concurned with 'iq' for now.

Re: OdbcDataReader calling IsDBNull changes the behaviour of Get methods

Dan Mos3-Jan-12 4:44

Dan Mos

3-Jan-12 4:44

OK. Heres what I would do. Run that query in SQLManagementStudio and check the results.
Are there any? Or some with a field null or...

All the best,

Dan

Re: OdbcDataReader calling IsDBNull changes the behaviour of Get methods

__John_3-Jan-12 4:58

__John_

3-Jan-12 4:58

Hi MDL=Moshu, thanks for looking at this.

The code now looks like this...

int nColName   = odbcDataReader.GetOrdinal(@"name");
    int nColNumber = odbcDataReader.GetOrdinal(@"iq");

    while (odbcDataReader.Read())
    {
        if(odbcDataReader.IsDBNull(nColNumber))
        {
            continue;
        }

        int nNumber = odbcDataReader.GetInt32(nColNumber);

        string strNumber = odbcDataReader.GetString(nColNumber);

    }

No nulls are returned and the behaviour is the same ie. with the call to IsDBNull() GetString() fails, and without it, GetInt32() fails.

Re: OdbcDataReader calling IsDBNull changes the behaviour of Get methods

Dan Mos3-Jan-12 5:06

Dan Mos

3-Jan-12 5:06

This is my final reply. First check my post below.

Second you're using nColNumber for both the name and the iq.
That's not right and I pointed it out on my first post to this thread.

If for some reason you want a string representation of the number/iq then
just use strNumber = nNumber.ToString();

Hope it helps. I'm out of CP for today. Going to program some beers now.
Smile | :)

All the best,

Dan

Re: OdbcDataReader calling IsDBNull changes the behaviour of Get methods

Dan Mos3-Jan-12 4:53

Dan Mos

3-Jan-12 4:53

Also since you are not using a Select * statement, I would drop the lines of code that get the ordinals.

You know them name is 0 and iq is 1 so in a nutshell:

     while(dr.Read()){
     string name = dr[0].ToString();
     int iq = (int)dr[1];
}

All the best,

Dan

Re: OdbcDataReader calling IsDBNull changes the behaviour of Get methods

__John_3-Jan-12 5:02

__John_

3-Jan-12 5:02

Thanks dan,

But I think my way is more robust and future proof.
ie. it will still work even with 'SELECT *'.

- John

Re: OdbcDataReader calling IsDBNull changes the behaviour of Get methods

Dan Mos3-Jan-12 5:09

Dan Mos

3-Jan-12 5:09

True. Then use dr["name"] and dr["iq"] and it will be future proof.

All the best,

Dan

Re: OdbcDataReader calling IsDBNull changes the behaviour of Get methods

__John_3-Jan-12 5:29

__John_

3-Jan-12 5:29

Hi MDL=>Moshu,

Using an indexer as you sagest (with column name) is fine if you only have a small number of fields, but if the number of fields is higher then I think you will take a performance hit.

Re: OdbcDataReader calling IsDBNull changes the behaviour of Get methods

Luc Pattyn3-Jan-12 5:50

Luc Pattyn

3-Jan-12 5:50

__John_ wrote:
my way is more robust

That doesn't sound very convincing as your code does not produce the expected results to begin with.

Have you already figured nColNumber is appearing too many times in your code? (courtesy MDL)

OMG | :OMG:

Luc Pattyn [My Articles] Nil Volentibus Arduum

Re: OdbcDataReader calling IsDBNull changes the behaviour of Get methods

PIEBALDconsult4-Jan-12 13:41

PIEBALDconsult

4-Jan-12 13:41

MDL=>Moshu wrote:
string name = dr[0].ToString();

It's already a string, just cast it -- string name = (string) dr[0]; , no need to call a method.

Re: OdbcDataReader calling IsDBNull changes the behaviour of Get methods

Dan Mos7-Jan-12 12:07

Dan Mos

7-Jan-12 12:07

True.

Is there a performance reason too, or just style?
Cause if there is one, I might need to change some lines of code. I've always happily used ToString() on
datareader where the element is a text/string.
Smile | :)

All the best,

Dan

Re: OdbcDataReader calling IsDBNull changes the behaviour of Get methods

PIEBALDconsult7-Jan-12 13:22

PIEBALDconsult

7-Jan-12 13:22

There must be at least a small overhead in calling the method.

Re: OdbcDataReader calling IsDBNull changes the behaviour of Get methods

Ennis Ray Lynch, Jr.3-Jan-12 7:41

Ennis Ray Lynch, Jr.

3-Jan-12 7:41

Your code changed. When you test things you should change 1 item at a time, however, you have also changed order and type. In your last example, you are calling GetString and GetInt32 on nColNumber which refers to an integer, your code should read:

int nameColumn = odbcDataReader.GetOrdinal(@"name");
int idColumn = odbcDataReader.GetOrdinal(@"iq");
while (odbcDataReader.Read()){
        odbcDataReader.IsDBNull(idColumn);
        int iqValue= odbcDataReader.GetInt32(idColumn);
        string nameValue = odbcDataReader.GetString(nameColumn );
}

I just typed it in the post window but you can see how different variable names make it easier to see.

Need custom software developed? I do custom programming based primarily on MS tools with an emphasis on C# development and consulting. I also do Android Programming as I find it a refreshing break from the MS.

"And they, since they Were not the one dead, turned to their affairs" -- Robert Frost

Deployment

arkiboys3-Jan-12 2:52

arkiboys

3-Jan-12 2:52

Hi,
We are using VSS for our windows c# projects...
What is the best and easiest way to monitor the changeRequests with the deployment versions?
Currently, each developer is working on a changerequest i.e. add a print button to FormA, etc...
Should there be a link between the version of the vss and the changerequest?, etc?

Thanks

Re: Deployment

manognya kota3-Jan-12 18:38

manognya kota

3-Jan-12 18:38

The general version control would be to use a Rationalclearcase tool or UCM.Are you looking at something other than version control tools?

Re: Deployment

arkiboys4-Jan-12 10:47

arkiboys

4-Jan-12 10:47

no, just using vss

Re: Deployment

manognya kota4-Jan-12 18:04

manognya kota

4-Jan-12 18:04

For a efficient version control and tracking its better to use the version control tools(Rational clear case/UCM ) this will give us the right code deployed at any instance.We can even roll back the CR's or merge the code.

Dealing only with VS is little tricky..as i doubt whether we can rollback the changes if you dont require the CR?

-Manognya.K
_________________________
$ God gives what is best.Not what all you wish Smile | :)

C# Asp.net application to run on all Internet browsers

MduduziM3-Jan-12 2:35

MduduziM

3-Jan-12 2:35

Hi All

I have a website developed using C# Asp.Net Framework 4.0 and currently the site menus are not functioning on Mozilla Firefox, Opera & Safari browsers. On these browsers I can see the site but Menus are not showing. The site is working perfect on Internet Explorer but I want it to work the same on other Internet browsers, please help.

Re: C# Asp.net application to run on all Internet browsers

Richard MacCutchan3-Jan-12 2:50

Richard MacCutchan

3-Jan-12 2:50

I do not think this is likely to be a C# issue.

Unrequited desire is character building. OriginalGriff
I'm sitting here giving you a standing ovation - Len Goodman

Re: C# Asp.net application to run on all Internet browsers

BillWoodruff3-Jan-12 2:54

BillWoodruff

3-Jan-12 2:54

This may sound like a "blow you off" response, but I assure you that's not my intent: you are proposing the equivalent of blood transfusions between different species.

The answer, I think you'll find, will be found in implementing client-side UI using jQuery, and/or other libraries that extend JavaScript.

best, Bill

"It is the mark of an educated mind to be able to entertain a thought without accepting it." Aristotle

Re: C# Asp.net application to run on all Internet browsers

BobJanova3-Jan-12 7:53

BobJanova

3-Jan-12 7:53

Make sure you use a menu control or partial view that is coded correctly. Unfortunately various parts of the non-MVC Microsoft control library isn't and it doesn't provide a good user experience (including some browser problems like this), and many third party controls are dubious, though if you have the source at least you can fix those.

Alternatively, write a menu control yourself, it's not that hard (I've done it in PHP before).

Use Ctrl+Left/Right to switch messages, Ctrl+Up/Down to switch threads, Ctrl+Shift+Left/Right to switch pages.