Klassen Deklaration und Initialisierung

[Joob]

ich habe verschiedene Klassen für den Datenzugriff und würde gerne über eine Systemvariable entscheiden welche Klasse ich deklariere und iniatlisiere.
Das habe ich über eine if Schleife versucht, geht aber nicht das Java davon ausgeht das etwas in der if Schleife nicht unbedingt ausgeführt wird.

Wie kann ich das steuern :

if (System.getProperty("DAT").equalsIgnoreCase("LOC")) {
                    DATA_LOC data = new DATA_LOC();
                } else if (System.getProperty("DAT").equalsIgnoreCase("HOS")) {
                    DATA_PHP data = new DATA_PHP();
                } else {
                    DATA_LOC data = new DATA_LOC();
                    JOptionPane.showMessageDialog(null,"DataAccess unbekannt lokale wird verwendet! [Datalock]","Datenzugriff !", JOptionPane.ERROR_MESSAGE);
                }

 

[mrBrown]

if Schleife

http://if-schleife.de

Das habe ich über eine if Schleife versucht, geht aber nicht das Java davon ausgeht das etwas in der if Schleife nicht unbedingt ausgeführt wird.

Und was geht daran nicht?
der gezeigte Code ist richtig so...

 

[truesoul]

Und es macht eher Sinn die if Abfrage so zu schreiben:

if("Test".equalsIgnoreCase(.....)){}

So tritt dann auch kein NullPointerException auf. Kommt denn eine Fehlermeldung?

 

[mrBrown]

Und es macht eher Sinn die if Abfrage so zu schreiben:

Unsinn, das verschleiert nur Fehler und verschlechtert die Lesbarkeit.

 

[truesoul]

Unsinn, das verschleiert nur Fehler und verschlechtert die Lesbarkeit.

Geschmacksache. Verschleiert den Fehler lass ich noch so gelten aber verschlechtert die Lesbarkeit?

Das ist also schlechter zu lesen als:

if(variable!= null && variable. equalsIgnoreCase("test") ) {}

Aber es ist total legitim meine Schreibweise anzuwende. Und wenn, dann schreibt man den Wert in eine Variable. Sprichwort: Debugging

Es geht aber auch mit Objects.equals. Nicht ohne Grund

 

[mrBrown]

Das ist also schlechter zu lesen als:

Was hört sich besser an: "ich habe ein Auto und mein Auto ist Rot" oder "Rot ist mein Auto" bzw "ich habe kein Auto" oder "nicht Rot ist mein Auto"?

Aber es ist total legitim meine Schreibweise anzuwende

Aber eben auch total legitim verpönt 😉

 

[truesoul]

Aber eben auch total legitim verpönt 😉

Sorry. Muss jeder deine Sichtweise für richtig ansehen?
Ich würde sagen nein.

Grüße

 

[mrBrown]

Sorry. Muss jeder deine Sichtweise für richtig ansehen?
Ich würde sagen nein.

Das als nur meine Sichtweise zu sehen ist aber etwas vermessen, das ist eine ziemlich verbreitete Sichtweise 😉

Es enttäuscht mich, dass du nicht mal meine Frage beantwortest :'(

 

[Harry Kane]

@truesoul und @mrBrown : Könntet ihr diese Grundsatzdiskussion bitte woanders führen?
@Joob Dein wirkliches Problem ist nicht klar erkennbar. Ich vermute mal, dass du mit dem Objekt, was in einer der if-Blöcke initialisiert wird, etwas machen möchtest. Dazu musst du das Objekt VOR den if-Abfragen deklarieren und nach den Abfragen testen, ob es in einer der if-Blöcke initialisiert wurde, also != null ist.
Haben DATA_LOC und DATA_PHP eine gemeinsame Oberklasse?

 

[truesoul]

Ignoriert das

 

[mrBrown]

Genauso wie meine Sichtweise. Und die als Unsinn abzutun zeigt mir schon das alles was deiner Sichtweise nicht entspricht Unsinn ist. Von daher ist das Thema für mich gegessen.

Na wenn nicht eine der beiden Varianten Unsinn ist, muss doch deine Aussage, dass die eine sinnvoller ist, Unsinn sein?
Meinem einem Argument hast du ja auch zugestimmt und das andere ignoriert, dann kann ja nicht allzu viel dafür sprechen? 😉

 

[JStein52]

Aber eben auch total legitim verpönt

Wieso soll das verpönt sein ? Ist gängige Praxis ...

 

[mrBrown]

Wieso soll das verpönt sein ? Ist gängige Praxis ...

Das warum steht doch oben...und eben deshalb ist es bei vielen verpönt

 

[JStein52]

Das warum steht doch oben

Ist aber Unsinn. Es kommt halt immer auf die Situation an. Manchmal will man halt wissen ob ein String ungleich einem anderen ist und null ist evtl. ein zulässiger unterschiedlicher Wert. Dann würge ich das nicht über eine NPE sondern drehe die Abfrage genau so um wie von @truesoul angegeben.

 

[mrBrown]

Ist aber Unsinn. Es kommt halt immer auf die Situation an. Manchmal will man halt wissen ob ein String ungleich einem anderen ist und null ist evtl. ein zulässiger unterschiedlicher Wert. Dann würge ich das nicht über eine NPE sondern drehe die Abfrage genau so um wie von @truesoul angegeben.

Oder man schreibt den Fall einfach explizit hin 😉

 

[JStein52]

Oder man schreibt den Fall einfach explizit hin

Auch das wenn es dir so lieber ist. Mich stört nur, dies als verpönt abzukanzeln. Es ist halt einfach eine Möglichkeit einen solchen Fall zu behandeln.