Boldemort
Σεβαστός
- Εγγρ.
- 4 Σεπ 2009
- Μηνύματα
- 47.529
- Κριτικές
- 12
- Like
- 42
- Πόντοι
- 1.220
Στο server.c χρησιμοποιείς global μεταβλητές για παράδειγμα ο δείκτης ip οπως και τους αλλους αυτο λογικα το έκανες για να μην δηλώσεις πολλα πραγματα στις παραμετρους της sender. Δεν ειναι καλη πρακτικη η χρηση global στα προγραμματα ομως.
Το 32 που περνάς σαν μέγεθος στην bzero αλλα και στον πίνακα που θα παίρνει το μήνυμα που θα στελνεις στο δικτυο και θα το επεξεργαζεται σαν συμβολοσειρα θα μπορούσε εξαρχης να δηλωθει με μια σταθερά #define ωστε αν χρειαστει να αλλαξει κάποια στιγμη να το αλλαξεις μια φορα και οχι σε πολλα σημεια του προγραμματος.
Το while(5) γιατι 5 και οχι 1? Φαινεται λιγο μυστήριο και τα 2 για ατερμονα βροχο το πάνε αλλα το 2ο το χρησιμοποιούν ολοι ενω το πρωτο ? Επισης ελπιζω να έχεις παρει τα μετρα σου με την strcat γιατι εσυ ο ιδιος θα πρεπει να χεις δηλωσει αρκετο χωρο για την συνένωση αλλιως θα έχεις ζήτημα.
Αυτα ειδα ... δεν έκατσα να κανω trace τον κώδικα να δω πως δουλευει.
Το 32 που περνάς σαν μέγεθος στην bzero αλλα και στον πίνακα που θα παίρνει το μήνυμα που θα στελνεις στο δικτυο και θα το επεξεργαζεται σαν συμβολοσειρα θα μπορούσε εξαρχης να δηλωθει με μια σταθερά #define ωστε αν χρειαστει να αλλαξει κάποια στιγμη να το αλλαξεις μια φορα και οχι σε πολλα σημεια του προγραμματος.
Το while(5) γιατι 5 και οχι 1? Φαινεται λιγο μυστήριο και τα 2 για ατερμονα βροχο το πάνε αλλα το 2ο το χρησιμοποιούν ολοι ενω το πρωτο ? Επισης ελπιζω να έχεις παρει τα μετρα σου με την strcat γιατι εσυ ο ιδιος θα πρεπει να χεις δηλωσει αρκετο χωρο για την συνένωση αλλιως θα έχεις ζήτημα.
Αυτα ειδα ... δεν έκατσα να κανω trace τον κώδικα να δω πως δουλευει.