Wert aus Combo in OnSelchangeCombo1() verwenden

[estartu]

Ja, die macht das selbe, nur eben mit einer Zeichenkette und einem kleinen Nachteil:
Stell dir vor, du hast eben etwas gefunden, das ist der letzte Datensatz.
Nun suchst du das gleich nochmal.
So, wie du das machst, fängt der blind wieder vorne an und rennt bis ganz hinten.

Ich habe das erst vor ein paar Tagen eingebaut, mit der Zusatzprüfung - das spart ENORM viel Zeit.

Die Suche nach der ID brauchst du, weil das SetAbsolutePosition ja nicht stimmen muss. (s.o.)
Also statt

m_pSet->SetAbsolutePosition(nID);

besser

m_pSet->Suche(nID);

Noch ein kleiner Hinweis in Sachen Klassendesign:
Pack das suchen besser in die Recordsetklasse. Sonst musst du das, wenn du es an einer anderen Stelle (in einem anderen View) brauchst, nochmal tippen bzw. kopieren. Sowas ist unschön, wartungshemmend und sehr fehleranfällig.
(Oder hast du da Sachen drin, die unbedingt im View passieren müssen und nicht ausgelagert werden können?)

[Polofreak]

A L S O!!
Ich habe jetzt meine Suche angepasst, dass ich auch nach ner ID suchen kann. Problem war nämlich ich hab ja immer nur eine Spalte mit dem Suchwert verglichen denn ích hab mehrere Spalten, wo z.B. 2 drin vor kommt. naja egal das geht ja auch alles ABER. Ich hab mal Spasseshalber einen Datensatz raus gelöscht und ans Ende mit ner neuen ID eingefügt, und ich hab genau das von dir angesprochene Problem, und das ohne dass ich moveabsoluteposition mach, sondern suchen (ID). Woran liegt das denn schon wieder ??

[estartu]

Zeig den Quellcode bitte.

Einmal vom suchen(id) und einmal, wie du es aufrufst.

[Polofreak]

öhm ich hab mir mal angeschaut was du sehen willst, und mir ist schon klar wo der fehler sitzt. Also:

void CDokumentenVerwaltungView::suchen(CString Anfrage, CString typ)
{
	bool Suchergebnis=FALSE;
	CDBVariant varValue;

	UpdateData(TRUE);

	short nFields = m_pSet->GetODBCFieldCount();

	for( short index = 0; index < nFields; index++ )
		{
			m_pSet->MoveFirst();
			while (!m_pSet->IsEOF())
			{
				m_pSet->GetFieldValue( index, varValue );

				if (typ=="ID")
				{
					int i = atoi(Anfrage);
					if (m_pSet->m_ID == i)
					{
						Suchergebnis = TRUE;
						break;
					}
				}
				m_pSet->MoveNext();
			}
			if (m_pSet->IsEOF())
			{
				m_pSet->MoveFirst();
				break;
			}
		}

		if (!Suchergebnis)
		 MessageBox("Die angeforderte Nummer ist nich vorhanden !","ERROR",MB_ICONERROR); 

	UpdateData(FALSE);

}

und dann noch

int nID =0;
	nID = m_PCD_CMB.GetCurSel(); 
	if (nID != -1) // es muss was gewählt sein 
	{ 
	    nID = m_PCD_CMB.GetItemData(nID); 
	} 
	suchen(nID,"ID");//Die Funktion erwartet ja aber einen String, muss ich das casten?
//also wenn ich "7" schreibe macht er mir tatsächlich das Feld mit dem Index 7 auf

kann ich nur über mich selbst mit den Augen rollen, da fehlt mir immer wieder grundlegendes

[estartu]

Wieso machst du das so "komisch"?
Also, warum gibst du den Feldnamen mit - wo es doch eh keine andere Möglichkeit zu geben scheint (oder soll das noch kommen?).

Naja, ein Vorschlag meinerseits wäre: Du überlädst die Funktion.
Kurze Grundlage: Es kann pro Klasse mehrere Funktionen mit dem selben Namen geben, die Parameter müssen sich aber unterscheiden. (Der Rückgabetyp ist leider kein Unterschied.)
Du kannst also ein
suchen(int id)
und ein
suchen(CString Anfrage)
haben.

Und entweder kümmerst du dich da drinne um die Umwandlung in CString - oder du sucht da, dann kannst du dir auch das atoi sparen.

Okay?

PS:

CString strZahl;
strZahl.Format("%d");

[Polofreak]

hm mehr oder weniger Okay ich hab die ganzen anderen If abfragen raus genommen weil die ja in dem Moment nicht interessieren, ich hab sonst schon noch mehr drin. Aber das mit dem Überladen, naja dann hab ich ja aber nachher 6 suchen Funktionen. So hab ich nur eine und hau da halt den suchtyp mit rein (macht man das aus Programmesthetischen Gründen nicht?) naja ich schau mal morgen früh nochmal denn für heute mache ich feierabend, es hat ja doch kein wert mehr heute.
Vielen Dank Estartu_de

[estartu]

Dann schönen Feierabend - morgen früh ist auch der Kopf wieder frei.

Das mit dem Überladen würde das Konvertieren sparen.
Ich gucke mal, in welchem Buch ne Erklärung dazu war...

[Polofreak]

bist echt ein schatz dir auch nen schönen Feierabend und bis morgen

[estartu]

Guten Morgen!

Also, ich habe in zwei meiner Bücher was gefunden.
Die C++-Programmiersprache | ISBN: 3827320585
Kapitel 7.4 erklärt das sehr ausführlich. Das Buch gibt es hin und wieder als Special Edition und dann ist es recht billig, schau dich mal um, das ist DAS Standardwerk für C++.
OOP mit C++ | ISBN: 3898423328
Das Buch hat nur doppelseitige Kapitel, ist also als Nachschlagewerk genial. Ich fasse das Kapitel mal kurz zusammen:

Oft braucht man mehrere Funktionen, die mit verschiedenen Parametern das "Gleiche" tun, z.B. das Minimum von Zahlen zurückgeben. Da gibt es ja ganze Zahlen und Kommazahlen.
In C++ kann man (und manchmal sollte man auch) verschiedenen Funktionen den gleichen Namen geben:

int min(int x, int y);
double min(double x double y);

Die Funktionen werden anhand ihrer "Signatur" unterschieden. Das sind die Typen und die Anzahl der Parameter.
Der Rückgabetyp einer Funktion gehört nicht zur Signatur.

So, das war jetzt die ultrakurze Version. Nun musst du fragen.

[Polofreak]

Das mit dem Kopf frei hat voll und ganz hingehauen, ich hab mir das heute morgen nochmal angeschauen, deine Ratschläge beherzigt, und schon geht es sowas von einwandfrei, und ich habe bemerkt dass ich so auch die Stringsuche gar nicht brauche! Ich hatte ursprünglich anstelle der Combo ein Edit Feld, und drum musste ich auf Strings suchen, jetzt kann ich aber gleich auf die IDs suchen und wenn ich das schön durchziehe, gibt es in meiner Anwendung keinen Drund nach Strings zu suchen (naja bisher zumindest). Wie fange ich jetzt aber am besten ne Falscheingabe des Combos ein? Es kann doch jetzt jemand nen Scheiß in die Combo tippen (statt auswählen) und dann bekomm ich das ja gar nicht mit.

Kann ich das irgendwie machen wie bei der Klassenauswahl im Klassenassi, dass sobald ich ne 2 eingib er gleich die ganzen Werte wo vorn ne 1 steht überspringt und zum ersten Wert der an erster Stelle ne zwei hat springt??
Wie kann ich elegant ne Standard auswahl machen? (OnInit den Index eintragen??)

[estartu]

Polofreak schrieb:

Wie fange ich jetzt aber am besten ne Falscheingabe des Combos ein? Es kann doch jetzt jemand nen Scheiß in die Combo tippen (statt auswählen) und dann bekomm ich das ja gar nicht mit.

Kann ich das irgendwie machen wie bei der Klassenauswahl im Klassenassi, dass sobald ich ne 2 eingib er gleich die ganzen Werte wo vorn ne 1 steht überspringt und zum ersten Wert der an erster Stelle ne zwei hat springt??

Das kannst du mit ein paar einfachen Mausklicks erschlagen:
Im Resourceeditor ie Eigenschaften der Combobox aufmachen, Tab Formate anklicken und in der oberen Combobox "Dropdown-Listenfeld" auswählen.
Fertig.
Leider kann man das nicht als Standard festlegen, ich nutze NUR diesen Typ.

Polofreak schrieb:

Wie kann ich elegant ne Standard auswahl machen? (OnInit den Index eintragen??)

Fast. Mach ein SetCurSel in OnInitDialog.

[Polofreak]

Genau das wollte ich! Klasse vielen Dank Estartu_de hast mir mal wieder richtig auf die Beine geholfen, doch laufen versuch ich jetzt selbst!