Hi Leute,

ich möchte gerne Variablen definieren, die über jede Quellcodedatei erreichbar und änderbar ist.

Ist das mit 'extern' möglich?

ich würde das ganze gerne etwas eleganter schreiben, aber ich finde zur zeit leider
nichts.

greetz johnc.

ich möchte gerne Variablen definieren, die über jede Quellcodedatei erreichbar und änderbar ist.
Ist das mit 'extern' möglich?

Ja, vllt hättest du es einfach ausprobieren sollen

JohnC. schrieb:

ich würde das ganze gerne etwas eleganter schreiben, aber ich finde zur zeit leider nichts.

Schau dir eventuell mal das Singelton an. Mir kommt ein so allgemeiner Zugriff auf Variablen, wie du ihn planst, aber wenig sinnvoll vor. Ein Singelton, das nur zur Kapselung von globalen Variablen dient, ist auch nicht gerade optimal. Meist gibt es da bessere Alternativen.

Gruß
Don06[url][/url]

Ein Singelton, das nur zur Kapselung von globalen Variablen dient, ist auch nicht gerade optimal.

Manchmal aber notwendig, Stichwort „Registry“.

folgender Code:
step.h

#ifndef _STEP_
#define _STEP_

#include <iostream>
#include "variabeln.h"
#include "tabelle.h"
#include "ctrlStepArray.h"
#include "routine.h"
using namespace std;

void step(int wordpos)
{
bool ctrl;
   for (int i=1; i<=8; ++i)
      {
      switch(i)
         {
         case 1:  if (stepArray[wordpos].y-1 >= 0)
                     {
                     ctrl = ctrlStAr(0,-1, wordpos);
                     if (ctrl == false)
                        {
                        int x = stepArray[wordpos].x;
                        int y = stepArray[wordpos].y-1;
                        routine(x,y,wordpos);
                        }
                     }
                  break;
         case 2: break;
         case 3: break;
         case 4: break;
         case 5: break;
         case 6: break;
         case 7: break;
         case 8: break;    
         }

      }
};

#endif

routine.h

#ifndef _ROUTINE_
#define _ROUTINE_

#include "variabeln.h"
#include "step.h"
#include "tabelle.h"
using namespace std;

void routine(int x, int y, int wordpos)
{
   tabelle lolol;
   lolol.ctrl[0] = false;
   lolol.ctrl[1] = false;
   word = word + tab[x][y];
   ++wordpos;
   lolol = db.control(word, wordpos);

   if (lolol.ctrl[0])
	   {
	   if (lolol.ctrl[1])
		   {
			cout << word << endl;
		   }
	   }
   else
	   {
		int sfrtg = word.size();
      word.erase(sfrtg-1,1);
      --wordpos;
	   }
   step(wordpos);
};
#endif

wenn ich kompilieren möchte kommt
error C3861: 'routine': identifier not found

ich frag mich bloß warum

in der main habe ich routine.h genauso inlcudet, und dort findet er 'routine'

greetz johnc.

Ein Beispiel wie es gemacht wird.
routine.h

#ifndef _ROUTINE_
#define _ROUTINE_

#include "variabeln.h"
#include "step.h"
#include "tabelle.h"

void routine(int x, int y, int wordpos);

#endif

routine.cpp

#include "routine.h"

void routine(int x, int y, int wordpos)
{
   tabelle lolol;
   lolol.ctrl[0] = false;
   lolol.ctrl[1] = false;
   word = word + tab[x][y];
   ++wordpos;
   lolol = db.control(word, wordpos);

   if (lolol.ctrl[0])
       {
       if (lolol.ctrl[1])
           {
            cout << word << endl;
           }
       }
   else
       {
        int sfrtg = word.size();
      word.erase(sfrtg-1,1);
      --wordpos;
       }
   step(wordpos);
}

Du solltest alle deine Dateien in eine *.h Datei (Header)
und eine *.cpp Datei (implementation) teilen.

PS: verwende nie using namespace std; oder andere in einer Hederdatei,
verwende sie nur in der implementationdatei.

greetz

ctrlStepArray.obj : error LNK2005: "class datenbank db" (?db@@3Vdatenbank@@A) already defined in racer_main.obj

main.cpp

#include <iostream>
#include <fstream>
#include <string>
#include "datenbank.h"
#include "tabelle.h"
#include "variabeln.h"
#include "routine.h"
using namespace std;

routine.h

#ifndef _ROUTINE_
#define _ROUTINE_

#include "variabeln.h"
#include "step.h"
#include "tabelle.h"
#include <iostream>
#include <string>

void routine(int x, int y, int wordpos);

#endif

step.h

#ifndef _STEP_
#define _STEP_

#include <iostream>
#include "variabeln.h"
#include "tabelle.h"
#include "ctrlStepArray.h"
#include "routine.h"

void step(int wordpos);

#endif

ctrlStepArray.h

#ifndef _ctrlStepArray_
#define _ctrlStepArray_

#include "variabeln.h"
#include <iostream>

bool ctrlStAr(int x, int y, int wordpos);

#endif

variabeln.h

#ifndef _VARIABELN_
#define _VARIABELN_
#include "datenbank.h"
#include "tabelle.h"

const int ct = 2;
datenbank db;
string word = "";
tabelle stepArray[10];
char tab[ct][ct];

#endif

brauch ich für variablen.h auch eine cpp?

brauch ich für variablen.h auch eine cpp?

Ja

Was machst du eigentlich für ein programm?
Wieviel Dateien sind in dem Project?

Was machst du eigentlich für ein programm?

nehmen wir an es gibt eine Tabelle bestehend aus buchstaben

01
0 en
1 si

das ist jetzt mal ne kleine Tabelle

ich programmiere gerade ein programm, was die wörter herausfindet, welche mit diesen buchstaben möglich sind.

Allerdings darf jeder buchstaben nur einmal vorkommen, und nur benachbarte Buchstaben folgen (diagonal, vert,horizont)

bei diesem beispiel gehen:
ein
eins
eis
ins
nie
sei
sein
sie
sin

min. 3 B.staben

Wieviel Dateien sind in dem Project?

es besteht aus der main
dann aus der routine(func) datei die dazu dient selbst aufgerufen zu werden
die step(func) datei, die die schritte 1 bis 8 durchgeht(oben, obenrechts, rechts) etc.
die datenbank Klasse, zum einlesen, der datenbank txt-file, und kontrollieren der wörter
und die ctrlStepArray(func) die kontrolliert, wenn ein schritt möglich ist, ob der buchstabe nicht schon im Wort vorhanden ist

greetz johnc.

const int ct = 2;
extern char tab[ct][ct];

sagen wir du verwendest die Variable " tab "
nur in der datei routine.cpp, dann kannst du dort

const int ct = 2;
char tab[ct][ct];

schreiben und sost nirgend wo.

PS:
Giebt es eine Datei namens racer_main.cpp oder racer_main.h?
Was ist in der tabelle.h drin?

Giebt es eine Datei namens racer_main.cpp oder racer_main.h?
Was ist in der tabelle.h drin?

es gibt eine racer_main.cpp

in der tabelle.h habe ich eine Klasse

#ifndef _TABELLE_
#define _TABELLE_

class tabelle
{ 
public:
   int x,y,iStep;
   bool ctrl[2];

   tabelle();
};
#endif

gewöhn Dir an, nur die wirklich notwendigen Includes in eine Headerdatei zupacken. Wenn Du in jeden Header <iostream> einfügst, dann verlangsamt das die Compilezeit von Deinem Projekt nur unnötig, ohne etwas zu nützen, und so wie's scheint, benötigst Du den Header in Wahrheit nirgendwo.

(Wenn Du einfach auf eine Deklaration aus <iostream> zurückgreifen willst, z.B. 'std::basic_ostream', dann reicht es außerdem, wenn Du statt <iostream> den Header <iosfwd> einbindest, der enthält nur die notwendigen Vorwärtsdeklarationen. Dadurch beschleunigst Du die Compilezeit auch.)

Und noch etwas: nie, nie 'using namespace …' in einem Header! Das führt zu Namenskonflikten.

ich benutze an und an cout zum debuggen, deswegen binde ich das überall ein. Aber ich merke gerade, dass das mit dem Debugger wesentlich, einfacher ist

apropos: Gerade bin ich voll dabei und es gibt da eine Sache(wesentlich mehr=), die ich nicht verstehe

#include "step.h"

using namespace std;

void step(int wordpos)
{
extern tabelle stepArray[10];
const int ct = 2;

bool ctrl;
   for (int i=stepArray[wordpos].iStep; i<=8; ++i)
      {
      switch(i)
         {
         case 1:  if (stepArray[wordpos].y-1 >= 0)
                     {
                     ctrl = ctrlStAr(0,-1, wordpos);
                     if (ctrl == false)
                        {
                        int x = stepArray[wordpos].x;
                        int y = stepArray[wordpos].y-1;
                        stepArray[wordpos].iStep = i;
                        routine(x,y,wordpos);
                        }
                     }
                  else
                     {
                     break;
                     }
         case 2: if ((stepArray[wordpos].x+1 <= ct)&&(stepArray[wordpos].y-1 >= 0))
                     {
                     ctrl = ctrlStAr(+1,-1, wordpos);
                     if (ctrl == false)
                        {
                        int x = stepArray[wordpos].x+1;
                        int y = stepArray[wordpos].y-1;
                        routine(x,y,wordpos);
                        }
                     }
                 break;
         case 3: if (stepArray[wordpos].x+1 <= ct)/*&&(stepArray[wordpos].y >= 0)*/
                     {
                     ctrl = ctrlStAr(+1,0, wordpos);
                     if (ctrl == false)
                        {
                        int x = stepArray[wordpos].x+1;
                        int y = stepArray[wordpos].y;
                        routine(x,y,wordpos);
                        }
                     }
                 break;
         case 4: if ((stepArray[wordpos].x+1 <= ct)&&(stepArray[wordpos].y+1 <= ct))
                     {
                     ctrl = ctrlStAr(+1,+1, wordpos);
                     if (ctrl == false)
                        {
                        int x = stepArray[wordpos].x+1;
                        int y = stepArray[wordpos].y+1;
                        routine(x,y,wordpos);
                        }
                     }
                 break;
         case 5: if /*((stepArray[wordpos].x+1 <= ct)&&*/(stepArray[wordpos].y+1 <= ct)
                     {
                     ctrl = ctrlStAr(0,+1, wordpos);
                     if (ctrl == false)
                        {
                        int x = stepArray[wordpos].x;
                        int y = stepArray[wordpos].y+1;
                        routine(x,y,wordpos);
                        }
                     } 
                 break;
         case 6:  if ((stepArray[wordpos].x-1 >= 0)&&(stepArray[wordpos].y+1 <= ct))
                     {
                     ctrl = ctrlStAr(-1,+1, wordpos);
                     if (ctrl == false)
                        {
                        int x = stepArray[wordpos].x-1;
                        int y = stepArray[wordpos].y+1;
                        routine(x,y,wordpos);
                        }
                     }
                  break;
         case 7:  if (stepArray[wordpos].x-1 >= 0)/*&&(stepArray[wordpos].y-1 >= 0))*/
                     {
                     ctrl = ctrlStAr(-1,0, wordpos);
                     if (ctrl == false)
                        {
                        int x = stepArray[wordpos].x-1;
                        int y = stepArray[wordpos].y;
                        routine(x,y,wordpos);
                        }
                     }
                  break;
         case 8:  if ((stepArray[wordpos].x-1 >= 0)&&(stepArray[wordpos].y-1 >= 0))
                     {
                     ctrl = ctrlStAr(-1,-1, wordpos);
                     if (ctrl == false)
                        {
                        int x = stepArray[wordpos].x-1;
                        int y = stepArray[wordpos].y-1;
                        routine(x,y,wordpos);
                        }
                     }
                  break;    
         }

      }
};

bei break, springt er nicht aus switch raus in die schleife zurück sondern, geht einfach auf die nächste case-Bedingung zu, obwohl ich da break; stehen habe

greet johnc.

Mein Tipp als Erstes: Lege im switch-Block nur fest, ob und mit welchen Offsets später routine() aufgerufen werden soll und mache den Aufruf nach dem Block.

Gruß,

Simon2.

if(...)
//kein break; fehler
else
    break;

Kommt diese fehler immer noch

ctrlStepArray.obj : error LNK2005: "class datenbank db" (?db@@3Vdatenbank@@A) already defined in racer_main.obj

Nee die ganzen linkerfehler sind jetzt weg(dank extern), VielenDank!!! dafür

greetz johnc.

greetz johnC++

ich hab einfach nur dei dateien ins projekt "geaddet".

greetz johnC++

for (int i=stepArray[wordpos].iStep; i<=8; i++)

oh dann muss der fehler hier liegen.

Sorry, aber das ist Schwachsinn und mit 100% Sicherheit nicht das Problem... Und davon abgesehen ist im Zweifel ++i immer i++ vorzuziehen. Dazu gibt es aber einen eigenen Thread.

cu André

Sorry, aber das ist Schwachsinn und mit 100% Sicherheit nicht das Problem... Und davon abgesehen ist im Zweifel ++i immer i++ vorzuziehen. Dazu gibt es aber einen eigenen Thread.

1. Naja werden wir sehen!

2. Das ist ein C Array, diese beiden operatoren werden von Compiler
bereitgestellt ( ++i und i++ ), die anderen die du meinst sind die überladene
Operatoren der STL ( std::vector, std::map, ... ), dort gilt was du gesagt hast,
bei C Arrays ist es ein bischen anders.

zu 1.
Begründe doch bitte mal wieso der Fehler bei i++ liegt.

die anderen die du meinst sind die überladene
Operatoren der STL

... Wo sind hier Beim Inkrement/Deinkrement überladene Operatoren der STL (oder wo bitte ist hier das Thema STL gefallen)? Und das i kein Array ist, wurde ja auch schon erwähnt.

cu André

#include <iostream>
using namespace std;

int main(void)
{   

	for( int i = 0; i<=4; i++)
	{
		switch(i)
		{
		case 0:
			cout << "case 0" << endl;
			break;

		case 1:
			cout << "case 1" << endl;
			break;

		case 2:
			cout << "case 2" << endl;
			break;
		case 3:
			cout << "case 3" << endl;
			break;
		case 4:
			cout << "case 4" << endl;
			break;

		}
	}

	cout << endl << endl;

	for( int i = 0; i<=4; ++i)
	{
		switch(i)
		{
		case 0:
			cout << "case 0" << endl;
			break;

		case 1:
			cout << "case 1" << endl;
			break;

		case 2:
			cout << "case 2" << endl;
			break;
		case 3:
			cout << "case 3" << endl;
			break;
		case 4:
			cout << "case 4" << endl;
			break;

		}
	}

    return 0;
}