akikos
Επιφανές μέλος
Ο Ανδρέας αυτή τη στιγμή δεν είναι συνδεδεμένος. Επαγγέλεται Η.Μ.Μ.Υ. και μας γράφει απο Αθήνα (Αττική). Έχει γράψει 11,859 μηνύματα.
08-10-18
13:50
Μικρή διαφορά η συνάρτηση μου δεν αναφέρει ότι όποιος δεν ικανοποιεί τις παραπάνω συνθήκες αυτομάτως δεν είναι και μη ποιοτικός. Συνεπώς λείπει ένα if. Στο παράδειγμά μας το body του if (που δεν έβαλες) θα ναι μικρό αλλά σε πραγματικά project μπορεί να είναι μεγάλο κάτι που κάνει τον κώδικα δυσανάγνωστο.
Ας σουλουπώσω και το δικό μου.
Υπάρχει κανένας που θεωρεί το κώδικα του post πάνω πιο ευανάγνωστο από αυτό που έγραψα εγώ εδώ.
Ας σουλουπώσω και το δικό μου.
Code:
for (int i = 0; i < count(eStekiUsers) ; i++){
if (eStekiUsers[i]->age < 25 || strcmp(eStekiUsers[i]->name,"Fockos") || strcmp(eStekiUsers[i]->sxoli,"ΠΛΗΡΟΦΟΡΙΚΗ") || strcmp(eStekiUsers[i]->sxoli,"HMMY") || (eStekiUsers[i]->postCount <= 500 )) continue;
if (time_exec = 600) break;
eStekiUsers[i]->poiotikosXristis = true;
}
Υπάρχει κανένας που θεωρεί το κώδικα του post πάνω πιο ευανάγνωστο από αυτό που έγραψα εγώ εδώ.
Σημείωση: Το μήνυμα αυτό γράφτηκε 5 χρόνια πριν. Ο συντάκτης του πιθανόν να έχει αλλάξει απόψεις έκτοτε.
akikos
Επιφανές μέλος
Ο Ανδρέας αυτή τη στιγμή δεν είναι συνδεδεμένος. Επαγγέλεται Η.Μ.Μ.Υ. και μας γράφει απο Αθήνα (Αττική). Έχει γράψει 11,859 μηνύματα.
08-10-18
10:40
Για μενα κανει τα πραγματα περιπλοκα εχω καιρο να χρησιμοποιησω σε κατι που δεν ειναι παρα πολυ απλο
Έτσι μας μάθαιναν στις σχολές αλλά βλέπω αρκετοί χρησιμοποιύν τα beaκ και continue για να δουν αν ικανοποιούνται οι συνθήκες για να προχωρήσουμε στη συνέχεια.
Παράδειγμα
Code:
for (int i = 0; i < count(eStekiUsers) ; i++){
if (eStekiUsers[i]->age < 25) continue;
if (strcmp(eStekiUsers[i]->name,"Fockos")) continue;
if (strcmp(eStekiUsers[i]->sxoli,"ΠΛΗΡΟΦΟΡΙΚΗ") || strcmp(eStekiUsers[i]->sxoli,"HMMY")) continue;
if (eStekiUsers[i]->postCount <= 500 continue;
if (time_exec = 600) break;
eStekiUsers[i]->poiotikosXristis = true;
}
Μπορεί να κάνει κάποιος αυτό χωρίς continue; και break; και να ναι εξίσου ευανάγνωστο;
Σημείωση: Το μήνυμα αυτό γράφτηκε 5 χρόνια πριν. Ο συντάκτης του πιθανόν να έχει αλλάξει απόψεις έκτοτε.