Junior Code Review: 5 Tips for Over-Engineered Example

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

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

  • @tusharnain6652
    @tusharnain6652 2 месяца назад +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 2 месяца назад +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 2 месяца назад

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

    • @dannjohnrem
      @dannjohnrem Месяц назад

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

  • @aamirsohailkmas
    @aamirsohailkmas 2 месяца назад +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 2 месяца назад +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 2 месяца назад +5

    I think a Trait should have been used here.

  • @Aymen-y8j
    @Aymen-y8j 2 месяца назад +2

    Do more of these pleass

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

    It’s really interesting

  • @iamriwash7943
    @iamriwash7943 2 месяца назад

    flashToast can use inside helper funtion also

  • @logudotcom
    @logudotcom 2 месяца назад

    Thank you

  • @AndiSyafrianda
    @AndiSyafrianda 2 месяца назад +1

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

  • @rihardstirums6698
    @rihardstirums6698 2 месяца назад

    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 2 месяца назад +1

      The same way as with earlier laravel versions

    • @rihardstirums6698
      @rihardstirums6698 2 месяца назад

      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  2 месяца назад +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 2 месяца назад

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

    • @LaravelDaily
      @LaravelDaily  2 месяца назад

      Email povilas@laraveldaily.com

  • @NeverCodeAlone
    @NeverCodeAlone 2 месяца назад +1

    Very good video.

  • @keyvanakbarzadeh
    @keyvanakbarzadeh 2 месяца назад

    nice

  • @HieuNguyen-pm9uf
    @HieuNguyen-pm9uf 2 месяца назад

    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  2 месяца назад

      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 2 месяца назад

      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 2 месяца назад

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

    • @HieuNguyen-pm9uf
      @HieuNguyen-pm9uf 2 месяца назад

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