Conversion From Decimal To Octal

Question

1.00/5 (1 vote)

See more:

Guys I have made a convertor that converts a decimal number to a octal number in VB.NET,but it converts only upto 7 digits of decimal number to Octal.If it exceeds 7 digits "Memory Overflow" message is displayed.How To solve this problem guys?

Here Is The Code:

VB

Dim s1,s2 As Atring
Dim i1,i2 As Integer

s1=textbox1.Text
i1=Convert.ToInt64(s1, 10)
s2=Convert.ToString(i1, 8)
i2=Convert.ToInt64(s2, 8)
MsgBox(s2)

but this code is limited only up to 7 characters of input.

Posted 21-Sep-13 19:07pm

srinivasankrishnaa

Updated 21-Sep-13 22:56pm

Akinmade Bond

v4

Add a Solution

Comments

phil.o 22-Sep-13 1:49am

You may begin with showing us the code you wrote :)

Sergey Alexandrovich Kryukov 22-Sep-13 12:11pm

Do not use "Convert" is such cases. Use Integer.Parse and System.Object.ToString (with parameters is required). Even if it works the same, it tells you explicitly what you are doing, which is not really "conversion".
—SA

2 solutions

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

OriginalGriff · Answer 1 · 2013-09-21T20:45:00

Solution 2

Without seeing your code, we can't be that helpful.
phil.o has given you instruction in how to start debugging your application, but...did you realise that .NET can do this for you?

C#

Dim inp As Integer = 1234567890
Dim oct As String = Convert.ToString(inp, 8)
Console.WriteLine("The Octal value of {1} is {2}", inp, oct)

Posted 21-Sep-13 20:45pm

OriginalGriff

Comments

phil.o 22-Sep-13 3:25am

Fair enough ; but solving (and learning to solve) the actual problem could be more useful for the OP than using a predefined method :)

OriginalGriff 22-Sep-13 3:29am

Oh, I totally agree - that's why I referred to your post - but it's worth knowing when you are reinventing the wheel as well! :laugh:

phil.o 22-Sep-13 3:33am

Unless someone feels a ~~elliptical~~ chaotic random-shaped wheel could be useful to humanity ;)

OriginalGriff 22-Sep-13 4:30am

Well, now you mention it... :laugh:
http://www.velobase.com/CompImages/CrankRing/B7C758B9-C6D9-4D86-B3BD-9E82CB4ED9CC.jpeg
A keen cyclist friend of mine used to swear by these in the 80s - effectively it changed the gearing to match the angle of your leg to the pedal and evened out the power of the stroke.

phil.o 22-Sep-13 4:48am

Wow! Did you friend survive? :D

OriginalGriff 22-Sep-13 4:55am

Regrettably not: the Big C got him in the 90s... :sigh:

phil.o · Answer 2 · 2013-09-21T20:37:00

Without any code provided, the only solution left for you is to:
- put a beakpoint at the beginning of the method handling the conversion.
- lauch your project in debug mode (F5 in Visual Studio).
- go on from line to line and watch carefully for the values of your variables ; you have to wonder what it is you expect for them, and what you actually get; the difference will point the problem.

[edit]

Seeing your code, then Solution 2 by Original Griff is the one that does exactly what you want to do the way you want to do it, but without limitations.

I would just change the way you get your Int32 value from your textBox:

VB

If (Int32.TryParse(s1, i1))
   s2 = Convert.ToString(i1, 8)
End If

[/edit]