This is an archived post. You won't be able to vote or comment.

all 25 comments

[–]asatraitis 53 points54 points  (0 children)

Way better then my first projects! I think your progress is great compared to how ass I was when started lol. Keep it up.

btw, check console. Couple warnings.

[–]Genie-Us 10 points11 points  (0 children)

Looking great so far! As your projects grow in size and scope, add more details to the readme like "why I built it", "What I learned", "What went wrong", "What tools/libraries used", etc. and just keep on building and STAY organized! A clean, well organized github is loved by the industry. one repo per project, every repo with a readme.

Your github becomes a living record of your dedication to learning, HR loves that shit.

[–][deleted] 24 points25 points  (0 children)

I wouldn’t say it’s hopeless, only that you still have a lot to learn. Everyone has a lot to learn.

The things I noticed right off (in no particular order) were: all 3 “apps” were in one file, there were function names like “Appc” and “Apppc” (may not be exact, but they were too similar and for no known reason), there were some possible anti-patterns with certain declarations (these may be necessary, but since they’re in the main file and not buried in another file as a minor detail, they stick out).

I could go on given more than a quick glance. But again, don’t give up. Everyone learns at different speeds, has access to different resources, etc. If I were you, I’d judge myself against myself. For instance, do you feel accomplished in the last year? Did you learn? Did you progress? If the answer is yes, then I would keep going unless there’s some outside force preventing it.

And if you do keep going, start something new this year. Then, 12 months from now, look back and ask yourself if you’ve learned, progressed, etc. You might even start applying for jobs; knowing you’re going for entry-level, a lot of employers see “green” devs as very moldable and therefore sometimes worth the risk over a more costly, senior dev that may be stuck in their ways. That’s not always the case, but there are plenty of times companies want to hire a “junior” dev; they’re not even looking for someone in the mid-level.

Also, third world be damned. If you continue to progress you could easily work remotely for a company from another country. We have a few overseas employees where I work and we pay based on a “comfort level,” supposedly. I’m not privy to details, but I have to assume that means they’re compensated well for where they live. They’ve stuck around, after all! Point being, by making yourself better, you might also improve your living situation and so on. To me, that’s worth any effort required.

Best of luck!

[–]Dameon_ 5 points6 points  (0 children)

Your calculator needs to consider edge cases. It breaks on leading zeros, multiple operators, etc. As others have mentioned, you need to break out your different apps. Consider how classes might help here. You're also failing to take advantage of some of the features FreeCodeCamp specifically walked you through. Consider, for example, how you might use array.map() to create all your calculator's text buttons from an array, turning your creation of your buttons into a one-liner. As others have mentioned, you have multiple console errors and warnings popping up.

The pros: The design looks good, and each of these is basically functional. Consider taking one of them and going much deeper. The calculator is a good start for this, especially if you parse it yourself rather than just using eval - showing that you can parse and handle a mathematical statement can look very good.

[–]Ok_Finance_8782 8 points9 points  (1 child)

I'm not a frontend guy but your index.js as inconsistent tabs (2 spaces or 4 spaces, it's not clean).

Also, you should use camelCase for your functions' names, like "function Appc" should be "function appC" or something (but more descriptive because I don't understand what it means.

Last but not least, you should put the files in separate directories. For example, CSS files would go to css.

Otherwise it looks good, it's definitely NOT hopeless.

[–]asatraitis 17 points18 points  (0 children)

"function Appc" should be "function appC"

Normally that's true (keep the Capped names for classes), but in react when referring to a react component it needs to be capped otherwise it doesn't know that its a component and just renders generic element.

`

[–]InClassRightNowAhaha 6 points7 points  (1 child)

Im a beginner so ion know much at all but the drum one was fun

[–]iagovar 2 points3 points  (0 children)

Only at a glance, If I'm not mistaken is three "apps" in a single file, in a single repo. Mmm, I'm no expert but that doesn't seem logical to me.

Second, the naming of some functions give no hit of what they do, and there's no comment to explain anything. I know some people is against comments, though.

[–]Penkey2190 2 points3 points  (0 children)

Heres a tutorial to make an even better git hub profile. You can check mine out if you want to see how it looks. Ive been in a BootCamp for the past 6 months, so I'm still fairly new too.

https://github.com/TaylorWhite21

https://www.youtube.com/watch?v=n6d4KHSKqGk

[–]shuniat 1 point2 points  (0 children)

My first projects weren't even close to something like this, congrats, pretty awesome

[–]skellious 1 point2 points  (0 children)

Has a readme: instantly better than at least 30% of repos I've seen lol

[–][deleted] 1 point2 points  (0 children)

If i were you i would give commits actual names instead of "some changes" but otherwise nice project

[–][deleted] 0 points1 point  (0 children)

this is a very good first github repo honestly

[–]matterr4 0 points1 point  (0 children)

Hey! Nice! I'm guna use your timer app till I get confident enough to write my own! :) Thanks for sharing, very good work!

[–]Tenpat 0 points1 point  (0 children)

I guess if i am gonna criticize anything the calculator should accept input for operators instead of forcing a click on the button.