Ошибка сегментирования

[_PROGRAMM_]

Есть реализованный класс, который отвечает за рисование круга, окружности.
Хедер:

class CircleCos2D
{
   public:
    CircleCos2D(double s, double r);
    void Reshape(double s,double r);
    void Paint();
    void Coords(float x, float y);
   private:
    int      count;
    double   r;
    double   xc,yc;
    double * x;
    double * y;
};

Описание

CircleCos2D::CircleCos2D(double s,double r)
{
    x=new double[1]; // Создаем динамический массив, чтобы небыло ошибки
    y=new double[1]; // сегментирования
    if ((double)r/(double)s<0.6)
    {
      printf("Отношение радиуса к отрезку превышает критический минимум");
    }
    Reshape(s,r);
}

void CircleCos2D::Reshape(double s,double r)
{
    delete[] x;
    delete[] y;
    double angle = acos((r*r+r*r-s*s)/(2*r*r));
    count = ((double)pi*2.0/(double)angle)+1;
    x=new double[count];
    y=new double[count];
    int ct = count/2;
    for (int i=0;i<=ct;i++)
    {
       x[i]=r*cos(angle*i);
       x[i+ct]=-x[i];
       y[i]=r*sin(angle*i);
       y[i+ct]=-y[i];
    }
   
}

void CircleCos2D::Paint()
{
    glBegin(GL_POLYGON);
     for (int i=0;i<=count;i++)
     {
         glVertex2d(x[i]+xc,y[i]+yc);
     }
     glVertex2d(x[0]+xc,y[0]+yc);
    glEnd();
  
}
void CircleCos2D::Coords(float x, float y)
{
    xc=x;
    yc=y; 
}

Я создаю указатель на класс(в моем коде ниже это "cr") и соответственно использую его. При вращении колеса мыши, круг масштабируется.

if((button==4))
        {
          Env.r*=1.01;
          cr->Reshape(0.01,Env.r);
        }

На cr->Reshape(0.01,Env.r); программа валится, а так как она в полноэкранном режиме, все зависает(Диспетчер задач не может ничего закрыть). Если закомментировать, то никаких ошибок не наблюдается, но и масштаб объекта, главное в приложении, не работает Самое удивительное, что эта программа полностью работоспособна на ОС с ядром Linux. Все это возникло из-за того, что я захотел перенести ее на Windows. Написано с использованием GLUT. Кроссплатформенно компилируется, но на Windows, как я говорил, вылетает.

Еще один факт. В конструкторе класса также вызывается Reshape, но приложение не вылетает. Вызов конструктора происходит в main.

P.S. пользуясь случаем, хотелось бы узнать о существовании русских информационных сайтов или статей о OpenGL 3+.

[Shad0wF1rst]

На cr->Reshape(0.01,Env.r); программа валится, а так как она в полноэкранном режиме, все зависает(Диспетчер задач не может ничего закрыть). Если закомментировать, то никаких ошибок не наблюдается, но и масштаб объекта, главное в приложении, не работает Самое удивительное, что эта программа полностью работоспособна на ОС с ядром Linux. Все это возникло из-за того, что я захотел перенести ее на Windows. Написано с использованием GLUT. Кроссплатформенно компилируется, но на Windows, как я говорил, вылетает.

Приведенный код класса это хорошо, но хотелось бы видеть больше кода предшествующего тому месту где у вас валится приложение.

P.S. В конструкторе класса вместо того что бы выделять 1 байт массива динамически, просто приравняй указатели нулю и ошибки сегментирования не будет при удалении нулевого указателя.

[_PROGRAMM_]

Приведенный код класса это хорошо, но хотелось бы видеть больше кода предшествующего тому месту где у вас валится приложение.

Как-то так. На Linux ведь работает! Причем отлично.

void MouseDown(int button,int state,int x, int y)
{
     Xf = XScreen*2*(float)x/(float)width-(XScreen);
     Yf = -(YScreen*2*(float)y/(float)height-(YScreen));

     if (button==GLUT_RIGHT_BUTTON)
     {
        bool select=false;
        for (int i=0;i<=count_object;i++)
        {
           double s = sqrt(pow(Planet[i]->Pos.x-Xf,2)+pow(Planet[i]->Pos.y-Yf,2));
           if (Planet[i]->r > s)
           {
             select_object=i;
             select=true;
           }
        }
        if (select==false)
        select_object = -1;
     }

     if((button==GLUT_LEFT_BUTTON)&&(state==GLUT_DOWN)&&(select_object!=-1))
     {
          Planet[select_object]->Speed.x+=(Xf-Planet[select_object]->Pos.x)/200.0;
          Planet[select_object]->Speed.y+=(Yf-Planet[select_object]->Pos.y)/200.0;

          if (select_object>count_object)
          {
             Planet[select_object]->m       = Env.p*rpi*pow(Env.r,3);
             count_object++;
          }
     }

     if((keys[78]==true)||(keys[110]==true))
     {

        if((button==GLUT_LEFT_BUTTON)&&(state==GLUT_DOWN))
        {
           for(int i=0;i<=count_object;i++)
           {
             printf("Номер %i \n",i);
             printf("Масса    %f\n",Planet[i]->m);                 
             printf("Скорость X:%f.3 Y:%f.3\n",Planet[i]->Speed.x,Planet[i]->Speed.y);      
             printf("Позиция  X:%f.3 Y:%f.3\n\n",Planet[i]->Pos.x,Planet[i]->Pos.y);      
           }
           int s = count_object+1;
           Planet[s]               = new Body(Env.r);
           Planet[s]->Pos.x        = Xf;
           Planet[s]->Pos.y        = Yf;
           select_object=s;
        }
        if((button==4))
        {
          Env.r*=1.01;
          cr->Reshape(0.01,Env.r);
        }
        if((button==3)&&(Env.r>0.1))
        {
          Env.r/=1.01;
          cr->Reshape(0.01,Env.r);
        }
     }
     else
     {
        if(button==4)
          Env.dScale*=1.05;
        if(button==3)
          Env.dScale/=1.05;
        Reshape(width,height);
     }
     
}

P.S. В конструкторе класса вместо того что бы выделять 1 байт массива динамически, просто приравняй указатели нулю и ошибки сегментирования не будет при удалении нулевого указателя.

Будет сделано. Спасибо.

---

Вот еще факт.
Инициализация класса у меня происходит в main вот так.

cr = new  CircleCos2D(0.01,0.1);

Если добавить строку с пересчетом

cr = new  CircleCos2D(0.01,0.1);
	cr->Reshape(0.01,Env.r);

, то сразу же вылетает. Замечу, что поставил я их в самое начало main. Скриншот прилагается.

[Shad0wF1rst]

Я так и не нашел чему же все таки равно начальное значение Env.r. Думаю в нем дело. Так как вы в конструкторе вызываете свою функцию:

CircleCos2D::CircleCos2D(double s,double r)
{
    x=new double[1]; // Создаем динамический массив, чтобы небыло ошибки
    y=new double[1]; // сегментирования
    if ((double)r/(double)s<0.6)
    {
      printf("Отношение радиуса к отрезку превышает критический минимум");
    }
    Reshape(s,r);
}

И оно реагирует нормально когда вы задаете числа, и у вас возникает ошибка только когда вы пытаетесь второй аргумент передать Env.r

[Igor95]

С чего это Вы взяли, что при использовании динамической памяти не будет ошибки сегментирования?

void CircleCos2D::Reshape(double s,double r)
{
    delete[] x;
    delete[] y;
    double angle = acos((r*r+r*r-s*s)/(2*r*r));
    count = ((double)pi*2.0/(double)angle)+1;
    x=new double[count];
    y=new double[count];
    int ct = count/2;
    for (int i=0;i<=ct;i++)
    {
       x[i]=r*cos(angle*i);
       x[i+ct]=-x[i];
       y[i]=r*sin(angle*i);
       y[i+ct]=-y[i];
    }
   
}

Вот здесь подозрительно. Проверьте, а не выходите ли Вы за пределы выделенной памяти под массив.

[_PROGRAMM_]

Проблему решил, смотрите конец сообщения.

Сделал так

Env.r=Env.r * 1.01;
printf("%f\n",Env.r);
//cr->Reshape(0.01,Env.r);

Ошибки нет. Env.r выводится исправно (см. вложение). Еще когда отписывался, такая мысль возникла. Я провел несколько экспериментов и был уверен, что проблема в Env.r.

С чего это Вы взяли, что при использовании динамической памяти не будет ошибки сегментирования?

1. Потому что в Linux ее нет.
2. Я такого не говорил.

Вот здесь подозрительно. Проверьте, а не выходите ли Вы за пределы выделенной памяти под массив.

Будем смотреть...

А теперь самое интересное. Код в начале

cr = new  CircleCos2D(0.01,0.1);
cr->Reshape(0.01,Env.r);

вылетал из-за того, что я не инициализировал Env.r. После инициализации он работает. НО!
Не соответствует никакой логике:
Этот код работает:

cr = new  CircleCos2D(0.01,0.1);
	cr->Reshape(0.01,Env.r);
	cr->Reshape(0.01,0.1);
	cr->Reshape(0.01,0.1);
	cr->Reshape(0.01,0.1);
	cr->Reshape(0.01,0.1);
	cr->Reshape(0.01,0.1);
	cr->Reshape(0.01,0.1);
	cr->Reshape(0.01,0.1);

Этот тоже:

cr = new  CircleCos2D(0.01,0.1);
	cr->Reshape(0.01,Env.r);
	cr->Reshape(0.01,0.1);
	cr->Reshape(0.01,0.1*1.01);

Этот НЕТ!

cr->Reshape(0.01,Env.r);
	cr->Reshape(0.01,0.1);
	cr->Reshape(0.01,0.1*1.01);
	cr->Reshape(0.01,0.1*1.01*1.01);

А этот опять работает

cr = new  CircleCos2D(0.01,0.1);
	cr->Reshape(0.01,Env.r);
	cr->Reshape(0.01,0.1*1.01*1.01);

Такой тоже функционирует

cr = new  CircleCos2D(0.01,0.1);
	cr->Reshape(0.01,Env.r);
	cr->Reshape(0.01,0.1*1.01*1.01);
	cr->Reshape(0.01,0.1*1.01*1.01);
	cr->Reshape(0.01,0.1*1.01*1.01);
	cr->Reshape(0.01,0.1*1.01*1.01);

Проблема все-таки в классе. Т.к. другие элементы структуры Env исправно работают, храня масштаб сцены и шкалу.

---

Проверьте, а не выходите ли Вы за пределы выделенной памяти под массив.

Вы были правы. На один элемент запрыгнул. Получается g++ сам за меня округлил ct в меньшую сторону, а вот Microsoft, к сожалению, не смог.

Shad0wF1rst и Igor95, огромное спасибо, я вам очень благодарен.

[Igor95]

Пожалуйста