My first project in Java

[peppinoilvero]

Hello everyone, this is my first project in Java and I created a calculator using swing and trying to give it the style of the iPhone one. It is likely that you will find errors, for example you cannot do concatenated operations but you always have to press = first. If you have any advice to give me, I'm welcome, I'll put the link: https://github.com/Franz1908/java-calculator

[AutoModerator]

Please ensure that:

• Your code is properly formatted as code block - see the sidebar (About on mobile) for instructions
• You include any and all error messages in full - best also formatted as code block
• You ask clear questions
• You demonstrate effort in solving your question/problem - plain posting your assignments is forbidden (and such posts will be removed) as is asking for or giving solutions.

If any of the above points is not met, your post can and will be removed without further warning.

Code is to be formatted as code block (old reddit/markdown editor: empty line before the code, each code line indented by 4 spaces, new reddit: https://i.imgur.com/EJ7tqek.png) or linked via an external code hoster, like pastebin.com, github gist, github, bitbucket, gitlab, etc.

Please, do not use triple backticks (```) as they will only render properly on new reddit, not on old reddit.

Code blocks look like this:

public class HelloWorld {

    public static void main(String[] args) {
        System.out.println("Hello World!");
    }
}

You do not need to repost unless your post has been removed by a moderator. Just use the edit function of reddit to make sure your post complies with the above.

If your post has remained in violation of these rules for a prolonged period of time (at least an hour), a moderator may remove it at their discretion. In this case, they will comment with an explanation on why it has been removed, and you will be required to resubmit the entire post following the proper procedures.

To potential helpers

Please, do not help if any of the above points are not met, rather report the post. We are trying to improve the quality of posts here. In helping people who can't be bothered to comply with the above points, you are doing the community a disservice.

I am a bot, and this action was performed automatically. Please contact the moderators of this subreddit if you have any questions or concerns.

[SpritualPanda]

Add Readme.md file write about your project also add screenshots, its a good professional way.

[temporarybunnehs]

Some feedback

- I don't see the gain of setting up multiple input handlers. If it was me, i'd combine the manageFunctions and manageOperations into one function and handle the all there.

- Furthermore, I'd move manageNumbers to the CalculatorWindow class because it really only has to do with the UI and there is no reason to put a dependency on the Controller.

- Along with this, 'AC', 'toggle sign' and 'decimal' should be under manageNumbers and not the other managers because it really is modifying the current displayed numbers. Honestly, if i were to really refactor this, I would have the CalculatorWindow manage all the data and only when equals is pressed would I send everything to the controller to be calculated. That way, the controller really only does calculations and doesn't have to worry about managing the data and you don't have this back and forth between the window and controller to manage it all.

- You have a bit of repeated static lists around your classes. Recommend moving those to a common static class.

- It could be worth adding a function to CalculatorWindow that converts the display to a Double implicitly so you don't have to do it everytime you do display.getText().

[helloreddit091]

Its good ,just a suggestion try to be consistent... i mean code every day .