Code Verständnisproblem

[Silvercreast]

Das ist eigentlich eine Antwort und eine Frage in einem das Achso ja war darauf bezogen das ich es verstanden habe und das andere die neue frage meine frage ist

[cpp]	    // Ist was markiert?
        POSITION pos = m_ListDataFound.GetFirstSelectedItemPosition();
        if (pos == NULL)
        { // Nichts gewählt, Fehlermeldung oder nix machen
			MessageBox("Es wurde kein Eintrag zum Löschen ausgewählt","Fehler",MB_OK);
        }
        else
        {
            while (pos)
            {
                // Index des Items holen und gleich weitersetzen
                int nItem = m_ListDataFound.GetNextSelectedItem(pos);
                // nItem ist der Index des Eintrags
[b]                //m_ListDataFound.DeletItem; würde gehen

                // nItem ???? Wie kann ich das Item löschen ???
				[/b]
            }
        }[/cpp]

[estartu]

//m_ListDataFound.DeletItem; würde gehen

Zeig mal wie.

[Silvercreast]

Ich mein jetzt rein von der Syntax. Also das ich bei der list DeleteItem auswählen kann

[estartu]

Und? Was musst du der Funktion so als Parameter geben?

[Silvercreast]

Was ich der Funktion als Parameter übergeben muss?? Ähm nichts das setht doch alles in meiner CtrlList drin

[estartu]

Und das kompiliert?
Hast du den Befehl denn schon mal in der MSDN nachgeschlagen?

Weil, wenn ich mal

m_ListDataFound.DeletItem();

so angucke, würde ich mich fragen: Welches Item denn nun?

[Silvercreast]

Jetzt funktioniert das FAST : m_ListDataFound.DeleteItem(nItem); Nur er speichert das nicht ab er löscht es aber wenn ich das Programm nochmals aufrufe dann ist der gelöschte Eintrag wieder da.

[Silvercreast]

Muss ich das irgendwie nochmal abspecihern ??

[estartu]

Du musst es noch aus der Datenbank löschen. (Du hast doch eine?)

Um dir zu zeigen, wie das geht, musst du mit aber zeigen, wo du die Liste füllst.

[Silvercreast]

Ja ich habe eine kleine Access Datenbank . Ich speicher hier:

void MusicManagement ::StoreDB(MusicPiece NewSong)
{
	CDatabase music_data;
	CString SqlString;
	CString strErgebnis;

		// Open the database
		music_data.Open( "MUSIK" );

		music_data.BeginTrans();
		SqlString.Format( "('%s','%s','%s','%s','%s')", NewSong.Shelf.c_str(),NewSong.Interpreter.c_str(),NewSong.Title.c_str(),NewSong.Rubric.c_str(),NewSong.CD.c_str());
		strErgebnis = "Insert into [tbl_Datenbank](Regal,Interpret,Titel,Rubrik,CD) values "; //conection with the databse
		strErgebnis += (LPCTSTR)SqlString;
		strErgebnis.Replace("''","NULL");
		music_data.ExecuteSQL(strErgebnis); // send to database
		music_data.CommitTrans(); // stor data

		music_data.Close();

}

Den Code müsstest du eigentlich noch kennen

[estartu]

Ja, da war was...

Aber ich wollte doch gar nicht den Code für das Speichern. Ich möchte den, wo du das Listcontrol füllst. Wo liest du aus der Datenbank?

Weitere Frage: Hat deine Tabelle keinen Primärschlüssel? Woran identifizierst du denn die Datensätze eindeutig? Denn, du musst ja rausfinden, welche genau du löschen willst.

[Silvercreast]

Achso Sorry Hab das falsch verstanden.

Hier lese ich aus der Datenbank:

void MusicManagement::ReadDB()
{

	CDatabase Musik_Daten;
	CString DB_File = "MUSIK";
	CString sDsn;
	CString sDriver = "MICROSOFT ACCESS DRIVER (*.mdb)";
	CString SqlString;

	// The ODBC Connection String

	sDsn.Format("ODBC;DRIVER={%s};DSN='MUSIK';DBQ=%s",sDriver,DB_File);

	TRY
	{
		// Open the database
		Musik_Daten.Open( "MUSIK" );

		// Allocate the recordset
		CRecordset recset( &Musik_Daten );

		// Build the SQL statement
		SqlString =  "SELECT Regal, Interpret, Titel, Rubrik, CD "
				"FROM tbl_Datenbank";

		// Execute the query
		recset.Open(CRecordset::forwardOnly,SqlString,CRecordset::readOnly);

		// Loop through each record
		CString Shelf;
		CString Interpreter;
		CString Title;
		CString Rubric;
		CString CD;

		while( !recset.IsEOF() )	
		{
			// Copy each column into a variable
			recset.GetFieldValue("Regal",Shelf);
			recset.GetFieldValue("Interpret",Interpreter);
			recset.GetFieldValue("Titel",Title);
			recset.GetFieldValue("Rubrik",Rubric);
			recset.GetFieldValue("CD",CD);

			MusicPiece music;

			music.Shelf		  = Shelf;
			music.Interpreter = Interpreter;
			music.Title		  = Title;
			music.Rubric	  = Rubric;
			music.CD		  = CD;

			m_MusicCollection.MusicData.push_back( music );

			// go to next record
			recset.MoveNext();
		}

		// Close the database
		Musik_Daten.Close();

	}
	CATCH(CDBException, e)
	{
			// If a database exception occured, show error msg
			AfxMessageBox("Database error: "+e->m_strError);
	}

	END_CATCH;

	}

Und natürlich habe ich einen Primärschlüssel , der wird doch automatisch gesetzt. Momentan identifiziere ich die nirgendswo

[estartu]

Dann lade den mal mit, den brauchst du für das Löschen.

Und dann zeig bitte noch die Stelle, wo du die geladenen Daten für die Anzeige ins Listcontrol packst.
Denn da musst du mit SetItemData den Autowert an jede Zeile anhängen, damit du ihn beim Löschen abfragen kannst.

[Silvercreast]

Hier ist der Code:

for ( ; it != itend; ++it )
		{
			MusicPiece& music = *it; 

			TRACE( "%s\r\n", music.Interpreter.c_str() );

			int item = m_ListDataFound.InsertItem(0,0,0);
			m_ListDataFound.SetItem( item , 0 , LVIF_TEXT, music.Shelf.c_str(), 100 , 0 , 0 , 0 );
			m_ListDataFound.SetItem( item , 1, LVIF_TEXT , music.Interpreter.c_str(), 100 , 0 , 0 , 0 );
			m_ListDataFound.SetItem( item , 2 , LVIF_TEXT , music.Title.c_str(), 100 , 0 , 0 , 0  );
			m_ListDataFound.SetItem( item , 3 , LVIF_TEXT , music.Rubric.c_str(),100 , 0 , 0 , 0 );
			m_ListDataFound.SetItem( item , 4 , LVIF_TEXT , music.CD.c_str(),100 , 0 , 0 , 0 );

		}

Dann soll ich also auch die ID mit anzeigen lassen also überall ID hinzufügen??

[estartu]

Dann soll ich also auch die ID mit anzeigen lassen also überall ID hinzufügen??

Wie genau liest du eigentlich, was ich so schreibe?

Denn da musst du mit SetItemData den Autowert an jede Zeile anhängen, damit du ihn beim Löschen abfragen kannst.

Du musst die Zahl nicht anzeigen, das sähe ja doof aus.

Das hier meinte ich:

m_ListDataFound.SetItemData( item , music.ID );

[Silvercreast]

Wie genau liest du eigentlich, was ich so schreibe?

Genauso wie jeder andere auch

Du musst die Zahl nicht anzeigen, das sähe ja doof aus.

OK hast recht

Das hier meinte ich:

estartu_de schrieb:

p]m_ListDataFound.SetItemData( item , music.ID );

das meinte ich dann muss ich jetzt überall erstmal nen ID hinzufügen.

[Silvercreast]

OK erledigt ID ist hinzugefügt

[Silvercreast]

Was ich jetzt noch nicht so ganz verstehe was habe ich davon wenn jeder Datensatz weiss was für ne ID er hat, aber ich später bei m_ListDataFound.DeleteItem(nItem) doch diese ID gar nicht nutzen kann??

[estartu]

Doch, kannst du.
Du musst ja jede Zeile zweimal löschen.

Einmal im Control - das tust du schon.
Und dann noch in der Datenbank, das musst du noch einbauen.

Dazu brauchst du dann das ItemData von dem markierten Eintrag, damit du das where basteln kannst.

[estartu]

Ich bastel mal zusammen, wie ich das meine:

SqlString.Format( "('%s','%s','%s','%s','%s')", NewSong.Shelf.c_str(),NewSong.Interpreter.c_str(),NewSong.Title.c_str(),NewSong.Rubric.c_str(),NewSong.CD.c_str());
        strErgebnis.Format("Delete from [tbl_Datenbank] where [ID] = %d", m_ListDataFound.GetItemData( item )); //conection with the databse
        music_data.ExecuteSQL(strErgebnis); // send to database

Das packst du VOR das Löschen aus dem Listcontrol, aber in die Schleife.
Variablendeklaration musst du noch machen, aber die solltest du ja kennen.