lab10grades Άσκηση 2η: - Δεν κάνει compile: Δείτε την λύση και ξανα προσπαθήστε τα υπόλοιπα στο σπίτι.

Transcript

1 ΑΕΜ ΒΑΘΜΟΣ ΣΧΟΛΙΑ 449???? Άσκηση 1η: - Καλή δουλειά! Μπράβο - Σωστή στοίχιση - Σωστά σχόλια - Σωστός ορισµός των πεδίων struct - Σωστή υλοποίηση της init_books - Σωστή υλοποίηση της add_book_info: Εναλλακτικά, αντί να ψάχνετε την 1η κενή θέση θα µπορούσατε να χρησιµοποιήσετε ένα counter στη main και να περνάτε την διεύθυνση της 1ης κενής θέσης στον πίνακα των structs. - Σωστή υλοποίηση της strip_dashes, όµως, εφόσον χρησιµοποιείτε j+1, τα όρια του πίνακα έπρεπε να είναι µέχρι ISBN10-1, γιατί έτσι όπως είναι τώρα, στην τελευταία επανάληψη θα προσπελάσετε θέση εκτός των ορίων του πίνακα. - Σωστή υλοποίηση της print_books Άσκηση 2η: - Δεν κάνει compile: Δείτε την λύση και ξανα προσπαθήστε τα υπόλοιπα στο σπίτι. 451???? Άσκηση 1η: - Καλή δουλειά! Μπράβο - Σωστή στοίχιση - Σωστά σχόλια - Σωστός ορισµός των πεδίων struct - Σωστή υλοποίηση της init_books - Σωστή υλοποίηση της add_book_info: Εναλλακτικά, αντί να ψάχνετε την 1η κενή θέση θα µπορούσατε να χρησιµοποιήσετε ένα counter στη main και να περνάτε την διεύθυνση της 1ης κενής θέσης στον πίνακα των structs. - Σωστή υλοποίηση της strip_dashes, όµως, εφόσον χρησιµοποιείτε j+1, τα όρια του πίνακα έπρεπε να είναι µέχρι ISBN10-1, γιατί έτσι όπως είναι τώρα, στην τελευταία επανάληψη θα προσπελάσετε θέση εκτός των ορίων του πίνακα. - Σωστή υλοποίηση της print_books Άσκηση 2η: - Δεν κάνει compile: Δείτε την λύση και ξανα προσπαθήστε τα υπόλοιπα στο σπίτι. 476???? -Καλά σχόλια -Καλά ονόµατα µεταβλητών,σωστά πεδία στην struct -Σωστη η strip dashes -Σωστή χρήση της add books -Σωστό τύπωµα των βιβλίων -Σωστό init του πίνακα. -Είχε χαζό error στο compile. Να κάνετε πάντα ένα τελικό compile πριν στείλετε την άσκηση Απουσία 1733 PASS -Στοίχιση οκ -Δεν υπάρχουν σχόλια -Καλά ονόµατα µεταβλητών και τύποι πεδίων στη struct -Σωστά υλοποιείται την strip_dashes όµως στην Τελευταία επανάληψη του for κάνετε προσπέλαση εκτός ορίων του πίνακα -Σωστά υλοποιείται την add_book_info -Σωστά υλοποιείται την print_books και την init -Σωστά υλοποιείται τις sprintf -Σωστά καλείται την print_book -Το πρόγραµµα τερµατίζει µε Segmentation fault -Δεν αρχικοποιείται τον πίνακα pointers σε NULL -Σωστά χρησιµοποιείται τo argv[], δεν ελέγχετε πουθενά αν δόθηκε στη γραµµή εντολών 2 string -Δεν υλοποιείται σωστά το πρόγραµµα - Αντί να βάζετε τυχαία printf στο πρόγραµµα, µάθετε να χρησιµοποιείτε gdb. Page 1

2 1779???? -Δεν έχετε σχόλια -Σωστα ονόµατα µεταβλητών,σωστά πεδία στην struct -Το πρόγραµµα τρέχει σωστά -Η strip dashes δουλεύει σωστά -Σωστή η add_books, θα µπορούσατε αντί να ψάχνετε το πρώτο ελεύθερο πεδίο του πίνακα των structs να στέλνετε στην συνάρτηση τον δείκτη προς την διεύθυνση που θέλετε να περάσετε. -Σωστό τύπωµα των βιβλίων, θα µπορούσατε αντί να κάνετε break όταν βρίσκετε το πρώτο κενό πεδίο του πίνακα να στέλνετε το count αντί του MAX_LENGTH Που στέλνετε εσείς. -Σωστή αρχικοποιήση του πίνακα books. -'Eβγαζε error στο compile,(δεν είχατε δηλώσει την µεταβλητή ptr) -Φαίνεται οτι την προσπαθήσατε όµως. 1899???? -Δεν έχετε σχόλια -Καλά ονόµατα µεταβλητών, σωστά πεδία στην struct -Το πρόγραµµα δεν τρέχει σωστά, ίσως κάποιο πρόβληµα στην add_books (µαλλον δεν περνάτε σωστά τον πίνακα των structs) -Η strip dashes φαίνεται σωστή, αλλά η τελευταία επανάληψη βγαίνει εκτός ορίων του πίνακα µε το j+1 -Η print δεν τυπώνει τα περιεχόµενα του πίνακα σωστά, µάλλον διότι ο πίνακας δεν έχει στοιχέια µέσα. -Η init είναι σωστή -Δεν έχει σταλεί 1900???? -Δεν έχετε σχόλια -Καλά ονόµατα µεταβλητών, σωστά πεδία στην struct -Το πρόγραµµα δεν τρέχει σωστά, ίσως κάποιο πρόβληµα στην add_books (µαλλον δεν περνάτε σωστά τον πίνακα των structs) -Η strip dashes φαίνεται σωστή, αλλά η τελευταία επανάληψη βγαίνει εκτός ορίων του πίνακα µε το j+1 -Η print δεν τυπώνει τα περιεχόµενα του πίνακα σωστά, µάλλον διότι ο πίνακας δεν έχει στοιχέια µέσα. -Η init είναι σωστή -Δεν έχει σταλεί Διαδικαστικά: ΟΚ 1901 FAIL - Σχόλια: OK - Στοίχιση: OK - Σταθερές: OK - Ονόµατα µεταβλητών: OK - Τύποι πεδίων struct: OK - ορθότητα create OK - ορθότητα print: OK - ορθότητα add: OK, αν και θα µπορούσατε να περνούσατε την διεύθυνση της επόµενης θέσης που µπορείτε να γράψετε αντί για το students - ορθότητα init: OK - ορθότητα sprintf: OK - κλήση συναρτήσεων: OK - Γενικά: Το πεδίο used δεν χρειάζεται, αλλά δεν είναι και κακό. Η ιδέα των αρχικοποιήσεων που κάνατε είναι να ψάχνετε να βρείτε που υπάρχει το κενό string και εκεί να κάνετε την ένθεσή σας. Αλλά και έτσι που το έχετε καλό είναι. - Compiler errors Page 2

3 Διαδικαστικά: ΟΚ 1904 PASS - Σχόλια: κανένα σχόλιο... - Στοίχιση: ΟΚ - Σταθερές: ΟΚ - Ονόµατα µεταβλητών: ΟΚ - Τύποι πεδίων struct: ΟΚ - ορθότητα create ΟΚ - ορθότητα print: ΟΚ - ορθότητα add: προσθέσατε ένα παραπάνω πεδίο, int count. Δεν σας ζητήσαµε τέτοια συνάρτηση. Θα µπορούσατε να περνούσατε την διεύθυνση της επόµενης θέσης που µπορείτε να γράψετε αντί για το students - ορθότητα init: ΟΚ - ορθότητα sprintf: ΟΚ - κλήση συναρτήσεων: ΟΚ - Γενικά: Κλέψατε χρησιµοποιόντας ένα ακόµα πεδίο. Κανονικά θα έπρεπε να κάνατε αναζήτηση για να βρείτε κενή θέση - Αρχικοποίηση πίνακα pointers: Δεν αρχικοποιείται - Χρήση argc, argv: ΟΚ - Παράκαµψη NULL θέσεων στην τελική εκτύπωση: Δεν εκτυπώνει κάτι. - Γενικά: Αν έχω καταλάβει σωστά, πάτε να δηµιουργήσετε στο input (που είναι ένα string) πολλά string βάζοντας σε κάθε θέση που ξεκινάει ένα delimeter το '\0'. Αυτή δεν είναι άσχηµη ιδέα. Θα πρέπει λοιπόν στον πίνακα µε του δείκτες να βάζετε τις αρχές αυτών των strings όπως και κάνετε. Άρα µετά γιατί δεν τα τυπώνετε µε %s? Στοίχιση µε πάρα πολλές αλλαγές γραµµής FAIL - Καθόλου σχόλια! - Η init_students() πρέπει να αρχικοποιεί όλα τα πεδία του struct και όχι µόνο το firstname. - Η add_student_info() δεν εισάγει στην πρώτη ελεύθερη θέση του πίνακα µαθητών, αλλά κάνει overwrite τις ήδη γεµάτες θέσεις µε τα στοιχεία της νέας καταχώρησης. - Η strcpy() παίρνει ως παραµέτρους δυο pointers σε χαρακτήρα. Μέσα στην add_student_info(), µόνο η πρώτη χρήση της είναι σωστή. Για τις άλλες σας βγάζει warning ο compiler, τα οποία ΔΕΝ πρέπει να αγνοείτε! - Δεν υλοποιήσατε την print_students(). - Το format_str για την εισαγωγή του ΑΕΜ δεν είναι σωστό. Το µέγεθος του ΑΕΜ είναι AEM_LEN. - Η κλήση της συνάρτησης add_student_info() µέσα στην main δεν είναι σωστή. Διαβάστε ξανά πως καλούµε συναρτήσεις. - Καθόλου σχόλια! - Η άσκηση που στείλατε είναι η lab9a.c µε αλλαγµένο όνοµα και δηλώσεις µεταβλητών. - Οι γραµµές του πίνακα split είναι STRLEN, οπότε η συνθήκη για το i µέσα στο εξωτερικό for statement πρέπει να είναι i<strlen. - Η θέση εισαγωγής πρέπει να είναι το µέγεθος της συµβολοσειράς των χαρακτήρων '*' FAIL - Τα σχόλια µέσα στην main() δεν προσφέρουν κάτι (απλά λένε το προφανές). - Σωστή init_students(), θα µπορούσατε να αρχικοποιήσετε τα strings απλά µε ένα '\0' σαν πρώτο χαρακτήρα. - Λάθος add_student_info() γιατί δεν ελέγχετε για προσπέλασή του µέγιστου αριθµού µαθητών. Επίσης τι θα γίνει αν κάποιος φοιτητής είχε όνοµα που αποτελούνταν από 1 γράµµα. Η αναζήτηση µπορεί να παραληφθεί εάν περνάτε σαν παράµετρο κατευθείαν τον pointer στην πρώτη ελεύθερη θέση (&students[count-1]). - Σωστή print_students() και σωστά χρησιµοποιείτε την print_students() σε συνδυασµό µε το students_count. - Καλή δουλειά. - Δεν κάνει compile. Page 3

4 Διαδικαστικά: ΟΚ 1910 PASS - Σχόλια: OK - Στοίχιση: OK - Σταθερές: OK - Ονόµατα µεταβλητών: ΟΚ - Τύποι πεδίων struct: ΟΚ - ορθότητα create ΟΚ - ορθότητα print: ΟΚ - ορθότητα add: ΟΚ - ορθότητα init: ΟΚ - ορθότητα sprintf: ΟΚ - κλήση συναρτήσεων: ΟΚ - Γενικά: Σωστός τρόπος περάσµατος της επόµενης κενής θέσης. - Αρχικοποίηση πίνακα pointers: OK - Χρήση argc, argv: OK - Παράκαµψη NULL θέσεων στην τελική εκτύπωση: OK - Γενικά: Έχετε ένα πολύ σοβαρό λάθος που καταστρέφει το πρόγραµµά σας. Στη γραµµή endptr=strstr(string,delimiter) δοκιµάστε αντί για string το startptr. Κατά τα άλλα δουλεύει ικανοποιητικά. Το string[100] είναι αρκετά άσχηµο. Επίσης αν δεν υπάρχει τελικός delimeter, το πρόγραµµά σας δεν θα εκτυπώσει την τελευταία σειρά µε αστεράκια Διαδικαστικά ΟΚ Άσκηση 1η 1911 PASS - Η θέση των σχολίων είναι πάνω από τις συναρτήσεις και όχι δίπλα από αυτές, έτσι ώστε να διατηρούµε ένα πλάτος κώδικα περίπου στις 100 στήλες. Περιεχόµενο σχολίων ΟΚ. - ορισµός πεδίων struct student: ΟΚ - init_students: ΟΚ, σωστή αρχικοποίηση. Θα µπορούσατε να αρχικοποιήσετε µόνο τον 1 χαρακτήρα σε '\0', καθώς µόνο αυτό ελέγχετε στην add_student_info. - create_ ΟΚ - add_student_info: ΟΚ. Εναλλατικά, αντί να ψάχνετε την 1η κενή θέση θα µπορούσατε να χρησιµοποιήσετε ένα counter στη main και να περνάτε την διεύθυνση της 1ης κενής θέσης στον πίνακα των structs. - print_students: ΟΚ. Εναλλακτικά (οµοίως µε την add_student_info), αντί να ψάχνετε για τις κατηλληµένες θέσεις θα µπορούσατε να περνάτε τον αριθµό των θέσεων που έχουν καταληφθεί. Άσκηση 2η - Για τα σχόλια ισχύει ότι και στην άσκηση 1. Αν έχετε απορίες ρωτήστε µας. - Σωστή χρήση argc, argv. Θα µπορούσατε να ελέγχετε εάν πράγµατι έχει βάλει ο χρήστης τουλάχιστον 2 ορίσµατα και σε διαφορετική περίπτωση να τερµατίζετε το πρόγραµµα. - Αλγοριθµικά ΟΚ, το πρόγραµµα δουλεύει καλά. Καλή προσπάθεια. Εξοικειωθείτε στη χρήση του το gdb, ώστε να εντοπίζετε τα λάθη σας στο µέλλον FAIL -Το πρόγραµµα δεν κάνει compile -Δεν έγινε Page 4

5 1914 PASS` - Δεν υπάρχουν σχόλια. - Σωστή init_students(). Καλύτερο θα ήταν να αρχικοποιούσατε ολόκληρο το string µε \0 έτσι ώστε όταν κάνετε copy κάποιο άλλο να µην εξαφανιστεί αυτό που θα βρίσκεται στην πρώτη θέση. - Σωστή add_student_info(). Μια βελτιστοποίηση είναι οτι η αναζήτηση µπορεί να παραληφθεί εάν περνάτε σαν παράµετρο κατευθείαν τον pointer στην πρώτη ελεύθερη θέση (&students[students_count-1]). - Σωστή print_students(). Επίσης για παράµετρο θα µπορούσατε να περνάτε το students_count αντί του MAX_STUDENTS για να απαλείψετε τον έλεγχο κενής θέσης. - Δεν κάνει σωστά αυτό που ζητάµε. Ενα βασικό λάθος το οποίο υπάρχει και σας το τονίζει και ο compiler είναι οτι στο star_group µέσα στο 2ο for loop κάνετε ανάθεση έναν χαρακτήρα ενώ έπρεπε να αναθέσετε το δείκτη απο εκεί που ξεκινάει το string. Μπορείτε να δείτε πως διορθώνεται το πρόβληµα; Πρέπει να εισάγετε ένα επιπλέον σύµβολο. -Προσπαθήστε να βρείτε γιατί δεν ολοκληρώνεται το loop και συνεπώς το πρόγραµµα για να λειτουργήσει η άσκηση σας. Χρησιµοποιείτε το δείκτη i και στα 2 loop. Μπορείτε να δείτε γιατί αυτό είναι πρόβληµα και προκαλεί ατέρµονο loop; Διαδικαστικά: ΟΚ 1915 PASS - Σχόλια: κανένα σχόλιο... - Στοίχιση: ΟΚ - Σταθερές: ΟΚ - Ονόµατα µεταβλητών: ΟΚ - Τύποι πεδίων struct: ΟΚ - ορθότητα create ΟΚ - ορθότητα print: ΟΚ - ορθότητα add: προσθέσατε ένα παραπάνω πεδίο, int count. Δεν σας ζητήσαµε τέτοια συνάρτηση. Θα µπορούσατε να περνούσατε την διεύθυνση της επόµενης θέσης που µπορείτε να γράψετε αντί για το students - ορθότητα init: ΟΚ - ορθότητα sprintf: ΟΚ - κλήση συναρτήσεων: ΟΚ - Γενικά: Κλέψατε χρησιµοποιόντας ένα ακόµα πεδίο. Κανονικά θα έπρεπε να κάνατε αναζήτηση για να βρείτε κενή θέση - Αρχικοποίηση πίνακα pointers: Δεν αρχικοποιείται - Χρήση argc, argv: ΟΚ - Παράκαµψη NULL θέσεων στην τελική εκτύπωση: Δεν εκτυπώνει κάτι. - Γενικά: Αν έχω καταλάβει σωστά, πάτε να δηµιουργήσετε στο input (που είναι ένα string) πολλά string βάζοντας σε κάθε θέση που ξεκινάει ένα delimeter το '\0'. Αυτή δεν είναι άσχηµη ιδέα. Θα πρέπει λοιπόν στον πίνακα µε του δείκτες να βάζετε τις αρχές αυτών των strings όπως και κάνετε. Άρα µετά γιατί δεν τα τυπώνετε µε %s? Στοίχιση µε πάρα πολλές αλλαγές γραµµής. Page 5

6 1917 FAIL, σας έχουν ξεφύγει µερικά σηµεία. -Δεν έχετε καθόλου σχόλια -Καλά ονόµατα µεταβλητών, σωστα πεδία στην struct -Το πρόγραµµα δεν έτρεχε σωστά γιατί έχετε λάθος και στην αναζήτηση που κάνετε στην add_book_info και στον τρόπο που µετακινείτε τα κελιά του string στην strip_dashes.επίσης, µετά την αφαίρεση των dashes, πάτε και αντιγράφετε µέσα στο str το άδειο temp_string και το χαλάτε. Προσπαθήστε να βρείτε το λάθος σας τρέχοντας τον κώδικα της συνάρτησης µέσα από gdb (δώστε πρώτα πιο µικρές τιµές στο TITLE_LEN και στο MAX_BOOKS για να τελειώνουν πιο γρήγορα τα loops. -Είχε σχόλια - Να χρησιµοποιείτε τις συναρτήσεις που ήδη υπάρχουν (πχ strlen). - Η δουλειά της print είναι να εκτυπώνει. Γιατί θέτετε το point[i] σε NULL µέσα σε αυτή? - Ο delimiter έχει µέχρι 3 στοιχεία, όχι ακριβώς 3, εποµένως ο έλεγχος για delimiter[0], [1] και [2] δεν είναι πάντα σωστός. - Για την αρχικοποίηση του point[i] έπρεπε να χρησιµοποιείτε διαφορετική θέση (πχ. i) για τον point και διαφορετική (πχ κάποιο j) για τον string. Γενικά φαίνεται να έχετε σοβαρό πρόβληµα στο αλγοριθµικό κοµµάτι. Τα προγράµµατά σας κάνουν µεν compile αλλά καµία από τις συναρτήσεις σας δεν είναι γραµµένη σωστά. Χρειάζεται να λύσετε πολλές παλιές ασκήσεις για εξάσκηση, και να ζητάτε βοήθεια όταν δυσκολεύεστε σε αυτές FAIL - Τα σχόλια µέσα στην main() δεν προσφέρουν κάτι (απλά λένε το προφανές). - Ένα σηµαντικό λάθος που κάνατε ήταν να µην ορίσετε σωστά το format string µε αποτέλεσµα να µην διαβάζει σωστά την είσοδο που δίνει ο χρήστης. Μπορείτε να δείτε ποιό είναι το λάθος σας; Μπορείτε να εκτυπώσετε το format string και να δείτε τι λείπει. - Σωστή init_students(). Καλύτερο θα ήταν να αρχικοποιούσατε ολόκληρο το string µε \0 έτσι ώστε όταν κάνετε copy κάποιο άλλο να µην εξαφανιστεί αυτό που θα βρίσκεται στην πρώτη θέση. - Λάθος δηµιουργία . Σχηµατίζετε το mail µε τον σωστό τρόπο αλλά δεν το αποθηκεύετε στη σωστή διεύθυνση. Έπρεπε να το βάλετε στη διεύθυνση που σας δίνεται σαν παράµετρο και όχι σε έναν τοπικό πίνακα καθώς το αποτέλεσµα δεν θα περάσει στη main. - Σωστή add_student_info(). Μια βελτιστοποίηση είναι οτι η αναζήτηση µπορεί να παραληφθεί εάν περνάτε σαν παράµετρο κατευθείαν τον pointer στην πρώτη ελεύθερη θέση (&students[students_count-1]). - Σωστή print_students(). - Επίσης για παράµετρο θα µπορούσατε να περνάτε το students_count αντί του MAX_STUDENTS για να απαλείψετε τον έλεγχο κενής θέσης. - Δεν κάνει compile. Έχετε πολλά λάθη. Θα τονίσω οτι µε NULL αρχικοποιούνται διευθύνσεις. Εσείς αρχικοποιείτε µε NULL έναν ολόκληρο πίνακα απο διευθύνσεις. Διαδικαστικά ΟΚ Άσκηση 1η 1922???? - Στοίχιση: ΟΚ - Σχόλια: Η θέση των σχολίων είναι πάνω από τις συναρτήσεις και όχι δίπλα από αυτές, έτσι ώστε να διατηρούµε ένα πλάτος κώδικα περίπου στις 100 στήλες. Επίσης δεν γράφουµε σχόλια µε ελληνικούς χαρακτήρες. Περιεχόµενο σχολίων ΟΚ. - ορισµός πεδίων struct student: ΟΚ - init_students: ΟΚ - create_ ΟΚ - add_student_info: ΟΚ. Εναλλακτικά, αντί να ψάχνετε την 1η κενή θέση θα µπορούσατε να χρησιµοποιήσετε ένα counter στη main και να περνάτε την διεύθυνση της 1ης κενής θέσης στον πίνακα των structs. - print_students: ΟΚ. Εναλλακτικά (οµοίως µε την add_student_info), αντί να ψάχνετε για τις κατειληµµένες θέσεις θα µπορούσατε να περνάτε τον αριθµό των θέσεων που έχουν καταληφθεί. Άσκηση 2η - Στοίχιση: ΟΚ - Ποιός σας είπε ότι ο delimiter είναι πάντα $$; - Πολύ λάθος σκέψη για τη λύση. Τι θα κάνατε αν αντί για 10 ο πίνακας είχε µέγεθος 100? Page 6

7 1925 FAIL Άσκηση 1η: - Σωστή στοίχιση - Σωστά σχόλια - Σωστός ορισµός των πεδίων struct - Σωστή υλοποίηση της init_books - Σωστή υλοποίηση της add_book_info (την καλείτε δυο φορές στην main, µάλλον καταλάθος): Εναλλακτικά, αντί να ψάχνετε την 1η κενή θέση θα µπορούσατε να χρησιµοποιήσετε ένα counter στη main και να περνάτε την διεύθυνση της 1ης κενής θέσης στον πίνακα των structs. - Σωστή υλοποίηση της strip_dashes - Σχεδόν σωστή υλοποίηση της print_books αλλά έχετε ένα περίεργο bug, δεν εκτυπώνετε σωστά τα αποτελέσµατα. Άσκηση 2η: -Δεν κάνει compile, δείτε την λύση και ξαναπροσπαθείστε στο σπίτι. Περισσότερο συµµετοχή την επόµενη φορά PASS -Καλά σχόλια -Σωστά ονόµατα µεταβλητών, σωστά πεδία στην struct. Πιστέυω δεν χρειαζόταν να δηλώσετε LENGTH+1 για τα πεδία, το µέγεθος 100 είναι αρκετά µεγάλο για να περιέχει και το '/0' -Το πρόγραµµα τρέχει σωστά -Η strip dashes δουλεύει σωστά. -Η add books δουλεύει σωστά. Θα µπορούσατε αντί να ψάχνετε το πρώτο ελεύθερο πεδίο του πίνακα των structs να στέλνετε στην συνάρτηση τον δείκτη προς την διεύθυνση που θέλετε να περάσετε. -Σωστό τύπωµα των βιβλίων Θα µπορούσατε να στέλνετε το count αντι για το MAX_LENGTH και να είχατε αποφύγει µια σύνθετη συνθήκη για τον τερµατισµο της for. -Σωστή init. -Δεν έχετε σχόλια -Η άσκηση τρέχει, παίρνει σωστά input από την γραµµή εντολών -Δεν βγάζει σωστά αποτελέσµατα όµως.τυπώνει το string ίδιο αλλά αντί για το delimiter έχει αριθµούς PASS Άσκηση 1η: - Πολύ καλή δουλειά! Μπράβο - Σωστή στοίχιση - Σωστά σχόλια - Σωστός ορισµός των πεδίων struct - Σωστή υλοποίηση της init_books - Σωστή υλοποίηση της add_book_info: Εναλλακτικά, αντί να ψάχνετε την 1η κενή θέση θα µπορούσατε να χρησιµοποιήσετε ένα counter στη main και να περνάτε την διεύθυνση της 1ης κενής θέσης στον πίνακα των structs. - Σωστή υλοποίηση της strip_dashes - Σωστή υλοποίηση της print_books Άσκηση 2η: - Καλή δουλειά - Σωστή αρχικοποίηση πίνακα pointers - Σωστή χρήση argc, argv - Σωστά παρακάµπτετε τις NULL θέσεις κατά την εκτύπωση αλλά εκτυπώνετε κάποια επιπλέον strings µε 'σκουπίδια'. Εναλλακτικά θα µπορούσατε να χρησιµοποιήσετε την strpbrk. Δείτε την λύση. Page 7

8 1928 PASS - Δεν υπάρχουν σχόλια. - Σωστή init_students(). Καλύτερο θα ήταν να αρχικοποιούσατε ολόκληρο το string µε \0 έτσι ώστε όταν κάνετε copy κάποιο άλλο να µην εξαφανιστεί αυτό που θα βρίσκεται στην πρώτη θέση. - Σωστή add_student_info(). Μια βελτιστοποίηση είναι οτι η αναζήτηση µπορεί να παραληφθεί εάν περνάτε σαν παράµετρο κατευθείαν τον pointer στην πρώτη ελεύθερη θέση (&students[students_count-1]). - Σωστή print_students(). Επίσης για παράµετρο θα µπορούσατε να περνάτε το students_count αντί του MAX_STUDENTS για να απαλείψετε τον έλεγχο κενής θέσης. - Δεν κάνει σωστά αυτό που ζητάµε. Ενα βασικό λάθος το οποίο υπάρχει και σας το τονίζει και ο compiler είναι οτι στο star_group µέσα στο 2ο for loop κάνετε ανάθεση έναν χαρακτήρα ενώ έπρεπε να αναθέσετε το δείκτη απο εκεί που ξεκινάει το string. Μπορείτε να δείτε πως διορθώνεται το πρόβληµα; Πρέπει να εισάγετε ένα επιπλέον σύµβολο. -Προσπαθήστε να βρείτε γιατί δεν ολοκληρώνεται το loop και συνεπώς το πρόγραµµα για να λειτουργήσει η άσκηση σας. Χρησιµοποιείτε το δείκτη i και στα 2 loop. Μπορείτε να δείτε γιατί αυτό είναι πρόβληµα και προκαλεί ατέρµονο loop; 1929 FAIL - Τα σχόλια µέσα στην main() δεν προσφέρουν κάτι (απλά λένε το προφανές). - Σωστή init_students(). - Σωστή add_student_info(). Η αναζήτηση µπορεί να παραληφθεί εάν περνάτε σαν παράµετρο - Σωστή print_students(). Η κλήση της έπρεπε να γίνεται έξω από την do-while. - Επίσης για παράµετρο θα µπορούσατε να περνάτε το students_count αντί του MAX_STUDENTS για να απαλείψετε τον έλεγχο κενής θέσης. - Δεν κάνει compile. Διαδικαστικά ΟΚ Άσκηση 1η 1931 PASS - Σωστή στοίχιση και σχόλια. - Ονόµατα πεδίων struct: σωστά. - init_students: σωστή αρχικοποίηση. Θα µπορούσατε να αρχικοποιήσετε µόνο τον 1 χαρακτήρα σε '\0', καθώς µόνο αυτό ελέγχετε στην add_student_info. - create_ OK, ωραία ιδέα η χρήση της sprintf(), αν και στην εκτέλεση της είναι πιο αρχή από strcat/strcpy. - add_student_info: σωστή. Αντί να ψάχνετε την 1η κενή θέση θα µπορούσατε να χρησιµοποιήσετε τον counter sum στη main και να περνάτε την διεύθυνση της 1ης κενής θέσης στον πίνακα των structs. - print_students: OK Άσκηση 2η - Σωστή στοίχιση, τα σχόλια καλό είναι να µπαίνουν πάνω από τον κώδικα και όχι δεξιά. - Σωστή χρήση argc, argv. Θα µπορούσατε να ελέγχετε εάν πράγµατι έχει βάλει ο χρήστης τουλάχιστον 2 ορίσµατα και σε διαφορετική περίπτωση να τερµατίζετε το πρόγραµµα. - Αλγοριθµικά καθαρή λύση. Πολύ καλά! Συνεχίστε έτσι. Page 8

9 1932 PASS - Καλά σχόλια. - Σωστοί τύποι και περιγραφικά ονόµατα πεδίων του struct. - Σωστή init_students(). - Σωστή add_student_info(). Αντί για την έξτρα παράµετρο που περνάτε, µπορείτε να περνάτε - Σωστή print_students() και εκτύπωση µόνο των γεµάτων θέσεων. - Καλά σχόλια. - Σωστή print_asterisks(). - Η προσθήκη του delimiter στο τέλος του αρχικού string, µε τον τρόπο που είναι γραµµένη δεν είναι ασφαλής εντολή. Εκτός του ότι µπορεί η είσοδος να έχει ήδη στο τέλος delimiter, το βασικότερο πρόβληµα είναι πως τροποποιούµε µνήµη που δεν γνωρίζουµε το µέγεθος της. Δεν υπάρχει τρόπος να εγγυηθούµε ότι µετά το τέλος του argv[i] υπάρχει διαθέσιµος κενός χώρος ή εάν θα βγούµε εκτός ορίων του πίνακα όπου βρίσκεται αποθηκευµένη η συµβολοσειρά PASS - Καλά σχόλια. - Σωστοί τύποι και περιγραφικά ονόµατα πεδίων του struct. - Σωστή init_students(). - Σωστή add_student_info(). Αντί για την έξτρα παράµετρο που περνάτε, µπορείτε να περνάτε - Σωστή print_students() και εκτύπωση µόνο των γεµάτων θέσεων. - Καλά σχόλια. - Σωστή print_asterisks(). - Η προσθήκη του delimiter στο τέλος του αρχικού string, µε τον τρόπο που είναι γραµµένη δεν είναι ασφαλής εντολή. Εκτός του ότι µπορεί η είσοδος να έχει ήδη στο τέλος delimiter, το βασικότερο πρόβληµα είναι πως τροποποιούµε µνήµη που δεν γνωρίζουµε το µέγεθος της. Δεν υπάρχει τρόπος να εγγυηθούµε ότι µετά το τέλος του argv[i] υπάρχει διαθέσιµος κενός χώρος ή εάν θα βγούµε εκτός ορίων του πίνακα όπου βρίσκεται αποθηκευµένη η συµβολοσειρά FAIL - Σχόλια OK. - Σωστή init_students(), θα µπορούσατε να αρχικοποιήσετε τα strings απλά µε ένα '\0' σαν πρώτο χαρακτήρα. - Λάθος add_student_info(). Η αναζήτηση µπορεί να παραληφθεί εάν περνάτε σαν παράµετρο - Σωστή print_students() αλλά δεν χρησιµοποιείτε σωστά την print_students(). Έπρεπε να το κάνετε σε συνδυασµό µε το students_count. Δεν χρειαζόταν να χρησιµοποιήσετε κάποιο έλεγχο για '\0'. - Καλή δουλειά. - Δεν κάνει compile. Page 9

10 Διαδικαστικά:ΟΚ 1936 FAIL - Σχόλια: Αρκετά, αν και θα µπορούσατε να περιγράψετε και άλλα πράγµατα (πχ τι κάνουν οι µεταβλητές suffix, dot_str κτλ) - Στοίχιση: ΟΚ - Σταθερές: ΟΚ - Ονόµατα µεταβλητών: ΟΚ - Τύποι πεδίων struct: ΟΚ - ορθότητα create ΟΚ - ορθότητα add: θέλετε να συγκρίνετε το firstname[0] και όχι σκέτο το firstname! Το πρώτο είναι ο πρώτος χαρακτήρας (όπως σωστά αρχικοποιήσατε) το δεύτερο είναι διεύθυνση. Οπότε η διεύθυνση δεν είναι ποτέ!=0 και πέφτετε σε άπειρο loop - ορθότητα print: Δείτε και παραπάνω - ορθότητα init: ΟΚ - ορθότητα sprintf: ΟΚ - κλήση συναρτήσεων: ΟΚ - Γενικά: Έχετε ένα πολύ σοβαρό λάθος που περιέγραψα παραπάνω. Κατά τα άλλα το πρόγραµµά σας δουλεύει. - Compiler errors 1942 FAIL - Τα σχόλια µέσα στην main() δεν προσφέρουν κάτι (απλά λένε το προφανές). - Σωστή init_students(), θα µπορούσατε να αρχικοποιήσετε τα strings απλά µε ένα '\0' σαν πρώτο χαρακτήρα. - Λάθος add_student_info() γιατί δεν ελέγχετε για προσπέλασή του µέγιστου αριθµού µαθητών. Επίσης τι θα γίνει αν κάποιος φοιτητής είχε όνοµα που αποτελούνταν από 1 γράµµα. Η αναζήτηση µπορεί να παραληφθεί εάν περνάτε σαν παράµετρο κατευθείαν τον pointer στην πρώτη ελεύθερη θέση (&students[count-1]). - Σωστή print_students() και σωστά χρησιµοποιείτε την print_students() σε συνδυασµό µε το students_count. - Καλή δουλειά. - Δεν κάνει compile. Παπαγάτσια Πανάγου Διαδικαστικά: ΟΚ 1946 PASS - Σχόλια: Μόνο στην αρχή - Στοίχιση: ΟΚ - Σταθερές: ΟΚ - Ονόµατα µεταβλητών: ΟΚ - Τύποι πεδίων struct: ΟΚ - ορθότητα create ΟΚ - ορθότητα print: ΟΚ - ορθότητα add: ΟΚ - ορθότητα init: ΟΚ - ορθότητα sprintf: ΟΚ - κλήση συναρτήσεων: ΟΚ - Γενικά: Σωστός τρόπος περάσµατος της επόµενης κενής θέσης. - Αρχικοποίηση πίνακα pointers: OK - Χρήση argc, argv: OK - Παράκαµψη NULL θέσεων στην τελική εκτύπωση: OK - Γενικά: Το πρόβληµα είναι ότι υπάρχει ένα ερωτηµατικό στο τέλος του for που βρίσκεται µέσα στην else! Επίσης δεν βλέπω κανένα σχόλιο... Page 10

11 1947 FAIL - Ελάχιστα σχόλια. Αυτά που έχετε θα µπορούσαν να µπούν πάνω απο την κάθε συνάρτηση. - Σωστή init_books(). Καλύτερο θα ήταν να αρχικοποιούσατε ολόκληρο το string µε \0 έτσι ώστε όταν κάνετε copy κάποιο άλλο να µην εξαφανιστεί αυτό που θα βρίσκεται στην πρώτη θέση. - Σωστή αφαίρεση των '-' απο τον κωδικό ISBN10. - Σωστή add_book_info(). Μια βελτιστοποίηση είναι οτι η αναζήτηση µπορεί να παραληφθεί εάν περνάτε σαν παράµετρο κατευθείαν τον pointer στην πρώτη ελεύθερη θέση (&books[count-1]). - Σωστή print_students(). Επίσης για παράµετρο θα µπορούσατε να περνάτε το count αντί του MAX_BOOKS για να απαλείψετε τον έλεγχο κενής θέσης. - Δεν κάνει compile. Προσπαθείστε να διορθώσετε τη συνάρτηση που έχετε δηµιουργήσει και να της βάλετε σωστά ορίσµατα για να την καλείτε. Αν η συνάρτηση µπεί σε σχόλια τότε κάνει compile. Αν έχετε τέτοιου είδους προβλήµατα σε δευτερεύουσες συναρτήσεις καλό είναι να τις βάζετε σε σχόλια µε κάποια υποσηµείωση για να µπορεί να κάνει το προγραµµά σας compile. - Σωστά αφαιρείτε τα delimeters απο το string που δίνει ο χρήστης αλλά δεν ορίζετε σωστά τη κάθε αρχή του κάθε substring στον πίνακα array_ptr. Ενα βασικό λάθος που έχετε κάνει είναι οτι δεν χρησιµοποιείτε το δείκτη j στο εσωτερικό loop. Προσπαθήστε να ολοκληρώσετε τη σκέψη σας και να διορθώσετε την άσκηση. 1949???? -Το πρόγραµµα τερµατίζει µε Segmentation fault -Στοίχιση και σχόλια οκ -Καλά ονόµατα µεταβλητών και τύποι πεδίων στη struct -Δεν υλοποιείται σωστά την strip_dashes -Σωστά υλοποιείται την add_book_info - Δεν υλοποιείται σωστά την init αν θέλετε να αρχικοποιήσετε όλα τα στοιχεία µε \0 θα πρέπει να πάρετε ενα for εξωτερικό για το book και 3 εσωτερικά for για κάθε πεδίο όπου θα τρέχουν µέχρι το max µέγεθος του καθενός. -Σωστά υλοποιείται την συνάρτηση print -Σωστά υλοποιείται τις sprintf -Σωστά καλείται τις συναρτήσεις - -Το πρόγραµµα σας βγάζει warnings και τερµατίζει µε Segmentation fault -Δεν ελέγχετε πουθενά αν δόθηκε από τη γραµµή εντολών 2 string -Δεν κάνετε σωστή χρήση του argv -Δεν υλοποιείται σωστά το πρόγραµµα 1950 PASS - Καλά σχόλια - Σωστή init_students(), θα µπορούσατε να αρχικοποιήσετε τα strings απλά µε ένα '\0' σαν πρώτο χαρακτήρα. - Στην create_ θα µπορούσατε να γράψετε απευθείας στο " ". Δεν υπάρχει λόγος για τη fullname. - Στην add_student_info() η αναζήτηση µπορεί να παραληφθεί εάν περνάτε σαν παράµετρο - Στην print_students() θα ήταν πιο λογικό να περνάτε ως δεύτερη παράµετρο το students_count. - Καλή δουλειά. - Σωστή αρχικοποίηση. - Λάθος έλεγχος µεταξύ string και χαρακτήρα. ( το -Wall το είδατε; ) - Τελειώσετε την άσκηση Page 11

12 1955 FAIL - Σχόλια OK. - Σωστή init_students(), θα µπορούσατε να αρχικοποιήσετε τα strings απλά µε ένα '\0' σαν πρώτο χαρακτήρα. - Λάθος add_student_info(). Η αναζήτηση µπορεί να παραληφθεί εάν περνάτε σαν παράµετρο - Σωστή print_students() και χρησιµοποιείτε σωστά την print_students() σε συνδυασµό µε το students_count. - Καλή δουλειά. - Δεν κάνει compile Απουσία Διαδικαστικά OK Άσκηση 1η 1957 PASS - Στοίχιση: Σωστή. - Σχόλια: Έχετε σχόλια στην αρχή του προγράµµατος σας. Απουσιάζουν σχόλια πριν από κάθε συνάρτηση σχετικά µε την λειτουργία της (τι κάνει, ορίσµατα, επιστρεφόµενη τιµή αν δεν επιστρέφει void). - ορισµός πεδίων struct student: ΟΚ - init_students: ΟΚ - create_ ΟΚ - add_student_info: ΟΚ. Εναλλακτικά, αντί να ψάχνετε την 1η κενή θέση θα µπορούσατε να χρησιµοποιήσετε ένα counter στη main και να περνάτε την διεύθυνση της 1ης κενής θέσης στον πίνακα των structs. - print_students: Η συνάρτηση θα πρέπει να εκτυπώνει όλους τους φοιτητές σε µία κλήση και όχι ένα-ένα. Η δοµή επανάληψης θα έπρεπε να υπάρχει µέσα στην print_students και όχι µέσα στη main. Άσκηση 2η - Τραγική στοίχιση! - Απουσία χρήσης argc, argv. - Αντιγράψατε κοµµάτια της άσκησης 1 του lab9. - O αλγόριθµος δεν δουλεύει. 1959???? -Δεν έχετε σχόλια -Σωστα ονόµατα µεταβλητών,σωστά πεδία στην struct -Το πρόγραµµα τρέχει σωστά -Η strip dashes δουλεύει σωστά -Σωστή η add_books, θα µπορούσατε αντί να ψάχνετε το πρώτο ελεύθερο πεδίο του πίνακα των structs να στέλνετε στην συνάρτηση τον δείκτη προς την διεύθυνση που θέλετε να περάσετε. -Σωστό τύπωµα των βιβλίων, θα µπορούσατε αντί να κάνετε break όταν βρίσκετε το πρώτο κενό πεδίο του πίνακα να στέλνετε το count αντί του MAX_LENGTH Που στέλνετε εσείς. -Σωστή αρχικοποιήση του πίνακα books. -'Eβγαζε error στο compile,(δεν είχατε δηλώσει την µεταβλητή ptr) -Φαίνεται οτι την προσπαθήσατε όµως. - Πολύ καλή δουλειά, µπράβο PASS - Πριν προσπελάσετε το argv πρέπει να ελέγξετε την τιµή του argc ώστε να είστε σίγουροι ότι δόθηκαν τα ζητούµενα strings στη γραµµή εντολής. - Σωστή κατά τα άλλα Page 12

13 1962 FAIL - Ελάχιστα σχόλια. Αυτά που έχετε θα µπορούσαν να µπούν πάνω απο την κάθε συνάρτηση. - Σωστή init_books(). Καλύτερο θα ήταν να αρχικοποιούσατε ολόκληρο το string µε \0 έτσι ώστε όταν κάνετε copy κάποιο άλλο να µην εξαφανιστεί αυτό που θα βρίσκεται στην πρώτη θέση. - Σωστή αφαίρεση των '-' απο τον κωδικό ISBN10. - Σωστή add_book_info(). Μια βελτιστοποίηση είναι οτι η αναζήτηση µπορεί να παραληφθεί εάν περνάτε σαν παράµετρο κατευθείαν τον pointer στην πρώτη ελεύθερη θέση (&books[count-1]). - Σωστή print_students(). - Έχετε κάνει λάθος στη χρήση του πίνακα δεικτών. Ο πίνακας αυτός δεν είναι string αλλά είναι πίνακας απο δείκτες που δείχνουν σε string. - Σωστά διαβάζετε την είσοδο µέσω του argv αλλά δεν την αντιγράφετε σωστά µε την strcopy γεγονός που σας τονίζει και ο compiler. - Κάνετε σύγκριση µεταξύ strings χρησιµοποιώντας ==. Θα έπρεπε να γίνει χρήση της strcmp. Διαδικαστικά ΟΚ Άσκηση 1η 1964 PASS - Σωστή στοίχιση και σχόλια. - Ονόµατα πεδίων struct: σωστά. - init_students: σωστή αρχικοποίηση. Θα µπορούσατε να αρχικοποιήσετε µόνο τον 1 χαρακτήρα σε '\0', καθώς µόνο αυτό ελέγχετε στην add_student_info. - create_ OK, ωραία ιδέα η χρήση της sprintf(), αν και στην εκτέλεση της είναι πιο αρχή από strcat/strcpy. - add_student_info: σωστή. Αντί να ψάχνετε την 1η κενή θέση θα µπορούσατε να χρησιµοποιήσετε τον counter sum στη main και να περνάτε την διεύθυνση της 1ης κενής θέσης στον πίνακα των structs. - print_students: OK Άσκηση 2η - Σωστή στοίχιση, τα σχόλια καλό είναι να µπαίνουν πάνω από τον κώδικα και όχι δεξιά. - Σωστή χρήση argc, argv. Θα µπορούσατε να ελέγχετε εάν πράγµατι έχει βάλει ο χρήστης τουλάχιστον 2 ορίσµατα και σε διαφορετική περίπτωση να τερµατίζετε το πρόγραµµα. - Αλγοριθµικά καθαρή λύση. Πολύ καλά! Συνεχίστε έτσι. 1965???? -Το πρόγραµµα τερµατίζει µε Segmentation fault -Στοίχιση και σχόλια οκ -Καλά ονόµατα µεταβλητών και τύποι πεδίων στη struct -Δεν υλοποιείται σωστά την strip_dashes -Σωστά υλοποιείται την add_book_info - Δεν υλοποιείται σωστά την init αν θέλετε να αρχικοποιήσετε όλα τα στοιχεία µε \0 θα πρέπει να πάρετε ενα for εξωτερικό για το book και 3 εσωτερικά for για κάθε πεδίο όπου θα τρέχουν µέχρι το max µέγεθος του καθενός. -Σωστά υλοποιείται την συνάρτηση print -Σωστά υλοποιείται τις sprintf -Σωστά καλείται τις συναρτήσεις - -Το πρόγραµµα σας βγάζει warnings και τερµατίζει µε Segmentation fault -Δεν ελέγχετε πουθενά αν δόθηκε από τη γραµµή εντολών 2 string -Δεν κάνετε σωστή χρήση του argv -Δεν υλοποιείται σωστά το πρόγραµµα -Δεν κάνει compile, και φαίνεται να είναι από λάθος απροσεξίας. Να κάνετε πάντα ένα τελικό compile πριν στείλετε τις ασκήσεις σας FAIL -Το πρόγραµµα τερµατίζει µε Segmentation fault -Σωστά χρησιµοποιείται τo argv[], δεν ελέγχετε πουθενά αν δόθηκε στη γραµµή εντολών 2 string -Δεν υλοποιείται σωστά το πρόγραµµα Page 13

14 1975???? -Στοίχιση και σχόλια οκ -Καλά ονόµατα µεταβλητών και τύποι πεδίων στη struct -Σωστά υλοποιείται την strip_dashes και την add_book_info Έπρεπε να ελέγχετε το πρώτο στοιχείο books[i].author[0]!='\0' ο ιδανικός τρόπος θα ήτανε να περνούσατε ως τελευταία παράµετρο το &books[count-1] -Σωστά υλοποιείται την print_books και την init -Σωστά υλοποιείται τις sprintf -Σωστά καλείται τις συναρτήσεις -Το πρόγραµµά σας βγάζει warnings. Τι ίδιο σχόλιο σας είχε γίνει και στο lab9. Πριν το επόµενο εργαστήριο προσπαθήστε να διορθώσετε τα λάθη σας και να κάνετε περισσότερη εξάσκηση σε κώδικα και σε αναγνώριση µηνυµάτων του compiler. -Δε χρησιµοποιείτε πίνακα 100 θέσεων από pointers που ζητάει η άσκηση -Σωστά πάτε να ελέγξετε για το argc, θα µπορούσατε να ελέγχετε στην αρχή απλά αν argc<3 να τερµατίζει το πρόγραµµα - Δεν αποθηκεύεται πουθενά το string που διαβάζετε από την γραµµή εντολών argv[1] - Δεν υλοποιείται σωστά το πρόγραµµα FAIL - Ελάχιστα σχόλια. Αυτά που έχετε θα µπορούσαν να µπούν πάνω απο την κάθε συνάρτηση. - Σωστή init_books(). Καλύτερο θα ήταν να αρχικοποιούσατε ολόκληρο το string µε \0 έτσι ώστε όταν κάνετε copy κάποιο άλλο να µην εξαφανιστεί αυτό που θα βρίσκεται στην πρώτη θέση. - Σωστή αφαίρεση των '-' απο τον κωδικό ISBN10. - Σωστή add_book_info(). Μια βελτιστοποίηση είναι οτι η αναζήτηση µπορεί να παραληφθεί εάν περνάτε σαν παράµετρο κατευθείαν τον pointer στην πρώτη ελεύθερη θέση (&books[count-1]). - Σωστή print_students(). Επίσης για παράµετρο θα µπορούσατε να περνάτε το count αντί του MAX_BOOKS για να απαλείψετε τον έλεγχο κενής θέσης. - Δεν κάνει compile. Προσπαθείστε να διορθώσετε τη συνάρτηση που έχετε δηµιουργήσει και να της βάλετε σωστά ορίσµατα για να την καλείτε. Αν η συνάρτηση µπεί σε σχόλια τότε κάνει compile. Αν έχετε τέτοιου είδους προβλήµατα σε δευτερεύουσες συναρτήσεις καλό είναι να τις βάζετε σε σχόλια µε κάποια υποσηµείωση για να µπορεί να κάνει το προγραµµά σας compile. - Σωστά αφαιρείτε τα delimeters απο το string που δίνει ο χρήστης αλλά δεν ορίζετε σωστά τη κάθε αρχή του κάθε substring στον πίνακα array_ptr. Ενα βασικό λάθος που έχετε κάνει είναι οτι δεν χρησιµοποιείτε το δείκτη j στο εσωτερικό loop. Προσπαθήστε να ολοκληρώσετε τη σκέψη σας και να διορθώσετε την άσκηση PASS -Καλά σχόλια -Σωστά ονόµατα µεταβλητών, σωστά πεδία στην struct. Πιστέυω δεν χρειαζόταν να δηλώσετε LENGTH+1 για τα πεδία, το µέγεθος 100 είναι αρκετά µεγάλο για να περιέχει και το '/0' -Το πρόγραµµα τρέχει σωστά -Η strip dashes δουλεύει σωστά. -Η add books δουλεύει σωστά. Θα µπορούσατε αντί να ψάχνετε το πρώτο ελεύθερο πεδίο του πίνακα των structs να στέλνετε στην συνάρτηση τον δείκτη προς την διεύθυνση που θέλετε να περάσετε. -Σωστό τύπωµα των βιβλίων Θα µπορούσατε να στέλνετε το count αντι για το MAX_LENGTH και να είχατε αποφύγει µια σύνθετη συνθήκη για τον τερµατισµο της for. -Σωστή init. -Δεν έχετε σχόλια -Η άσκηση τρέχει, παίρνει σωστά input από την γραµµή εντολών -Δεν βγάζει σωστά αποτελέσµατα όµως.τυπώνει το string ίδιο αλλά αντί για το delimiter έχει αριθµούς. Page 14

15 1984 FAIL -Στοίχιση οκ -Δεν υπάρχουν σχόλια -Καλά ονόµατα µεταβλητών -Μέσα στη struct το µέγεθος του isbn δεν είναι MAX_BOOKS αλλά ISBN10 -Δεν υλοποιείται σωστά τη συνάρτηση strip_dashes -Δεν υλοποιείται σωστά τη συνάρτηση add_book -Στην print_books σωστά περνάτε σα παράµετρο το count και δεν εκτυπώνεται µέχρι το MAX -Στην init αφού περνάτε το ΜAX_BOOKS σα παράµετρο στο len έπρεπε να είχατε στο for (i=0; i<len; i++) -Σωστά υλοποιείται τις sprintf -Πρώτα έπρεπε να καλέσετε την strip_dashes και µετά την add_book_info -Σωστά καλείται την print_book -Δεν έγινε Παπαγάτσια Πανάγου Διαδικαστικά: ΟΚ 1985 PASS - Σχόλια: Μόνο στην αρχή - Στοίχιση: ΟΚ - Σταθερές: ΟΚ - Ονόµατα µεταβλητών: ΟΚ - Τύποι πεδίων struct: ΟΚ - ορθότητα create ΟΚ - ορθότητα print: ΟΚ - ορθότητα add: ΟΚ - ορθότητα init: ΟΚ - ορθότητα sprintf: ΟΚ - κλήση συναρτήσεων: ΟΚ - Γενικά: Σωστός τρόπος περάσµατος της επόµενης κενής θέσης. - Αρχικοποίηση πίνακα pointers: OK - Χρήση argc, argv: OK - Παράκαµψη NULL θέσεων στην τελική εκτύπωση: OK - Γενικά: Το πρόβληµα είναι ότι υπάρχει ένα ερωτηµατικό στο τέλος του for που βρίσκεται µέσα στην else! Επίσης δεν βλέπω κανένα σχόλιο PASS - Καλά σχόλια - Σωστή init_students(), θα µπορούσατε να αρχικοποιήσετε τα strings απλά µε ένα '\0' σαν πρώτο χαρακτήρα. - Στην create_ θα µπορούσατε να γράψετε απευθείας στο " ". Δεν υπάρχει λόγος για τη fullname. - Στην add_student_info() η αναζήτηση µπορεί να παραληφθεί εάν περνάτε σαν παράµετρο - Στην print_students() θα ήταν πιο λογικό να περνάτε ως δεύτερη παράµετρο το students_count. - Καλή δουλειά. - Σωστή αρχικοποίηση. - Λάθος έλεγχος µεταξύ string και χαρακτήρα. ( το -Wall το είδατε; ) - Τελειώσετε την άσκηση Page 15

16 1988 PASS -Καλά σχόλια -Σωστο αποτέλεσµα της strip_dashes. Αλλά επειδή χρησιµοποιέιτε την for µέχρι j<isbn10 το [j+1] θα βγαίνει εκτος ορίων του πίνακα, πράγµα που δεν είναι σωστό. -Σωστη η add books. Θα µπορούσατε αντί να ψάχνετε το πρώτο ελεύθερο πεδίο του πίνακα των structs να στέλνετε στην συνάρτηση τον δείκτη προς την διεύθυνση που θέλετε να περάσετε. -Σωστή η init books -Σωστό τύπωµα των βιβλίων -Σωστά ελέγχετε άν ο χρήστης έδωσε στην γραµµή εντολών είσοδο. -Πολύ λίγα σχόλια -Δεν βγάζει σωστό αποτέλεσµα, αλλά φαίνεται οτι την δουλέψατε 1992 PASS - Καλά σχόλια. - Σωστοί τύποι και περιγραφικά ονόµατα πεδίων του struct. Καλό είναι να δηλώνετε σε διαφορετική γραµµή το κάθε πεδίο. - Στην init_students() καλό είναι να αρχικοποιείτε όλες τις θέσεις µνήµης. - Σωστή add_student_info(), αν και θα µπορούσατε να την γράψετε πιο απλά. - Επίσης αντί να κάνετε αναζήτηση για την πρώτη κενή θέση µέσα στην συνάρτηση, θα µπορούσατε να περνάτε την πρώτη κενή θέση σαν πρώτη παράµετρο (&students[count-1]) από την main. - Σωστή print_students() και εκτύπωση µόνο των γεµάτων θέσεων. - Καλά σχόλια. - Το πρόγραµµα τερµατίζει µε sgmentation fault. - Δεν ελέγχετε την τιµή επιστροφής της strstr(). Εάν ο delimiter δεν βρεθεί η επιστρεφόµενη τιµή είναι NULL. - Τα στοιχεία του πίνακα table_ptr είναι pointers αρχικοποιηµένοι σε NULL. Οπότε όταν βρεθεί µία συµβολοσειρά χαρακτήρων '*' µεγέθους len, έπρεπε να αποθηκεύετε την διεύθυνση που ξεκινά (start_ptr) στην θέση table_ptr[len-1]. - Η print_aterisks() πρέπει να εκτυπώνει µόνο '*'. Διαδικαστικά ΟΚ Άσκηση 1η 1994 PASS - Στοίχιση: OK - Σχόλια: Έχετε σχόλια στην αρχή του προγράµµατος σας. Απουσιάζουν σχόλια πριν από κάθε συνάρτηση σχετικά µε την λειτουργία της (τι κάνει, ορίσµατα, επιστρεφόµενη τιµή αν δεν επιστρέφει void). - ορισµός πεδίων struct student: OK - init_students: OK - create_ OK - add_student_info: γραµµή 29, γράφετε if(students[i].firstname =='\0'). Όµως η µεταβλητή students[i].firstname δεν λαµβάνει ποτέ τη τιµή 0 διότι πάντα δείχνει σε µία ισχύουσα διεύθυνση. Το πρόγραµµα σας δεν µπαίνει ποτέ στο if. Το σωστό είναι if(students[i].firstname[0] =='\0') ή if( strlen(students[i].firstname) == 0 ). - print_students: ΟΚ. Έχετε ξεχάσει να βάλετε '\n' στο τέλος κάθε γραµµής. Άσκηση 2η - Στοίχιση: ΟΚ - Σχόλια: ΟΚ - Χρήση argc,argv: OK - Αρχικοποίηση πίνακα δεικτών: ΟΚ - Το πρόγραµµα σας υποθέτει λανθασµένα ότι πάντοτε µετά τον τρέχον delimiter υπάρχει και άλλος ένας delimiter. Απουσιάζει ένας έλεγχος για το αν όντως υπάρχει delimiter ή όχι. Υπολογίζει σωστά όλες τις οµάδες από αστεράκια εκτός της τελευταίας, όπου ο υπολογισµός του n είναι λανθασµένος και το η εντολή pinakas_deiktwn[n -1] = start βγαίνει εκτός ορίων του προγράµµατος. - Στο τέλος τυπώνετε τις διευθύνσεις pinakas_deiktwn[i], όπου θα σας τυπώσει όλο το string από pinakas_deiktwn[i] και πέρα και όχι µόνο τα αστεράκια. Δουλέψτε σε προγράµµατα µε pointers. Και στα δύο προγράµµατα του lab10 είχατε προβλήµατα µε διάβασµα ή εγγραφή pointers. Page 16

17 - Σωστή. Αντί για χρήση static στην add_book_info, θα µπορούσατε να περνάτε ως τελευταία παράµετρο το &books[count-1] 1996 PASS - Πριν προσπελάσετε το argv πρέπει να ελέγχετε το argc για να είστε σίγουροι ότι έχουν δοθεί τα ζητούµενα στη γραµµή εντολής. - Δεν είναι σωστή η χρήση strcmp. Πιθανώς να εννοούσατε strstr αλλά ούτε αυτό θα ήταν σωστό, γιατί θελουµε να τσεκάρουµε έναν-έναν τους χαρακτήρες του delimiter, όχι όλους µαζί. Για παράδειγµα, στο xyabzbw µε delimiter BA θα έπρεπε να αντικατασταθούν µε \0 όλες οι εµφανίσεις του Β και η εµφάνιση του Α FAIL - Δεν υπάρχουν σχόλια. + Ακριβής χώρος. - Δεν χρειάζεται να αρχικοποιείτε τα constant strings σε πίνακες και το const char suffix[] = "@inf.uth.gr" είναι σωστό. - Επίσης το "." διαφέρει από το '.'. Το δεύτερο δεν έχει τερµατικό "\0" και δεν είναι string γιατί αποτελείται από ακριβώς 1 χαρακτήρα. - Σωστή init_students(), θα µπορούσατε να αρχικοποιήσετε τα strings απλά µε ένα '\0' σαν πρώτο χαρακτήρα. - Λάθος add_student_info(). Η αναζήτηση µπορεί να παραληφθεί εάν περνάτε σαν παράµετρο - Σωστή print_students() αλλά δεν χρησιµοποιείτε σωστά την print_students(). Έπρεπε να το κάνετε σε συνδυασµό µε το students_count. Δεν χρειαζόταν να χρησιµοποιήσετε κάποιο έλεγχο για '\0'. - Καλή δουλειά. - Δεν κάνει compile FAIL - Σχόλια OK. - Σωστή init_students(), θα µπορούσατε να αρχικοποιήσετε τα strings απλά µε ένα '\0' σαν πρώτο χαρακτήρα. - Λάθος add_student_info(). Η αναζήτηση µπορεί να παραληφθεί εάν περνάτε σαν παράµετρο - Σωστή print_students() και χρησιµοποιείτε σωστά την print_students() σε συνδυασµό µε το students_count. - Καλή δουλειά. - Δεν κάνει compile. 2001???? -Καλά σχόλια -Καλά ονόµατα µεταβλητών,σωστά πεδία στην struct -Σωστη η strip dashes -Σωστή χρήση της add books -Σωστό τύπωµα των βιβλίων -Σωστό init του πίνακα. -Είχε χαζό error στο compile. Να κάνετε πάντα ένα τελικό compile πριν στείλετε την άσκηση. Page 17

18 2004 FAIL - Δεν υπάρχουν σχόλια. + Ακριβής χώρος. - Επίσης το "." διαφέρει από το '.'. Το δεύτερο δεν έχει τερµατικό "\0" και δεν είναι string γιατί αποτελείται από ακριβώς 1 χαρακτήρα. - Σωστή init_students(), θα µπορούσατε να αρχικοποιήσετε τα strings απλά µε ένα '\0' σαν πρώτο χαρακτήρα. - Λάθος add_student_info(). Η αναζήτηση µπορεί να παραληφθεί εάν περνάτε σαν παράµετρο - Σωστή print_students() και χρησιµοποιείτε σωστά την print_students() σε συνδυασµό µε το students_count. - Καλή δουλειά. - Δεν κάνει compile. 2007???? - Ελάχιστα σχόλια. - Σωστή init_books(). - Σωστή αφαίρεση των '-' απο τον κωδικό ISBN10. - Σωστή add_book_info(). Μια βελτιστοποίηση είναι οτι η αναζήτηση µπορεί να παραληφθεί εάν περνάτε σαν παράµετρο κατευθείαν τον pointer στην πρώτη ελεύθερη θέση (&books[count-1]). - Σωστή print_students(). Επίσης για παράµετρο θα µπορούσατε να περνάτε το count αντί του MAX_BOOKS για να απαλείψετε τον έλεγχο κενής θέσης. - Δεν έχει γίνει κάτι σηµαντικό. Προσπαθήστε να την ολοκληρώσετε στο σπίτι. Κορδής Διαδικαστικά: γράφει section 3 αλλά νοµίζω είναι section PASS - Σχόλια: Ελάχιστα - Στοίχιση: ΟΚ - Σταθερές: ΟΚ - Ονόµατα µεταβλητών: ΟΚ - Τύποι πεδίων struct: ΟΚ - ορθότητα create ΟΚ - ορθότητα print: ΟΚ - ορθότητα add: ΟΚ - ορθότητα init: ΟΚ - ορθότητα sprintf: ΟΚ - κλήση συναρτήσεων: ΟΚ - Γενικά: Σωστός τρόπος περάσµατος της επόµενης κενής θέσης. - Compiler Errors - Πολύ καλή δουλειά, µπράβο PASS - Πριν προσπελάσετε το argv πρέπει να ελέγξετε την τιµή του argc ώστε να είστε σίγουροι ότι δόθηκαν τα ζητούµενα strings στη γραµµή εντολής. - Σωστή κατά τα άλλα Page 18

19 2013 FAIL -Στοίχιση οκ -Δεν υπάρχουν σχόλια -Καλά ονόµατα µεταβλητών -Μέσα στη struct το µέγεθος του isbn δεν είναι MAX_BOOKS αλλά ISBN10 -Δεν υλοποιείται σωστά τη συνάρτηση strip_dashes -Δεν υλοποιείται σωστά τη συνάρτηση add_book -Στην print_books σωστά περνάτε σα παράµετρο το count και δεν εκτυπώνεται µέχρι το MAX -Στην init αφού περνάτε το ΜAX_BOOKS σα παράµετρο στο len έπρεπε να είχατε στο for (i=0; i<len; i++) -Σωστά υλοποιείται τις sprintf -Πρώτα έπρεπε να καλέσετε την strip_dashes και µετά την add_book_info -Σωστά καλείται την print_book -Δεν έγινε 2017 PASS Άσκηση 1η: - Πολύ καλή δουλειά! Μπράβο - Σωστή στοίχιση - Σωστά σχόλια - Σωστός ορισµός των πεδίων struct - Σωστή υλοποίηση της init_books - Σωστή υλοποίηση της add_book_info: Εναλλακτικά, αντί να ψάχνετε την 1η κενή θέση θα µπορούσατε να χρησιµοποιήσετε ένα counter στη main και να περνάτε την διεύθυνση της 1ης κενής θέσης στον πίνακα των structs. - Σωστή υλοποίηση της strip_dashes - Σωστή υλοποίηση της print_books Άσκηση 2η: - Σωστή αρχικοποίηση πίνακα pointers - Σωστή χρήση argc, argv - Σωστά έχετε υλοποιήσει την εύρεση υποέκφρασης. Δείτε την λύση και ξανα προσπαθήστε το σπίτι τα υπόλοιπα στο σπίτι. -Δεν κάνει compile, και φαίνεται να είναι από λάθος απροσεξίας. Να κάνετε πάντα ένα τελικό compile πριν στείλετε τις ασκήσεις σας FAIL -Το πρόγραµµα τερµατίζει µε Segmentation fault -Σωστά χρησιµοποιείται τo argv[], δεν ελέγχετε πουθενά αν δόθηκε στη γραµµή εντολών 2 string -Δεν υλοποιείται σωστά το πρόγραµµα Διαδικαστικά ΟΚ Άσκηση 1η 2022 FAIL - Στοίχιση: Σωστή - Σχόλια: ΟΚ - ορισµός πεδίων struct student: ΟΚ, το πρόθεµα str_ δεν ήταν απαραίτητο. - init_students: ΟΚ, σωστή αρχικοποίηση. Θα µπορούσατε να αρχικοποιήσετε µόνο τον 1 χαρακτήρα σε '\0', καθώς µόνο αυτό ελέγχετε στην add_student_info. - create_ ΟΚ - add_student_info: Εναλλακτικά, αντί να ψάχνετε την 1η κενή θέση θα µπορούσατε να χρησιµοποιήσετε ένα counter στη main και να περνάτε την διεύθυνση της 1ης κενής θέσης στον πίνακα των structs. - print_students: ΟΚ. Εναλλακτικά (οµοίως µε την add_student_info), αντί να ψάχνετε για τις κατειληµµένες θέσεις θα µπορούσατε να περνάτε τον αριθµό των θέσεων που έχουν καταληφθεί. Άσκηση 2η - Στοίχιση: ΟΚ - Έγινε µόνο η αρχικοποίηση του πίνακα δεικτών. Page 19

20 2024 PASS -Καλά σχόλια -Σωστο αποτέλεσµα της strip_dashes. Αλλά επειδή χρησιµοποιέιτε την for µέχρι j<isbn10 το [j+1] θα βγαίνει εκτος ορίων του πίνακα, πράγµα που δεν είναι σωστό. -Σωστη η add books. Θα µπορούσατε αντί να ψάχνετε το πρώτο ελεύθερο πεδίο του πίνακα των structs να στέλνετε στην συνάρτηση τον δείκτη προς την διεύθυνση που θέλετε να περάσετε. -Σωστή η init books -Σωστό τύπωµα των βιβλίων -Σωστά ελέγχετε άν ο χρήστης έδωσε στην γραµµή εντολών είσοδο. -Πολύ λίγα σχόλια -Δεν βγάζει σωστό αποτέλεσµα, αλλά φαίνεται οτι την δουλέψατε 2026???? - Ελάχιστα σχόλια. - Σωστή init_books(). - Σωστή αφαίρεση των '-' απο τον κωδικό ISBN10. - Σωστή add_book_info(). Μια βελτιστοποίηση είναι οτι η αναζήτηση µπορεί να παραληφθεί εάν περνάτε σαν παράµετρο κατευθείαν τον pointer στην πρώτη ελεύθερη θέση (&books[count-1]). - Σωστή print_students(). Επίσης για παράµετρο θα µπορούσατε να περνάτε το count αντί του MAX_BOOKS για να απαλείψετε τον έλεγχο κενής θέσης. - Δεν έχει γίνει κάτι σηµαντικό. Προσπαθήστε να την ολοκληρώσετε στο σπίτι. Διαδικαστικά ΟΚ Άσκηση 1η 2028 PASS - Η θέση των σχολίων είναι πάνω από τις συναρτήσεις και όχι δίπλα από αυτές, έτσι ώστε να διατηρούµε ένα πλάτος κώδικα περίπου στις 100 στήλες. Περιεχόµενο σχολίων ΟΚ. - ορισµός πεδίων struct student: ΟΚ - init_students: ΟΚ, σωστή αρχικοποίηση. Θα µπορούσατε να αρχικοποιήσετε µόνο τον 1 χαρακτήρα σε '\0', καθώς µόνο αυτό ελέγχετε στην add_student_info. - create_ ΟΚ - add_student_info: ΟΚ. Εναλλατικά, αντί να ψάχνετε την 1η κενή θέση θα µπορούσατε να χρησιµοποιήσετε ένα counter στη main και να περνάτε την διεύθυνση της 1ης κενής θέσης στον πίνακα των structs. - print_students: ΟΚ. Εναλλακτικά (οµοίως µε την add_student_info), αντί να ψάχνετε για τις κατηλληµένες θέσεις θα µπορούσατε να περνάτε τον αριθµό των θέσεων που έχουν καταληφθεί. Άσκηση 2η - Για τα σχόλια ισχύει ότι και στην άσκηση 1. Αν έχετε απορίες ρωτήστε µας. - Σωστή χρήση argc, argv. Θα µπορούσατε να ελέγχετε εάν πράγµατι έχει βάλει ο χρήστης τουλάχιστον 2 ορίσµατα και σε διαφορετική περίπτωση να τερµατίζετε το πρόγραµµα. - Αλγοριθµικά ΟΚ, το πρόγραµµα δουλεύει καλά. Καλή προσπάθεια. Εξοικειωθείτε στη χρήση του το gdb, ώστε να εντοπίζετε τα λάθη σας στο µέλλον. Page 20

21 2029???? Άσκηση 1η: - Σωστή στοίχιση - Έλλειψη σχολίων - Σωστός ορισµός των πεδίων struct - Σωστή υλοποίηση της init_books - Σωστή υλοποίηση της add_book_info: Εναλλακτικά, αντί να ψάχνετε την 1η κενή θέση θα µπορούσατε να χρησιµοποιήσετε ένα counter στη main και να περνάτε την διεύθυνση της 1ης κενής θέσης στον πίνακα των structs. - Έχετε 2 bugs Καλείτε strip_dashes( format_str ); Μάλλον εννοείτε strip_dashes( isbn ); add_book_info(&title[count-1], &author[count-1], &isbn[count-1], &books_array[count-1]); Γιατί [count -1] ; Μάλλον θέλατε το εξης: add_book_info(title, author, isbn, &books_array[count-1]); Δείτε την λύση. Άσκηση 2η: - Δεν χρησιµοποιείτε argc, argv - Δείτε την λύση και ξανα προσπαθήστε το σπίτι τα υπόλοιπα στο σπίτι FAIL - Σχόλια OK. - Σωστή init_students(), θα µπορούσατε να αρχικοποιήσετε τα strings απλά µε ένα '\0' σαν πρώτο χαρακτήρα. - Λάθος add_student_info(). Η αναζήτηση µπορεί να παραληφθεί εάν περνάτε σαν παράµετρο - Σωστή print_students() αλλά δεν χρησιµοποιείτε σωστά την print_students(). Έπρεπε να το κάνετε σε συνδυασµό µε το students_count. Δεν χρειαζόταν να χρησιµοποιήσετε κάποιο έλεγχο για '\0'. - Καλή δουλειά. - Δεν κάνει compile. 2031???? Άσκηση 1η: - Σωστή στοίχιση - Έλλειψη σχολίων - Σωστός ορισµός των πεδίων struct - Σωστή υλοποίηση της init_books - Σωστή υλοποίηση της add_book_info: Εναλλακτικά, αντί να ψάχνετε την 1η κενή θέση θα µπορούσατε να χρησιµοποιήσετε ένα counter στη main και να περνάτε την διεύθυνση της 1ης κενής θέσης στον πίνακα των structs. - Έχετε 2 bugs Καλείτε strip_dashes( format_str ); Μάλλον εννοείτε strip_dashes( isbn ); add_book_info(&title[count-1], &author[count-1], &isbn[count-1], &books_array[count-1]); Γιατί [count -1] ; Μάλλον θέλατε το εξης: add_book_info(title, author, isbn, &books_array[count-1]); Δείτε την λύση. Άσκηση 2η: - Δεν χρησιµοποιείτε argc, argv - Δείτε την λύση και ξανα προσπαθήστε το σπίτι τα υπόλοιπα στο σπίτι. - Σωστή. Αντί για χρήση static στην add_book_info, θα µπορούσατε να περνάτε ως τελευταία παράµετρο το &books[count-1] 2032 PASS - Πριν προσπελάσετε το argv πρέπει να ελέγχετε το argc για να είστε σίγουροι ότι έχουν δοθεί τα ζητούµενα στη γραµµή εντολής. - Δεν είναι σωστή η χρήση strcmp. Πιθανώς να εννοούσατε strstr αλλά ούτε αυτό θα ήταν σωστό, γιατί θελουµε να τσεκάρουµε έναν-έναν τους χαρακτήρες του delimiter, όχι όλους µαζί. Για παράδειγµα, στο xyabzbw µε delimiter BA θα έπρεπε να αντικατασταθούν µε \0 όλες οι εµφανίσεις του Β και η εµφάνιση του Α. Page 21

22 2033 PASS - Καλά σχόλια. - Σωστοί τύποι και περιγραφικά ονόµατα πεδίων του struct. Καλό είναι να δηλώνετε σε διαφορετική γραµµή το κάθε πεδίο. - Στην init_students() καλό είναι να αρχικοποιείτε όλες τις θέσεις µνήµης. - Σωστή add_student_info(), αν και θα µπορούσατε να την γράψετε πιο απλά. - Επίσης αντί να κάνετε αναζήτηση για την πρώτη κενή θέση µέσα στην συνάρτηση, θα µπορούσατε να περνάτε την πρώτη κενή θέση σαν πρώτη παράµετρο (&students[count-1]) από την main. - Σωστή print_students() και εκτύπωση µόνο των γεµάτων θέσεων. - Καλά σχόλια. - Το πρόγραµµα τερµατίζει µε sgmentation fault. - Δεν ελέγχετε την τιµή επιστροφής της strstr(). Εάν ο delimiter δεν βρεθεί η επιστρεφόµενη τιµή είναι NULL. - Τα στοιχεία του πίνακα table_ptr είναι pointers αρχικοποιηµένοι σε NULL. Οπότε όταν βρεθεί µία συµβολοσειρά χαρακτήρων '*' µεγέθους len, έπρεπε να αποθηκεύετε την διεύθυνση που ξεκινά (start_ptr) στην θέση table_ptr[len-1]. - Η print_aterisks() πρέπει να εκτυπώνει µόνο '*' PASS - Καλά σχόλια. - Σωστοί τύποι και περιγραφικά ονόµατα πεδίων του struct. - Σωστή init_students(). - Σωστή add_student_info(). Αντί για την έξτρα παράµετρο που περνάτε, µπορείτε να περνάτε - Σωστή print_students() και εκτύπωση µόνο των γεµάτων θέσεων. - Καλά σχόλια. - Σωστή print_asterisks(). - Η προσθήκη του delimiter στο τέλος του αρχικού string, µε τον τρόπο που είναι γραµµένη δεν είναι ασφαλής εντολή. Εκτός του ότι µπορεί η είσοδος να έχει ήδη στο τέλος delimiter, το βασικότερο πρόβληµα είναι πως τροποποιούµε µνήµη που δεν γνωρίζουµε το µέγεθος της. Δεν υπάρχει τρόπος να εγγυηθούµε ότι µετά το τέλος του argv[i] υπάρχει διαθέσιµος κενός χώρος ή εάν θα βγούµε εκτός ορίων του πίνακα όπου βρίσκεται αποθηκευµένη η συµβολοσειρά. 2036???? Άσκηση 1η: - Πολύ καλή δουλειά! Μπράβο! - Σωστή στοίχιση - Σωστά σχόλια - Σωστός ορισµός των πεδίων struct - Σωστή υλοποίηση της init_books - Σωστή υλοποίηση της add_book_info: Πολύ σωστά χρησιµοποιήσετε τον counter στη main για να περνετε την διεύθυνση της 1ης κενής θέσης στον πίνακα των structs. - Σωστή υλοποίηση της strip_dashes - Σωστή υλοποίηση της print_books Άσκηση 2η: - Σωστή αρχικοποίηση πίνακα pointers - Σωστή χρήση argv, αλλά αν το argc δεν είναι 3, τότε θα εκτελεστεί η δεύτερη for ενώ δε θα έπρεπε γιατί το delimiter δεν έχει πάρει τιµή. - Ξανα προσπαθήστε τα υπόλοιπα στο σπίτι. Page 22