Check the value of dgi.Cells(2).Text, e.g. write the value out to the screen see what that value is.

-------------------------
I will only tell you how to do it, not do it for you.
Unless, of course, you want to hire me to do work for you.

^^Thats my signature

it gives the number only,
ie, for eg: 12

Hmmm not sure. How many times does your loop execute? It could be that there is an instance where you are passing a null string into decimal.parse or convert.todecimal and that is what is causing the errror.

-------------------------
I will only tell you how to do it, not do it for you.
Unless, of course, you want to hire me to do work for you.

^^Thats my signature

it just go through the datagrid items
but however, how can the unitprice be passed as zero?

but the dgi.cell(2).text returns number only(not without a $ sign)

I dont understand what you are asking.

-------------------------
I will only tell you how to do it, not do it for you.
Unless, of course, you want to hire me to do work for you.

^^Thats my signature

what i meant was how can there be chance of passing a null string? as the unit prices are all set
can you please help me to proceed from this error

You are making the assumption that your program logic (in regards to looping through the datagrid) is correct but now you are getting an error, you need to go at the problem as though your logic is incorrect and figure out why.

-------------------------
I will only tell you how to do it, not do it for you.
Unless, of course, you want to hire me to do work for you.

^^Thats my signature

sarah lee:
In addition to the above replies, one issue I noticed is your call to the FillCartFromDB(). In your source you have this routine declared as a FUNCTION, which by definition MUST return an answer, as in: MyCart=FillCartFromDB()
According to your source code listed it would be easier to change your FUNCTION statement to a SUBROUTINE statement.

Hope this helps.

thanks to all for all ur valuable advises

Expanding further on peace95s reply, in C# you CANNOT get away with declaring a function and not returning something as the compiler will spit out an error:

"Not all code paths return a value"


-------------------------
I will only tell you how to do it, not do it for you.
Unless, of course, you want to hire me to do work for you.

^^Thats my signature