Место записи о нарушении прав доступа ... Где ошибка?(Диаграмма Вейча) - PullRequest
Новая
       53

Место записи о нарушении прав доступа ... Где ошибка?(Диаграмма Вейча)

0 голосов
/ 23 марта 2012

ПРИМЕЧАНИЕ: у меня было две переменные с одинаковым именем ... Большое спасибо Стефану Бирладяну и Хенрику за то, что заметил это! *

Недавно я начал писать код, который помогает мне вводитьзначения функции bool для диаграммы Вейча (Карно) с 4 переменными.Код должен записывать элементы в матрицу размером 4x4, но с такими индексами:

  1. element - индекс 3,3
  2. element - индекс 2,3
  3. element - index3,2
  4. элемент - индекс 2,2
  5. элемент - индекс 0,3
  6. элемент - индекс 1,3
  7. элемент - индекс 0,2
  8. элемент - индекс 1,2
  9. элемент - индекс 3,0
  10. элемент - индекс 2,0
  11. элемент - индекс 3,1
  12. элемент - индекс 2,1
  13. элемент - индекс 0,0
  14. элемент - индекс 1,0
  15. элемент - индекс 0,1

  16. элемент - индекс 1,1 Это код main (): 

        void main()
    {
        int n;

    n=4;

    int **VeitchDiagram;

    //allocate memory for Veitch diagram
    VeitchDiagram = new int *[n];
    for(int i=0; i<n; i++)
        VeitchDiagram[i]=new int [n];

    //enter the elements
    for(int i=0; i<n; i++)
    {
        int j, k;
        if(i%2==1)
        {
            k=0;
            if(i<2)
                j=4;
            else
                j=-1;
            for(int k=0; k<2; k++)
            {
                if(i<2)
                    j--;
                else
                    j++;
                cin >> VeitchDiagram[k][j];     //this part writes the input to elements with index (at least it should do that):
                k++;                            //0,3     1,3     0,2     1,2     if i%2==1 and i<2
                cin >> VeitchDiagram[k][j];     //0,0     1,0     0,1     1,1     if i%2==1 and i>=2
                k--;
            }
        }
        else
        {
            k=3;
            if(i<2)
                j=4;
            else
                j=-1;
            for(int k=0; k<2; k++)
            {
                if(i<2)
                    j--;
                else
                    j++;
                cin >> VeitchDiagram[k][j];     //this part writes the input to elements with index (at least it should do that):
                k--;                            //3,3     2,3     3,2     2,2    if i%2==0 and i<2
                cin >> VeitchDiagram[k][j];     //3,0     2,0     3,1     2,1    if i%2==0 and i>=2
                k++;
            }
        }
    }

    //free memory allocated for VeitchDiagram
    for(int i=0; i<n; i++)
        delete [] VeitchDiagram[i];
    delete [] VeitchDiagram;
}

Ответы [ 3 ]

2 голосов
/ 23 марта 2012

для i = 0 вы достигнете этой ветви 

else
            {
                k=3;
                if(i<2)
                    j=4;
                else
                    j=-1;
                for(int k=0; k<2; k++)
                {
                    if(i<2)
                        j--;
                    else
                        j++;
                    cin >> VeitchDiagram[k][j];     //this part writes the input to elements with index (at least it should do that):
                    k--;                            //3,3     2,3     3,2     2,2    if i%2==0 and i<2
                    cin >> VeitchDiagram[k][j];     //3,0     2,0     3,1     2,1    if i%2==0 and i>=2
                    k++;
                }
            }

когда k = 0 

cin >> VeitchDiagram[k /* = 0  OK */][j];     //this part writes the input to elements with index (at least it should do that):
                    k--; //decrease it                            //3,3     2,3     3,2     2,2     if i%2==0 and i<2
                    cin >> VeitchDiagram[k /* here k = -1 BAD!!! */][j];     //3,0     2,0     3,1     2,1    if i%2==0 and i>=2
                    k++;
2 голосов
/ 23 марта 2012 
        for(int k=0; k<2; k++)
        {
            if(i<2)
                j--;
            else
                j++;
            cin >> VeitchDiagram[k][j];     //this part writes the input to elements with index (at least it should do that):
            k--;                            //3,3     2,3     3,2     2,2    if i%2==0 and i<2
            cin >> VeitchDiagram[k][j];     //3,0     2,0     3,1     2,1    if i%2==0 and i>=2
                                 ^ k == -1

Но вы действительно должны научиться пользоваться отладчиком.

1 голос
/ 23 марта 2012

Как уже было отмечено, вы индексируете за пределами массива.
Так же, как совет, табличная версия может быть менее сложной, чтобы получить правильную информацию: 

const size_t k_index[] = {3,2,3,2,0,1,0,1,3,2,3,2,0,1,0,1};
const size_t j_index[] = {3,3,2,2,3,3,2,2,0,0,1,1,0,0,1,1};

int main()
{
    const int n = 4;
    int VeitchDiagram[n][n]; // No need for dynamic allocation here.

    //enter the elements
    for(int i = 0; i < n * n; i++)
    {
        cin >> VeitchDiagram[k_index[i]][j_index[i]];
    }
}

Это пара строккороче.

...