Items aus einer Liste ansprechen

SaschaMois · 21. Februar 2017

Hey

Ich hab eine Liste gemacht in der beim Serverstart alle Häuser geladen werden.

C

public static List<House> Haeuser = new List<House>();

Die Klasse "House" sieht wie folgt aus:

C

public class House
    {
        public int dbid; 
        public string Name;
        public int Besitzer;
        public bool Rentable;
        public int Rent;
        public int Rent_People;
        public TextLabel label;
        public Marker marker;
        public Vector3 Position;
        public int Preis; 
    }

Alles anzeigen

Wenn ich nun das Haus update und es daher neu Laden möchte hab ich momentan folgendes gemacht um das "alte" Haus zu löschen:

C

foreach (var hous in Haeuser)
            {
                if (hous.dbid == id)
                {
                    API.shared.deleteEntity(hous.label);
                    API.shared.deleteEntity(hous.marker); 
                    Haeuser.Remove(hous); 
                }
            }

Gibt es nun irgend eine "einfachere" Möglichkeit das Item aus der Liste zu bekommen ohne das ich mit foreach alle Häuser durchlaufen muss? Das ich vielleicht speziell nach der "dbid" suchen kann oderso?

[DT]Sniper · 21. Februar 2017

Was du auf jedenfall machen kannst ist eine funktion schreiben um die id aus der dbid heraus zu erhalten. Dann geht das ganze etwas schneller und du musst iwann mal eine schleife ändern und nicht hunderte.

ChrisXD · 21. Februar 2017

Du kannst ein Dictionary verwenden, es benutzt das KeyValue Prinzip das wäre in dem Fall am einfachsten

SaschaMois · 21. Februar 2017

Also ca so:

C

Dictionary<int, House> Houses = new Dictionary<int, House>();

Wie spreche ich dann beim hinzufügen von einem neuen Haus die Objekte in der "House" Klasse an?

Aktuell funktionierts ja so:

C

House haus = new House(); 
haus.name = "blabla"; 
haus.preis = 5000; 
Haeuser.Add(haus);

// Edit: Ok habs
Danke das Dictionary ist wirklich praktischer

TutNichts · 21. Februar 2017

Zitat von ChrisXD

Du kannst ein Dictionary verwenden, es benutzt das KeyValue Prinzip das wäre in dem Fall am einfachsten

wozu?

C

// wenn du die Häuser in einer List<House> hast:
var house = list.FirstOrDefault(x => x.dbid == id);
// house ist nun entweder vom Typ "House" oder null (weil es kein Haus mit der ID gibt)
if(house == null) {
	// Haus gibt es nicht
}
list.Remove(house);


// =====


// durchaus könntest du aber auch direkt mit der DB kommunizieren (würde ich bevorzugen), sprich mit dem DbSet<House>
// Houses ist hierbei dann also "DbSet<House>", sprich vom Context
var house = Houses.FirstOrDefault(x => x.dbid == id);
// house ist nun entweder vom Typ "House" oder null (weil es kein Haus mit der ID gibt)
if(house == null) {
	// Haus gibt es nicht
}


Houses.Remove(house);
Houses.SaveChanges();

Alles anzeigen

jetzt aus'm Kopf geschrieben, hab leider keine gescheite Umgebung hier. Würde dir grundsätzlich empfehlen ein Repository in Erwägung zu ziehen.

edit:

Zitat von SaschaLois

Ich hab eine Liste gemacht in der beim Serverstart alle Häuser geladen werden.

wieso?

ChrisXD · 21. Februar 2017

Zitat von TutNichts

wozu?

C

// wenn du die Häuser in einer List<House> hast:
var house = list.FirstOrDefault(x => x.dbid == id);
// house ist nun entweder vom Typ "House" oder null (weil es kein Haus mit der ID gibt)
if(house == null) {
	// Haus gibt es nicht
}
list.Remove(house);


// =====


// durchaus könntest du aber auch direkt mit der DB kommunizieren (würde ich bevorzugen), sprich mit dem DbSet<House>
// Houses ist hierbei dann also "DbSet<House>", sprich vom Context
var house = Houses.FirstOrDefault(x => x.dbid == id);
// house ist nun entweder vom Typ "House" oder null (weil es kein Haus mit der ID gibt)
if(house == null) {
	// Haus gibt es nicht
}


Houses.Remove(house);
Houses.SaveChanges();

Alles anzeigen

jetzt aus'm Kopf geschrieben, hab leider keine gescheite Umgebung hier. Würde dir grundsätzlich empfehlen ein Repository in Erwägung zu ziehen.

edit:

wieso?

Alles anzeigen

Wie gesagt es ist am einfachsten und noch dazu die schnellste/resourcen sparende Methode

IPrototypeI · 21. Februar 2017

Naja Chris es kommt drauf an wie man das aufbaut alternativ kann man auch ein array verwenden, der zugiff auf den jeweiligen index ist schneller als ein binärbaum

SaschaMois · 21. Februar 2017

Also ich hab´s jetzt mit dem Dictionary gemacht.
Hauptsächlich aus dem Grund weil ich dann folgendes machen kann:

C

if(Houses.ContainsKey(id)) 
{
    // existiert
}

@TutNichts Wieso ich speziell ne Liste gemacht hab oder warum ich das überhaupt irgndwo speichern möchte?
Der ursprüngliche Hauptgrund für die Liste war das ich dann die TextLabels und Marker gut löschen kann wenn das Haus aktualisiert wird. Des weiteren habe ich angenommen das es eventuell etwas schneller geht als wenn der jedes mal wenn ich irgendwas mit dem Haus mache das alles von der Datenbank abfragen muss.

Items aus einer Liste ansprechen

dennismitzwein 9. November 2020

breadfish.de 12. März 2022