Frage zu Typedef

Bashar

Das hier? https://www.slideshare.net/princebhau/programming-c-and-c-in-five-days

Ich verstehe den Unterschied zwischen C und C++ noch nicht ganz.

Musst du doch auch nicht. Willst du C oder C++ lernen?

Wie kann man die swap-Funktion:

void swap(char* string1, char* string2, int str_size)
{
	char* temp = (char*) malloc(str_size);
	*temp      = *string2;
	*string2   = *string1;
	*string1   = *temp;
}

denn besser machen?

Das Interface lässt keine richtige Lösung zu, da für die beiden Strings nur eine gemeinsame Länge angegeben werden kann. Es wäre ohnehin besser, std::string statt char-Arrays zu verwenden, dann löst sich das Problem auch ganz von selbst.

H.P.Wurst

@H-P-Wurst Habs jetzt so:

void swap(char* string1, char* string2, int str_size)
{
	char* temp = (char*) malloc(str_size);
	temp       = string2;
	strcpy_s(string2, str_size, string1);
	strcpy_s(string1, str_size, temp);
}

DirkB

@H-P-Wurst sagte in Frage zu Typedef:

char* temp = (char*) malloc(str_size);
temp = string2;

wozu das malloc, wenn du das Ergebnis gleich wieder mit string2 überschreibst?
Da wird kein Inhalt kopiert.
zudem gehört zu jedem malloc auch ein free!

Wutz

@Bashar sagte in Frage zu Typedef:

ich lerne gerade C++ neu und bin gerade bei typedef und structen angekommen.

Du lernst C.
Du lernst kein C++ (zumindest deinen Codebeispielen zufolge).

@H-P-Wurst sagte in Frage zu Typedef:

#include <iostream>

Das lässt vermuten, dass du einen C++ Compiler zum Lernen von C benutzt; das ist komplett sinnfrei und gerade für einen Lernenden schlecht.

H.P.Wurst

@Wutz Also ich benutze Visual Studio 2020. Wie kann ich denn bei VS einen C-Compiler benutzen?

Swordfish

@Bashar sagte in Frage zu Typedef:

Willst du C oder C++ lernen?

DirkB

@H-P-Wurst sagte in Frage zu Typedef:

@Wutz Also ich benutze Visual Studio 2020. Wie kann ich denn bei VS einen C-Compiler benutzen?

Indem du die Source-Datei mit der Endung .c abspeicherst oder in den Compiler Optionen das entsprechend einstellst.

H.P.Wurst

@Swordfish Das Skript mit dem ich arbeite ist aufgeteilt in einen C- und einen C++ Teil. Momentan bin ich noch im C-Teil

H.P.Wurst

Also so funktioniert es jetzt:

#include <stdio.h>

typedef struct emp_struct
{
	char name[10];
	int employee_no;
	float salary, tax_to_date;
} Employee;

typedef Employee Database[10];

void swap(Employee* emp1, Employee* emp2)
{
	Employee temp;
	temp = *emp1;
	*emp1 = *emp2;
	*emp2 = temp;
}

void sort(Database people, int n, int str_size)
{
	int i, j;

	for (i = 0; i < n - 1; i++)
		for (j = i + 1; j > 0; j--)

			if (strcmp(people[j].name, people[j - 1].name) < 0)
				swap(&people[j], &people[j - 1]);

			else if (strcmp(people[j].name, people[j - 1].name) == 0)
				if (people[j].employee_no < people[j - 1].employee_no)
					swap(&people[j], &people[j - 1]);
}

int main()
{
	int i, n, str_size;
	n = 10;
	str_size = 10;

	Database people = /*initializer: real DB would read from disk*/
	{ {"Fred", 10, 10000, 3000},
	{"Jim", 9, 12000, 3100.5},
	{"Fred", 13, 1000000, 30},
	{"Mary", 11, 170000, 40000},
	{"Judith", 45, 130000, 50000},
	{"Nigel", 10, 5000, 1200},
	{"Trevor", 10, 20000, 6000},
	{"Karen", 10, 120000, 34000},
	{"Marianne", 10, 50000, 12000},
	{"Mildred", 10, 100000, 30000}
	};

	sort(people, n, str_size);

	for (i = 0; i < n; i++)
		printf("Name%d: %s, Mitarbeiternummer: %d \n", i, people[i].name, people[i].employee_no);
}

SeppJ

@H-P-Wurst sagte in Frage zu Typedef:

@Swordfish Das Skript mit dem ich arbeite ist aufgeteilt in einen C- und einen C++ Teil. Momentan bin ich noch im C-Teil

Dein Script ist falsch. Such dir bitte was anderes. Wurde dir schon gesagt, hast du aber nicht ernst genommen. Musst du aber. Falsches Lehrmaterial ist das schlimmst, was du dir antun kannst. Und ganz wichtig: Entscheide dich für eine Sprache. Wenn ein Buch dir C und C++ beibringen möchte, dann bringt es dir keines davon bei.

*john 0

@H-P-Wurst sagte in Frage zu Typedef:

Also so funktioniert es jetzt:
void sort(Database people, int n, int str_size)

Ohje, wieder ein Skript was nicht die korrekten Typen für die Größe von Strings verwendet. int ist dafür nicht ausreichend. Früher (in der guten alten Zeit mit 16Bit Computern oder den ersten 32Bit Systemen) mag das noch funktioniert haben. Mittlerweile kann man in Intels bzw. AMD aktuelle Desktop Plattform 128GB RAM verbauen, da ist es sehr einfach mehr als 2GB für einen String zu verwenden. Der korrekte Typ ist size_t bzw. wenn man es Vorzeichen behaftet haben will ssize_t.

@john-0 sagte in Frage zu Typedef:

Ohje, wieder ein Skript was nicht die korrekten Typen für die Größe von Strings verwendet.

Nicht alles, was str im Namen hat, wird für Strings verwendet. Und hier wird die Variable gar nicht benutzt.

*john 0

@manni66 sagte in Frage zu Typedef:

Nicht alles, was str im Namen hat, wird für Strings verwendet. Und hier wird die Variable gar nicht benutzt.

Es ändert nichts daran, dass im gesamten Code Variablen des Typs int verwendet werden, um über dynamische allozierte Felder zu iterieren. Das ist ein Programmfehler.

Wutz

@john-0 sagte in Frage zu Typedef:

Der korrekte Typ ist size_t bzw. wenn man es Vorzeichen behaftet haben will ssize_t.

Rede nur von Sachen, von denen du Ahnung hast. ssize_t ist nur POSIX und kein Bestandteil von ISO C.

*john 0

@Wutz sagte in Frage zu Typedef:

Rede nur von Sachen, von denen du Ahnung hast. ssize_t ist nur POSIX und kein Bestandteil von ISO C.

Das von jemanden, der das int Problem unkommentiert durchgehen lässt. Das ergibt UB. Ja, ssize_t ist aus POSIX, aber der Compiler würde das Problem anmerken. Deshalb hatte ich das nicht korrigiert.

Wutz

@john-0 sagte in Frage zu Typedef:

aber der Compiler würde das Problem anmerken

LOL
Leute die nach dem Motto programmieren "der Compiler wirds schon merken" - wann hast du das letzte Mal produktionsrelevanten Code geschrieben, der das mehrstufige Codereview überstanden hat?
Trolle dich.

*john 0

@Wutz sagte in Frage zu Typedef:

Leute die nach dem Motto programmieren "der Compiler wirds schon merken" - wann hast du das letzte Mal produktionsrelevanten Code geschrieben, der das mehrstufige Codereview überstanden hat?

Entweder ist ssize_t definiert, oder es ist nicht definiert. Da kann nichts passieren. Aber du reißt hier die Klappe gewaltig auf und lässt UB durchgehen! Wer ist denn hier der Totalversager was produktionsrelevanten Code anbelangt?

Wutz

@john-0
Nonsens.
Depp.
Mit deinen Ansichten wärst du bei mir im Review gleich rausgeflogen und ich hätte dich zum FirstLevel-Support versetzt, da kannst du mit deinem aufgeschnappten Halbwissen nicht soviel Unheil anrichten.
Hier ist bzgl. int nirgendwo UB.
Array-subscript zerfällt in Pointerarithmetik, wobei der eine Operand immer ein Zeiger auf einen (vollständigen) Objekttyp und der andere immer von einem beliebigen Integraltyp (oder enum) ist, nirgends wird hier im Standard size_t gefordert, das hast du irgendwo aufgeschnappt und propagierst es hier als UB. (dass hierbei höchstens noch die arithmetic promotion mal mit reinspielt, ändert nichts am Grundsatz, d.h. wenn überhaupt wird in diesem Zusammenhang (implizit) int genannt aber niemals size_t).
Du hast Pointerarithmetik nicht verstanden und propagierst dein aufgeschnapptes Halbwissen als UB - ein Begriff, den du auch irgendwo aufgeschnappt hast und in deinem beschränkten Horizont einfach nachplapperst.
Ich empfehle dir N1570 und dort die Kapitel 6.*; wenn du das gelesen, verinnerlicht und in 30 Jahren C-Programmierung nachgewiesen hast es verstanden zu haben, kannst du hier mal wieder vorstellig werden.
Du hast keine Ahnung.
Trolle dich.

wob

@Wutz sagte in Frage zu Typedef:

@john-0
Nonsens.
Depp.
Ich empfehle dir N1570 (...)
Du hast keine Ahnung.
Trolle dich.

Ach. Ad hominem. Toll.

Sagt der, der aus @john-0 s Aussage, dass int nicht ausreichend ist, wenn int 32 bit hat und man mehr als 2GB große Objekte haben möchte, irgendwas über Array-Subscript herleitet?

Ich zitiere mal aus N1570. Kapitel 7.24.1 [String function conventions] - aber eigentlich auch egal, wo:

1 The header <string.h> declares one type and several functions, and defines one
macro useful for manipulating arrays of character type and other objects treated as arrays
of character type.307) The type is size_t and the macro is NULL (both described in
7.19).

So, und deiner Meinung nach soll man da lieber int nutzen? Warum wird dann überall in den C-Funktionen wie malloc, memcpy etc. nicht der Parametertyp int gewählt, sondern size_t?

DirkB

@wob Der Programmierer darf ruhig sein Expertenwissen mit ein bringen, ob int reicht oder size_t nötig ist.