C# objektorientering

Du får nog pli på det om du skapar en metod som skapar upp hela "banken" med testdata. Det är ju svårt att "använda" banken om du inte har något att leka med. Oftast så brukar det lösa sig själv eftersom man kommer på vad som saknas.

Jag har inte orkat titta på din kod men ungefär på detta sätt borde det ju fungera:

Testdata - Ett privatkonto som tillhör en kund. Kontot har ett kort kopplat.

var kund1 = new Kund();
kund1.FirstName = "Olle";
kund1.LastName = "Ekman";

var kort1 = new Kort();

var konto1 = new PrivatKonto();
konto1.Kund = kund1;

konto1.KortLista.Add(kort1);

Angående kontohistoriken så borde du verkligen inte ha en "jagged array string: index, belopp, tidpunkt. "
Du borde ha en List<Transaction> som består av alla transaktioner som har gjorts mot detta konto. En transaktion borde ju ändå innehålla det du beskriver.

Dina "deposit" och "withDraw"-metoded så måste du ju skapa transaktioner, annars är den klassen meningslös.
Ex "deposit", lägg till denna rad:
var transaction = new Transaction(this,amount);

Jag vet inte hur du lagrar alla transaktioner. Gör du det globalt i en statisk klass? Gör du det per konto? Oavsett vad så måste transaktionen läggas in i listan.

Jag vet inte om du har läst något om enums än, men på något sätt måste man ju säga vad som har hänt med kontot. I dagsläget kanske du bara tolkar en negativ summa som om det är en WITHDRAW och alla positiva summor är DEPOSIT?

//insättning
        public double Deposit(double amount)
        {
            if (amount > 0.0)
            {
                dBalance += amount;
            }
            return dBalance;
        }

        //uttag
        public double Withdraw(double amount)
        {
            if (dBalance >= amount)
            {
                dBalance -= amount;
            }
            return dBalance;
        }

Enligt mig så bör Kort-klassen vara hyfsat tom. Några egenskaper kanske men dessa egenskaper är ju meningslösa om de inte kommer till användning. Enligt ditt klassdiagram så är det ju bara två mindre metoder. Jag ser inte hur det är meningen att Kort-klassen ens kommer att användas. Då borde man ha någon BankTransactionHandler-klass som fungerar typ som en bankomat.
1. Kortet in
2. Validera pinkod
3. Kolla upp vem kortet tillhör och hämta rätt konto
4. Utföra transaktion

Med det sagt så vet jag/vi inte hur du egentligen ska använda alla klasser.

List<int> och List<string> fungerar på exakt samma sätt som List<Transaction> eller List<Kund>.
Defintionen är List<T> där T kan vara vilken klass/typ som helst. Med andra ord, du använder List<Transaction> på samma sätt som du använder List<int>. En till anledning till varför loggen bör vara av en List är för att en array har ett bestämt antal index-positioner. Men självklart kan du använda någon annan "Collection" typ ArrayList eller så. Jag föredrar List<T>.

Jag ser ju att du utför själva transaktionhändelsen men du måste fortfarande spara en logg över transaktionen (med tanke på vad du har skrivit tidigare). Av att döma från ditt klassdiagram så ser det ut som att du borde ha en lista med transaktioner på varje konto. En List<Transaction>-property som du lägger till en transaktion av det som nyss gjordes.

Du verkar inte ha gjort alla klasser än. Det blir väldigt svårt att "se" lösningen om man inte har allt material framför sig om man sitter fast.
Skapa alla egenskaper/properties och gör tomma metoder (om du inte vet hur själva logiken i dessa ska vara än) för allt som ska kunna göras. Sen när du försöker använda klasserna så borde du se vad det är som saknas och hur det ska byggas upp.