Wert einer neuen Variable wird verändert - obwohl eine Andere geändert wird

mpmichael · 16. Februar 2021, 17:41

Hallo,

ich habe ein Problem - das mich vor ein echtes Rätsel stellt. Vielleicht kann mir ja jemand helfen.

Ich habe ein Programm in dem ich immer wieder einen LDAP-Login benötige. Deshalb habe ich eine Class erstellt die unter anderem folgendes beinhaltet:

VB.NET-Quellcode

Public Class Setup
Public LDAPAccess As New DirectoryEntry With {
.username = "myname"
.password = "Mypass"
.path = "hattps:\\myldamp\usw"
}

Dann habe ich eine weitere Class in der ich eine Funktion aufrufe:

VB.NET-Quellcode

Public Shared MySetup As New Setup
...
Private Sub MachdenChange()
TestObject.WriteAttributeToLDAP(LDAPAccess, meinwert, meinvalue)
End Sub

In der Funktion selbst mache ich folgendes:

VB.NET-Quellcode

Public Function WriteAttributtoLDAP(LDAPObject As DirectoryEntry, Prop As String, NewValue As String) As LDAPFunx.MsgResults
Dim searcher As New DirectorySearcher(LDAPObject) With {
.Filter = CliFilter,
.SearchScope = SearchScope.Subtree
}
Dim resultObject As SearchResult = searcher.FindOne()
LDAPObject.Path = resultObject.Path
...
End Function

Und hier in Zeile 7 wird der Pfad geändert - und das ABER auch in der ursprünglichen Variable (also in Mysetup.LDAPAccess) - und das verstehe ich nicht.
Es wirk als wäre die Variable LDAPObject keine eigene Variable sondern das durchgereichte Objekt...
Jemand eine Idee was ich falsch mache?
Und wie kann ich es anders - bzw. besser machen?

Code-Tags korrigiert ~VaporiZed

mrMo · 16. Februar 2021, 18:06

Ja das ist so weil es die gleiche Instanz des Objekts ist. Will man das nicht, muss man eine eigene Instanz erstellen.

mpmichael · 16. Februar 2021, 19:53

Hallo,

danke für die Antwort. Ich hatte so etwas bereits vermutet.
Ich hatte bereits folgendes versucht - aber immer mit identischem Ergebnis:

VB.NET-Quellcode

Dim NeueLDAP as new DirectoryEntry
NeueLDAP = Masetup.LDAPAccess

Dur das "NEW" wäre ich der Meinung gewesen das eine neue Instanz geschaffen wird?! Aber nein
Auch

VB.NET-Quellcode

Public Function WriteAttributtoLDAP(ByVal LDAPObject As DirectoryEntry, Prop As String, NewValue As String) As LDAPFunx.MsgResults

ByVal sollte doch eine Kopie erstellen... hätte ich gedacht
Wie handhabt Ihr so etwas?

Code-Tags korrigiert ~VaporiZed

Naifu · 16. Februar 2021, 19:58

Du könntest ICloneable implementieren.

mpmichael · 16. Februar 2021, 20:06

Hallo,

das ist gut und könnte mein Problem lösen. Ist das ein Standardvorgehen oder eher eine Art Workaround?

Naifu · 16. Februar 2021, 20:09

Beschäftige dich mal mit Interfaces, da tun sich Welten auf, und deine Frage beatwortet sich von selbst.

ErfinderDesRades · 16. Februar 2021, 20:16

ähm - Interfaces sind total überbewertet.
Ich selbst hab noch nie ein Interface anlegen müssen.
In jedem Falle waren Basisklassen oder Delegaten die bessere Alternative.

wie dem auch sei - ich verstehe garnet die Fragestellung. Weil die Codeschnipsel sind sowas von aus jedem Zusammenhang gerissen...

Welchen DAtentyp hat TestObject?
In welcher Klasse/Modul liegt WriteAttributtoLDAP()?

Naifu · 16. Februar 2021, 20:22

ErfinderDesRades schrieb:

ähm - Interfaces sind total überbewertet.

Hmm, ausgerechnet von dir hätte ich das nun nicht erwartet, aber wie du meinst.

Gruß Robert

mpmichael · 16. Februar 2021, 20:28

Hallo

ich habe 3 Klassen:

Setup
Main
und LDAPFunc

Das Testobject ist ein String. Da Problem welches ich habe bezieht sich auf LDAPAccess As New DirectoryEntry und auf die Tatsache, das ich das Object LDAPAccess an die Function WriteAttributtoLDAP übergebe (die Function habe ich geschrieben). Dort wird es durch den 2ten Search geändert und diese Änderung wird quasi im "Orginalobjekt" gemacht. Ich hätte gedacht das ließe sich durch den Einsatz von ByVal verhindern - aber es ist schon so, wie MrMo schreibt - ist ist wohl immer das gleiche Objekt.

Ich verwende LDAPAccess später wieder und finde dann in LDAPAccess.Path - das Ergebnis aus dem LDAPObject.Path = resultObject.Path

ErfinderDesRades · 16. Februar 2021, 21:02

mpmichael schrieb:

Das Testobject ist ein String.

kann nicht sein:

mpmichael schrieb:

VB.NET-Quellcode

TestObject.WriteAttributeToLDAP(LDAPAccess, meinwert, meinvalue)

String hat keine Methode .WriteAttributeToLDAP()

ISliceUrPanties · 17. Februar 2021, 10:07

Hallo,

mpmichael schrieb:

ByVal sollte doch eine Kopie erstellen... hätte ich
gedacht

Klassen sind Referenztypen docs.microsoft.com/de-de/dotne…types-and-reference-types. Änderungen in einer Methode an deinem Objekt wirken sich auch auf den Aufrufer aus - trotz ByVal.

mpmichael · 17. Februar 2021, 12:49

Hallo,

ja - da habe ich jetzt wieder einige Basics gelernt, die mir so nicht ganz klar waren. -> Klassen sind Referenztypen !
Vielen Dank!

Ich mache es jetzt so, das ich in der Klasse nicht mehr das DirectoryEntry Objekt speichere und übergebe - sondern nur noch die Credentials. Dann baue ich in den Funktionen halt jedes mal ein frisches DirectoryEntry-Objekt zusammen.

Naifu · 17. Februar 2021, 16:03

Mit nem Clone kannst du dir das sparen und nur an einer Stelle Fehler machen.

mpmichael · 17. Februar 2021, 17:18

Hi,

ja - das sehe ich auch als Sinnvoll an. Vor allem der Grundsatz innerhalb eines Programmes eine "Tätigkeit" nur einmalig zu tun.
Ich habe das erstellen der neuen Instanz und die Zuweisung der Credentials in ein Sub ausgelagert und rufe das dann auf.
Das mit den Clones muss ich noch testen - wird sicherlich noch wertvoll sein!

ErfinderDesRades · 17. Februar 2021, 21:47

Jo, das Klonen ist auch bischen Wissenschaft.
Zunächstmal musst du wissen, was du klonen willst.
Dann kannste gucken, ob das vielleicht ICloneable ist - dann hat es nämlich schon eine Clone-Methode (wenn auch eine mies designete).
Wenns nicht ICloneable ist, musste selbst iwas schreiben, was einen Klon erschafft.
Wenns deine eigene Klasse ist, kannste dort ja ICloneable implementieren - dann muss deine Clone-Methode genauso mies designed sein, wie vom Interface vorgeschrieben.

Jo, bei Fragen fragen - ich nehme nicht an, dass du diese fachsimpelige Einlassung auf Anhieb vollumfänglich verstehst

Naifu · 17. Februar 2021, 21:54

@ErfinderDesRades :thumbsup:

mpmichael · 18. Februar 2021, 08:32

Vollzitat eines Vorposts durch Anredefunktion ersetzt ~VaporiZed

@ErfinderDesRades:
Danke - doch das habe ich verstanden. Es ist meine Klasse - aber die vier Codezeilen die ich mir spare ist es nicht "wert" einen Clone-Methode zu implementieren.
Ich hatte einen "strategischen Denkfehler" und der konnte gelöst werden!
Merci!

Wert einer neuen Variable wird verändert - obwohl eine Andere geändert wird

Wert einer neuen Variable wird verändert - obwohl eine Andere geändert wird

VB.NET-Quellcode

VB.NET-Quellcode

VB.NET-Quellcode

VB.NET-Quellcode

VB.NET-Quellcode

ErfinderDesRades schrieb:

mpmichael schrieb:

mpmichael schrieb:

VB.NET-Quellcode

mpmichael schrieb:

Tags

Ähnliche Themen

5 Benutzer haben hier geschrieben