Sun Java Compiler Bug ?

[Zifnab]

Code: Select all

class serialtest {
	public static void main(String args[]) {	
		if (true) 	
			String str;
	}
}

Έφτιαξα αυτό που φαίνεται σωστό αλλά μου βγάζει:

Code: Select all

C:\OPA\2nd semister\Papaioannou\bug report\serialtest.java:4: not a statement
			String str;
                        ^
C:\OPA\2nd semister\Papaioannou\bug report\serialtest.java:4: ';' expected
			String str;
                               ^
2 errors

Process completed.

Αν όμως κάνω compile το παρακάτω όλα ΟΚ !

Code: Select all

class serialtest {
	public static void main(String args[]) {	
		if (true) 	{
			String str;
                }
	}
}

To πρόβλημα εντοπίζεται εδώ:
if (true) {
String str;
}Αν αφαιρέσω τα άγκυστρα πάπαλα ο compiler


Πρόκειται για bug?

[Einherjar]

Δε καταλαβαίνω το πρόβλημά σου, τι αντικαθιστάς δηλαδή με τι και παίζει.
Θα σου πρότεινα πάνως να χρησιμοποιείς τον τελεστή instanceof σε καταστάσεις όπως αυτή

Code: Select all

(obj.getClass() == String.class) 

[Zifnab]

Το obj instanceof classname, θα μου επιστρέψει true ακόμα και αν το obj δεν είναι τύπου classname αλλά κάποιου απογόνου της classname. Ενώ το getClass() μου επιστρέφει ένα αντικείμενο τύπου Class που είναι πραγματικού τύπου. Δεν είναι εκεί το πρόβλημα...

Έκανα edit το παραπάνω post για να φανεί καλύτερα η διαφορά

[Babedacus]

Αυτό μου το είχε κάνει κι εμένα σε κάποια εργασία,μόνο που ήταν 2 for και μια if χωρίς καθόλου άγκιστρα.Όταν έβαλα τα άγκιστρα δούλεψε.Μήπως ο JCreator επιβάλλει τα code conventions της Sun; Θα συμφωνήσω πως δεν έχει να κάνει με την getClass(),πάντως.

[Zifnab]

Babedacus++

Περιόρισα και άλλο το πιθανό bug. Ισχύει για οποιοδήποτε είδος αντικειμένου
και για τα primitive types int,short,...,float,double, κτλ
Δείτε πάνω πάνω τον κώδικα....

[Einherjar]

Λογικά, μετά από ένα if περιμένει ένα statement. Αυτό που κάνεις εσύ χωρίς τις αγγύλες είναι declaration ενώ με τις αγγύλες γίνεται statement.
Δεν είμαι απόλυτα σίγουρος για την ορθότητα του πράγματος, αλλά μου φαίνεται λογικό.

[Zifnab]

Thanks Einherjar!
Αν ισχύει αυτό τότε οι καθηγητές πρέπει να αλλάξουν τον τρόπο που διδάσκουν την if, δηλαδή τον γνωστό: αν έχουμε μία εντολή παραλείπονται οι αγκύλες ενώ αν έχουμε πολλές βάζουμε αγκύλες...

[tsilochr]

χα, αν παίξεις με τις λέξεις τότε μια χαρά τα λένε οι καθηγητές διότι εσύ δεν βάζεις εντολή αλλά κάνεις δήλωση και αρχικοποιήση μεταβλητής ταυτόχρονα

[Zifnab]

εσύ δεν βάζεις εντολή αλλά κάνεις δήλωση και αρχικοποιήση μεταβλητής ταυτόχρονα

Μόνο δήλωση έκανα...

Code: Select all

class serialtest {
   public static void main(String args[]) {   
      if (true)    
         String str;
   }
}

Α και παρεπιπτόντως το String str; το ονομάζουν declaration statement μέσα στα βιβλία...

[adam98]

Στην minijava αυτό είναι λάθος πάντως για πολλούς λόγους.Αν έχεις πάρει το μάθημα και έχεις κάνει το σχετικό project θα καταλάβεις γιατί λέω αυτό το κακόγουστο αστείο!

[PaP]

Thanks Einherjar!
Αν ισχύει αυτό τότε οι καθηγητές πρέπει να αλλάξουν τον τρόπο που διδάσκουν την if, δηλαδή τον γνωστό: αν έχουμε μία εντολή παραλείπονται οι αγκύλες ενώ αν έχουμε πολλές βάζουμε αγκύλες...

Ασε τι λένε οι καθηγητές... καλό είναι να βάζουμε πάντα αγκύλες για code readability

[HdkiLLeR]

εσύ δεν βάζεις εντολή αλλά κάνεις δήλωση και αρχικοποιήση μεταβλητής ταυτόχρονα

Μόνο δήλωση έκανα...

Code: Select all

class serialtest {
   public static void main(String args[]) {   
      if (true)    
         String str;
   }
}

Α και παρεπιπτόντως το String str; το ονομάζουν declaration statement μέσα στα βιβλία...

Δεν κάνεις κάτι λάθος στην λογική σου. Το String str; είναι declaration statement και πολύ σωστά έχεις ένα μόνο statement και μπορείς να βγάλεις τις αγγύλες. Επειδή αυτό είναι λάθος όμως λογικά εάν συμβεί, δηλαδή σε ένα block statement να έχεις μόνο ένα declaration, γιατί δηλώνεις κάτι που δεν θα χρησιμοποιηθεί ποτέ (θα έχει scope μόνο μέσα στο if το οποίο δεν έχει body και συνεπώς είναι μια δήλωση άκυρη) το συντακτικό της Java δεν σου επιτρέπει δηλώσεις ανούσιες με αυτόν τον τρόπο.