'n'a pas été déclaré dans cette étendue' erreur

Donc j'ai écrit ce programme simple pour calculer le jour d'une date à l'aide de l'algorithme de Gauss trouvé ici.

#include <iostream>
using namespace std;

//Using the Gaussian algorithm
int dayofweek(int date, int month, int year ){
    int d=date;
    if (month==1||month==2)
        {int y=((year-1)%100);int c=(year-1)/100;}
    else
        {int y=year%100;int c=year/100;}
    int m=(month+9)%12+1;
    int product=(d+(2.6*m-0.2)+y+y/4+c/4-2*c);
    return product%7;
}

int main(){
    cout<<dayofweek(19,1,2054);
    return 0;
}

C'est un programme très simple et ce qui est encore plus étonnant est la sortie.

:In function  dayofweek(int, int, int)’:
:19: warning:  unused variable ‘y’
:19: warning: unused variable ‘c’
:21: warning: unused variable ‘y’
:21: warning: unused variable ‘c’
:23: error: ‘y’ was not declared in this scope
:25: error: ‘c’ was not declared in this scope

Il dit que ma variable est inutilisée mais dit alors qu'il n'est pas déclaré? Quelqu'un pourrait-il me dire de quoi de mal.

source d'informationauteur cortex

  1. 9

    La portée d'une variable est toujours le bloc c'est à l'intérieur. Par exemple si vous faites quelque chose comme

    if(...)
{
     int y = 5; //y is created
} //y leaves scope, since the block ends.
else
{
     int y = 8; //y is created
} //y leaves scope, since the block ends.

cout << y << endl; //Gives error since y is not defined.

    La solution est de définir y en dehors de la si les blocs

    int y; //y is created

if(...)
{
     y = 5;
} 
else
{
     y = 8;
} 

cout << y << endl; //Ok

    Dans votre programme, vous devez déplacer la définition de y et c du si de blocs dans la plus haute portée. Votre Fonction serait alors ressembler à ceci:

    //Using the Gaussian algorithm
int dayofweek(int date, int month, int year )
{
    int y, c;
    int d=date;

    if (month==1||month==2)
    {
         y=((year-1)%100);
         c=(year-1)/100;
    }
    else
    {
         y=year%100;
         c=year/100;
    }
int m=(month+9)%12+1;
int product=(d+(2.6*m-0.2)+y+y/4+c/4-2*c);
return product%7;
}
  2. 3

    Voici un exemple simplifié sur la base de votre problème:

    if (test) 
{//begin scope 1
    int y = 1; 
}//end scope 1
else 
{//begin scope 2
    int y = 2;//error, y is not in scope
}//end scope 2
int x = y;//error, y is not in scope

    Dans la version ci-dessus, vous avez une variable appelée y qui est limitée à une portée de 1, et d'autre variable appelée y qui est limitée à la portée 2. Vous essayez alors de se référer à une variable nommée y après la fin de la ifet non de la variable y peut être vu parce qu'aucune variable existe dans ce champ d'application.

    Vous résoudre le problème en plaçant y dans le contexte le plus externe qui contient toutes les références:

    int y;
if (test) 
{
    y = 1; 
}
else 
{
    y = 2;
}
int x = y;

    J'ai écrit l'exemple simplifié composé de code pour le rendre plus clair pour vous de comprendre le problème. Vous devriez maintenant être en mesure d'appliquer le principe de votre code.

  3. 2

    Vous devez déclarer y et c en dehors de la portée de la if/else. Une variable est uniquement valable à l'intérieur de la portée, il est déclaré (et d'un champ est marqué par des { })

    #include <iostream> 
using namespace std; 
//Using the Gaussian algorithm 
int dayofweek(int date, int month, int year ){ 
int d=date; 
int y, c;
if (month==1||month==2) 
        {y=((year-1)%100);c=(year-1)/100;} 
else 
        {y=year%100;c=year/100;} 
int m=(month+9)%12+1; 
int product=(d+(2.6*m-0.2)+y+y/4+c/4-2*c); 
return product%7; 
} 

int main(){ 
cout<<dayofweek(19,1,2054); 
return 0; 
}
  4. 1

    Ici

    {int y=((year-1)%100);int c=(year-1)/100;}

    vous déclarez et initialisez les variables y, cmais vous n'avez pas utilisé à tous avant de courir hors de portée. C'est pourquoi vous obtenez le unused message.

    Plus tard en la fonction, y, c sont non déclaré, parce que les déclarations que vous faites seulement le maintenir à l'intérieur du bloc elles ont été faites dans (le bloc entre accolades {...}).

  5. -2
    #include <iostream>
using namespace std;
class matrix
{
int a[10][10],b[10][10],c[10][10],x,y,i,j;
public :
void degerler();
void ters();
};
void matrix::degerler()
{
cout << "Satırları giriniz: "; cin >> x;
cout << "Sütunları giriniz: "; cin >> y;
cout << "İlk matris elamanlarını giriniz:\n\n";
for (i=1; i<=x; i++)
{
for (j=1; j<=y; j++)
{
cin >> a[i][j];
}
}
cout << "İkinci matris elamanlarını giriniz:\n\n";
for (i=1; i<=x; i++)
{
for (j=1; j<=y; j++)
{
cin >> b[i][j];
}
}
}
void matrix::ters()
{
cout << "matrisin tersi\n";
for (i=1; i<=x; i++)
{
for (j=1; j<=y; j++)
{
if(i==j)
{
b[i][j]=1;
}
else
b[i][j]=0;
}
}
float d,k;
for (i=1; i<=x; i++)
{
d=a[i][j];
for (j=1; j<=y; j++)
{
a[i][j]=a[i][j]/d;
b[i][j]=b[i][j]/d;
}
for (int h=0; h<x; h++)
{
if(h!=i)
{
k=a[h][j];
for (j=1; j<=y; j++)
{
a[h][j]=a[h][j]-(a[i][j]*k);
b[h][j]=b[h][j]-(b[i][j]*k);
}
}
count << a[i][j] << "";
}
count << endl;
}  
}
int main()
{
int secim;
char ch;    
matrix m;
m.degerler();
do
{
cout << "seçiminizi giriniz\n";
cout << " 1. matrisin tersi\n";
cin >> secim;
switch (secim)
{
case 1:
m.ters();
break;
}
cout << "\nBaşka bir şey yap/n?";
cin >> ch;
}
while (ch!= 'n');
cout << "\n";
return 0;
}