Jump to content

Elfocrash

VIP Member
  • Content Count

    1,654
  • Joined

  • Last visited

  • Days Won

    29
  • Feedback

    0%

Elfocrash last won the day on October 16

Elfocrash had the most liked content!

Community Reputation

160 Excellent

About Elfocrash

  • Rank
    Marquis

Contact Methods

  • Website URL
    http://chapsas.com

Profile Information

  • Current Mood
    Angry
  • Gender
    Male
  • Country
    United Kingdom

Recent Profile Visitors

4,794 profile views
  1. Εεε μπορώ να πω counter argument οτι δεν "σε κάνει και κατι αλλο" αλλα σου δίνει το μέσο για να εκφράσεις το class σου πάνω σε μια συγκεκριμένη επιφάνεια. Τουλάχιστον στo δικό μου περιβάλλον είναι έτσι. Επίσης πλέον το inheritance αποφεύγεται πλέον πολύ για πολλούς λόγους, μπορείς να το ψάξεις. Good read https://codeburst.io/inheritance-is-evil-stop-using-it-6c4f1caf5117 Anyway κατάλαβες τι παίζει και αυτό μετράει @Kara`
  2. Δεν είπε κανείς οτι δεν λειτουργεί. Σου λέμε απλά μια εναλλακτική για το πώς μπορείς να τον κάνεις καλύτερο. Έχεις δυο επιλογές. Ή να το δεχτεις οτι μπορεί να βελτιωθεί και να μάθεις απο αυτό, ή να συνεχίσεις να λες "τον έκανα σε 5 λεπτά στο μπάνιο" και να παραμείνεις ο Kara του mxc. Για μένα θα έπρεπε να είναι interface για πολλούς λόγους με τον βασικότερο, το οτι δεν ειναι το usecase για abstract class, αλλα δεν θα μπώ σε argument γιατι θα πω πράγματα όπως SOLID, KISS, YAGNI και unit testing και θα τα ακούσω μόνο εγώ και κανα δύο άλλοι. Το βασικότερο προβλημα ειναι στο field readability στα extended classes.
  3. μα κάνει implement πάνω απο ένα Επισης δεν βλέπω κανένα λόγο για inheritance στο συγκερκιμένο σενάριο. Γενικά να αποφεύγεται
  4. Κάνεις ένα string field στο PvpTitle class. Δεν υπάρχει λόγος να κάνεις generalisation στο επίπεδο που το κάνεις για δύο λόγους. Πρώτον κάνεις violate single responsibility γιατί δίνεις στο values field ιδιότητες που δεν θα έπρεπε να έχει. Δεύτερον, δεν χρειάζεσαι κάν abstract class, μπορείς να το κάνεις με interface πολύ πιο απλά. public class PvPSkill extends PEvent { public PvPSkill(int id, int level) { values = new Object[2] { id, level }; } @Override public void getEffect(L2PcInstance player) { Skill skill = SkillData.getInstance().getSkill((int) values[0], (int) values[1]); if (skill == null) { return; } player.addSkill(skill); } } public class PvPSkill implements PEvent { private final int id; private final int level; public PvPSkill(int id, int level) { this.id = id; this.level = level; } @Override public void getEffect(L2PcInstance player) { Skill skill = SkillData.getInstance().getSkill(id, level); if (skill == null) { return; } player.addSkill(skill); } } Εσυ ποιό απο τα δύο βλέπεις σωστό?
  5. Όλοι φυσικα λολ. Άν κλείσει το stackoverflow πρέπει να αλλάξω επάγγελμα
  6. [GR] Ρε άσε τις δικαιολογίες και μην κάνεις σαν 10αχρονο. Δεν έχει σημασία πως έγραψες τον κώδικα, σημασία έχει το αποτέλεσμα που παρουσιάζεις και μόνο.
  7. I think he means https://gist.github.com/ It was worse, check previous comments. Still, good effort.
  8. Σιγά μην ρίξουμε και χλέπα να δούμε ποιός την πάει πιο μακριά. Πήγαινε διάβασε λίγο design patterns μπας και κάνεις κανα share της προκοπής, γιατι είσαι η τελευταία ελπίδα αυτού του φόρουμ.
  9. Αρκετά καλή περιγραφή μου για 2015
  10. Περιμένω black friday να βάλει εκπτώσεις
  11. Stay tuned for 20% https://www.asos.com
  12. ΜΑ ΔΕΝ ΕΙΝΑΙ Η ΖΩΗ ΜΟΥ ΟΛΟΚΛΗΡΗ ΧΩΡΙΣ LEGENDARY ΡΑΝΚ!!!111ΕΝΑΕΝΑ
  13. You can also shorthand it like this: values = new Object[]{ id, level }; There are multiple ways to clean up the code.
×