Probleme bei der Kompilierung meines Programms unter Linux

dachschaden

Mangan2504 schrieb:

Das hier ist das Ergebnis: http://pastebin.com/VxeirRzV

Das ist eine ganz kaputte Mischung aus C und C++. Ja, mit einem C++-Compiler kompiliert es, aber das ist dennoch C ( fopen , fscanf ...)

Wenn ich das beim Dozenten gelehrt bekommen hätte, hätte ich ihm erst mal auf den Tisch gekotzt.

Ich habe jetzt nur nach dem ersten Fehler gesucht, den mir gdb gezeigt hat. Und bin dann darüber gestolpert:

int main (int x, char *pups[]){

Traditionell sollte das:

int main(int argc,char*argv[])

sein. Das eigentliche Problem aber ist: du solltest beim Programmaufruf einen Parameter übergeben, der einen Dateinamen beinhaltet, dessen Datei dann später geöffnet werden soll.

Jetzt haben wir diese Datei allerdings nicht. Blöd gelaufen.

Und du prüfst auch nicht, ob dein Programm die Datei einladen kann. Blöd gelaufen die zweite.

if(argc < 2)
{
    fprintf(stderr,"Nicht genug Argumente: %s <Dateiname>\n",argv[0]);
    return -1;
}

text = fopen (pups[1], "r");
if(!text)
{
    fprintf(stderr,"Datei \"%s\" kann nicht geöffnet werden!\n",pups[1]);
    return -1;
}

DirkB hat schon nachgefragt, wie rufst du das Programm auf?
Und ich frage nach der Datei, die du da öffnest.

Wenn ich übrigens mal die Warnungen anstelle, mit denen ich alles kompiliere, was ich schreibe:

-W -Wextra -Wall -Wimplicit -Wunused -Wunused-parameter -Wshadow -Wempty-body -Wignored-qualifiers -Wvla -Wvariadic-macros -Wdisabled-optimization -Winline -Wsign-compare -Wuninitialized -Wparentheses -Wswitch

, dann bekomme ich diese Warnungen:

kata.c: In Funktion »void ausgabe(domino*)«:
kata.c:43:27: Warnung: Deklaration von »kopf« überdeckt eine globale Deklaration [-Wshadow]
 void ausgabe (domino *kopf){                  // Funktion zum ausgeben einer Liste, soll auch Ringliste ausgeben können
                           ^
kata.c:16:13: Anmerkung: verdeckte Deklaration ist hier
 verwaltung *kopf, *ende;             // Listenkopf und Listenende erstellen
             ^
kata.c: In Funktion »void ausgabeRing(domino*)«:
kata.c:48:34: Warnung: Deklaration von »ringkopf« überdeckt eine globale Deklaration [-Wshadow]
 void ausgabeRing(domino *ringkopf){
                                  ^
kata.c:18:9: Anmerkung: verdeckte Deklaration ist hier
 domino *ringkopf, *ringmitte, *ringende; // Ringlistenzeiger (soll nach Ausgabe recyclet werden)
         ^
kata.c: In Funktion »void bautRing(domino*)«:
kata.c:58:11: Warnung: Anweisung hat keinen Effekt [-Wunused-value]
     for (n; n != NULL; n = n->next){
           ^
kata.c: In Funktion »int main(int, char**)«:
kata.c:96:9: Warnung: Variable »z« wird nicht verwendet [-Wunused-variable]
     int z = 0; // zeigerzähler
         ^
kata.c: Im globalen Gültigkeitsbereich:
kata.c:82:15: Warnung: unbenutzter Parameter »x« [-Wunused-parameter]
 int main (int x, char *pups[]){         // pups beinhaltet die Parameter (Anzahl übergebener Werte und Datei)

EDIT: besserer Check, sodass wir den potentiellen NULL-Pointer nicht an fopen übergeben.

Mangan2504

dachschaden schrieb:

Das ist eine ganz kaputte Mischung aus C und C++. Ja, mit einem C++-Compiler kompiliert es, aber das ist dennoch C ( fopen , fscanf ...)

Wenn ich das beim Dozenten gelehrt bekommen hätte, hätte ich ihm erst mal auf den Tisch gekotzt.

Wie gesagt haben wir in der Vorgänger-Veranstaltung nur mit C programmiert, deswegen ist der Fehler da eher bei mir zu suchen.

Gast3 schrieb:

wenn ich deinen Code direkt bei

http://gcc.godbolt.org/

reinkopieren und mit dem gcc 4.x-6.x oder clang kompilieren Optionen: -O2 -Wall

kompilierts - und jetzt?

was ist dein Kompileraufruf: also gcc ... , oder g++ ....

Ok? Das ist komisch.

Wir sollen g++ verwenden.

In der Anleitung steht:

Beispiel (Übersetzen des Quellcodes einer Datei „test.cpp“)
g++ -o test test.cpp

Ich gebe also ein: g++ -o programm programm.cpp
auf dem Terminal unter Linux.

Wenn es laufen würde, dann mache ich weiter mit ./programm.cpp

Wenn es laufen würde, dann mache ich weiter mit ./programm.cpp

nein machst du nicht sondern mit ./programm (nicht .cpp) - die erste Angabe von "programm" in der Kommandozeile sagt wie die Applikation heissen soll

btw: die vorherigen Post haben den Fehler schon eingegrenzt also nur noch fixen - die Datei einstellen dann läufts - alles nur Schlurifehler nichts wildes

kompiliert unter gcc.godbolt, und jedem anderen Linux das gerade offen ist - so ein Trivial-Programm muss kompilieren - sonst ist was mit deinen Einstellungen faul (hast du an etwas rumgespielt nachdem das 1. mal der Fehler gekommen ist?)

also 1.

"g++ -o programm programm.cpp"

eingeben - Ausgabe hier posten

"./programm dateiname"

eingeben - Ausgabe hier posten

solltest du den dateinamen als Parameter weglassen stürzt dein Programm ab (fehlenden Prüfung deinerseits)

die Datei mal hier einstellen die du als Parameter angibst - oder solltest

Mangan2504

Gast3 schrieb:

kompiliert unter gcc.godbolt, und jedem anderen Linux das gerade offen ist - so ein Trivial-Programm muss kompilieren - sonst ist was mit deinen Einstellungen faul (hast du an etwas rumgespielt nachdem das 1. mal der Fehler gekommen ist?)

also 1.

"g++ -o programm programm.cpp"

eingeben - Ausgabe hier posten

"./programm dateiname"

eingeben - Ausgabe hier posten

solltest du den dateinamen als Parameter weglassen stürzt dein Programm ab (fehlenden Prüfung deinerseits)

die Datei mal hier einstellen die du als Parameter angibst - oder solltest

Ehrlich gesagt weiß ich nich was für ein Compiler die Uni benutzt. Ich habe daran sicher nichts umgestellt, ich wüsste ja gar nicht wie... Das einzige was ich getan habe ist mit "cd" Verzeichnisse wechseln. Aber natürlich NICHT während ich am Program gearbeitet habe, sondern davor.

Die Dateien, die ich als Parameter angebe beinhalten einfach nur willkürlich gewählte Zahlenpaare, die die Dominosteine darstellen sollen.

Die Eingabe/Ausgabe kann ich im Moment erstmal nicht mehr beisteuern, denn für heute habe ich erstmal Schluss gemacht. Ich bin schon seit 9 Uhr dran und gerade an einem Punkt, wo nichs mehr geht... Ich wage dann morgen in aller Frische einen neuen Versuch.

Vielen Dank schonmal für eure Geduld und Zeit.

Ehrlich gesagt weiß ich nich was für ein Compiler die Uni benutzt

g++ --version

1. was hast du für einen gcc/g++

g++ --version

2. und ein Trivial-Test ob dein gcc/g++ richtig funktioniert

--main.cpp--

#include<stdio.h>

int main() 
{
  int* x = new int();
  delete x;
  printf("Hello World\n");
  return 0;
}

---

g++ -o main main.cpp

sollte "Hello World" ausgeben

Ich finde es unerhört, dass auf diese Weise versucht wird, sich Punkte in der Prüfungsvorleistung zu erschleichen. Bitte versuchen Sie selbstständig die Aufgaben zu lösen. Bei Fragen können Sie sich gerne an mich wenden.

Wutz

...

Das Programm ist bis auf new pures C, entferne schrittweise alle Compilerwarnungen (von oben nach unten), entferne alle globalen Variablen und mache sie funktions- bzw. main-lokal evtl. übergib sie als (Zeiger)Parameter.
Speicherzugriffsfehler haben ihre Ursache meist bei Array- bzw. allg. Speichergrenzenüberschreitungen oder uninitialisierten Variablen (würde der Compiler melden) oder falschem Zeigergebrauch.

dachschaden

Wutz schrieb:

Das Programm ist bis auf new pures C

Noch nicht ganz. Die Art, wie die Typen angegeben werden, ist in reinem C fehlerhaft (benötigt den Modifier union, struct , oder enum ).

Aber das ist nur für's Archiv.

War nur ein kleiner Spaß. Wusste ja nicht, dass ich damit gleich solche Emotionen auslöse^^

Sitze selber noch an der Aufgabe, bin aber recht zuversichtlich. Was die Schlussfolgerung betrifft, dass der Fehler gleich beim Herr Dr. zu suchen ist, bin ich geteilter Meinung. Es ist wirklich nicht sehr vorteilhaft in der Vorgängervorlesung die Grundlagen einer Programmiersprache in C zu lehren und dann anschließend im nächsten Semester hauptsächlich auf C++ zu wechseln.

Kann mir gut vorstellen, dass das den ein oder anderen verwirrt. Das andere Problem ist, dass zur Vereinfachung auf viele Sachen nicht geachtet werden soll (oftmals einiges an Exceptionhandling), wobei gerade das wichtig sein wird.

Andererseits muss man auch beachten, dass das eine Vorlesung ist, die von allen Informatik-Studiengängen besucht wird. Ist eben dann schwierig einen Mittelweg zu finden, dass alle es verstehen. Leute die vorher noch garnicht oder kaum programmiert haben, haben es da sicher am schwersten.

Ich würde also nicht mal den Fehler beim Dr. suchen sondern, wie schon gesagt wurde, bei der Strukturierung des neuen Studiengangs. Es müsste eben eine Vorlesung für all die geben, die Informatik als Nebenfach absolvieren. Ist eben doch ungünstig alle in die selbe Vorlesung zu stecken.

Beste Grüße und viel Erfolg

Mangan2504

Gast42 schrieb:

War nur ein kleiner Spaß. Wusste ja nicht, dass ich damit gleich solche Emotionen auslöse^^

Der war aber leider nicht besonders lustig. Wenn du wüsstest, was du damit bei mir heute Morgen angerichtet hast...

Allerdings, Dozenten beleidigen hilft keinem wirklich weiter. Zumal gewisse Sachen offensichtlich ein Missverständnis waren oder ich habe mich nicht deutlich ausgedrückt.

Aber, die Aufgabe ist nunmal da, daran kann man nichts ändern und bearbeiten muss ich sie ja trotzdem.

Wie dem auch sei. Ich habe eure Tipps beherzigt und ich denke ich bin ebenfalls auf einem guten Weg, dieses Programm noch zum laufen zu bringen.

Ich möchte mich nochmal bedanken für die Geduld und die Tipps.