Thread: Passing an array to a function

  1. #1
    Registered User
    Join Date
    Feb 2010
    Posts
    15

    Passing an array to a function

    Im trying to pass the array A into the function 'determinant' in order to calculate its determinant. However I keep getting the following error: cannot convert `float (*)[((unsigned int)((int)x))]' to `float (*)[2]' for argument `1' to `float determinant(float (*)[2])'

    Here is my code:

    #include <stdio.h>

    float determinant(float a[2][2]);

    main ()
    {
    int col, row, x=2, y=2;
    float A[y][x], B[y],detA;


    printf("Enter Matrix A: ");

    for(row=0 ; row<y; row++)
    {
    for(col=0 ; col<x ; col++)
    scanf("%f", &A[row][col]);
    }

    printf("You entered:\n");

    for(row=0 ; row<y; row++)
    {
    for(col=0 ; col<x ; col++)
    printf("[%f]", A[row][col]);
    putchar('\n');
    }

    printf("Enter Matrix B: ");

    for(row=0 ; row<x; row++)
    scanf("%f", &B[row]);


    printf("You entered:\n");

    for(row=0 ; row<y; row++)
    printf("[%f]\n", B[row]);

    detA = determinant(A);
    printf("%f", detA);

    }


    /* Calculate determinant of matrix A */

    float determinant(float a[2][2])
    {
    float det;
    det = a[0][0]*a[1][1] - a[0][1]*a[1][0];
    return det;
    }


    Im editing and compiling using Dev-C++ 4.9.9.2
    Any help would be great, Thanks.

  2. #2
    C++ Witch laserlight's Avatar
    Join Date
    Oct 2003
    Location
    Singapore
    Posts
    28,413
    The problem could be that you are inadvertently using variable length arrays. Examine:
    Code:
    int col, row, x=2, y=2;
    float A[y][x], B[y],detA;
    x and y are not constants, yet you use them to create A and B. Therefore, A and B are variable length arrays. As a quick fix, you could write:
    Code:
    float A[2][2], B[2], detA;
    Quote Originally Posted by Bjarne Stroustrup (2000-10-14)
    I get maybe two dozen requests for help with some sort of programming or design problem every day. Most have more sense than to send me hundreds of lines of code. If they do, I ask them to find the smallest example that exhibits the problem and send me that. Mostly, they then find the error themselves. "Finding the smallest program that demonstrates the error" is a powerful debugging tool.
    Look up a C++ Reference and learn How To Ask Questions The Smart Way

  3. #3
    Registered User
    Join Date
    Feb 2010
    Posts
    15
    Ah you're right, it works now when I replace x and y with 2.
    Thanks

  4. #4
    Registered User
    Join Date
    Feb 2010
    Posts
    6
    i would of used a pointer for it and then passed the length of the x and y.

  5. #5
    C++まいる!Cをこわせ!
    Join Date
    Oct 2007
    Location
    Inside my computer
    Posts
    24,654
    Quote Originally Posted by Adak View Post
    io.h certainly IS included in some modern compilers. It is no longer part of the standard for C, but it is nevertheless, included in the very latest Pelles C versions.
    Quote Originally Posted by Salem View Post
    You mean it's included as a crutch to help ancient programmers limp along without them having to relearn too much.

    Outside of your DOS world, your header file is meaningless.

  6. #6
    Registered User
    Join Date
    Feb 2010
    Posts
    15
    Mostly the same code as before but I'm trying to use functions and calling by reference to input the matrices. But I'm still getting a similar error as before, and I can't see where I'm going wrong. Thanks.

    12 cannot convert `float (*)[2][2]' to `float* (*)[2]' for argument `1' to `void getmatrixA(float* (*)[2])'

    13 cannot convert `float (*)[2]' to `float**' for argument `1' to `void getmatrixB(float**)'

    Code:
    #include <stdio.h>
    
    void getmatrixA(float *A[2][2]);
    void getmatrixB(float *B[2]);
    float determinant(float a[2][2]);
    
    main ()
    {
         int col, row, x=2, y=2;
         float A[2][2], B[2], Ak[2][2], detA; 
         
         getmatrixA(&A);
         getmatrixB(&B);
         
         detA = determinant(A);
         printf("Determinant of A: %f\n", detA);
         
         for(row=0 ; row<y; row++)
         {
          for(col=0 ; col<x ; col++)
           {
            if(col=0)
            Ak[row][col]=B[row];
            else
            Ak[row][col]=A[row][col];
           }
         }
    
         printf("Ak:\n");
      
         for(row=0 ; row<y; row++)
         {
          for(col=0 ; col<x ; col++)
            printf("[%f]", Ak[row][col]);
          putchar('\n');
         }
    }
    
    void getmatrixA(float *A[2][2])
    {    
         int col, row;
         
         printf("Enter Matrix A: ");
         
         for(row=0 ; row<2; row++)
         { 
          for(col=0 ; col<2 ; col++)
            scanf("%f", &A[row][col]);
         }
    
         printf("You entered:\n");
      
         for(row=0 ; row<2; row++)
         {
          for(col=0 ; col<2 ; col++)
            printf("[%f]", A[row][col]);
          putchar('\n');
         }
    }
    
    void getmatrixB(float *B[2])
    {
         int col, row;
         
         printf("Enter Matrix B: ");
         
         for(row=0 ; row<2; row++)
          scanf("%f", &B[row]);
         
    
         printf("You entered:\n");
      
         for(row=0 ; row<2; row++)
            printf("[%f]\n", B[row]);
    }
          
    
    /* Calculate determinant of matrix A */ 
    
    float determinant(float a[2][2])
    {
      float det;          
      det = a[0][0]*a[1][1] - a[0][1]*a[1][0];
      return det;
    }

  7. #7
    C++まいる!Cをこわせ!
    Join Date
    Oct 2007
    Location
    Inside my computer
    Posts
    24,654
    The types are wrong. You are trying to send x by y arrays to the functions, whose type should be float [x][y], not float* [x][y]. Furthermore, don't pass the address of the array. That would create a pointer to an array, float (*)[x][y], which is not the same as a pointer to the first element.
    Quote Originally Posted by Adak View Post
    io.h certainly IS included in some modern compilers. It is no longer part of the standard for C, but it is nevertheless, included in the very latest Pelles C versions.
    Quote Originally Posted by Salem View Post
    You mean it's included as a crutch to help ancient programmers limp along without them having to relearn too much.

    Outside of your DOS world, your header file is meaningless.

Popular pages Recent additions subscribe to a feed

Similar Threads

  1. Compiling sample DarkGDK Program
    By Phyxashun in forum Game Programming
    Replies: 6
    Last Post: 01-27-2009, 03:07 AM
  2. Replies: 7
    Last Post: 11-21-2008, 04:27 PM
  3. function passing argument..array ?
    By jochen in forum C Programming
    Replies: 2
    Last Post: 09-30-2007, 11:53 AM
  4. Problem with Visual C++ Object-Oriented Programming Book.
    By GameGenie in forum C++ Programming
    Replies: 9
    Last Post: 08-29-2005, 11:21 PM
  5. Interface Question
    By smog890 in forum C Programming
    Replies: 11
    Last Post: 06-03-2002, 05:06 PM