- Σωστή χρήση του προσδιοριστή const για την σταθερά pi. Καλό θα ήταν επιλέξετε κεφαλαία γράµµατα για το όνοµα της σταθεράς.

Transcript

1 ΑΕΜ ΒΑΘΜΟΣ ΣΧΟΛΙΑ - Σωστή χρήση του προσδιοριστή const για την σταθερά p. Καλό θα ήταν επιλέξετε κεφαλαία γράµµατα για το όνοµα της σταθεράς. - Στην printf("enter radius: \n"); ο χαρακτήρας αλλαγής γραµµής είναι περιττός. - Η εκτύπωση της printf είναι σωστή (%10.3lf), αλλά δεν χρησιµοποιήσατε στην έξοδο τα ονόµατα που γράφονταν στην εκφώνηση. 449 Radius: XXXΧXX.xxx Area : YYYYΥY.yyy - Η στοίχιση της εξόδου σας είναι σωστή. - Η µορφοποίηση του κώδικα σας είναι σωστή. - Τα σχόλια σας είναι σωστά. Ούτε πολλά, ούτε λίγα, διευκολύνουν την ανάγνωση. - Συνολικά πολύ καλή δουλειά. 451 Στην 2η άσκηση αντιγράψατε το lab2a.c στο lab2b.c. Δεν τον κώδικα της 2ης άσκησης. ΠΡΟΣΟΧΗ!!! Από το errors.txt φαίνεται να έχετε βρει τα συντακτικά λάθη και το 1 από τα 2 λογικά λάθη. - Σωστή χρήση του προσδιοριστή const για την σταθερά p. Καλό θα ήταν επιλέξετε κεφαλαία γράµµατα για το όνοµα της σταθεράς. - Στην printf("enter radius: \n"); ο χαρακτήρας αλλαγής γραµµής είναι περιττός. - Η εκτύπωση της printf είναι σωστή (%10.3lf), αλλά δεν χρησιµοποιήσατε στην έξοδο τα ονόµατα που γράφονταν στην εκφώνηση. Radius: XXXΧXX.xxx Area : YYYYΥY.yyy - Η στοίχιση της εξόδου σας είναι σωστή. - Η µορφοποίηση του κώδικα σας είναι σωστή. - Τα σχόλια σας είναι σωστά. Ούτε πολλά, ούτε λίγα, διευκολύνουν την ανάγνωση. - Συνολικά πολύ καλή δουλειά Στην 2η άσκηση αντιγράψατε το lab2a.c στο lab2b.c. Δεν τον κώδικα της 2ης άσκησης. ΠΡΟΣΟΧΗ!!! Από το errors.txt φαίνεται να έχετε βρει τα συντακτικά λάθη και το 1 από τα 2 λογικά λάθη. - Μπράβο - Σωστό θέµα , attachment και tgz αρχείο. - Καλή προσπάθεια στα σχόλια. Κάποια είναι περιττά γιατί δε δίνουν επιπλέον πληροφορία ή δεν είναι ακριβή. πχ στις δηλώσεις των µεταβλητών λέτε /*Arxikopoihsh tis metavlitis Area */. Αυτό δεν είναι ακριβές, γιατί δε γίνεται αρχικοποίηση εκεί, και είναι περιττό γιατί είναι προφανές ότι αυτή η γραµµή κάνει απλά δήλωση µιας µεταβλητής που λέγεται Area. Δείτε τη σελίδα 10 στο φυλλάδιο " Αρχές καλού προγραµµατισµού " στο site του εργαστηρίου για περισσότερες λεπτοµέρειες - Σωστοί τύποι και ονοµασία µεταβλητών και χρησιµοποιήσατε πολύ σωστά το const. Κατά σύµβαση, τα ονόµατα των σταθερών γράφονται µε κεφαλαία (δηλαδή PI) για να ξεχωρίζουν, ενώ τα ονόµατα των µεταβλητών ξεκινούν µε µικρό γράµµα (lowercase). - Σωστή στοίχιση - Πολύ σωστά στην printf ο κωδικός είναι %10.3lf (6 ακέραια + 1 υποδιαστολή + 3 δεκαδικά). Σωστή µορφοποίηση output. - Δεν διορθώσατε την ονοµατολογία των µεταβλητών calcul και programmig (αυτή δεν είναι η βέλτιστη ονοµατολογία). - Διορθώσατε όλους τους µορφοποιητές τύπου της scanf εκτός από έναν (%s αντί για %d στο digital_design) αλλά δεν προσθέσατε τον τελεστή & σε όλες τις scanf κάτι που εξηγεί τα πολλά warnings που σας πετάει ο compiler. - Δεν αφαιρέσατε τον τελεστή & από την printf. - Δεν βρήκατε τα 2 λογικά λάθη που είχε το πρόγραµµα (διαίρεση αντί για πολλαπλασιασµό και return 1 αντί για 1.2) - Μπράβο - Σωστό θέµα , attachment και tgz αρχείο. - Καλή προσπάθεια στα σχόλια. Κάποια είναι περιττά γιατί δε δίνουν επιπλέον πληροφορία ή δεν είναι ακριβή. πχ στις δηλώσεις των µεταβλητών λέτε /*Arxikopoihsh tis metavlitis Area */. Αυτό δεν είναι ακριβές, γιατί δε γίνεται αρχικοποίηση εκεί, και είναι περιττό γιατί είναι προφανές ότι αυτή η γραµµή κάνει απλά δήλωση µιας µεταβλητής που λέγεται Area. Δείτε τη σελίδα 10 στο φυλλάδιο " Αρχές καλού προγραµµατισµού " στο site του εργαστηρίου για περισσότερες λεπτοµέρειες - Σωστοί τύποι και ονοµασία µεταβλητών και χρησιµοποιήσατε πολύ σωστά το const. Κατά σύµβαση, τα ονόµατα των σταθερών γράφονται µε κεφαλαία (δηλαδή PI) για να ξεχωρίζουν, ενώ τα ονόµατα των µεταβλητών ξεκινούν µε µικρό γράµµα (lowercase). - Σωστή στοίχιση - Πολύ σωστά στην printf ο κωδικός είναι %10.3lf (6 ακέραια + 1 υποδιαστολή + 3 δεκαδικά). Σωστή µορφοποίηση output. - Δεν διορθώσατε την ονοµατολογία των µεταβλητών calcul και programmig (αυτή δεν είναι η βέλτιστη ονοµατολογία). - Διορθώσατε όλους τους µορφοποιητές τύπου της scanf εκτός από έναν (%s αντί για %d στο digital_design) αλλά δεν προσθέσατε τον τελεστή & σε όλες τις scanf κάτι που εξηγεί τα πολλά warnings που σας πετάει ο compiler. - Δεν αφαιρέσατε τον τελεστή & από την printf. - Δεν βρήκατε τα 2 λογικά λάθη που είχε το πρόγραµµα (διαίρεση αντί για πολλαπλασιασµό και return 1 αντί για 1.2) Απουσία από το εργαστήριο Σωστή δήλωση του pi ως σταθερά. Συνήθως τις σταθερές τις ορίζουµε µε κεφαλαία γράµµατα για να τις ξεχωρίζουµε µέσα στο πρόγραµµα από τις µεταβλητές. - Δεν έχετε γράψει σχόλια για την λειτουργία του προγράµµατος. Δείτε το σχετικό φυλλάδιο στο - Το µήνυµα εισαγωγής θα έπρεπε να είναι "Enter radius: " δηλαδή µε ένα κενό (space) στο τέλος. Προσέχετε καλά την εκφώνηση Σωστό θέµα , ΕΝΤΕΛΩΣ λάθος attachment και tgz αρχείο. Στείλατε όλες τις ασκήσεις χωρίς φάκελο µε διάφορα τυχαία και κενά tgz αρχεία. Επιπλέον δεν κάνατε cc και τα 2 µέλη της οµάδας. Δώστε µεγάλη προσοχή στις οδηγίες αποστολής. - Έλλειψη σχολίων! - Σωστοί τύποι και ονοµασία µεταβλητών και χρησιµοποιήσατε πολύ σωστά το const. - Σωστή στοίχιση - Στην printf ο κωδικός έπρεπε να είναι %10.3lf (6 ακέραια + 1 υποδιαστολή + 3 δεκαδικά). Επίσης, σας λείπουν κάποια tabs. Να δώσετε επιπλέον προσοχή την επόµενη φορά για τη σωστή εµφάνιση µε βάση το format που σας ζητάµε. Οι ":" δεν είναι στοιχισµένες Δεν διορθώσατε την ονοµατολογία των µεταβλητών calcul και programmig (αυτή δεν είναι η βέλτιστη ονοµατολογία). - Σωστά διορθώσατε τους µορφοποιητές τύπου της scanf και προσθέσατε τον τελεστή &. - Σωστά αφαιρέσατε τον τελεστή & από την printf. - Βρήκατε το πρώτο λογικό λάθος που είχε το πρόγραµµα (διαίρεση αντί για πολλαπλασιασµό) αλλά δεν βρήκατε το δεύτερο που ήταν το return 1 αντί για Mη σωστή επιλογή ονοµάτων για τις µεταβλητές. Τα ονόµατα δεν είναι περιγραφικά. Επιπλέον συνήθως επιλέγουµε µεταβλητές που περιέχουν µη κεφαλαία γράµµατα, ενώ τις σταθερές τις γράφουµε µε κεφαλαία. Αντιστρέψατε τον παραπάνω άτυπο κανόνα. - Σωστή χρήση του προσδιοριστή const για την σταθερά pi. Καλό θα ήταν επιλέξετε κεφαλαία γράµµατα για το όνοµα της σταθεράς. - Στην printf δεν ακολουθήσατε τις οδηγίες και για αυτό το αποτέλεσµα δεν είναι στοιχισµένο. H χαρακτήρας tab ζητήθηκε να µπει µετά τον χαρακτήρα : και όχι στη αρχή της κάθε γραµµής. - Σωστή µορφοποίηση του κώδικα. - Συνολικά καλή δουλειά. - Σωστή διόρθωση errors και warnings από τον compiler. - Επιπλέον βρήκατε και τα 2 λογικά λάθη που είχε ο κώδικας. - Συγχαρητήρια, καλή δουλειά.

2 1899 -Έχετε βάλει σχόλια µε την περιγραφή του προγράµµατος, αλλά µην χρησιµοποιείτε ελληνικούς χαρακτήρες µέσα σε ένα πρόγραµµα. -Σωστοί τύποι µεταβλητών. Πολύ σωστά δηλώσατε και το π σαν const double. Καλό είναι οι σταθερές να δηλώνονται χρησιµοποιώντας κεφαλαίους χαρακτήρες και οι µεταβλητές µε lowercase χαρακτήρες. Δεν χρειάζεται να βάζετε κεφαλαίο στο πρώτο γράµµα της µεταβλητής. -Δεν τυπώνετε ακριβώς αυτό που ζητούσε η άσκηση. Για να τυπώσετε 6 χαρακτήρες πριν την υποδιαστολή + µια τελεία + 3 δεκαδικά µετά (σύνολο 10 ψηφία) θα έπρεπε να χρησιµοποιήσεις %10.3 ενώ εσείς χρησιµοποιείτε %6.3. Επίσης λανθασµένα κάνεις return 1.2 στην main. Η main είναι δηλωµένη ως int οπότε πρέπει να επιστρέφει ακέραιο. Προφανώς το είδες από την άσκηση 2 και το θεώρησες σωστό ενώ ήταν λάθος. -Διορθώσατε όλα τα λάθη και το 1 από τα 2 λογικά λάθη που υπήρχαν. Το 2ο λάθος ήταν ότι η main επιστρέφει 1.2 (δεκαδικό αριθµό) ενώ είναι δηλωµένη σαν int. Οπότε το σωστό είναι να επιστρέφει ακέραιο. Δεν πειράζει που δεν το βρήκατε διότι δεν έχετε διδαχτεί ακόµα συναρτήσεις. -Σωστό format του errors.txt Έπρεπε να βάζετε όµως και µια πρόταση µε επεξήγηση για τα λάθη που διορθώσατε. Επίσης δεν βάλατε επεξήγηση για τα λογικά λάθη που βρήκατε Έχετε βάλει σχόλια µε την περιγραφή του προγράµµατος, αλλά µην χρησιµοποιείτε ελληνικούς χαρακτήρες µέσα σε ένα πρόγραµµα. -Σωστοί τύποι µεταβλητών. Πολύ σωστά δηλώσατε και το π σαν const double. Καλό είναι οι σταθερές να δηλώνονται χρησιµοποιώντας κεφαλαίους χαρακτήρες και οι µεταβλητές µε lowercase χαρακτήρες. Δεν χρειάζεται να βάζετε κεφαλαίο στο πρώτο γράµµα της µεταβλητής. -Δεν τυπώνετε ακριβώς αυτό που ζητούσε η άσκηση. Για να τυπώσετε 6 χαρακτήρες πριν την υποδιαστολή + µια τελεία + 3 δεκαδικά µετά (σύνολο 10 ψηφία) θα έπρεπε να χρησιµοποιήσεις %10.3 ενώ εσείς χρησιµοποιείτε %6.3. Επίσης λανθασµένα κάνεις return 1.2 στην main. Η main είναι δηλωµένη ως int οπότε πρέπει να επιστρέφει ακέραιο. Προφανώς το είδες από την άσκηση 2 και το θεώρησες σωστό ενώ ήταν λάθος. -Διορθώσατε όλα τα λάθη και το 1 από τα 2 λογικά λάθη που υπήρχαν. Το 2ο λάθος ήταν ότι η main επιστρέφει 1.2 (δεκαδικό αριθµό) ενώ είναι δηλωµένη σαν int. Οπότε το σωστό είναι να επιστρέφει ακέραιο. Δεν πειράζει που δεν το βρήκατε διότι δεν έχετε διδαχτεί ακόµα συναρτήσεις. -Σωστό format του errors.txt Έπρεπε να βάζετε όµως και µια πρόταση µε επεξήγηση για τα λάθη που διορθώσατε. Επίσης δεν βάλατε επεξήγηση για τα λογικά λάθη που βρήκατε Το πρόγραµµα δεν έχει σχόλια. Διαβάστε το φυλλάδιο για σχολιασµό στο - Σωστές πράξεις -Σωστή scanf - Σωστή printf - Σωστές οι διορθώσεις σας στα συντακτικά και λογικά λάθη. Γενικά: Καλή δουλειά Είναι κακή πρακτική να αφήνετε κενό µετά το όνοµα συναρτήσεων κατά την κλήση τους. Π.χ printf ("Hello!"). - Οι µεταβλητές πρέπει να έχουν περιγραφικά ονόµατα, οι "radius", "pi" είναι σωστές αλλά θα έπρεπε να είχατε γράψει "perimeter/perimetros" αντί για "per". - Το pi θα µπορούσατε να το είχατε δηλώσει ως σταθερά - Σωστά σχόλια. - Στην printf ο κωδικός έπρεπε να είναι %10.3lf (6 ακέραια + 1 υποδιαστολή + 3 δεκαδικά). Επίσης έχετε βάλει έναν κενό χαρακτήρα µετά από κάθε tab στην εκτύπωση των αριθµών. Γενικά: Καλή δουλειά, αλλά προσέξτε τα ονόµατα των µεταβλητών/σταθερών Δηλώσατε σχεδόν 100% σωστά την σταθερά pi, χρησιµοποιούµε κεφαλαία γράµµατα όταν δηλώνουµε σταθερές. - Παρότι δηλώσατε µε σχόλια τι σηµαίνουν τα ονόµατα µεταβλητών "r/p" δεν είναι περιγραφικά. Μία σωστή επιλογή ονοµάτων είναι "radius/perimeter". - Τα σχόλια καλό είναι να βρίσκονται σε δική τους γραµµή. - Στην printf ο κωδικός έπρεπε να είναι %10.3lf (6 ακέραια + 1 υποδιαστολή + 3 δεκαδικά). Επίσης έχετε βάλει ένα κενό χαρακτήρα µετά το tab στην εκτύπωση της περιµέτρου. Τέλος δεν έχετε ευθυγραµµίσει τα ":" όπως σας ζητήθηκε στην εκφώνηση. - Δεν έχετε σηµειώσει τα λάθη που εµφανίζονται κατά την µεταγλώττιση του προγράµµατος. - Διαδικαστικά: ΟΚ : Σχόλια: ΟΚ - Ονόµατα Μεταβλητών: Μονογραµµατικά. Τα ονόµατά σας πρέπει να είναι πιο περιγραφικά (πχ area ή embadon αντί για E) - Σταθερά π: Θα µπορούσατε το π να το είχατε δηλώσει ως σταθερά όµως - Στοίχιση και γενικότερη µορφή κώδικα: Το τελευταίο άγκιστρο του κώδικα δεν είναι σωστά στοιχισµένο. - scanf format: OK - printf format: Δεν έχετε χαρακτήρα tab. Ξαναδείτε την εκφώνηση. : - λάθη που εξακολουθεί να βγάζει ο compiler: KANENA - warnings που συνδέονται µε format specifiers της scanf: ΚΑΝΕΝΑ - επιστρέφει floating point αντί για ακέραιο: Προσέξτε ότι η main πρέπει να επιστρέφει ακέραιο Απουσία από το εργαστήριο Γενικά: Καλή δουλειά Είναι κακή πρακτική να αφήνετε κενό µετά το όνοµα συναρτήσεων κατά την κλήση τους. Π.χ printf ("Hello!"). - Οι µεταβλητές πρέπει να έχουν περιγραφικά ονόµατα, οι "radius", "pi" είναι σωστές αλλά θα έπρεπε να είχατε γράψει "perimeter/perimetros" αντί για "per". - Το pi θα µπορούσατε να το είχατε δηλώσει ως σταθερά - Σωστά σχόλια. - Στην printf ο κωδικός έπρεπε να είναι %10.3lf (6 ακέραια + 1 υποδιαστολή + 3 δεκαδικά). Επίσης έχετε βάλει έναν κενό χαρακτήρα µετά από κάθε tab στην εκτύπωση των αριθµών.

3 1909 ΓΕΝΙΚΑ: Μέτρια δουλειά - Σωστό όνοµα attachment, σωστή δηµιουργία του tgz αρχείου. Το θέµα του είναι σωστό, αλλά δεν υπάρχει λόγος να το γράφετε µε κεφαλαία. - Το πρόγραµµά σας δεν έχει καθόλου σχόλια. Διαβάστε το φυλλάδιο για σχολιασµό στο και την επόµενη φορά γράψτε περιγραφικά σχόλια. - Σωστοί τύποι αλλά άσχηµα ονόµατα µεταβλητών. Μη χρησιµοποιείτε κοµµένες λέξεις γιατί µειώνουν την αναγνωσιµότητα και κατανόηση του προγράµµατος. Πείτε radius και perimeter αντί για rad και per. Σωστά δηλώσατε το pi ως const. Κατά σύµβαση, τα ονόµατα των σταθερών γράφονται µε κεφαλαία (δηλαδή PI) για να ξεχωρίζουν. Είναι καλό να χωρίζετε µε κενές γραµµές τις διάφορες ενότητες του προγράµµατος όπως κάνατε, αλλά όχι µε περισσότερες από µία κενές γραµµές. - Στην printf έχετε κάποια περίεργα κόµµατα, επιπλέον κενά, και λείπει ο χαρακτήρας tab. Επίσης, ο κωδικός έπρεπε να είναι %10.3lf (6 ακέραια + 1 υποδιαστολή + 3 δεκαδικά). Είχατε σχόλιο και στο lab1 σχετικά µε την παρουσίαση των αποτελεσµάτων, οπότε προσέξτε το! - Στις διορθώσεις των ονοµάτων των µεταβλητών, θα ήταν πιο λογικό να είχατε αλλάξει το calcul σε calculus αντί για το calculus σε calcul (οµοίως και για τα υπόλοιπα) ώστε τα ονόµατα των µεταβλητών να είναι γραµµατικά σωστά. Την άσκηση πρέπει να τη στέλνει ο ένας µόνο από τους δύο Σωστό θέµα , σωστό attachment (τη δεύτερη φορά), σωστή δηµιουργία του tgz αρχείου - Καλή δουλειά µε τα σχόλια προγράµµατος (στην αρχή), µπράβο. Τα υπόλοιπα δέχονται βελτίωση όµως. Δείτε τη σελίδα 10 στο φυλλάδιο µε τίτλο " Αρχές καλού προγραµµατισµού". Αποφεύγουµε σχόλια που µας λένε το προφανές (πχ "Δήλωση µεταβλητών", "Εκτύπωση µηνύµατος") και προτιµάµε σχόλια που κάνουν πιο ξεκάθαρο τον κώδικα (πχ "Υπολογισµός περιµέτρου"). - Σωστοί τύποι, αλλά πολύ άσχηµα ονόµατα µεταβλητών. Τα ονόµατα πρέπει να είναι περιγραφικά, και γραµµένα µε πεζούς χαρακτήρες. Καλά ονόµατα στη συγκεκριµένη άσκηση θα ήταν radius, perimeter ή aktina, perimetros. Κατά σύµβαση, γράφουµε µε κεφαλαία µόνο τις σταθερές για να ξεχωρίζουν. Το π θα µπορούσατε να το είχατε δηλώσει ως σταθερά (µε όνοµα PI) - Αποφεύγετε να έχετε κενή γραµµή µετά από κάθε εντολή. Συνήθως βάζουµε µια κενή γραµµή ανάµεσα σε διαφορετικές ενότητες του προγράµµατος. Για παράδειγµα, µια µετά τις δηλώσεις µεταβλητών και µία µετά τον υπολογισµό και πριν την εκτύπωση των αποτελεσµάτων. - Γενικά σωστή η printf, αλλά το tab έπρεπε να είναι µετά το : - Το πρόγραµµα παράγει ένα warning για το ότι η εντολή return στο τέλος επιστρέφει αριθµό κινητής υποδιαστολής αντί για ακέραιο. Στο 90% των περιπτώσεων πρέπει να - Οι αλλαγές των λάθος ονοµάτων µεταβλητών που αναφέρετε στο errors.txt δεν είναι ακριβώς αυτό που κάνατε τελικά. Λέτε ότι θα αλλάξετε το όνοµα calculus σε calcul αλλά κάνατε το αντίθετο. Παρεµπιπτόντως, αυτό που τελικά κάνατε είναι πιο καλή λύση, γιατί το calculus είναι πλήρης, γραµµατικά σωστή λέξη. Την άσκηση πρέπει να τη στέλνει ο ένας µόνο από τους δύο Σωστό θέµα , σωστό attachment (τη δεύτερη φορά), σωστή δηµιουργία του tgz αρχείου - Καλή δουλειά µε τα σχόλια προγράµµατος (στην αρχή), µπράβο. Τα υπόλοιπα δέχονται βελτίωση όµως. Δείτε τη σελίδα 10 στο φυλλάδιο µε τίτλο " Αρχές καλού προγραµµατισµού". Αποφεύγουµε σχόλια που µας λένε το προφανές (πχ "Δήλωση µεταβλητών", "Εκτύπωση µηνύµατος") και προτιµάµε σχόλια που κάνουν πιο ξεκάθαρο τον κώδικα (πχ "Υπολογισµός περιµέτρου"). - Σωστοί τύποι, αλλά πολύ άσχηµα ονόµατα µεταβλητών. Τα ονόµατα πρέπει να είναι περιγραφικά, και γραµµένα µε πεζούς χαρακτήρες. Καλά ονόµατα στη συγκεκριµένη άσκηση θα ήταν radius, perimeter ή aktina, perimetros. Κατά σύµβαση, γράφουµε µε κεφαλαία µόνο τις σταθερές για να ξεχωρίζουν. Το π θα µπορούσατε να το είχατε δηλώσει ως σταθερά (µε όνοµα PI) - Αποφεύγετε να έχετε κενή γραµµή µετά από κάθε εντολή. Συνήθως βάζουµε µια κενή γραµµή ανάµεσα σε διαφορετικές ενότητες του προγράµµατος. Για παράδειγµα, µια µετά τις δηλώσεις µεταβλητών και µία µετά τον υπολογισµό και πριν την εκτύπωση των αποτελεσµάτων. - Γενικά σωστή η printf, αλλά το tab έπρεπε να είναι µετά το : - Το πρόγραµµα παράγει ένα warning για το ότι η εντολή return στο τέλος επιστρέφει αριθµό κινητής υποδιαστολής αντί για ακέραιο. Στο 90% των περιπτώσεων πρέπει να - Οι αλλαγές των λάθος ονοµάτων µεταβλητών που αναφέρετε στο errors.txt δεν είναι ακριβώς αυτό που κάνατε τελικά. Λέτε ότι θα αλλάξετε το όνοµα calculus σε calcul αλλά κάνατε το αντίθετο. Παρεµπιπτόντως, αυτό που τελικά κάνατε είναι πιο καλή λύση, γιατί το calculus είναι πλήρης, γραµµατικά σωστή λέξη. - Σωστή χρήση του προσδιοριστή const για την σταθερά pi. Καλό θα ήταν επιλέξετε κεφαλαία γράµµατα για το όνοµα της σταθεράς. - Η σωστή εκτύπωση στην printf είναι %10.3lf (6 ακέραια + 1 υποδιαστολή + 3 δεκαδικά) - Προσοχή στην µορφοποίηση του κώδικα σας. Αν και η τελική µορφή είναι σωστή, αλλού χρησιµοποιείται tab και αλλού spaces. Αν ανοίξει κάποιος τον κώδικα µε διαφορετικό editor θα εµφανίζεται χωρίς µορφοποίηση. Επιλέξτε έναν από τους δύο τρόπους στα προγράµµατα σας. - Συνολικά καλή δουλειά. - Σωστή διόρθωση errors και warnings από τον compiler. - Τα µαθήµατα θα µπορούσατε να τα είχατε ορίσει και ως int. Η επιλογή float δεν είναι λάθος. - Επιπλέον βρήκατε και τα 2 λογικά λάθη που είχε ο κώδικας. - Συγχαρητήρια, καλή δουλειά. Γενικά: Καλή δουλειά Σωστά ονόµατα µεταβλητών. - Δηλώσατε σχεδόν 100% σωστά την σταθερά pi, χρησιµοποιούµε κεφαλαία γράµµατα όταν δηλώνουµε σταθερές. - Δεν έχετε γράψει σχόλια για την λειτουργία του προγράµµατος. Δείτε το σχετικό φυλλάδιο στο - Στην printf ο κωδικός έπρεπε να είναι %10.3lf (6 ακέραια + 1 υποδιαστολή + 3 δεκαδικά). Επίσης προσθέσατε ένα κενό χαρακτήρα µετά το χαρακτήρα tab. - Δεν έχετε σηµειώσει τα λάθη που εµφανίζονται κατά την µεταγλώττιση του προγράµµατος Το συνηµµένο αρχείο του δεύτερου mail δεν περιέχει τίποτα. Οπότε βαθµολογηθήκατε µε αυτό που στείλατε στο πρώτο mail. Προσέχετε κατά τη συµπίεση και την αποστολή έτσι ώστε να στέλνετε µαζεµένη τη δουλειά σας σε ένα mail. - Σωστή δήλωση του p ως σταθερά. Συνήθως τις σταθερές τις ορίζουµε µε κεφαλαία γράµµατα για να τις ξεχωρίζουµε µέσα στο πρόγραµµα από τις µεταβλητές. - Το µήνυµα εισαγωγής θα έπρεπε να είναι "Enter radius: " δηλαδή µε ένα κενό (space) στο τέλος. Προσέχετε καλά την εκφώνηση. - Επίσης στο µήνυµα εξόδου δεν είναι ευθυγραµµισµένα τα colons ':' (αυτό γίνεται χρησιµοποιώντας τον χαρακτήρα tab '\t') και λείπουν οι χαρακτήρες αλλαγής γραµµής '\n' στο τέλος. - Λείπει το αρχείο lab2b.c

4 1916. Καλό θα είναι να κάνετε cc και τα 2 µέλη της οµάδας όταν στέλνετε το . - Έλλειψη σχολίων - Σωστοί τύποι µεταβλητών. Τα ονόµατα θα έπρεπε να είναι πιο περιγραφικά (πχ radius) και να ξεκινάν µε lowercase. Μη χρησιµοποιείτε ονόµατα του ενός γράµµατος, εκτός από µία εξαίρεση που θα µάθετε αργότερα. Το pi θα µπορούσε να δηλωθεί ως const. Κατά σύµβαση, τα ονόµατα των σταθερών γράφονται µε κεφαλαία (δηλαδή PI) για να ξεχωρίζουν, ενώ τα ονόµατα των µεταβλητών ξεκινούν µε µικρό γράµµα (lowercase). - Σωστή στοίχιση - Στην printf ο κωδικός έπρεπε να είναι %10.3lf (6 ακέραια + 1 υποδιαστολή + 3 δεκαδικά). Επίσης, σας λείπουν κάποια tabs. Να δώσετε επιπλέον προσοχή την επόµενη φορά για τη σωστή εµφάνιση µε βάση το format που σας ζητάµε. Οι ":" δεν είναι στοιχισµένες. - Δεν διορθώσατε την ονοµατολογία των µεταβλητών calcul και programmig (αυτή δεν είναι η βέλτιστη ονοµατολογία). - Σωστά διορθώσατε τους µορφοποιητές τύπου της scanf και προσθέσατε τον τελεστή &. - Δεν αφαιρέσατε τον τελεστή & από την printf κάτι που οδηγεί στην λάθος εκτύπωση αποτελεσµάτων. - Βρήκατε το πρώτο λογικό λάθος που είχε το πρόγραµµα (διαίρεση αντί για πολλαπλασιασµό) αλλά δεν βρήκατε το δεύτερο που ήταν το return 1 αντί για Έχετε βάλει σχόλια, αλλά να µην τα βάζετε µε ελληνικούς χαρακτήρες, επίσης καλό είναι να βάζεις και ένα σχόλιο πάνω πάνω στο πρόγραµµα µε την περιγραφή του προγράµµατος -Σωστή δήλωση µεταβλητών και πολύ σωστή δήλωση του π σαν σταθερά. Να χρησιµοποιείτε πιο περιγραφικά ονόµατα µεταβλητών από το A και το R. Επίσης στο µέλλον να χρησιµοποιείτε lowercase χαρακτήρες για την δήλωση µεταβλητών ("A" -> "area") και uppercase χαρακτήρες για την δήλωση των σταθερών ("pi" -> "PI"). -Σωστό output του προγράµµατος. Σωστά χρησιµοποιείς το %010.3lf για την printf. -Διορθώσατε όλα τα λάθη και το 1 από τα 2 λογικά λάθη που υπήρχαν. Το 2ο λάθος ήταν ότι η main επιστρέφει 1.2 (δεκαδικό αριθµό) ενώ είναι δηλωµένη σαν int. Οπότε το σωστό είναι να επιστρέφει ακέραιο. Δεν πειράζει που δεν το βρήκατε διότι δεν έχετε διδαχτεί ακόµα συναρτήσεις. -Σωστό format του errors.txt, σωστά βάλατε και επεξήγηση για τα λάθη που διορθώσατε Απουσία από το εργαστήριο Στείλατε ξανά το lab1 αντί για το lab2. Μετά την αποστολή της άσκησής 1920 FAIL σας πρέπει ΠΑΝΤΑ να ελέγχετε το αντίγραφο του που λαµβάνετε για να βεβαιώνεστε ότι δεν έχει γίνει λάθος Το πρόγραµµα δεν έχει σχόλια. Διαβάστε το φυλλάδιο για σχολιασµό στο - Σωστοί τύποι µεταβλητών, αν και συνήθως χρησιµοποιούµε double για αριθµούς κινητής υποδιαστολής, για µεγαλύτερη ακρίβεια. Τα ονόµατα είναι µέτρια. Καλό το area, αλλά αποφεύγετε να κόβετε λέξεις (πχ χρησιµοποιήστε radius αντί για rad), Τέλος, κατά σύµβαση τα γράφουµε όλα µε µικρά. Σωστά δηλώσατε το π ως σταθερά. Σε αυτή την περίπτωση, το ονοµάζουµε µε κεφαλαία (πχ PI) για να ξεχωρίζει από τις µεταβλητές. - Για τη στοίχιση χρησιµοποιείτε απλά ένα κενό και όχι tab όπως θα έπρεπε. Κάντε τις κατάλληλες ρυθµίσεις στο Kate και στο µέλλον χρησιµοποιήστε tab. -Σωστή scanf - Σωστή printf αλλά ο τρόπος που ορίσατε τη συµβολοσειρά προς εκτύπωση δεν είναι ο συνήθης και µπορεί να οδηγήσει εύκολα σε λάθος. Έπρεπε να είχατε γράψει printf("radius:\t %10.3lf", rad) και όχι printf("radius:\t" "%10.3lf", rad). Επίσης, δεν βάλατε \n στο τέλος των γραµµών. - Σωστά διορθώσατε την ονοµατολογία των µεταβλητών calcul και programmig - Δεν διορθώσατε τους µορφοποιητές τύπου της scanf και επιπλέον δεν προσθέσατε τον τελεστή & σε όλες τις scanf κάτι που εξηγεί τα πολλά warnings που σας πετάει ο compiler και το λανθασµένο τελικό αποτέλεσµα. - Δεν αφαιρέσατε τον τελεστή & από την printf. (παρόλο που το έχετε δηλωµένο στο errors.txt δεν έχετε κάνει την αλλαγή στον κώδικα) - Δεν βρήκατε τα 2 λογικά λάθη που είχε το πρόγραµµα (διαίρεση αντί για πολλαπλασιασµό και return 1 αντί για 1.2) (παρόλο που το έχετε δηλωµένο στο errors.txt δεν έχετε κάνει την αλλαγή στον κώδικα) ΓΕΝΙΚΑ: Μέτρια δουλειά. Στο µέλλον προσπαθήστε να είστε καλύτερα προετοιµασµένοι για να προλαβαίνετε και τις δύο ασκήσεις Καλή προσπάθεια µε τα σχόλια αλλά χρειαζόταν να έχετε και κάποια γενικά σχόλια για το τι κάνει το πρόγραµµα (γραµµένα στην αρχή του αρχείου). Το σχόλιο για τις δηλώσεις µεταβλητών είναι περιττό εφόσον είναι προφανές ότι σε εκείνο το σηµείο γίνονται δηλώσεις. - Σωστοί τύποι αλλά µέτρια ονόµατα µεταβλητών. Αποφεύγετε κοµµένα ονόµατα, καθώς και ονόµατα του ενός γράµµατος. Για παράδειγµα, χρησιµοποιήστε radius αντί για rad και PI αντί για p. Σωστά δηλώσατε το p ως const. Κατά σύµβαση, τα ονόµατα των σταθερών γράφονται µε κεφαλαία (δηλαδή PI) για να ξεχωρίζουν. Σωστά έχετε βάλει κενές γραµµές για να διαχωρίζουν οι επιµέρους ενότητες του προγράµµατος. Χρησιµοποιήστε παρόµοια τακτική για τις εκφράσεις. Για παράδειγµα, γράψτε perimeter = 2 * p * rad; και όχι perimeter= 2* p* rad ; - Στην printf ο κωδικός έπρεπε να είναι %10.3lf (6 ακέραια + 1 υποδιαστολή + 3 δεκαδικά). Επίσης, έχετε κάποια επιπλέον κενά. Το ζητούµενο ήταν Radius :\t%9.3lf\n - Δεν παραδόθηκε. Στο µέλλον θέλουµε να δουλεύετε και τη δεύτερη άσκηση Το πρόγραµµα δεν έχει σχόλια. Διαβάστε το φυλλάδιο για σχολιασµό στο - Σωστοί τύποι µεταβλητών, αν και συνήθως χρησιµοποιούµε double για αριθµούς κινητής υποδιαστολής, για µεγαλύτερη ακρίβεια. Τα ονόµατα είναι καλά, αλλά κατά σύµβαση τα γράφουµε όλα µε µικρά. Το π θα µπορούσατε να το είχατε δηλώσει ως σταθερά. Σε αυτή την περίπτωση, το ονοµάζουµε µε κεφαλαία (πχ PI) για να ξεχωρίζει από τις µεταβλητές. -Σωστή στοίχιση. Καλό είναι να αφήνετε µια κενή γραµµή κάτω από τα #include. -Σωστή scanf - Σωστή printf αλλά ξεχάσατε να εκτυπώσετε το tab κι έχετε κάποια επιπλέον κενά. - Λίγες οι διορθώσεις σας. Προσπαθήστε να ολοκληρώσετε την άσκηση και µετά δείτε τη λύση. Για το lab1 που έστειλες: Καλή δουλειά, µπράβο Έχετε βάλει σχόλια, αλλά να µην τα βάζετε µε ελληνικούς χαρακτήρες, επίσης καλό είναι να βάζεις και ένα σχόλιο πάνω πάνω στο πρόγραµµα µε την περιγραφή του προγράµµατος -Σωστή δήλωση µεταβλητών και πολύ σωστή δήλωση του π σαν σταθερά. Να χρησιµοποιείτε πιο περιγραφικά ονόµατα µεταβλητών από το A και το R. Επίσης στο µέλλον να χρησιµοποιείτε lowercase χαρακτήρες για την δήλωση µεταβλητών ("A" -> "area") και uppercase χαρακτήρες για την δήλωση των σταθερών ("pi" -> "PI"). -Σωστό output του προγράµµατος. Σωστά χρησιµοποιείς το %010.3lf για την printf. -Διορθώσατε όλα τα λάθη και το 1 από τα 2 λογικά λάθη που υπήρχαν. Το 2ο λάθος ήταν ότι η main επιστρέφει 1.2 (δεκαδικό αριθµό) ενώ είναι δηλωµένη σαν int. Οπότε το σωστό είναι να επιστρέφει ακέραιο. Δεν πειράζει που δεν το βρήκατε διότι δεν έχετε διδαχτεί ακόµα συναρτήσεις. -Σωστό format του errors.txt, σωστά βάλατε και επεξήγηση για τα λάθη που διορθώσατε. -Δεν έχετε βάλει καθόλου σχόλια στο πρόγραµµα. Κοιτάξτε το φυλλάδιο για τον σχολιασµό του προγράµµατος στο -Σωστά ονόµατα και δήλωση µεταβλητών σαν double. Πολύ σωστά δηλώσατε το π σαν const double. Στο µέλλον να χρησιµοποιείτε lowercase χαρακτήρες για την δήλωση µεταβλητών ("Area" -> "area") και uppercase χαρακτήρες για την δήλωση των σταθερών ("pi" -> "PI"). -Σωστό output του προγράµµατος. Σωστά χρησιµοποιείς το %10.3lf για την printf. -Διορθώσατε όλα τα λάθη και τα 2 λογικά λάθη που υπήρχαν. -Σωστό format του errors.txt Σωστά βάλατε και επεξήγηση για την διόρθωση των λαθών που βρήκατε.

5 Γενικά: Καλή δουλειά Σωστά ονόµατα µεταβλητών. - Δηλώσατε σχεδόν 100% σωστά την σταθερά pi, χρησιµοποιούµε κεφαλαία γράµµατα όταν δηλώνουµε σταθερές. - Δεν έχετε γράψει σχόλια για την λειτουργία του προγράµµατος. Δείτε το σχετικό φυλλάδιο στο - Στην printf ο κωδικός έπρεπε να είναι %10.3lf (6 ακέραια + 1 υποδιαστολή + 3 δεκαδικά). Επίσης προσθέσατε ένα κενό χαρακτήρα µετά το χαρακτήρα tab. - Δεν έχετε σηµειώσει τα λάθη που εµφανίζονται κατά την µεταγλώττιση του προγράµµατος. Γενικά: Οριακό pass Έστειλες 4 φορές το ίδιο mail. - Η µεταβλητή "p" θα έπρεπε να είχε δηλωθεί ως "pi" και να είχε τύπο double/float. Τέλος εφόσον την δηλώνετε ως σταθερά θα έπρεπε να την γράφατε µε κεφαλαία γράµµατα. Αυτή την πρακτική χρησιµοποιούµε για να ξεχωρίζουµε τις σταθερές από τις µεταβλητές ενός προγράµµατος. - Λίγα σχόλια στον κώδικά σας. - Δεν είναι καλή πρακτική να είναι κολληµένες οι δηλώσεις µιας συνάρτησης µε το κυρίως σώµα της. - Στην printf ο κωδικός έπρεπε να είναι %10.3lf (6 ακέραια + 1 υποδιαστολή + 3 δεκαδικά). Επίσης δεν τοποθετήσατε σωστά το χαρακτήρα tab και δεν έχετε ευθυγραµµίσει τα ":" όπως σας ζητήθηκε στην εκφώνηση Απουσία από το εργαστήριο Καλή προσπάθεια µε τα σχόλια προγράµµατος. Το σχόλιο για τη scanf είναι περιττό γιατί είναι προφανές από τη χρήση της scanf ότι σε αυτό το σηµείο γίνεται είσοδος ακτίνας. Χωρίσατε ωραία µε κενές γραµµές τις διάφορες ενότητες του προγράµµατος. - Σωστή η printf, αλλά προσέξτε λίγο την ορθογραφία. Έπρεπε να εκτυπώνει Radius, όχι Radious. Επίσης, δε χρειαζόταν επιπλέον κενό µετά το tab. - Στις διορθώσεις των ονοµάτων των µεταβλητών, θα ήταν πιο λογικό να είχατε αλλάξει το calcul σε calculus αντί για το calculus σε calcul (οµοίως και για τα υπόλοιπα) ώστε τα ονόµατα των µεταβλητών να είναι γραµµατικά σωστά. - Πολύ καλή δουλειά Καλά σχόλια. Συνήθως τα σχόλια προηγούνται των εντολών που περιγράφουν έτσι ώστε να εντοπίζουµε το σηµείο του προγράµµατος που µας ενδιαφέρει χωρίς να διαβάζουµε από πριν όλες τις λεπτοµέρειες. - Σωστή δήλωση του pi ως σταθερά. Συνήθως τις σταθερές τις ορίζουµε µε κεφαλαία γράµµατα για να τις ξεχωρίζουµε µέσα στο πρόγραµµα από τις µεταβλητές. - Σωστή µορφοποίηση εξόδου. - Σωστή διόρθωση του λογικού λάθους (διαίρεση αντί πολλαπλασιασµού µε τον αριθµό των µαθηµάτων). Γενικά: Καλή δουλειά Σωστά ονόµατα µεταβλητών. - Δηλώσατε σχεδόν 100% σωστά την σταθερά pi, χρησιµοποιούµε κεφαλαία γράµµατα όταν δηλώνουµε σταθερές. - Είναι λάθος να γράφετε σχόλια για κάθε γραµµή του κώδικα, ανατρέξτε στην ενδεικτική λύση. Μπράβο πάντως για τα αρχικά σχόλια προγράµµατος. - Στην printf ο κωδικός έπρεπε να είναι %10.3lf (6 ακέραια + 1 υποδιαστολή + 3 δεκαδικά). Επίσης δεν χρησιµοποιήσατε το χαρακτήρα tab και δεν έχετε ευθυγραµµίσει τα ":" όπως σας ζητήθηκε στην εκφώνηση. - Δεν έχετε σηµειώσει τα λάθη που εµφανίζονται κατά την µεταγλώττιση του προγράµµατος Δεν εκτυπώνετε τα αποτελέσµατα µε τη µορφή που σας ζητήθηκαν. -Δεν χρειάζεται να αφήνετε κενά πριν κλείσετε τις παρενθέσεις - Στην printf ο κωδικός έπρεπε να είναι %10.3lf (6 ακέραια + 1 υποδιαστολή + 3 δεκαδικά). ΓΕΝΙΚΑ: προσέξτε την έξοδο των προγραµµάτων σας. Γενικά: Οριακό Χρησιµοποιήσατε "καρφωτή" σταθερά για την τιµή pi. Έπρεπε να την αποθηκεύσετε σε κάποια σταθερά δηλωµένη µε const. - Παρότι δηλώσατε µε σχόλια τι σηµαίνουν τα ονόµατα µεταβλητών "rad/per" δεν είναι περιγραφικά. Μία σωστή επιλογή ονοµάτων είναι "radius/perimeter". - Στην printf ο κωδικός έπρεπε να είναι %10.3lf (6 ακέραια + 1 υποδιαστολή + 3 δεκαδικά). Επίσης δεν χρησιµοποιήσατε το χαρακτήρα tab όπως σας ζητήθηκε στην εκφώνηση. - Το αρχείο errors.txt είναι κενό. Σε επόµενα εργαστήρια αυτό θα είναι ικανή συνθήκη για FAIL.

6 - Εν µέρη σωστή επιλογή ονοµάτων για τις µεταβλητές. Το εµβαδό δεν έχει αρκετά περιγραφικό όνοµα. - Θα µπορούσατε να χρησιµοποιήσετε τον προσδιοριστή const για την σταθερά pi. Καλό θα ήταν επιλέξετε κεφαλαία γράµµατα για το όνοµα της σταθεράς. - Λάθος υπολογισµός του εµβαδού. ΔΕΝ ακολουθήσατε τον τύπο που σας δόθηκε. - Η σωστή εκτύπωση στην printf είναι printf("radius:\t %10.3lf\n", radius); και printf("area :\t %10.3lf\n", E); Απέχει πολύ από αυτό που γράψατε εσείς. - Προσοχή στην µορφοποίηση του κώδικα σας. Αν και η τελική µορφή είναι σωστή, αλλού χρησιµοποιείται tab και αλλού spaces. Αν ανοίξει κάποιος τον κώδικα µε διαφορετικό editor θα εµφανίζεται χωρίς µορφοποίηση. Επιλέξτε έναν από τους δύο τρόπους στα προγράµµατα σας Οριακό - Περνάει οριακά. - Διορθώσατε το λάθος της 1η γραµµής και την ονοµατολογία κάποιων µεταβλητών. - Δεν διορθώσατε ούτε τους µορφοποιητές τύπου της scanf, ούτε προσθέσατε τον τελεστή & πριν από τις µεταβλητές στις οποίες αποθηκεύει η scanf. - Προσθέσατε τις γραµµές, 11,12 "float * english ; float * calculus ;" που συντακτικά διορθώνουν το πρόβληµα σας, αλλά εισάγουν λογικά λάθη (αναφέρονται σε pointers) που δεν έχετε διδαχθεί ακόµη και έτσι όπως τις έχετε γράψει το πρόγραµµα σας εάν έκανα compile θα κράσαρε. - Το πρόγραµµα δεν κάνει compile και αποµένουν επιπλέον συντακτικά λάθη προς διόρθωση. - Τα 2 λογικά λάθη που είχε το πρόγραµµα δεν τα βρήκατε διότι θα έπρεπε πρώτα να δοκιµάσετε να το τρέξετε. Γενικά: Καλή δουλειά Δεν είναι σωστό να αφήνετε κενό µετά το # στα "#include" - Σωστά ονόµατα µεταβλητών. - Δηλώσατε σχεδόν 100% σωστά την σταθερά pi, χρησιµοποιούµε κεφαλαία γράµµατα όταν δηλώνουµε σταθερές. - Δεν έχετε βάλει σχόλια στον κώδικά σας. - Είναι κακή πρακτική να αφήνετε κενό µετά το όνοµα συναρτήσεων κατά την κλήση τους. Π.χ printf ("Hello!"). - Είναι λάθος να γράφετε σχόλια για κάθε γραµµή του κώδικα, ανατρέξτε στην ενδεικτική λύση. - Στην printf ο κωδικός έπρεπε να είναι %10.3lf (6 ακέραια + 1 υποδιαστολή + 3 δεκαδικά). Επίσης προσθέσατε ένα κενό χαρακτήρα µετά το χαρακτήρα tab. - Δεν έχετε σηµειώσει τα λάθη που εµφανίζονται κατά την µεταγλώττιση του προγράµµατος. - Διαδικαστικά: OK : Σχόλια: ΟΚ - Ονόµατα Μεταβλητών: ΟΚ - Στοίχιση και γενικότερη µορφή κώδικα: ΟΚ - scanf format: ΟΚ - printf format: Το output δεν έχει στοιχηθεί όπως θα θέλαµε, ξαναδείτε την εκφώνηση. Λάθος στο format specifier, θα έπρεπε να είναι 10.3lf όπου 10 είναι οι χαρακτήρες µαζί µε την υποδιαστολή : λάθη που εξακολουθεί να βγάζει ο compiler: ΚΑΝΕΝΑ - warnings που συνδέονται µε format specifiers της scanf: ΚΑΝΕΝΑ - επιστρέφει floating point αντί για ακέραιο: Προσέξτε ότι η main πρέπει να επιστρέφει ακέραιο. ΓΕΝΙΚΑ: Μέτρια δουλειά - Σωστό όνοµα attachment, σωστή δηµιουργία του tgz αρχείου. Το θέµα του είναι σωστό, αλλά δεν υπάρχει λόγος να το γράφετε µε κεφαλαία. - Το πρόγραµµά σας δεν έχει καθόλου σχόλια. Διαβάστε το φυλλάδιο για σχολιασµό στο και την επόµενη φορά γράψτε περιγραφικά σχόλια. - Σωστοί τύποι αλλά άσχηµα ονόµατα µεταβλητών. Μη χρησιµοποιείτε κοµµένες λέξεις γιατί µειώνουν την αναγνωσιµότητα και κατανόηση του προγράµµατος. Πείτε radius και perimeter αντί για rad και per. Σωστά δηλώσατε το pi ως const. Κατά σύµβαση, τα ονόµατα των σταθερών γράφονται µε κεφαλαία (δηλαδή PI) για να ξεχωρίζουν. Είναι καλό να χωρίζετε µε κενές γραµµές τις διάφορες ενότητες του προγράµµατος όπως κάνατε, αλλά όχι µε περισσότερες από µία κενές γραµµές. - Στην printf έχετε κάποια περίεργα κόµµατα, επιπλέον κενά, και λείπει ο χαρακτήρας tab. Επίσης, ο κωδικός έπρεπε να είναι %10.3lf (6 ακέραια + 1 υποδιαστολή + 3 δεκαδικά). Είχατε σχόλιο και στο lab1 σχετικά µε την παρουσίαση των αποτελεσµάτων, οπότε προσέξτε το! - Στις διορθώσεις των ονοµάτων των µεταβλητών, θα ήταν πιο λογικό να είχατε αλλάξει το calcul σε calculus αντί για το calculus σε calcul (οµοίως και για τα υπόλοιπα) ώστε τα ονόµατα των µεταβλητών να είναι γραµµατικά σωστά. - Διαδικαστικά: OK : Σχόλια: ΟΚ - Ονόµατα Μεταβλητών: ΟΚ - Στοίχιση και γενικότερη µορφή κώδικα: ΟΚ - scanf format: ΟΚ - printf format: Το output δεν έχει στοιχηθεί όπως θα θέλαµε, ξαναδείτε την εκφώνηση. : - λάθη που εξακολουθεί να βγάζει ο compiler: KANENA - warnings που συνδέονται µε format specifiers της scanf: KANENA - επιστρέφει floating point αντί για ακέραιο: ΔΙΟΡΘΩΘΗΚΕ Απουσία από το εργαστήριο Εκτός από την αρχή δεν έχετε σχόλια. Διαβάστε το φυλλάδιο για σχολιασµό στο και την επόµενη φορά γράψτε περιγραφικά σχόλια. - Σωστοί τύποι αλλά κάνατε λάθος στα ονόµατα µεταβλητών. Έχετε κάνει ορθογραφικό λάθος και γράψατε το "radius" σαν "radious". Θα µπορούσατε να δηλώσετε το pi ως const καθώς είναι σταθερά και δεν πρέπει να αλλάξει κατά τη διάρκεια του προγράµµατος. Κατά σύµβαση, τα ονόµατα των σταθερών γράφονται µε κεφαλαία (δηλαδή PI) για να ξεχωρίζουν. - Μέτρια στοίχιση, µπορείτε να δείτε τη λύση που έχει ανέβει και να παρατηρήσετε τα σηµεία στα οποία αφήνουµε κενή γραµµή. Για παράδειγµα µπορείτε να ξεχωρίσετε το σηµείο από το οποίο έχουµε αλληλεπίδραση µε το χρήστη(printf, scanf) µε το σηµείο που γίνονται οι πράξεις. - Στην printf έχετε αφήσει αρκετά κενά. Το ζητούµενο ήταν: Radius:\t%10.3lf\n και όχι Radius: \t %10.3lf \n"" καθώς και η στοίχιση δεν είναι ακριβώς όπως τη ζητούσαµε. Οι ":" Θα έπρεπε να είναι η µία κάτω από την άλλη. - Το πρόγραµµα έχει ένα λογικό λάθος το οποίο και διορθώσατε. Καλή δουλειά.

7 1947 Έβαλες λάθος ΑΕΜ στο tgz αρχείο : - Σχόλια: ΚΑΝΕΝΑ ΣΧΟΛΙΟ! Διαβάστε το - Ονόµατα Μεταβλητών: Μονογραµµατικά. Τα ονόµατά σας πρέπει να είναι πιο περιγραφικά - Στοίχιση και γενικότερη µορφή κώδικα: ΟΚ - scanf format: ΟΚ - printf format: Το output δεν έχει στοιχηθεί όπως θα θέλαµε (κενά, χρήση tab), ξαναδείτε την εκφώνηση. : - λάθη που εξακολουθεί να βγάζει ο compiler: Πολύ σοβαρό λάθος, η printf παίρνει ΔΙΕΥΘΥΝΣΗ µεταβλητής προς εκτύπωση, όχι ΤΙΜΗ µεταβλητής, θα έπρεπε να είναι το αντίθετο. - warnings που συνδέονται µε format specifiers της scanf: ΚΑΝΕΝΑ - επιστρέφει floating point αντί για ακέραιο: Προσέξτε ότι η main πρέπει να επιστρέφει ακέραιο. ΓΕΝΙΚΑ: Πολύ καλή δουλειά. Συνεχίστε έτσι Πολύ καλή η προσπάθεια σας στα σχόλια. Περιγράψατε πολύ σωστά στην αρχή του προγράµµατος τη λειτουργία του και βάλατε σχόλια στον υπολογισµό του εµβαδού που είναι και η σηµαντικότερη λειτουργία. - Σωστοί τύποι και καλά ονόµατα µεταβλητών. Πολύ σωστά δηλώσατε το pi σαν σταθερά αλλά τα ονόµατα των σταθερών κατά σύµβαση γράφονται µε κεφαλαία(δηλαδή PI) για να ξεχωρίζουν. Χωρίσατε ωραία µε κενές γραµµές τις διάφορες ενότητες του προγράµµατος. - Σωστή η printf, µπράβο. - Πολύ σωστά διορθώσατε όλα τα λογικά και συντακτικά λάθη - Στις διορθώσεις των ονοµάτων των µεταβλητών, θα ήταν πιο λογικό να είχατε αλλάξει το calcul σε calculus αντί για το calculus σε calcul (οµοίως και για τα υπόλοιπα) ώστε τα ονόµατα των µεταβλητών να είναι γραµµατικά σωστά. -Πολύ σωστά διορθώσατε την επιστροφή τιµής από τη main να είναι ένας ακέραιος αριθµός και όχι κινητής υποδιαστολής Καλή προσπάθεια µε τα σχόλια προγράµµατος. Το σχόλιο για τη σταθερά είναι περιττό γιατί δε λέει κάτι παραπάνω από αυτό που βλέπουµε στην εντολή. - Σωστοί τύποι και καλά ονόµατα µεταβλητών. Σωστά δηλώσατε το PI ως const και το γράψατε µε κεφαλαία. Μπράβο. Χωρίσατε ωραία µε κενές γραµµές τις διάφορες ενότητες του προγράµµατος. - Σωστή η printf, µπράβο. - Πολύ καλή δουλειά, µπράβο FAIL 1956 Στείλατε ξανά το lab1 αντί για το lab2. Μετά την αποστολή της άσκησής σας πρέπει ΠΑΝΤΑ να ελέγχετε το αντίγραφο του που λαµβάνετε για να βεβαιώνεστε ότι δεν έχει γίνει λάθος. - Το πρόγραµµα δεν έχει σχόλια. Διαβάστε το φυλλάδιο για σχολιασµό στο - Σωστοί τύποι µεταβλητών, αν και συνήθως χρησιµοποιούµε double για αριθµούς κινητής υποδιαστολής, για µεγαλύτερη ακρίβεια. Τα ονόµατα είναι µέτρια. Καλό το area, αλλά αποφεύγετε να κόβετε λέξεις (πχ χρησιµοποιήστε radius αντί για rad), Τέλος, κατά σύµβαση τα γράφουµε όλα µε µικρά. Σωστά δηλώσατε το π ως σταθερά. Σε αυτή την περίπτωση, το ονοµάζουµε µε κεφαλαία (πχ PI) για να ξεχωρίζει από τις µεταβλητές. - Για τη στοίχιση χρησιµοποιείτε απλά ένα κενό και όχι tab όπως θα έπρεπε. Κάντε τις κατάλληλες ρυθµίσεις στο Kate και στο µέλλον χρησιµοποιήστε tab. -Σωστή scanf - Σωστή printf αλλά ο τρόπος που ορίσατε τη συµβολοσειρά προς εκτύπωση δεν είναι ο συνήθης και µπορεί να οδηγήσει εύκολα σε λάθος. Έπρεπε να είχατε γράψει printf("radius:\t %10.3lf", rad) και όχι printf("radius:\t" "%10.3lf", rad). Επίσης, δεν βάλατε \n στο τέλος των γραµµών. - Σωστά διορθώσατε την ονοµατολογία των µεταβλητών calcul και programmig - Δεν διορθώσατε τους µορφοποιητές τύπου της scanf και επιπλέον δεν προσθέσατε τον τελεστή & σε όλες τις scanf κάτι που εξηγεί τα πολλά warnings που σας πετάει ο compiler και το λανθασµένο τελικό αποτέλεσµα. - Δεν αφαιρέσατε τον τελεστή & από την printf. (παρόλο που το έχετε δηλωµένο στο errors.txt δεν έχετε κάνει την αλλαγή στον κώδικα) - Δεν βρήκατε τα 2 λογικά λάθη που είχε το πρόγραµµα (διαίρεση αντί για πολλαπλασιασµό και return 1 αντί για 1.2) (παρόλο που το έχετε δηλωµένο στο errors.txt δεν έχετε κάνει την αλλαγή στον κώδικα) Γενικά: Καλή δουλειά, αλλά προσέξτε τα ονόµατα των µεταβλητών/σταθερών Δηλώσατε σχεδόν 100% σωστά την σταθερά pi, χρησιµοποιούµε κεφαλαία γράµµατα όταν δηλώνουµε σταθερές. - Παρότι δηλώσατε µε σχόλια τι σηµαίνουν τα ονόµατα µεταβλητών "r/p" δεν είναι περιγραφικά. Μία σωστή επιλογή ονοµάτων είναι "radius/perimeter". - Τα σχόλια καλό είναι να βρίσκονται σε δική τους γραµµή. - Στην printf ο κωδικός έπρεπε να είναι %10.3lf (6 ακέραια + 1 υποδιαστολή + 3 δεκαδικά). Επίσης έχετε βάλει ένα κενό χαρακτήρα µετά το tab στην εκτύπωση της περιµέτρου. Τέλος δεν έχετε ευθυγραµµίσει τα ":" όπως σας ζητήθηκε στην εκφώνηση Δεν έχετε σηµειώσει τα λάθη που εµφανίζονται κατά την µεταγλώττιση του προγράµµατος. - Mη σωστή επιλογή ονοµάτων για τις µεταβλητές. Τα ονόµατα δεν είναι περιγραφικά. Επιπλέον συνήθως επιλέγουµε µεταβλητές που περιέχουν µη κεφαλαία γράµµατα, ενώ τις σταθερές τις γράφουµε µε κεφαλαία. Αντιστρέψατε τον παραπάνω άτυπο κανόνα. - Σωστή χρήση του προσδιοριστή const για την σταθερά pi. Καλό θα ήταν επιλέξετε κεφαλαία γράµµατα για το όνοµα της σταθεράς. - Στην printf δεν ακολουθήσατε τις οδηγίες και για αυτό το αποτέλεσµα δεν είναι στοιχισµένο. H χαρακτήρας tab ζητήθηκε να µπει µετά τον χαρακτήρα : και όχι στη αρχή της κάθε γραµµής. - Σωστή µορφοποίηση του κώδικα. - Συνολικά καλή δουλειά. - Σωστή διόρθωση errors και warnings από τον compiler. - Επιπλέον βρήκατε και τα 2 λογικά λάθη που είχε ο κώδικας. - Συγχαρητήρια, καλή δουλειά. - Σωστή χρήση του προσδιοριστή const για την σταθερά pi. Καλό θα ήταν επιλέξετε κεφαλαία γράµµατα για το όνοµα της σταθεράς. - Η σωστή εκτύπωση στην printf είναι %10.3lf (6 ακέραια + 1 υποδιαστολή + 3 δεκαδικά) - Προσοχή στην µορφοποίηση του κώδικα σας. Αν και η τελική µορφή είναι σωστή, αλλού χρησιµοποιείται tab και αλλού spaces. Αν ανοίξει κάποιος τον κώδικα µε διαφορετικό editor θα εµφανίζεται χωρίς µορφοποίηση. Επιλέξτε έναν από τους δύο τρόπους στα προγράµµατα σας. - Συνολικά καλή δουλειά. - Σωστή διόρθωση errors και warnings από τον compiler. - Τα µαθήµατα θα µπορούσατε να τα είχατε ορίσει και ως int. Η επιλογή float δεν είναι λάθος. - Επιπλέον βρήκατε και τα 2 λογικά λάθη που είχε ο κώδικας. - Συγχαρητήρια, καλή δουλειά.

8 - Προσέχετε κατά τη συµπίεση και την αποστολή έτσι ώστε να στέλνετε µαζεµένη τη δουλειά σας σε ένα mail Ο τύπος double για µεταβλητές κινητής υποδιαστολής έχει µεγαλύτερη ακρίβεια από τον τύπο float οπότε είναι καλό να χρησιµοποιείτε αυτόν. Στη συγκεκριµένη άσκηση δεν έχει ιδιαίτερη διαφορά. - Δεν έχετε γράψει σχόλια για την λειτουργία του προγράµµατος. Δείτε το σχετικό φυλλάδιο στο - Το µήνυµα εισαγωγής θα έπρεπε να είναι "Enter radius: ". Προσέχετε καλά την εκφώνηση. - Επίσης στο µήνυµα εξόδου δεν είναι ευθυγραµµισµένα τα colons ':' (αυτό γίνεται χρησιµοποιώντας τον χαρακτήρα tab '\t') και λείπει ο χαρακτήρας αλλαγής γραµµής '\n' στο τέλος. ΓΕΝΙΚΑ: Καλή προσπάθεια Στείλατε 2 φορές το mail. - Δεν έχετε γράψει σχόλια για την λειτουργία του προγράµµατος. Δείτε το σχετικό φυλλάδιο στο - Σωστοί τύποι αλλά µέτρια ονόµατα µεταβλητών. Να προτιµάτε να µην κόβετε τα ονόµατα των µεταβλητών, δηλαδή να γράφετε "perimeter" αντί για "per" και "radius" αντί για "rad". To pi επειδή είναι σταθερός αριθµός µπορεί να δηλωθεί ως const double. Με το να τον δηλώνουµε σαν const δεν επιτρέπουµε να γίνει οποιαδήποτε αλλαγή πάνω του και έτσι λειτουργεί σαν σταθερά. Κατά σύµβαση, τα ονόµατα των σταθερών γράφονται µε κεφαλαία (δηλαδή PI) για να ξεχωρίζουν. - Μέτρια στοίχιση. Σωστά αφήσατε µια κενή γραµµή µεταξύ των δηλώσεων και των υπολοίπων εντολών αλλά χρειάζεται να αφήνετε περισσότερες ανάλογα µε το σηµείο του προγράµµατος. Δείτε τη σωστή λύση που έχει ανεβεί και παρατηρήστε πως αφήνουµε κενές γραµµές µεταξύ της αλληλεπίδρασης µε το χρήστη (printf,scanf) και των πράξεων έτσι ώστε να τα ξεχωρίζουµε πιο εύκολα. - Στην printf ο κωδικός έπρεπε να είναι %10.3lf (6 ακέραια + 1 υποδιαστολή + 3 δεκαδικά). Σωστά έχετε στοιχίσει τα αποτελέσµατα και έχετε βάλει \t εκεί που πρέπει. - Στις διορθώσεις των ονοµάτων των µεταβλητών, θα ήταν πιο λογικό να είχατε αλλάξει το calcul σε calculus αντί για το calculus σε calcul (οµοίως και για τα υπόλοιπα) ώστε τα ονόµατα των µεταβλητών να είναι γραµµατικά σωστά Καλή προσπάθεια µε τα σχόλια προγράµµατος. Το σχόλιο για τη scanf είναι περιττό γιατί είναι προφανές από τη χρήση της scanf ότι σε αυτό το σηµείο γίνεται είσοδος ακτίνας. Χωρίσατε ωραία µε κενές γραµµές τις διάφορες ενότητες του προγράµµατος. - Σωστή η printf, αλλά προσέξτε λίγο την ορθογραφία. Έπρεπε να εκτυπώνει Radius, όχι Radious. Επίσης, δε χρειαζόταν επιπλέον κενό µετά το tab. - Στις διορθώσεις των ονοµάτων των µεταβλητών, θα ήταν πιο λογικό να είχατε αλλάξει το calcul σε calculus αντί για το calculus σε calcul (οµοίως και για τα υπόλοιπα) ώστε τα ονόµατα των µεταβλητών να είναι γραµµατικά σωστά Πολύ καλή η προσπάθεια σας στα σχόλια. Περιγράψατε πολύ σωστά στην αρχή του προγράµµατος τη λειτουργία του και βάλατε σχόλια στον υπολογισµό του εµβαδού που είναι και η σηµαντικότερη λειτουργία. - Σωστοί τύποι και καλά ονόµατα µεταβλητών. Πολύ σωστά δηλώσατε τ pi σαν σταθερά αλλά τα ονόµατα των σταθερών κατά σύµβαση γράφονται µε κεφαλαία(δηλαδή PI) για να ξεχωρίζουν. Χωρίσατε ωραία µε κενές γραµµές τις διάφορες ενότητες του προγράµµατος. - Σωστή η printf, µπράβο. - Πολύ σωστά διορθώσατε όλα τα λογικά και συντακτικά λάθη - Στις διορθώσεις των ονοµάτων των µεταβλητών, θα ήταν πιο λογικό να είχατε αλλάξει το calcul σε calculus αντί για το calculus σε calcul (οµοίως και για τα υπόλοιπα) ώστε τα ονόµατα των µεταβλητών να είναι γραµµατικά σωστά. -Πολύ σωστά διορθώσατε την επιστροφή τιµής από τη main να είναι ένας ακέραιος αριθµός και όχι κινητής υποδιαστολής FAIL Δεν ελήφθη άσκηση Καλή προσπάθεια µε τα σχόλια προγράµµατος. Καλό είναι να αφήνετε µια κενή γραµµή µετά τα #include. - Σωστή στοίχιση - Στην printf ο κωδικός έπρεπε να είναι %10.3lf (6 ακέραια + 1 υποδιαστολή + 3 δεκαδικά). Επίσης, δεν εκτυπώσατε το χαρακτήρα tab όπως ζητήθηκε. -Έχετε βάλει σχόλια, αλλά λείπει ένα γενικό σχόλιο πάνω πάνω µε την περιγραφή του προγράµµατος. -Σωστοί τύποι και ονόµατα µεταβλητών. Δεν έχετε δηλώσει το π=3.14 σαν µεταβλητή ούτε σαν constant αλλά την χρησιµοποιείτε απ ευθείας στην πράξη. Καλό θα ήταν να την δηλώσετε σαν const double. -Σωστά χρησιµοποιείς το %10.3lf για την printf. -Διορθώσατε όλα τα λάθη και τα 2 λογικά λάθη που υπήρχαν. -Λάθος format του errors.txt Έπρεπε να διαχωρίζετε τα λάθη µε ##### να βάζετε την γραµµή του compiler που έβγαλε λάθος και την διόρθωση που κάνατε. Το δικό σας αρχείο δεν έχει ούτε στοίχιση

9 Δεν έχετε σχόλια. Διαβάστε το φυλλάδιο για σχολιασµό στο και την επόµενη φορά γράψτε περιγραφικά σχόλια. - Στην printf έχετε αφήσει αρκετά κενά. Το ζητούµενο ήταν: Radius:\t%10.3lf\n και όχι Radius: \t %10.3lf \n" - Πολύ καλή δουλειά, µπράβο Το πρόγραµµα δεν έχει σχόλια. Διαβάστε το φυλλάδιο για σχολιασµό στο -Δεν εκτυπώνετε τα αποτελέσµατα µε τη µορφή που σας ζητήθηκαν. ΓΕΝΙΚΑ: προσέξτε την έξοδο των προγραµµάτων σας. - Δεν διορθώσατε όλα τα warnings. Δεν διορθώσατε σωστά την τελευταία printf -H εντολή return στο τέλος επιστρέφει αριθµό κινητής υποδιαστολής αντί για ακέραιο. Στο 90% των περιπτώσεων πρέπει να επιστρέφει 0. Θα µάθετε αργότερα γιατί, και πότε επιστρέφουµε κάτι διαφορετικό! FAIL Καλή προσπάθεια µε τα σχόλια του προγράµµατος. Μπορείτε να κάνετε κάποιες βελτιώσεις. Δείτε τη σελίδα 10 στο φυλλάδιο µε τίτλο "Αρχές καλού προγραµµατισµού". Αποφεύγουµε σχόλια που µας λένε το προφανές (π.χ. "Δήλωση µεταβλητών", "Εκτύπωση ακτίνας"). Σωστά βάλατε σχόλιο για τον υπολογισµό του εµβαδού. -Σωστοί τύποι δεδοµένων και ονόµατα µεταβλητών. Αποφεύγουµε όµως ονόµατα του ενός γράµµατος. Για παράδειγµα, χρησιµοποιήστε PI αντί για p. Σωστά δηλώσατε το p ως const. Κατά σύµβαση τα ονόµατα των σταθερών γράφονται µε κεφαλαία(δηλαδή PI) για να ξεχωρίζουν -Σωστές πράξεις Σωστά έχετε βάλει κενές γραµµές για να διαχωρίζουν οι επιµέρους ενότητες του προγράµµατος - Σωστή η printf. Δεν χρειαζόταν κενό µετά από το \t. - Πολύ καλή δουλειά, µπράβο. - Σωστή χρήση του προσδιοριστή const για την σταθερά PI. Σωστή επιλογή κεφαλαίων γραµµάτων για το όνοµα της σταθεράς. - Σωστή εκτύπωση στην printf (γραµµές 17,18)) - Σωστή µορφοποίηση κώδικά. Δεν χρειάζεται να αφήνετε τόσο µεγάλα κενά µεταξύ των τµηµάτων του προγράµµατος σας. - Συνολικά καλή δουλειά. - Σωστή διόρθωση errors και warnings από τον compiler. - Επιπλέον βρήκατε και τα 2 λογικά λάθη που είχε ο κώδικας. - Συγχαρητήρια, καλή δουλειά. Στείλατε ξανά το lab1 αντί για το lab2. Μετά την αποστολή της άσκησής σας πρέπει ΠΑΝΤΑ να ελέγχετε το αντίγραφο του που λαµβάνετε για να βεβαιώνεστε ότι δεν έχει γίνει λάθος. - Σωστή χρήση του προσδιοριστή const για την σταθερά pi. Καλό θα ήταν επιλέξετε κεφαλαία γράµµατα για το όνοµα της σταθεράς. - Σωστή εκτύπωση στην printf. - Προσοχή στην µορφοποίηση του κώδικα σας. Αν και η τελική µορφή είναι σωστή, αλλού χρησιµοποιείται tab και αλλού spaces. Αν ανοίξει κάποιος τον κώδικα µε διαφορετικό editor θα εµφανίζεται χωρίς µορφοποίηση. Επιλέξτε έναν από τους δύο τρόπους στα προγράµµατα σας. - Συνολικά καλή δουλειά. - Σωστή διόρθωση errors και warnings από τον compiler. - Δεν βρήκατε και τα 2 λογικά λάθη που είχε ο κώδικας. Το πρόγραµµα σας δεν εκτυπώνει σωστά αποτελέσµατα. Προσέχουµε να στέλνουµε την άσκηση σωστά από την πρώτη φορά. - Διαδικαστικά: ΟΚ : Σχόλια: ΟΚ - Ονόµατα Μεταβλητών: ΟΚ - Στοίχιση και γενικότερη µορφή κώδικα: ΟΚ - scanf format: ΟΚ - printf format: Το output δεν έχει στοιχηθεί όπως θα θέλαµε, ξαναδείτε την εκφώνηση. Λάθος στο format specifier, θα έπρεπε να είναι 10.3lf όπου 10 είναι οι χαρακτήρες µαζί µε την υποδιαστολή : - λάθη που εξακολουθεί να βγάζει ο compiler: ΚΑΝΕΝΑ - warnings που συνδέονται µε format specifiers της scanf: ΚΑΝΕΝΑ - επιστρέφει floating point αντί για ακέραιο: Προσέξτε ότι η main πρέπει να επιστρέφει ακέραιο. - Διαδικαστικά: OK : Σχόλια: ΟΚ - Ονόµατα Μεταβλητών: ΟΚ - Στοίχιση και γενικότερη µορφή κώδικα: ΟΚ - scanf format: ΟΚ - printf format: Το output δεν έχει στοιχηθεί όπως θα θέλαµε, ξαναδείτε την εκφώνηση. : - λάθη που εξακολουθεί να βγάζει ο compiler: KANENA - warnings που συνδέονται µε format specifiers της scanf: KANENA - επιστρέφει floating point αντί για ακέραιο: ΔΙΟΡΘΩΘΗΚΕ

10 Καλή προσπάθεια µε τα σχόλια προγράµµατος. Το σχόλιο για τη σταθερά είναι περιττό γιατί δε λέει κάτι παραπάνω από αυτό που βλέπουµε στην εντολή. - Σωστοί τύποι και καλά ονόµατα µεταβλητών. Σωστά δηλώσατε το PI ως const και το γράψατε µε κεφαλαία. Μπράβο. Χωρίσατε ωραία µε κενές γραµµές τις διάφορες ενότητες του προγράµµατος. - Σωστή η printf, µπράβο. - Πολύ καλή δουλειά, µπράβο Έχετε καλό σχολιασµό, αλλά να µην χρησιµοποιείτε ελληνικούς χαρακτήρες στα σχόλια και το γενικό σχόλιο της περιγραφής του προγράµµατος µαζί µε τους συντάκτες καλό είναι να µπαίνει πάνω πάνω στο πρόγραµµα, πάνω και από τα #include. - Σωστά ονόµατα και τύποι µεταβλητών. Πολύ σωστά δηλώσατε το π σαν constant. Είναι σύµβαση να δηλώνουµε τις σταθερές µε κεφαλαία γράµµατα ("PI" αντί για "pi"). -Σωστά χρησιµοποιείς το %10.3lf για την printf. -Διορθώσατε όλα τα λάθη και τα 2 λογικά λάθη που υπήρχαν. -Σωστό format του errors.txt αλλά έπρεπε να βάλετε και µια πρόταση µε επεξήγηση για τα λάθη τα οποία διορθώσατε. - Καλή προσπάθεια µε τα σχόλια προγράµµατος. - Καλά ονόµατα µεταβλητών. - Θα µπορούσατε το p να το δηλώσετε ως const στην αρχή του προγράµµατος γιατί είναι σταθερά. Κατά σύµβαση, τα ονόµατα των σταθερών γράφονται µε κεφαλαία (δηλαδή PI) για να ξεχωρίζουν. -Δεν εκτυπώνετε τα αποτελέσµατα µε τη µορφή που σας ζητήθηκαν. Διαβάστε πως αλλάζετε γραµµή και πως δουλεύει το \t -Στην printf ο κωδικός έπρεπε να είναι %10.3lf (6 ακέραια + 1 υποδιαστολή + 3 δεκαδικά). ΓΕΝΙΚΑ: προσέξτε την έξοδο των προγραµµάτων σας. - Σωστές οι διορθώσεις σας στα συντακτικά και λογικά λάθη. - Προσέχετε κατά τη συµπίεση και την αποστολή έτσι ώστε να στέλνετε τα σωστά αρχεία FAIL - Παρότι δηλώσατε µε σχόλια τι σηµαίνουν τα ονόµατα µεταβλητών "r/p" δεν είναι περιγραφικά. Μία καλύτερη επιλογή ονοµάτων είναι "radius/perimeter". - Ο τύπος double για µεταβλητές κινητής υποδιαστολής έχει µεγαλύτερη ακρίβεια από τον τύπο float οπότε είναι καλό να χρησιµοποιείτε αυτόν. Στη συγκεκριµένη άσκηση δεν έχει ιδιαίτερη διαφορά. - Σωστή δήλωση του pi ως σταθερά. Συνήθως τις σταθερές τις ορίζουµε µε κεφαλαία γράµµατα για να τις ξεχωρίζουµε µέσα στο πρόγραµµα από τις µεταβλητές. - Το µήνυµα εισαγωγής θα έπρεπε να είναι "Enter radius: " χωρίς χαρακτήρα αλλαγής γραµµής στο τέλος. - Επίσης στο µήνυµα εξόδου δεν είναι ευθυγραµµισµένα τα colons ':' (αυτό γίνεται χρησιµοποιώντας τον χαρακτήρα tab '\t') και λείπει ο χαρακτήρας αλλαγής γραµµής '\n' στο τέλος. - Το πρόγραµµα παράγει ένα warning στην τελευταία printf. Για την εκτύπωση του µέσου όρου δεν περνάµε σαν παράµετρο την διεύθυνση της µεταβλητής (&mean) αλλά την τιµή της (mean). - Σωστές οι διορθώσεις σας στα υπόλοιπα συντακτικά λάθη. - Σωστό θέµα , attachment και tgz αρχείο - Δεν κάνει compile! - Έλλειψη δηλώσεων µεταβλητών - Λάθος σύνταξη της scanf - Δεν εστάλη Το πρόγραµµα δεν έχει ουσιαστικά σχόλια. Διαβάστε το φυλλάδιο για σχολιασµό στο και την επόµενη φορά γράψτε περιγραφικά σχόλια. Τα ονόµατα των µεταβλητών γράφονται µε µικρά (perimeter και όχι Perimeter) Χωρίσατε ωραία µε κενές γραµµές τις διάφορες ενότητες του προγράµµατος. - Σωστή η printf. Έλειπαν µόνο 3 κενά ανάµεσα στο Radius και το : 2000 ΓΕΝΙΚΑ: Μέτρια δουλειά. Στο µέλλον προσπαθήστε να είστε καλύτερα προετοιµασµένοι για να προλαβαίνετε και τις δύο ασκήσεις. - Καλή προσπάθεια µε τα σχόλια αλλά χρειαζόταν να έχετε και κάποια γενικά σχόλια για το τι κάνει το πρόγραµµα (γραµµένα στην αρχή του αρχείου). Το σχόλιο για τις δηλώσεις µεταβλητών είναι περιττό εφόσον είναι προφανές ότι σε εκείνο το σηµείο γίνονται δηλώσεις. - Σωστοί τύποι αλλά µέτρια ονόµατα µεταβλητών. Αποφεύγετε κοµµένα ονόµατα, καθώς και ονόµατα του ενός γράµµατος. Για παράδειγµα, χρησιµοποιήστε radius αντί για rad και PI αντί για p. Σωστά δηλώσατε το p ως const. Κατά σύµβαση, τα ονόµατα των σταθερών γράφονται µε κεφαλαία (δηλαδή PI) για να ξεχωρίζουν. Σωστά έχετε βάλει κενές γραµµές για να διαχωρίζουν οι επιµέρους ενότητες του προγράµµατος. Χρησιµοποιήστε παρόµοια τακτική για τις εκφράσεις. Για παράδειγµα, γράψτε perimeter = 2 * p * rad; και όχι perimeter= 2* p* rad ; - Στην printf ο κωδικός έπρεπε να είναι %10.3lf (6 ακέραια + 1 υποδιαστολή + 3 δεκαδικά). Επίσης, έχετε κάποια επιπλέον κενά. Το ζητούµενο ήταν Radius :\t%9.3lf\n Δεν παραδόθηκε. Στο µέλλον θέλουµε να δουλεύετε και τη δεύτερη άσκηση. - Σωστή χρήση του προσδιοριστή const για την σταθερά pi. Καλό θα ήταν επιλέξετε κεφαλαία γράµµατα για το όνοµα της σταθεράς. - Σωστή εκτύπωση στην printf. - Προσοχή στην µορφοποίηση του κώδικα σας. Αν και η τελική µορφή είναι σωστή, αλλού χρησιµοποιείται tab και αλλού spaces. Αν ανοίξει κάποιος τον κώδικα µε διαφορετικό editor θα εµφανίζεται χωρίς µορφοποίηση. Επιλέξτε έναν από τους δύο τρόπους στα προγράµµατα σας. - Συνολικά καλή δουλειά. - Σωστή διόρθωση errors και warnings από τον compiler. - Δεν βρήκατε και τα 2 λογικά λάθη που είχε ο κώδικας. Το πρόγραµµα σας δεν εκτυπώνει σωστά αποτελέσµατα.