• Status: Solved
  • Priority: Medium
  • Security: Public
  • Views: 1293
  • Last Modified:

Rounding of decimal to next highest whole number

This code is on a asp.net page using VB

I am trying to round of a fraction to the next highest whole number or integer. I found the following code on Experts, but I keep getting an error:
 BC30519: Overload resolution failed because no accessible 'InStr' can be called without a narrowing conversion
the doucumentation says:To correct this error Specify Option Strict Off.  how do I do this?

Is there better code?
my code:  
example: intshipunit will equal 1.15 in this test, I need it to change to 2

 dim intshipunit as decimal = strtotalweight / 20
Dim x As single
Dim y() As String
Dim z As single

x = intshipunit
If InStr(1, x, ".") Then
y() = Split(x, ".")
z = x - y(0)
x = x + (1 - z)
response.write (x)
End If


0
dplsr
Asked:
dplsr
  • 5
  • 4
  • 3
  • +2
1 Solution
 
AerosSagaCommented:
In the IDE Project->Properties->build
0
 
AerosSagaCommented:
there you will see option explicit, option strict, and option compare you can change these settings from here.
0
Never miss a deadline with monday.com

The revolutionary project management tool is here!   Plan visually with a single glance and make sure your projects get done.

 
dplsrAuthor Commented:
In the IDE Project->Properties->build

Is this something in Visual Studio?  I am not using visual studio if it is. Can i do this right on the page or in the sub?
0
 
AerosSagaCommented:
You can do this in the web.config file then:

<compilation> attribute

The <compilation> tag includes the following attributes:

    * defaultLanguage="C#|VB|C++" : Contains a list of language names to be used in dynamic compilation files.
    * debug="true|false" : Determines whether to compile in retail binaries or in debug binaries.
    * explicit="true|false" : Determines the setting of the Microsoft Visual Basic explicit compile option.
    * batch="true|false" : Determines whether batching is supported or not.
    * batchTimeout="number" : Determines the time-out period for batch compilation.
    * numRecompilesBeforeApprestart="number" : Determines the number of dynamic recompiles of resources that can happen before the application restarts.
    * strict="true|false" : Determines the setting of the Visual Basic strict compile option.

The code in Listing 1.6 shows compilation configuration settings:

<configuration>
   <system.web>
            <compilation defaultLanguage="C#"
                  debug="true"
                  numRecompilesBeforeAppRestart="10"
                  batchTimeout="10"
                  explicit="true"
                  strict="true"
                  batch="true"
            </compilation>
   </system.web>
</configuration>

http://www.codenotes.com/articles/articleAction.aspx?articleID=333
0
 
dplsrAuthor Commented:
I set strict="false" I still get the same error
0
 
raterusCommented:
Not to question the original poster of that code, but that is a horrid way to round up!

Use this instead: System.Math.Ceiling(intshipunit)

http://msdn.microsoft.com/library/default.asp?url=/library/en-us/cpref/html/frlrfsystemmathclassceilingtopic.asp

0
 
jnhorstCommented:
raterus is right (say that 10 times fast!).

InStr is old school VB and is replaced by IndexOf() in .NET, but for rounding up as you want to do, raterus' code is the way to do it.

John
0
 
dplsrAuthor Commented:
Hi raterus,

this looks better than what I had, but being a semi-newbie I  to VB I can't figure out how to apply it to

intshipunit
in
dim intshipunit as decimal = strtotalweight / 20

this is all used to set shipping charges on a web site
strtotalweight will vary per order, per number of items
0
 
Justin_WCommented:
System.Math.Ceiling(intshipunit) will only work if the number is non-negative.  A more general solution is:
   System.Math.Max(System.Math.Ceiling(intshipunit), System.Math.Floor(intshipunit))
0
 
Justin_WCommented:
I.e.:
   intshipunit = System.Math.Max(System.Math.Ceiling(intshipunit), System.Math.Floor(intshipunit))
0
 
dplsrAuthor Commented:
justin i should have split the points on that sorry! I did not notice the name changed!
0
 
Justin_WCommented:
That's OK.  I just remembered that Math.Ceiling is fine on its own unless you need the nearest whole number with the greatest absolute value.
0
 
dplsrAuthor Commented:
What I need is the next highest whole number. Like 1.15  changed to 2 . so far this works great
0

Featured Post

Free Tool: Subnet Calculator

The subnet calculator helps you design networks by taking an IP address and network mask and returning information such as network, broadcast address, and host range.

One of a set of tools we're offering as a way of saying thank you for being a part of the community.

  • 5
  • 4
  • 3
  • +2
Tackle projects and never again get stuck behind a technical roadblock.
Join Now