Junior Code Review: 5 Tips for Over-Engineered Example

Поделиться
HTML-код
  • Опубликовано: 24 ноя 2024

Комментарии • 28

  • @tusharnain6652
    @tusharnain6652 25 дней назад +17

    Hi everyone, the featured code was mine (just a starter for a big project), which featured a directory structure designed for larger projects, thinking ahead about scalability. Povilas provided great feedback on it being over-engineered for a smaller project, which I completely agree with. Honestly i think using laravel for any small project is an overkill in itself (its like using a sword to kill a bee). It's a valuable reminder to balance complexity with the project's size. Also for the admin authentication, i chose to go with custom solution instead for breeze, etc is because its going to be more complex later. Highly recommend checking out this series if you're looking to improve your coding skills!

    • @o_lobato
      @o_lobato 25 дней назад +5

      A project can be really small and still need authentication, authorization for resources control access, database management (migration, etc), notifications, queues, and so one! So its far from being a sword to kill a bee.

    • @tusharnain6652
      @tusharnain6652 25 дней назад

      @o_lobato Imo, for smaller projects, we got smaller framework like CI4, etc

    • @dannjohnrem
      @dannjohnrem 10 дней назад

      can I clone it too so that I can learn from you approach? thank you so much

  • @aamirsohailkmas
    @aamirsohailkmas 25 дней назад +6

    I remember when I was a beginner, my goal was to make a structure that can be extended without problems,
    for example using a global model class that extends Laravel's model and similar case for other vendor classes, 😅

  • @bboydarknesz
    @bboydarknesz 22 дня назад +1

    always remember coding about KISS but readable.
    If we always think every possibility, everything could happen and it is infinity.
    And then you will over handle the logic, and over engineered but the reality it could be just rarely to be used.
    You will spend too much time in testing, debugging, reading the code, teaching your junior how to read the code.
    Stop the possibility / make it less, and you won't get loss and focus the main thing.

  • @rock3rkis670
    @rock3rkis670 25 дней назад +5

    I think a Trait should have been used here.

  • @ЖеняХристенко-е3ы
    @ЖеняХристенко-е3ы 4 дня назад

    It’s really interesting

  • @LoganathanNatarajanlogudotcom
    @LoganathanNatarajanlogudotcom 20 дней назад

    Thank you

  • @Aymen-y8j
    @Aymen-y8j 25 дней назад +2

    Do more of these pleass

  • @iamriwash7943
    @iamriwash7943 24 дня назад

    flashToast can use inside helper funtion also

  • @AndiSyafrianda
    @AndiSyafrianda 24 дня назад +1

    "Premature optimization is the root of all evil."
    - Donald Knuth

  • @rihardstirums6698
    @rihardstirums6698 25 дней назад

    Hey how would you approche in laravel 11 translatable url slugs? When user switches pages language so it updates the slugs into url menu.

    • @krekas
      @krekas 25 дней назад +1

      The same way as with earlier laravel versions

    • @rihardstirums6698
      @rihardstirums6698 25 дней назад

      Hey @ can you please provide some guide (link, post) where i can learn more about this ? I wasn't able to find it.

    • @LaravelDaily
      @LaravelDaily  25 дней назад +2

      We have a full course on multi-language Laravel, so pretty sure you will find it somewhere here, depending on what package (if any) you're using: laraveldaily.com/course/multi-language-laravel

  • @RubensRocha
    @RubensRocha 25 дней назад

    Which contact channel can I use to send a code for analysis?

  • @NeverCodeAlone
    @NeverCodeAlone 25 дней назад +1

    Very good video.

  • @keyvanakbarzadeh
    @keyvanakbarzadeh 24 дня назад

    nice

  • @ZeinotGaming
    @ZeinotGaming 25 дней назад

    over engineering classic newbie mistake

    • @turculaurentiu91
      @turculaurentiu91 24 дня назад +1

      I've seen a lot of "seniors" doing similar mistakes.

  • @HieuNguyen-pm9uf
    @HieuNguyen-pm9uf 23 дня назад

    Hello sir, I have this kind of problem, maybe it's stupid but hey I'm just want to know that. For example, I have the menu header blade and need to return data from db, so create a Provider like this below. So can you answer that what I did is wrong or there is another way to do it

    • @LaravelDaily
      @LaravelDaily  23 дня назад

      Yeah, seems good to me! View Composers are a proper way to set such global variables.
      Maybe, two things to consider changing:
      1. Doing it in the AppServiceProvider instead of creating a separate one
      2. Separating ViewComposer into its own class, to shorten the Provider code to fit other features: laravel.com/docs/11.x/views#view-composers

    • @FallHealer2375
      @FallHealer2375 18 дней назад

      Hello! I have a project where I needed menu items and other data in the header, and after testing some solutions a bit, I decided to do a Provider just like you did.

    • @HieuNguyen-pm9uf
      @HieuNguyen-pm9uf 17 дней назад

      @@LaravelDaily thank you sir , and weird I dont see notification from youtube when someone reply to me

    • @HieuNguyen-pm9uf
      @HieuNguyen-pm9uf 17 дней назад

      @@FallHealer2375 cool , I did some research but not sure if my way is good or not