Sheet1_2 AEM 0001 0002 0003 0004 0006 COMMENTS - Segmentation fault λόγω λάθους στην αναδροµή. Δες τη λύση, αλλά προσπάθησε να το ξανακάνεις και στο σπίτι για εξ - Πρόσεχε λίγο τη στοίχιση - Καλή δουλειά στα σχόλια, µπράβο - Δεν υπάρχει αναδροµή στο πρόγραµµα. - Η συνάρτηση δεν είναι αναδροµική - Το '\0' είναι ο τερµατικός χαρακτήρας (ο χαρακτήρας µε ASCII µηδέν) ενώ το NULL είναι δείκτης (διεύθυνση µηδέν). Δεν είναι το ίδιο πράγµα. - Ο τελεστής * χρησιµοποιείται για να προσπελάσουµε τα περιεχόµενα σε µια διεύθυνση - δεν έπρεπε να τον χρησιµοποιείς στην κλήση της replace. - Δεν ελευθερώνεις τη δυναµικά δεσµευµένη µνήµη - Η έξοδος του προγράµµατός σου δεν είναι ακριβώς όπως ζητούσε η - Πρόσεχε λίγο τη στοίχιση - Δεν υπάρχει αναδροµή στο πρόγραµµα. - Λάθος στη scanf. Έπρεπε να διαβάζεις µε %100s (και να έχεις δεσµεύσει 101 bytes) - Στην strncpy έπρεπε να αντιγράφεις το replacement στο text, όχι στο to_go_away οριακά - Όταν η malloc επιστρέφει NULL, το πρόγραµµα πρέπει να τερµατίζει (γιατι αν δεν έχει δεσµευτεί µνήµη, θα γίνει segfault) - Δεν απελευθερώνεις τη δυναµικά δεσµευµένη µνήµη Page 1
Sheet1_2 0009 0010 0011 0012 0013 0014 - Δεν κάνει compile - Δεν έχει υλοποιηθεί η replace. Τα σχόλια που υπάρχουν δε δίνουν χρήσιµες πληροφορίες/. Διάβασε το φυλλάδιο σχολιασµού!!! - Λάθος στην αντιγραφή της συµβολοσειράς. Χρεάζεται ένα else πριν τη δεύτερη κλήση στη replace. - Πολύ καλή δουλειά αν και το παράκανες λίγο µε τα γενικά σχόλια :) Δε χρειάζεται να εξηγείς τι κάνουν οι συναρτήσεις της C. - Χρειάζεται να διαβάσεις και να καταλάβεις πώς λειτουργούν οι συναρτήσεις συµβολοσειρών, για να µπορείς να τις χρησιµοποιείς σωστά. - Να χρησιµοποιείς περιγραφικά ονόµατα µεταβλητών (όχι m, keime, ktl), αλλά έχεις καλά σχόλια προγράµµατος - Πολύ καλή δουλειά κατά τα άλλα. Page 2
Sheet1_2 οριακά 0015 - Ελλειπή σχόλια - Να χρησιµοποιείς συναρτήσεις συµβολοσειρών όπου χρειάζεται. Για παράδειγµα, το µήκος µιας συµβολοσειράς µπορεί να υπολογιστεί µε την strlen - Πρώτα δεσµεύουµε µνήµη για τις συµβολοσειρές και µετά διαβάζουµε τις τιµές τους. - Ξαναδιάβασε για την αναδροµή και κάνε πολλά παραδείγµατα - φαίνεται 0018 να µπερδεύεις την αναδροµή µε την επανάληψη. οριακά - Λάθος στην αναδροµή. Στην περίπτωση που η κλήση στην strncmp 0019 επιστρέφει διάφορο του µηδενός δεν έπρεπε να τερµατίζεις. 0020 - Πολύ καλή δουλειά κατά τα άλλα. - Καλή δουλειά, αλλά στο µέλλον να γράφεις τα σχόλια συναρτήσεων 0021 όπως περιγράφεται στο σχετικό φυλλάδιο 0022 οριακά - Δεν έχεις σωστό base case. Επρεπε να τερµατίζεις όταν η strstr 0023 επιστρέφει NULL. 0026 - Τι σηµαίνει j? Επρεπε να έχεις πιο περιγραφικό όνοµα. 0027 - Ο έλεγχος if((emfanisi+strlen(replacement)+1)=='\0') είναι περιττός 0028 - Στο τελευταίο printf δε χρειαζόταν το 100. Page 3
Sheet1_2 0030 0033 1072 1073 1074 1075 1077 1079 - Ελειπή σχόλια - Έχεις πολύ περιττό κώδικα. Δες τη λύση όταν ανακοινωθεί για να δεις πόσο πιο απλά θα µπορούσες να το έχεις κάνει. Αυτό θα σε βοηθήσει και να καταλάβεις καλύτερα πώς λειτουργεί η αναδροµή. - Όταν η malloc επιστρέφει NULL, το πρόγραµµα πρέπει να τερµατίζει (γιατι αν δεν έχει δεσµευτεί µνήµη, θα γίνει segfault) - Ωραία σχόλια προγράµµατος, αλλά τα πιο πολλά από αυτά έπρεπε να είναι στη συνάρτηση. - Πολύ καλή δουλειά κατά τα άλλα. - Κατα τα άλλα, καλή δουλειά - Πολύ καλή δουλειά- Δε χρειάζεται να σχολιάζεις συναρτήσεις της C, όπως η strstr οριακά - Γιατί υπολογίζεις τα µήκη των δύο πρώτων strings? - Να δίνεις πιο περιγραφικά ονόµατα στις µεταβλητές - Τα σχόλια συνάρτησης πρέπει να βρίσκονται πάνω από τη συνάρτηση Page 4
1080 1081 1084 1085 1086 1087 1088 1089 1091 1093 Sheet1_2 - Όταν η malloc επιστρέφει NULL, το πρόγραµµα πρέπει να τερµατίζει (γιατι αν δεν έχει δεσµευτεί µνήµη, θα γίνει segfault) - Καλή προσπάθεια στα σχόλια, αλλά πρέπει να αντιστοιχούν στον κώδικα (για παράδειγµα, η replace έχει τρεις όχι δυο παραµέτρους) - Λάθος στην κλήση της malloc - Δεν υπάρχουν σχόλια - Δεν απελευθερώνεις τη δυναµικά δεσµευµένη µνήµη - Να δίνεις περιγραφικά ονόµατα στις µεταβλητές - Μπράβο, εξαιρετική δουλειά! - Εξαιρετική δουλειά - Καλή δουλειά, αλλά στο µέλλον να γράφεις τα σχόλια συναρτήσεων όπως περιγράφεται στο σχετικό φυλλάδιο - Καλή δουλειά κατα τα άλλα - Καλή δουλειά οριακά - Δεν απελευθερώνεις όλη τη δυναµικά δεσµευµένη µνήµη. οριακά Page 5
1097 1098 1100 1101 1102 1104 1105 1109 1110 Sheet1_2 - Τα === παραπλανούν τον αναγνώστη : δίνουν την εντύπωση ότι ξεκινά κάποιο πολύ διαφορετικό τµήµα του προγράµµατος. Είναι καλή ιδέα να τα χρησιµοποιείς ανάµεσα σε συναρτήσεις, αλλά όχι µέσα στην ίδια συνάρτηση - Έχεις πολλά σχόλια σε κάθε γραµµή, τα οποία δε δίνουν κάποια χρήσιµη πληροφορία. Επίσης, δεν έχεις καλά σχόλια συναρτήσεων. Διάβασε το φυλλάδιο σχολιασµού. - Δε γίνεται δυναµική δέσµευση µνήµης - Δεν έχει υλοποιηθεί σχεδόν καθόλου η replace - Να χρησιµοποιείς περιγραφικά ονόµατα µεταβλητών. Τι σηµαίνει pch?. - Ξανακάνε την άσκηση στο σπίτι µε ακριβώς τις παραµέτρους που ζητούσε η εκφώνηση για να καταλάβεις καλύτερα πώς λειτουργούν οι αναδροµικές συναρτήσεις. - Το πρόγραµµα δεν κάνει compile - Δεν υπάρχουν σχόλια - Πολύ καλή δουλειά - Το loop στην getstring είναι περιττό και inefficient. Αρκούσε µια scanf. - Κατά τα άλλα, καλή δουλειά. - Εξαιρετική δουλειά Page 6
1111 1113 1114 1115 1116 1118 1119 1120 Sheet1_2 - Τα σχόλια συναρτήσεων δεν είναι σύµφωνα µε τις προδιαγραφές - Δε γίνεται δυναµική δέσµευση µνήµης - Να δίνεις περιγραφικά ονόµατα στις µεταβλητές - Το πρόγραµµα δε λειτουργεί σωστά: τα mikos3 και mikos1 έχουν πάντα την ίδια τιµή - Πολύ καλή δουλειά - Πολλά σχόλια αλλά δυστυχώς δε διαβάζονταν. Γράφε µε λατινικούς χαρακτήρες.. Επίσης, µην αφήνεις τόσο µεγάλα κενά ανάµεσα στις γραµµές του προγράµµατος. - Πρέπει κάθε εντολή να βρίσκεται σε νέα γραµµή - µη βάζεις πολλαπλές εντολές στην ίδια γραµµή - Πρώτα πρέπει να ελέγχεις αν το malloc επέστρεψε NULL και µετά να διαβάζεις κάτι στο string. Αν διαβάζεις πρώτα, τότε αν ήταν NULL θα έκανε segfault - Μη χρησιµοποιείς σχόλια C++ - Καλή δουλειά κατα τα άλλα - Λάθος στο base case και στην αναδροµική κλήση - Δεν απελευθερώνεις τη δυναµικά δεσµευµένη µνήµη. - Εξαιρετική δουλειά Page 7
Sheet1_2 1121 1122 1124 1126 1127 1128 1129 1130 1131 - Πολύ καλή δουλειά οριακά -Άσχηµη στοίχιση - Καλή δουλειά, αλλά προσπάθησε να το κάνεις και χωρίς να χρησιµοποιείς τη sum. Θα σε βοηθήσει να καταλάβεις καλύτερα την αναδροµή - Τα σχόλια συνάρτησης πρέπει να βρίσκονται πριν τη συνάρτηση - Δεν έχει υλοποιηθεί αναδροµή - Όταν η malloc επιστρέφει NULL, το πρόγραµµα πρέπει να τερµατίζει (γιατι αν δεν έχει δεσµευτεί µνήµη, θα γίνει segfault) - Ωραία σχόλια - Να δίνεις πιο περιγραφικά ονόµατα στις µεταβλητές οριακά - Δεν απελευθερώνεις τη δυναµικά δεσµευµένη µνήµη. Πολύ καλά! - Να χρησιµοποιείς πιο περιγραφικά ονόµατα για µεταβλητές. - Καλη δουλειά κατα τα άλλα. Page 8
1133 1135 1137 1138 1139 1141 1142 1144 1145 1148 Sheet1_2 - Τα σχόλια συνάρτησης πρέπει να βρίσκονται πάνω από τη συνάρτηση - Να χρησιµοποιείς περιγραφικά ονόµατα µεταβλητών. Τι σηµαίνει pch? - Το πρόγραµµα δεν κάνει compile οριακά - Στη C οι µεταβλητές πρέπει να δηλώνονται στην αρχή κάθε συνάρτησης, πριν από οποιεσδήποτε άλλες εντολές - Τα σχόλια συνάρτησης πρέπει να βρίσκονται πάνω από τη συνάρτηση - Να χρησιµοποιείς περιγραφικά ονόµατα µεταβλητών - Καλή δουλειά οριακά - Πρόσεχε λίγο τη στοίχιση - Καλή δουλειά στα σχόλια, µπράβο - Πολύ καλή δουλειά Page 9
1149 1150 1155 1157 1158 1159 1160 1161 Sheet1_2 - Δεν υπάρχουν σχόλια - Μην αγνοείς τα compilation warnings - σε προειδοποιούν για σηµαντικά λάθη. - Δε δεσµεύεται µνήµη για τα strings - Δεν υπάρχει τύπος void * - Το base case πρέπει πάντα να ελέγχεται στην αρχή της αναδροµικής συνάρτησης οριακά - Δεν υπάρχουν σχόλια - Δεν κάνει compile - Καλή δουλειά - Διάβασε το φυλλάδιο σχολιασµού - σχόλια όπως το "ελευθερώνει τη µνήµη" δεν προσφέρουν χρήσιµες πληροφορίες. Επιπλέον, πρέπει να βάζεις "σχόλια συναρτήσεων". - Τα σχόλια µέσα στη replace είναι καλά - Αντί για if (x>0 x<0) θα ήταν πιο "καθαρό" να είχες χρησιµοποιήσει else οριακά - Δεν απελευθερώνεις όλη τη δυναµικά δεσµευµένη µνήµη. Page 10
Sheet1_2 1171 1172 1177 1180 1181 1184 1185 - Το '\0' είναι ο τερµατικός χαρακτήρας (ο χαρακτήρας µε ASCII µηδέν) ενώ το NULL είναι δείκτης (διεύθυνση µηδέν). Δεν είναι το ίδιο πράγµα. - Δεν τερµατίζει ποτέ η αναδροµή - segmentation fault σε όλα εκτός από ένα testcase - To string είχε 100 χαρακτήρες και το τερµατικό, άρα έπρεπε να διαβάζεις µε %100s. - Όταν διαβάζεις string µε scanf πρέπει πάντα να προσδιορίζεις το µέγεθος - Τα σχόλια συναρτήσεων πρέπει να βρίσκονται πάνω από τη συνάρτηση. - Δεν απελευθερώνεις τη δυναµικά δεσµευµένη µνήµη οριακά - Πρώτα πρέπει να ελέγχεις αν το malloc επέστρεψε NULL και µετά να διαβάζεις κάτι στο string. Αν διαβάζεις πρώτα, τότε αν ήταν NULL θα έκανε segfault - Όταν διαβάζεις ένα string µε scanf πρέπει να προσδιορίζεις το µέγεθος. - Πρέπει κάθε εντολή να βρίσκεται σε νέα γραµµή - µη βάζεις πολλαπλές εντολές στην ίδια γραµµή - Τα σχόλια συνάρτησης πρέπει να βρίσκονται πάνω από τη συνάρτηση - Πολύ καλή δουλειά οριακά - Να χρησιµοποιείς περιγραφικά ονόµατα µεταβλητών. Τι σηµαίνει pch? Page 11
1186 1187 Sheet1_2 - Όταν η malloc επιστρέφει NULL, το πρόγραµµα πρέπει να τερµατίζει (γιατι αν δεν έχει δεσµευτεί µνήµη, θα γίνει segfault) - Πολύ καλή δουλειά - Δεν υπάρχουν σχόλια - Κατα τα άλλα, πολύ καλογραµµένο πρόγραµµα Page 12