Home Upgrade Search Memberlist Extras Hacker Tools Award Goals Help Wiki Contact
>

.NET 

Simple Calculator! (My First Project)

Considering this is my first project I wanted to share this in hopes you would give me feedback and if I should improve on anything.
I do like when I get received feedback weather it be good or bad.

Virus Scan: https://www.virustotal.com/en/file/b55da.../analysis/

The Download(Executable): http://ge.tt/5pjj4452
The Source: http://ge.tt/5v6O7452/v/0
Screenshot:
Spoiler (Click to View)
You're asking for feedback and only post an executable without code? Lol.

I'm not going to waste my time decompiling it, so I can give you the feedback you want.

Haven't downloaded, but;
Try finding out what the NumericUpDown control is for. Make sure you always check for valid input to prevent crashes, like I demonstrated in your other thread.
(Nov 22, 2014 05:31 PM)Dialog Wrote:  You're asking for feedback and only post an executable without code? Lol.

I'm not going to waste my time decompiling it, so I can give you the feedback you want.

Haven't downloaded, but;
Try finding out what the NumericUpDown control is for. Make sure you always check for valid input to prevent crashes, like I demonstrated in your other thread.

Sorry dude XD
Here is the source:
http://ge.tt/5v6O7452/v/0
Downloading now. Gonna see what its like. are you okay with me adding this to a compilation I am currently making?
All good rhyno
I'm fine with that xD
Try actually naming your textboxes, radio buttons, labels etc.
The code gets confusing pretty quickly when making larger applications if you don't.
It's a good habit to get into.
(Nov 22, 2014 06:22 PM)Achievement Hunter Wrote:  All good rhyno
I'm fine with that xD

Alright. Keep looking out for it, It should be done tonight.
I suggest using the Hungarian Notation naming method which works like this for a button- "btnName"

Also you can assign x and y at the beginning for the data from the text boxes and then reuse them for each operator.

If you want to be able to use decimals then I suggest using the double data type and then format your .ToString as .ToString("N2") which means number to two decimal places.
(Nov 22, 2014 06:40 PM)Heisenberg℠ Wrote:  I suggest using the Hungarian Notation naming method which works like this for a button- "btnName"

Also you can assign x and y at the beginning for the data from the text boxes and then reuse them for each operator.

If you want to be able to use decimals then I suggest using the double data type and then format your .ToString as .ToString("N2") which means number to two decimal places.

Although I agree controls should always be renamed, Hungarian notation is old and usage is discouraged by Microsoft's latest conventions.
I add a suffix to my identifiers for controls by convention, e.g. "registerButton" or "usernameTextBox".

See http://msdn.microsoft.com/en-us/library/...10%29.aspx


Possibly Related Threads...
Thread / Author Replies Views Last Post
[Python] by LSDJul 15, 2020 02:55 AM 2 194 Jul 15, 2020 02:55 AM
Last Post: greasyAlessandra
[C#] by RackenNov 8, 2019 12:59 PM 2 282 Nov 8, 2019 12:59 PM
Last Post: 0xDEAD10CC
[.NET] by Dir50Jun 5, 2018 08:56 AM 9 315 Jun 5, 2018 08:56 AM
Last Post: 0xDEAD10CC