ganz schlimmer fehler!!!

virtuell Realisticer

Sry, aber schreib ich etwa chinesich?

mfg
v R

halt noch ne überarbeitung, er fehler kommt nicht wenn ichs objekt erstelle, sondern wenn ich unter der klasse die matrix.cpp includieere.

wenn ich das include weglasse kommt wieder der gewohnte fehler:

main.cpp:(.text+0x2c): undefined reference to `matrix<int>::setze_in_matrix(int*, int)'
collect2: ld returned 1 exit status
make[2]: *** [anne] Fehler 1

trotzdem, wer nun weiß wie man das fehlerfrei kompilliertt finger HOCH. ich denke man kann den code leicht per copy and paste übernehmen und testen, ich hab es ja nicht umsonst hier ins forum gestellt...

achso, sry virtual,
ich kann mir denken was include guards sind, aber wie muss ich die machen?????

virtuell Realisticer

exigoner schrieb:

achso, sry virtual,
ich kann mir denken was include guards sind, aber wie muss ich die machen?????

Deine Header muss so aussehen:

#ifndef headernameH
#define headernameH

//jede menge code

#include "matrix.cpp"

#endif

Dein Problem war bis jetzt, dass du eine endlos Includierung erzeugt hast, denn

matrix.h includiert matrix.cpp
matrix.cpp includiert matrix.h
... usw. usf.

Der Includeguard sagt:

headernameH schon definiert? Wenn nicht, dann definiere ihn und liess den Rest
ein, ansonsten ignoriere den Rest.

mfg
v R

ok, ich hab jetzt meine includeguards gesetzt...

matrix.cpp:11: error: expected constructor, destructor, or type conversion before '<' token
matrix.cpp:11: error: expected `;' before '<' token
matrix.cpp:19: error: expected constructor, destructor, or type conversion before '<' token
matrix.cpp:19: error: expected `;' before '<' token
matrix.cpp:29: error: expected constructor, destructor, or type conversion before '<' token
matrix.cpp:29: error: expected `;' before '<' token
matrix.cpp:37: error: expected init-declarator before '<' token
matrix.cpp:37: error: expected `;' before '<' token
matrix.cpp:44: error: expected init-declarator before '<' token
matrix.cpp:44: error: expected `;' before '<' token

wenn ich jetzt die includeguards in der matrix.CPP weglasse, kommt aber das hier:

matrix.cpp:9: error: redefinition of `matrix<T>::matrix(unsigned int)'
matrix.cpp:9: error: `matrix<T>::matrix(unsigned int)' previously declared here
matrix.cpp:16: error: redefinition of `matrix<T>::matrix(unsigned int, unsigned int)'
matrix.cpp:16: error: `matrix<T>::matrix(unsigned int, unsigned int)' previously declared here
matrix.cpp:26: error: redefinition of `matrix<T>::~matrix()'
matrix.cpp:26: error: `matrix<T>::~matrix()' previously declared here
matrix.cpp:34: error: redefinition of `void matrix<T>::ausgabe_matrix()'
matrix.cpp:34: error: `void matrix<T>::ausgabe_matrix()' previously declared here
matrix.cpp:41: error: redefinition of `void matrix<T>::setze_in_matrix(T*)'
matrix.cpp:41: error: `void matrix<T>::setze_in_matrix(T*)' previously declared here

virtuell Realisticer

Die Includeguards kommen in die Headerdatei. Und in der .cpp-Datei brauchst du
die .h-Datei nicht mehr includieren. Nach der include-Zeile in der Header steht
der Code ja quasi direkt untereinander.

Auch hier ist es besser, den Namensraum nicht zu oeffnen und auch in der
.cpp-Datei mit dem std::-Prefix zu arbeiten.

mfg
v R

ok;
meine klasse sieht dann so aus:

#ifndef matrix_klasse
#define matrix_klasse

#include <iostream>
#include <cstdlib>

using namespace std;

template <class T>
class matrix
{
	public:
		matrix(unsigned int z); //eine dimension
		matrix(unsigned int z, unsigned int s);  //zwei dimemsionen
		//matrix(int, int, int);  //noch nicht
		~matrix();

		void ausgabe_matrix();

		void setze_in_matrix(T *feld); //schiebt hinten rein(matrix)
		void setze_in_matrix(T *feld, int pos_z);  //für eindim array
		void setze_in_matrix(T *feld, int pos_z, int pos_s); //für zweidim matrix

		void fuege_array_ein(T *array);

	private:
		T *matrix_eindim;
		T **matrix_zweidim;
		//T ***matrix_dreidim;

		unsigned int zeilen_matr;
		unsigned int spalten_matr;

		unsigned int index_z_eindim;  //damit verwechlungen ausgeschlossen sind
		unsigned int index_z_zweidim;
		unsigned int index_s_zweidim;
};
#include "matrix.cpp"

#endif

#include <iostream>
#include <cstdlib>

template <class T>
matrix<T>::matrix(unsigned int zeilen)
{
	matrix_eindim = new T[zeilen];
	zeilen_matr = zeilen;
}

template <class T>
matrix<T>::matrix(unsigned int zeilen, unsigned int spalten)
{
	matrix_zweidim = new T*[zeilen];
	for(int i=0; i < zeilen; i++)
		matrix_zweidim[i] = new T[spalten];
	zeilen_matr = zeilen;
	spalten_matr = spalten;
}

template <class T>
matrix<T>::~matrix()
{
	delete [] matrix_eindim;
	delete [] matrix_zweidim;
	//delete matrix_dreidim;
}

template <class T>
void matrix<T>::ausgabe_matrix()
{
	for(int i = 0; i < zeilen_matr; i++)
		cout<< "Position " << i << ": " << matrix_eindim[i] << endl;
}

template <class T>
void matrix<T>::setze_in_matrix(T *feld)
{
	int feld_zeilen = sizeof(feld) / sizeof(feld[0]);
	for(; index_z_eindim < feld_zeilen; index_z_eindim++)
		matrix_eindim[index_z_eindim] = feld[index_z_eindim];
}

matrix.cpp:5: error: expected constructor, destructor, or type conversion before '<' token
matrix.cpp:5: error: expected `;' before '<' token
matrix.cpp:12: error: expected constructor, destructor, or type conversion before '<' token
matrix.cpp:12: error: expected `;' before '<' token
matrix.cpp:22: error: expected constructor, destructor, or type conversion before '<' token
matrix.cpp:22: error: expected `;' before '<' token
matrix.cpp:30: error: expected init-declarator before '<' token
matrix.cpp:30: error: expected `;' before '<' token
matrix.cpp:37: error: expected init-declarator before '<' token
matrix.cpp:37: error: expected `;' before '<' token

mir scheint an erfahrung zu mangeln...

Caipi

@exigoner:
Dein Hauptprogramm sollte so aussehen:

#include "matrix.hpp"

int main()
{
      return 0;
}

Beachte das du in dein Hauptmodul den header matrix.hpp einbinden musst und nicht die matrix.cpp...

Caipi

also meine main:

#ifdef HAVE_CONFIG_H
#include <config.h>
#endif

#include <iostream>
#include <cstdlib>
#include "matrix.h"

using namespace std;

int main(int argc, char *argv[])
{
  int a[] = {1,2,3,4,5,6,7,8,9,8,7,6,5,4,3};

  matrix<int> mat(15);
  mat.setze_in_matrix(a, 15);
  mat.ausgabe_matrix();

  return EXIT_SUCCESS;
}

sieht ganz gut aus... außerdem hab ich meine klasse ganz normal matrix.h und meine getrennte implementation matrix.cpp genannt. (siehe oben)
der fehler liegt damit best. wieder in dieser beschissenen klasse.
[heul]

Caipi

My last try

Wenn deine Dateien matrix.hpp, matrix.cpp und main.cpp heißen und du zusätzlich die setze_in_matrix() Methode überarbeitest (Du rufst diese mit zwei Argumenten auf, sie erwartet aber nur eins), sollte sich der folgende Code (abgesehen von ein paar Warnungen) mit dem g++ compilieren lassen: (Von den logischen Fehlern spreche ich hier mal nicht...)

matrix.hpp

#ifndef matrix_klasse
#define matrix_klasse

#include <iostream>
#include <cstdlib>

using namespace std;

template <class T>
class matrix
{
    public:
        matrix(unsigned int z); //eine dimension
        matrix(unsigned int z, unsigned int s);  //zwei dimemsionen
        //matrix(int, int, int);  //noch nicht
        ~matrix();

        void ausgabe_matrix();

        void setze_in_matrix(T *feld); //schiebt hinten rein(matrix)
        void setze_in_matrix(T *feld, int pos_z);  //für eindim array
        void setze_in_matrix(T *feld, int pos_z, int pos_s); //für zweidim matrix

        void fuege_array_ein(T *array);

    private:
        T *matrix_eindim;
        T **matrix_zweidim;
        //T ***matrix_dreidim;

        unsigned int zeilen_matr;
        unsigned int spalten_matr;

        unsigned int index_z_eindim;  //damit verwechlungen ausgeschlossen sind
        unsigned int index_z_zweidim;
        unsigned int index_s_zweidim;
};
#include "matrix.cpp"

#endif

matrix.cpp

#include <iostream>
#include <cstdlib>

template <class T>
matrix<T>::matrix(unsigned int zeilen)
{
    matrix_eindim = new T[zeilen];
    zeilen_matr = zeilen;
}

template <class T>
matrix<T>::matrix(unsigned int zeilen, unsigned int spalten)
{
    matrix_zweidim = new T*[zeilen];
    for(int i=0; i < zeilen; i++)
        matrix_zweidim[i] = new T[spalten];
    zeilen_matr = zeilen;
    spalten_matr = spalten;
}

template <class T>
matrix<T>::~matrix()
{
    delete [] matrix_eindim;
    delete [] matrix_zweidim;
    //delete matrix_dreidim;
}

template <class T>
void matrix<T>::ausgabe_matrix()
{
    for(unsigned int i = 0; i < zeilen_matr; i++)
        cout<< "Position " << i << ": " << matrix_eindim[i] << endl;
}

template <class T>
void matrix<T>::setze_in_matrix(T *feld)
{
    unsigned int feld_zeilen = sizeof(feld) / sizeof(feld[0]);
    for(; index_z_eindim < feld_zeilen; index_z_eindim++)
        matrix_eindim[index_z_eindim] = feld[index_z_eindim];
}

main.cpp

#ifdef HAVE_CONFIG_H
#include <config.h>
#endif

#include <iostream>
#include <cstdlib>
#include "matrix.hpp"

using namespace std;

int main(int argc, char *argv[])
{
  int a[] = {1,2,3,4,5,6,7,8,9,8,7,6,5,4,3};

  matrix<int> mat(15);
  mat.setze_in_matrix(a);
  mat.ausgabe_matrix();

  return EXIT_SUCCESS;
}

Caipi

es kommen genau dieselben fehler..
und um auf die logischen fehler zu kommen... testen konnt ich die klasse noch nicht weiter

@caipi: hast du deinen code eigentlich getestet???

ich hoffe ihr seid glücklich! ich habe meine templates entfernt

dali

Ist glaub ich auch besser so...

Caipi

exigoner schrieb:

@caipi: hast du deinen code eigentlich getestet???

Von testen kann nicht die Rede sein. Aber ich habe ihn so wie von mir gepostet ohne (Syntax/Semantik)-Fehler durch den Compiler (g++) gejagt :).

Caipi

egal, ich hab die klasse nicht getrennt kompillieren können, ich musste unbedingt klasse und implementation in eine datei packen. naja wenigsten funzt es jetzt
trotzdem danke