Sheet1_2. - Δεν απελευθερώνεις τη δυναµικά δεσµευµένη µνήµη. - Η έξοδος του προγράµµατός σου δεν είναι ακριβώς όπως ζητούσε η άσκηση.

Transcript

1 Sheet1_2 AEM COMMENTS - Segmentation fault λόγω λάθους στην αναδροµή. Δες τη λύση, αλλά προσπάθησε να το ξανακάνεις και στο σπίτι για εξ - Πρόσεχε λίγο τη στοίχιση - Καλή δουλειά στα σχόλια, µπράβο - Δεν υπάρχει αναδροµή στο πρόγραµµα. - Η συνάρτηση δεν είναι αναδροµική - Το '\0' είναι ο τερµατικός χαρακτήρας (ο χαρακτήρας µε ASCII µηδέν) ενώ το NULL είναι δείκτης (διεύθυνση µηδέν). Δεν είναι το ίδιο πράγµα. - Ο τελεστής * χρησιµοποιείται για να προσπελάσουµε τα περιεχόµενα σε µια διεύθυνση - δεν έπρεπε να τον χρησιµοποιείς στην κλήση της replace. - Δεν ελευθερώνεις τη δυναµικά δεσµευµένη µνήµη - Η έξοδος του προγράµµατός σου δεν είναι ακριβώς όπως ζητούσε η - Πρόσεχε λίγο τη στοίχιση - Δεν υπάρχει αναδροµή στο πρόγραµµα. - Λάθος στη scanf. Έπρεπε να διαβάζεις µε %100s (και να έχεις δεσµεύσει 101 bytes) - Στην strncpy έπρεπε να αντιγράφεις το replacement στο text, όχι στο to_go_away οριακά - Όταν η malloc επιστρέφει NULL, το πρόγραµµα πρέπει να τερµατίζει (γιατι αν δεν έχει δεσµευτεί µνήµη, θα γίνει segfault) - Δεν απελευθερώνεις τη δυναµικά δεσµευµένη µνήµη Page 1

2 Sheet1_ Δεν κάνει compile - Δεν έχει υλοποιηθεί η replace. Τα σχόλια που υπάρχουν δε δίνουν χρήσιµες πληροφορίες/. Διάβασε το φυλλάδιο σχολιασµού!!! - Λάθος στην αντιγραφή της συµβολοσειράς. Χρεάζεται ένα else πριν τη δεύτερη κλήση στη replace. - Πολύ καλή δουλειά αν και το παράκανες λίγο µε τα γενικά σχόλια :) Δε χρειάζεται να εξηγείς τι κάνουν οι συναρτήσεις της C. - Χρειάζεται να διαβάσεις και να καταλάβεις πώς λειτουργούν οι συναρτήσεις συµβολοσειρών, για να µπορείς να τις χρησιµοποιείς σωστά. - Να χρησιµοποιείς περιγραφικά ονόµατα µεταβλητών (όχι m, keime, ktl), αλλά έχεις καλά σχόλια προγράµµατος - Πολύ καλή δουλειά κατά τα άλλα. Page 2

3 Sheet1_2 οριακά Ελλειπή σχόλια - Να χρησιµοποιείς συναρτήσεις συµβολοσειρών όπου χρειάζεται. Για παράδειγµα, το µήκος µιας συµβολοσειράς µπορεί να υπολογιστεί µε την strlen - Πρώτα δεσµεύουµε µνήµη για τις συµβολοσειρές και µετά διαβάζουµε τις τιµές τους. - Ξαναδιάβασε για την αναδροµή και κάνε πολλά παραδείγµατα - φαίνεται 0018 να µπερδεύεις την αναδροµή µε την επανάληψη. οριακά - Λάθος στην αναδροµή. Στην περίπτωση που η κλήση στην strncmp 0019 επιστρέφει διάφορο του µηδενός δεν έπρεπε να τερµατίζεις Πολύ καλή δουλειά κατά τα άλλα. - Καλή δουλειά, αλλά στο µέλλον να γράφεις τα σχόλια συναρτήσεων 0021 όπως περιγράφεται στο σχετικό φυλλάδιο 0022 οριακά - Δεν έχεις σωστό base case. Επρεπε να τερµατίζεις όταν η strstr 0023 επιστρέφει NULL Τι σηµαίνει j? Επρεπε να έχεις πιο περιγραφικό όνοµα Ο έλεγχος if((emfanisi+strlen(replacement)+1)=='\0') είναι περιττός Στο τελευταίο printf δε χρειαζόταν το 100. Page 3

4 Sheet1_ Ελειπή σχόλια - Έχεις πολύ περιττό κώδικα. Δες τη λύση όταν ανακοινωθεί για να δεις πόσο πιο απλά θα µπορούσες να το έχεις κάνει. Αυτό θα σε βοηθήσει και να καταλάβεις καλύτερα πώς λειτουργεί η αναδροµή. - Όταν η malloc επιστρέφει NULL, το πρόγραµµα πρέπει να τερµατίζει (γιατι αν δεν έχει δεσµευτεί µνήµη, θα γίνει segfault) - Ωραία σχόλια προγράµµατος, αλλά τα πιο πολλά από αυτά έπρεπε να είναι στη συνάρτηση. - Πολύ καλή δουλειά κατά τα άλλα. - Κατα τα άλλα, καλή δουλειά - Πολύ καλή δουλειά- Δε χρειάζεται να σχολιάζεις συναρτήσεις της C, όπως η strstr οριακά - Γιατί υπολογίζεις τα µήκη των δύο πρώτων strings? - Να δίνεις πιο περιγραφικά ονόµατα στις µεταβλητές - Τα σχόλια συνάρτησης πρέπει να βρίσκονται πάνω από τη συνάρτηση Page 4

5 Sheet1_2 - Όταν η malloc επιστρέφει NULL, το πρόγραµµα πρέπει να τερµατίζει (γιατι αν δεν έχει δεσµευτεί µνήµη, θα γίνει segfault) - Καλή προσπάθεια στα σχόλια, αλλά πρέπει να αντιστοιχούν στον κώδικα (για παράδειγµα, η replace έχει τρεις όχι δυο παραµέτρους) - Λάθος στην κλήση της malloc - Δεν υπάρχουν σχόλια - Δεν απελευθερώνεις τη δυναµικά δεσµευµένη µνήµη - Να δίνεις περιγραφικά ονόµατα στις µεταβλητές - Μπράβο, εξαιρετική δουλειά! - Εξαιρετική δουλειά - Καλή δουλειά, αλλά στο µέλλον να γράφεις τα σχόλια συναρτήσεων όπως περιγράφεται στο σχετικό φυλλάδιο - Καλή δουλειά κατα τα άλλα - Καλή δουλειά οριακά - Δεν απελευθερώνεις όλη τη δυναµικά δεσµευµένη µνήµη. οριακά Page 5

6 Sheet1_2 - Τα === παραπλανούν τον αναγνώστη : δίνουν την εντύπωση ότι ξεκινά κάποιο πολύ διαφορετικό τµήµα του προγράµµατος. Είναι καλή ιδέα να τα χρησιµοποιείς ανάµεσα σε συναρτήσεις, αλλά όχι µέσα στην ίδια συνάρτηση - Έχεις πολλά σχόλια σε κάθε γραµµή, τα οποία δε δίνουν κάποια χρήσιµη πληροφορία. Επίσης, δεν έχεις καλά σχόλια συναρτήσεων. Διάβασε το φυλλάδιο σχολιασµού. - Δε γίνεται δυναµική δέσµευση µνήµης - Δεν έχει υλοποιηθεί σχεδόν καθόλου η replace - Να χρησιµοποιείς περιγραφικά ονόµατα µεταβλητών. Τι σηµαίνει pch?. - Ξανακάνε την άσκηση στο σπίτι µε ακριβώς τις παραµέτρους που ζητούσε η εκφώνηση για να καταλάβεις καλύτερα πώς λειτουργούν οι αναδροµικές συναρτήσεις. - Το πρόγραµµα δεν κάνει compile - Δεν υπάρχουν σχόλια - Πολύ καλή δουλειά - Το loop στην getstring είναι περιττό και inefficient. Αρκούσε µια scanf. - Κατά τα άλλα, καλή δουλειά. - Εξαιρετική δουλειά Page 6

7 Sheet1_2 - Τα σχόλια συναρτήσεων δεν είναι σύµφωνα µε τις προδιαγραφές - Δε γίνεται δυναµική δέσµευση µνήµης - Να δίνεις περιγραφικά ονόµατα στις µεταβλητές - Το πρόγραµµα δε λειτουργεί σωστά: τα mikos3 και mikos1 έχουν πάντα την ίδια τιµή - Πολύ καλή δουλειά - Πολλά σχόλια αλλά δυστυχώς δε διαβάζονταν. Γράφε µε λατινικούς χαρακτήρες.. Επίσης, µην αφήνεις τόσο µεγάλα κενά ανάµεσα στις γραµµές του προγράµµατος. - Πρέπει κάθε εντολή να βρίσκεται σε νέα γραµµή - µη βάζεις πολλαπλές εντολές στην ίδια γραµµή - Πρώτα πρέπει να ελέγχεις αν το malloc επέστρεψε NULL και µετά να διαβάζεις κάτι στο string. Αν διαβάζεις πρώτα, τότε αν ήταν NULL θα έκανε segfault - Μη χρησιµοποιείς σχόλια C++ - Καλή δουλειά κατα τα άλλα - Λάθος στο base case και στην αναδροµική κλήση - Δεν απελευθερώνεις τη δυναµικά δεσµευµένη µνήµη. - Εξαιρετική δουλειά Page 7

8 Sheet1_ Πολύ καλή δουλειά οριακά -Άσχηµη στοίχιση - Καλή δουλειά, αλλά προσπάθησε να το κάνεις και χωρίς να χρησιµοποιείς τη sum. Θα σε βοηθήσει να καταλάβεις καλύτερα την αναδροµή - Τα σχόλια συνάρτησης πρέπει να βρίσκονται πριν τη συνάρτηση - Δεν έχει υλοποιηθεί αναδροµή - Όταν η malloc επιστρέφει NULL, το πρόγραµµα πρέπει να τερµατίζει (γιατι αν δεν έχει δεσµευτεί µνήµη, θα γίνει segfault) - Ωραία σχόλια - Να δίνεις πιο περιγραφικά ονόµατα στις µεταβλητές οριακά - Δεν απελευθερώνεις τη δυναµικά δεσµευµένη µνήµη. Πολύ καλά! - Να χρησιµοποιείς πιο περιγραφικά ονόµατα για µεταβλητές. - Καλη δουλειά κατα τα άλλα. Page 8

9 Sheet1_2 - Τα σχόλια συνάρτησης πρέπει να βρίσκονται πάνω από τη συνάρτηση - Να χρησιµοποιείς περιγραφικά ονόµατα µεταβλητών. Τι σηµαίνει pch? - Το πρόγραµµα δεν κάνει compile οριακά - Στη C οι µεταβλητές πρέπει να δηλώνονται στην αρχή κάθε συνάρτησης, πριν από οποιεσδήποτε άλλες εντολές - Τα σχόλια συνάρτησης πρέπει να βρίσκονται πάνω από τη συνάρτηση - Να χρησιµοποιείς περιγραφικά ονόµατα µεταβλητών - Καλή δουλειά οριακά - Πρόσεχε λίγο τη στοίχιση - Καλή δουλειά στα σχόλια, µπράβο - Πολύ καλή δουλειά Page 9

10 Sheet1_2 - Δεν υπάρχουν σχόλια - Μην αγνοείς τα compilation warnings - σε προειδοποιούν για σηµαντικά λάθη. - Δε δεσµεύεται µνήµη για τα strings - Δεν υπάρχει τύπος void * - Το base case πρέπει πάντα να ελέγχεται στην αρχή της αναδροµικής συνάρτησης οριακά - Δεν υπάρχουν σχόλια - Δεν κάνει compile - Καλή δουλειά - Διάβασε το φυλλάδιο σχολιασµού - σχόλια όπως το "ελευθερώνει τη µνήµη" δεν προσφέρουν χρήσιµες πληροφορίες. Επιπλέον, πρέπει να βάζεις "σχόλια συναρτήσεων". - Τα σχόλια µέσα στη replace είναι καλά - Αντί για if (x>0 x<0) θα ήταν πιο "καθαρό" να είχες χρησιµοποιήσει else οριακά - Δεν απελευθερώνεις όλη τη δυναµικά δεσµευµένη µνήµη. Page 10

11 Sheet1_ Το '\0' είναι ο τερµατικός χαρακτήρας (ο χαρακτήρας µε ASCII µηδέν) ενώ το NULL είναι δείκτης (διεύθυνση µηδέν). Δεν είναι το ίδιο πράγµα. - Δεν τερµατίζει ποτέ η αναδροµή - segmentation fault σε όλα εκτός από ένα testcase - To string είχε 100 χαρακτήρες και το τερµατικό, άρα έπρεπε να διαβάζεις µε %100s. - Όταν διαβάζεις string µε scanf πρέπει πάντα να προσδιορίζεις το µέγεθος - Τα σχόλια συναρτήσεων πρέπει να βρίσκονται πάνω από τη συνάρτηση. - Δεν απελευθερώνεις τη δυναµικά δεσµευµένη µνήµη οριακά - Πρώτα πρέπει να ελέγχεις αν το malloc επέστρεψε NULL και µετά να διαβάζεις κάτι στο string. Αν διαβάζεις πρώτα, τότε αν ήταν NULL θα έκανε segfault - Όταν διαβάζεις ένα string µε scanf πρέπει να προσδιορίζεις το µέγεθος. - Πρέπει κάθε εντολή να βρίσκεται σε νέα γραµµή - µη βάζεις πολλαπλές εντολές στην ίδια γραµµή - Τα σχόλια συνάρτησης πρέπει να βρίσκονται πάνω από τη συνάρτηση - Πολύ καλή δουλειά οριακά - Να χρησιµοποιείς περιγραφικά ονόµατα µεταβλητών. Τι σηµαίνει pch? Page 11

12 Sheet1_2 - Όταν η malloc επιστρέφει NULL, το πρόγραµµα πρέπει να τερµατίζει (γιατι αν δεν έχει δεσµευτεί µνήµη, θα γίνει segfault) - Πολύ καλή δουλειά - Δεν υπάρχουν σχόλια - Κατα τα άλλα, πολύ καλογραµµένο πρόγραµµα Page 12