Tabelle mit C ausgeben

a.b_om · 11. Dezember 2019, 13:45

Hallo zusammen

Ich komme nicht weiter.
Ich möchte meine doppelt verkettete Liste wie eine Tabelle ausgeben.

Die Zeichenlaenge sind unterschiedlich und deshalb, kann ich es nicht einfach mit fixen Werte machen.

Aber wie bekomme ich das hin, dass es wirklich eine Tabelle anzeigt, mit allen Einträgen in der Liste?

Mein Code zum ausgeben:

Spoiler anzeigen

C-Quellcode

YugiohKarte YugiohKarteAusgeben(YugiohKarte ersteYugiohKarte) {
printf("\nMoechtest du alle Karten ausgeben oder nur eine bestimmte? (Ja/Nein): ");
char antwort[MAX];
fgets(antwort, MAX, stdin);
if (antwort[0] == 'J' && antwort[1] == 'a')
{
do
{
if (ersteYugiohKarte.Beschreibung[0] == 'D')
{
printf("\nEs gibt keine Yugiohkarten zum ausgeben.");
}
else
{
YugiohKarte* jetzigeYugiohKarte = &ersteYugiohKarte;
printf("| Kartenname | Kartenart | Stufe | Rang | Pendelstufe | Link | ATK | DEF | Eigenschaft | Monstertyp | Kartentyp | Beschreibung |\n");
printf("------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------\n");
do
{
printf("| ");
printf(jetzigeYugiohKarte->Name);
} while (jetzigeYugiohKarte->pNext != NULL);
}
break;
} while (true);
}
else if (antwort[0] == 'N' && antwort[1] == 'e' && antwort[2] == 'i' && antwort[3] == 'n')
{
}
return ersteYugiohKarte;
}

*Topic verschoben*

Elanda · 11. Dezember 2019, 18:02

Hallo a.b_om,

Das erste das mir auffällt ist diese mir unverständliche do while Schleife, du setzt sie auf true und brichst sie gleich wieder ab.
Kann auch sein das ich den Punkt hier übersehe da ich nur mit C++ arbeite, villeicht ist das so ja ein C Ding.

Punkt 2, die zweite Schleife, wenn ich das richtig sehe änderst du nie die abzufragende Adresse von "jetzigeYugiohKarte", du frägst einfach nur ab ob "pNext" irgendwohin zeigt, lässt aber "jetzigeYugiohKarte" unverändert.
Das wird darin resultieren, dass für eine unbestimmt lange Zeit immer und immer wieder dieselbe Karte ausgegeben wird und dein Programm, sofern keine anderen Umstände bestehen, endlos laufen wird.
Soweit ich das verstehe ist pNext die Adresse zur nächsten Karte?
Dann mach doch einfach 'while(jetzigeYugiohKarte = jetzigeYugiohKarte->pNext)', dies wird den nächsten Wert zuweisen und gleichzeitig abfragen ob der Wert immer noch valide ist!

Und drittens, ich bin wie gesagt nicht sehr versiert in Sachen C und besonders string Management bringt mich immer wieder aus dem Konzept, ich hätte da aber einen Tipp.
Definiere irgendwo die verschiedenen Zeichenlängen der Tabellenheader, der Text der in die individuellen Zellen ausgegeben werden soll wird dann auf die Länge geprüft, ist es zu groß verkürzt du den Text mit '...' am Ende, ist es kürzer kannst du die Differenz der noch zu brauchenden Zeichen für die Headerlänge errechnen und mit Leerzeichen ausfüllen!

~ Elanda

Edit: Kleiner Nachtrag, bei der Abfrage Ja/Nein wird er nur validieren wenn Ja oder Nein anfänglich groß geschrieben wird, da 'j', 'J', 'n' und 'N' andere Code Points besitzen.
Sowieso würde ich eher nur auf J, j oder N, n überprüfen also "(J)a/(N)ein", aber wie du das bevorzugst bleibt dir überlassen.

a.b_om · 12. Dezember 2019, 08:10

@Elanda

Elanda schrieb:

Das erste das mir auffällt ist diese mir unverständliche do while Schleife, du setzt sie auf true und brichst sie gleich wieder ab.
Kann auch sein das ich den Punkt hier übersehe da ich nur mit C++ arbeite, villeicht ist das so ja ein C Ding.

Nein, das ist kein C Ding. Dort kommen noch mehr Codezeilen hin. Bin aber noch nicht so weit.

Elanda schrieb:

Punkt 2, die zweite Schleife, wenn ich das richtig sehe änderst du nie die abzufragende Adresse von "jetzigeYugiohKarte", du frägst einfach nur ab ob "pNext" irgendwohin zeigt, lässt aber "jetzigeYugiohKarte" unverändert.
Das wird darin resultieren, dass für eine unbestimmt lange Zeit immer und immer wieder dieselbe Karte ausgegeben wird und dein Programm, sofern keine anderen Umstände bestehen, endlos laufen wird.
Soweit ich das verstehe ist pNext die Adresse zur nächsten Karte?
Dann mach doch einfach 'while(jetzigeYugiohKarte = jetzigeYugiohKarte->pNext)', dies wird den nächsten Wert zuweisen und gleichzeitig abfragen ob der Wert immer noch valide ist!

Die Methode ist nur zur Ausgabe da und ich werde deine Methode mal ausprobieren.

Elanda schrieb:

Und drittens, ich bin wie gesagt nicht sehr versiert in Sachen C und besonders string Management bringt mich immer wieder aus dem Konzept, ich hätte da aber einen Tipp.
Definiere irgendwo die verschiedenen Zeichenlängen der Tabellenheader, der Text der in die individuellen Zellen ausgegeben werden soll wird dann auf die Länge geprüft, ist es zu groß verkürzt du den Text mit '...' am Ende, ist es kürzer kannst du die Differenz der noch zu brauchenden Zeichen für die Headerlänge errechnen und mit Leerzeichen ausfüllen!

Wie, bin auch nicht verisiert mit C. Ist ein Steilstart mit dem Projekt. Brauche dementsprechend auch viel Hilfe.

Elanda schrieb:

Edit: Kleiner Nachtrag, bei der Abfrage Ja/Nein wird er nur validieren wenn Ja oder Nein anfänglich groß geschrieben wird, da 'j', 'J', 'n' und 'N' andere Code Points besitzen.
Sowieso würde ich eher nur auf J, j oder N, n überprüfen also "(J)a/(N)ein", aber wie du das bevorzugst bleibt dir überlassen.

Was ist, wenn der Benutzer "Jey" oder etwas anderes eingibt?

Edit:
char Arrays in C sind schon etwas kompliziertes. Hätte es Strings, würde ich sie brauchen.

Elanda · 12. Dezember 2019, 15:48

a.b_om schrieb:

Was ist, wenn der Benutzer "Jey" oder etwas anderes eingibt?

Das ist dann das Problem des Nutzers, du gibst klare Anweisungen. (Ja/Nein)
Wenn der Benutzer die nicht einhält obwohl sie klar verständlich sind, stellt sich sowieso die Frage ob der Nutzer überhaupt dazu in der Lage ist das Programm zu verwenden.
Natürlich kannst du stapelweise Synonyme und Slang-terms einbauen, allerdings verstehe ich da den Sinn nicht dahinter wenn du klar und deutlich angeben kannst was der Benutzer zu tun hat!

Edit:
Du könntest, im Falle der falschen Eingabe, eine Antwort hinterlassen mit dem Hinweis zur Neueingabe!

a.b_om · 12. Dezember 2019, 15:58

@Elanda
Ja das ist mir klar. Aber wenn er es aus Spass etwas Falsches eingibt, kann ich das nicht abfangen.

Aber wie kriege ich das mit der Tabelle hin?

Elanda · 12. Dezember 2019, 18:18

a.b_om schrieb:

Ja das ist mir klar. Aber wenn er es aus Spass etwas Falsches eingibt, kann ich das nicht abfangen.

Natürlich kannst du es abfangen, mit einer einfachen If-else Anweisung.

a.b_om schrieb:

Aber wie kriege ich das mit der Tabelle hin?

Nun, nehmen wir mal an die Spalte Kartenname sieht so aus: |------Kartenname------|
Abzüglich der Seperatoren, wären das dann 22 Zeichen.
Nehmen wir an deine Karte wäre "Schwarzes Loch" (ja ich kenne mich ein wenig aus),
das wären dann 14 Zeichen, ergo fehlen noch 8 Zeichen bis zum nächsten Seperator, die füllst du mit Leerzeichen aus. (falls das funktioniert, es gibt genügende Anwendungen welche serielle Leerzeichen tragischerweise einfach löschen)
Nun angenommen der Name ist doch etwas üppiger, dann müssen wir die überschüssigen Zeichen entfernent.
Dies wird meistens, wenn nicht sogar immer, mit drei Punkten dargestellt.
Also zum Beispiel "Irgendein Drache aus dem Spiel" hat 30 Zeichen, demnach müsstest du dies auf 22 Zeichen trimmen.
Allerdings auf 19 Zeichen, da du für die 3 Punkte auch noch 3 Zeichen bräuchtest.

Natürlich kommt dann noch margin und alignment dazu, aber du solltest einmal die Grundausführung bereitmachen.
Das machst du dann mit allen Spalten so!

a.b_om · 13. Dezember 2019, 08:04

@Elanda
Ich werde es versuchen.

Edit:
Ich kann es nicht versuchen, weil ich einen Fehler habe.
Der Fehler passiert in Zeile 3-4 siehe Code in Post 1.
Der zweiter Datensatz, wird irgendwie verändert. Die Werte sind volkommener Blödsinn und deshalb kann ich es nicht versuchen.

Wenn ihr eine Idee habt, könnt ihr es sagen. Bin seit knapp 2 Tagen das dran. Bisher hat der Fehler keine Einschränkungen bei den Funktionen bewirkt.

Elanda · 13. Dezember 2019, 15:45

Ich hab es mal durch meinen Compiler gefüttert da funktionieren 3 - 4 Tadellos.
Wie groß ist denn MAX?
Was für ein Fehler tritt denn genau auf?

Oh und btw. mach doch anstatt von "string[index] == 'char'" einfach strcmp(string1, string2), dann brauchst du nicht jeden einzelnen character einzeln abfragen.
Sieht ein wenig unschön aus.

a.b_om · 13. Dezember 2019, 16:41

MAX = 265

Elanda · 13. Dezember 2019, 16:59

Wenn das größte Wort das du eingibst nein ist, sind 256 Zeichen ein wenig über dem Horizont, denkst du nicht?
Länge 5 müsste reichen, der Rest wird dann einfach abgeschnitten.

Das dürfte aber nicht das eigentliche Problem sein.
Ich habe irgendwie das Gefühl an deiner eigentlichen Fragestellung vorbeigerasselt zu sein.
Warscheinlich weil ich nicht genau verstanden habe was jetzt das eigentliche Problem hier ist.

a.b_om · 13. Dezember 2019, 17:09

Die Werte in der Struktur haben sich einfach plötzlich verändert. Das ist genau in dieser Zeile passiert.

Elanda · 13. Dezember 2019, 18:07

Also jetzt verstehe ich gar nichts mehr, von welcher Struktur redest du?
Vom Aufbau des codes oder einem struct?
Und wenn von einem struct, wo greifst du in Zeile 3 oder 4 auf ein struct zu?

Es wird ein wenig unübersichtlich.

a.b_om · 14. Dezember 2019, 11:06

@Elanda
https://www.vb-paradise.de/index.php/Thread/129957-Wie-kann-ich-die-Aufgabe-mit-verketteten-Listen-l%C3%B6sen/?postID=1124014#post1124014

Das ist ein Thread, der um das gleiche Projekt geht, aber um eine andere Frage. Das findest du die Struktur,

Elanda · 14. Dezember 2019, 16:05

Na wenn du das gleich mit in deinen ersten Beitrag genommen hättest, wären wir wahrscheinlich jetzt viel weiter.
Ich zum Beispiel wusste nichts von dem anderen Beitrag, auf das musst du schon hinweisen.

Egal,
Stell doch mal den gesamten Dateiinhalt hier rein, ich denke du wirst viel daran geändert haben.
Und dann zeigst du bitte auch was dir die Konsole ausgibt, wenn das nicht geht weil ein Problem aufgetreten ist, beschreibst du halt das Problem. (damit meine ich eine exception, wenn eine aufgetreten war)

Edit:
Jetzt fällt es mir erst auf!
Gibt es einen Grund wieso du "ersteYugiohKarte" by value und nicht by pointer übergibst? (ich weiß jetzt nicht ob C Referenzen unterstützt)
Das ist sinnlose Kopierarbeit für nichts.

a.b_om · 17. Dezember 2019, 08:46

@Elanda
Sorry, dass ich nicht zurückgeschrieben habe.

Nein, am Code hat sich nicht viel verändert.

Wie mache ich dann das mit dem Pointermitgabe.

Mein kompletter Code:

Spoiler anzeigen

C-Quellcode

#include "pch.h"
#include <iostream>
#include <stdio.h>
#include "LinkedList.h"
#define MAX 265
typedef struct YugiohKarte {
char Name[100];
char Kartenart[20];
int Stufe;
int Rang;
int Pendelstufe;
int Link;
char ATK[20];
char DEF[20];
char Eigenschaft[20];
char Monstertyp[40];
char Kartentyp[30];
char Beschreibung[500];
struct YugiohKarte* pNext;
struct YugiohKarte* pPrev;
} strYugiohKarte;
YugiohKarte YugiohKarteErstellen(YugiohKarte ersteYugiohKarte) {
char antwort[MAX];
int anzahlKarten;
YugiohKarte *vorhaerigeYugiohKarte = &ersteYugiohKarte;
do
{
YugiohKarte* pErsteYugiohKarte = &ersteYugiohKarte;
printf("\nWie viele Yugiohkarten moechten Sie erstellen? Bitte geben Sie nur Zahlen ein, sonst kommt es zu einem Absturz.");
fgets(antwort, MAX, stdin);
anzahlKarten = atoi(antwort);
for (int i = 1; i <= anzahlKarten; i++)
{
if (pErsteYugiohKarte->Beschreibung[0] == 'B' && pErsteYugiohKarte->Beschreibung[1] == 'l' && pErsteYugiohKarte->Beschreibung[2] == 'a' && pErsteYugiohKarte->Beschreibung[3] == 'B' && pErsteYugiohKarte->Beschreibung[4] == 'l' && pErsteYugiohKarte->Beschreibung[5] == 'a')
{
ersteYugiohKarte = { "Name", "Kartenart", 1, 0, 0, 0, "ATK", "DEF", "Eigenschaft", "Monstertyp", "Kartentyp", "Beschreibung" };
}
else
{
YugiohKarte naechsteYugiohKarte = { "Name", "Kartenart", 1, 0, 0, 0, "ATK", "DEF", "Eigenschaft", "Monstertyp", "Kartentyp", "Beschreibung2" };
do
{
if (vorhaerigeYugiohKarte->pNext != NULL)
{
vorhaerigeYugiohKarte = vorhaerigeYugiohKarte->pNext;
}
else
{
break;
}
} while (true);
naechsteYugiohKarte.pPrev = vorhaerigeYugiohKarte;
vorhaerigeYugiohKarte->pNext = &naechsteYugiohKarte;
if (vorhaerigeYugiohKarte->pPrev == NULL)
{
ersteYugiohKarte = *vorhaerigeYugiohKarte;
}
printf(" ");
}
}
return ersteYugiohKarte;
} while (true);
}
YugiohKarte YugiohKarteLoeschen(YugiohKarte ersteYugiohKarte) {
return ersteYugiohKarte;
}
YugiohKarte YugiohKarteAusgeben(YugiohKarte ersteYugiohKarte) {
YugiohKarte* pErsteYugiohKarte = &ersteYugiohKarte;
printf("\nMoechtest du alle Karten ausgeben oder nur eine bestimmte? (Ja/Nein): ");
char antwort[MAX];
fgets(antwort, MAX, stdin);
if (antwort[0] == 'J' && antwort[1] == 'a')
{
do
{
if (pErsteYugiohKarte == NULL)
{
printf("\nEs gibt keine Yugiohkarten zum ausgeben.");
}
else
{
int i = 1;
YugiohKarte* jetzigeYugiohKarte = pErsteYugiohKarte;
do
{
printf("\n%i. Yugioh-Karte", i);
printf("\nName: %s", jetzigeYugiohKarte->Name);
printf("\nKartenart: %s", jetzigeYugiohKarte->Kartenart);
printf("\nStufe: %i", jetzigeYugiohKarte->Stufe);
printf("\nRang: %i", jetzigeYugiohKarte->Rang);
printf("\nPendelstufe: %i", jetzigeYugiohKarte->Pendelstufe);
printf("\nLink: %i", jetzigeYugiohKarte->Link);
printf("\nATK: %s", jetzigeYugiohKarte->ATK);
printf("\nDEF: %s", jetzigeYugiohKarte->DEF);
printf("\nEigenschaft: %s", jetzigeYugiohKarte->Eigenschaft);
printf("\nMonstertyp: %s", jetzigeYugiohKarte->Monstertyp);
printf("\nKartentyp: %s", jetzigeYugiohKarte->Kartentyp);
printf("\nBeschreibung: %s\n", jetzigeYugiohKarte->Beschreibung);
i++;
jetzigeYugiohKarte = jetzigeYugiohKarte->pNext;
} while (jetzigeYugiohKarte != NULL);
}
break;
} while (true);
}
else if (antwort[0] == 'N' && antwort[1] == 'e' && antwort[2] == 'i' && antwort[3] == 'n')
{
}
return ersteYugiohKarte;
}
int main()
{
YugiohKarte ersteYugiohKarte = { "N", "K", 1, 0, 0, 0, "A", "D", "E", "M", "K", "BlaBla" };
printf("\nWICHTIG: Bitte maximieren Sie das Fenster, weil es sonst nicht alles richtig darstellt.");
do
{
printf("\nMoechten Sie eine Yugioh-Karten erstellen(YKE), eine Yugioh-Karte loeschen(YKL), eine Yugioh-Karte ausgeben(YKA) oder das Programm schliessen(Prsc):");
char antwort[MAX];
fgets(antwort, MAX, stdin);
if (antwort[0] == 'Y' && antwort[1] == 'K' && antwort[2] == 'E') {
ersteYugiohKarte = YugiohKarteErstellen(ersteYugiohKarte);
}
else if (antwort[0] == 'Y' && antwort[1] == 'K' && antwort[2] == 'L') {
YugiohKarteLoeschen(ersteYugiohKarte);
}
else if (antwort[0] == 'Y' && antwort[1] == 'K' && antwort[2] == 'A') {
YugiohKarteAusgeben(ersteYugiohKarte);
}
else if (antwort[0] == 'P' && antwort[1] == 'r' && antwort[2] == 's' && antwort[3] == 'c') {
return 0;
}
else
{
printf("Bitte geben Sie eine Abkuerzung ein!");
}
} while (true);
}

VaporiZed · 17. Dezember 2019, 09:02

Indem Du die Adresse an die Prozeduren übergibst:

C-Quellcode

Void Main() {
//...
if (antwort[0] == 'Y' && antwort[1] == 'K' && antwort[2] == 'E') {
ersteYugiohKarte = YugiohKarteErstellen(&ersteYugiohKarte);
}
}
YugiohKarte YugiohKarteAusgeben(YugiohKarte* ersteYugiohKarte) {
YugiohKarte* pErsteYugiohKarte = ersteYugiohKarte;
//...
}

a.b_om · 17. Dezember 2019, 09:08

@VaporiZed
Danke, ich versuche es. Ich melde mich, beim nächsten Problem

Elanda · 17. Dezember 2019, 18:29

Ach du liebes lieschen, dass nenne ich mal mit dem Kopf durch die Wand.

Ich habe ehrlich gesagt keine Ahnung wie ich dir da helfen soll, es sind so viele Dinge die mir hier ins Auge stechen, dass es echt schwierig ist da weiterzuhelfen.
Aber ich kann dir sagen, dass "YugiohKarteErstellen" sehr isoliert ist.

Wenn du dort eine neue Karte erstellst und sie als pointer an die zurückgegebene Karte zuweist, wird beim verlassen der Funktion die Karte aus dem Stack wieder entfernt.
Da du nur den Pointer in pNext zurückgibst ist das auch sehr verständlich, du müsstest die neu erstellte Karte also entweder dynamisch am Heap erzeugen und dessen Addresse als pointer ablegen oder das neu erzeugte struct by value zurückgeben, und in einem Array, welches während des gesamten Programmablaufes besteht, abspeichern und in "main()" dann als pNext zur letzten Karte im Array übergeben.
Derzeitiges Ergebnis liefert dir nur UB*, was nicht sehr zu empfehlen ist.

Edit:
*UB: Undefined Behaviour, heißt soviel wie keiner weiß was passieren wird, du begibst dich auf unbekannte Gefilde

a.b_om · 17. Dezember 2019, 18:39

@Elanda
Wie mache ich das denn richtig.

a.b_om · 7. Januar 2020, 14:27

Ich weiss jetzt wie das geht.

Ein Beispiel:

C-Quellcode

system("cls");
printf("%10s %10s %10s\n", "Marke", "Km-Stand", "Jahrgang");
for (struAuto* pOut = pStart; pOut != NULL; pOut = pOut->pNext)
{
printf("%10s %10i %10i\n", pOut->Marke, pOut->KmStand, pOut->Jahrgang);
}

Tabelle mit C ausgeben

Tabelle mit C ausgeben

C-Quellcode

Elanda schrieb:

Elanda schrieb:

Elanda schrieb:

Elanda schrieb:

a.b_om schrieb:

a.b_om schrieb:

a.b_om schrieb:

C-Quellcode

C-Quellcode

C-Quellcode

Benutzer online 1

Tags

Ähnliche Themen

3 Benutzer haben hier geschrieben