Thread: simple matrix

Read the row and value from the user before allocating memory for the matrix. Just don't tell the user what you're doing.

Is this what you meant?
Its still not compiling

Code:

#include<stdio.h>
#include <stdlib.h>


int main (void)
{
  int row_size1,col_size1,row_size2,col_size2;
  int **A;
  int i = 0, j = 0;


  //allocate space for 1 matrix
  A = malloc(1 * sizeof (int *));


     /* printf("Enter the number of rows and columns:");
  scanf("%d %d",&row_size1,&col_size1);


  A= malloc(row_size1*sizeof(int*));
  for(i=0;i<row_size1;i++)
    {
      A[i]=(int*)malloc(col_size1*sizeof(int));
    }
  */
  printf("Enter the elements of the first matrix write 99 when done:");
  row_size1= A[0][0];
  col_size1= A[0][1];


  A= malloc(row_size1*sizeof(int*));
  for(i=0;i<row_size1;i++)
    {
      A[i]=(int*)malloc(col_size1*sizeof(int));


    }




  for(i=0;i<row_size1;i++)
    {
      for(j=0;j<col_size1;j++)
        {
          scanf("%d",&A[i][j]);
          if (A[i][j] == 99) // 'x' is character variable I declared to \
 break;
        }
    }


printf("The required matrix is\n");
for(i=0;i<row_size1;i++)
    {
      for(j=0;j<col_size1;j++)
        printf("%d ",A[i][j]);
      printf("\n");
    }
 free(col_size1);
    }

Also could you please explain what you meant by not casting malloc

Wait.. so the problem I have is if I uncomment what I have commented how will the program know what row_size1 and col_size1 are?
This is what I think you mean

Code:

#include<stdio.h>
#include <stdlib.h>


int main (void)
{
  int row_size1,col_size1,row_size2,col_size2;
  int **A;
  int i = 0, j = 0;




  printf("Enter the elements of the first matrix write 99 when done:");
  row_size1= A[0][0];
  col_size1= A[0][1];


  A= malloc(row_size1*sizeof(int*));
  for(i=0;i<row_size1;i++)
    {
      A[i]=malloc(col_size1*sizeof(int));


    }
  //allocate space for 1 matrix
   A = malloc(1 * sizeof (int *));

for(i=0;i<row_size1;i++)
    {
      for(j=0;j<col_size1;j++)
        {
          scanf("%d",&A[i][j]);
          if (A[i][j] == 99) // 'x' is character variable I declared to \
            break;
        }
    }


printf("The required matrix is\n");
for(i=0;i<row_size1;i++)
 {
      for(j=0;j<col_size1;j++)
        printf("%d ",A[i][j]);
      printf("\n");
    }
 free(col_size1);
}

p.s please bear with me I'm a bit slow but I really want to learn this stuff

I dont want to seperately ask the user for the number of rows and columns. I want the program to run like this

Code:

printf("Enter the elements of the first matrix write 99 when done:");
(user) 2 2 1 2 3 4 

then I want 2 and 2 to be the number of rows and columns and 1 2 3 4 to be the 2 by 2 matrix printed

Originally Posted by zafy

I dont want to seperately ask the user for the number of rows and columns.

Then don't ask the user. Just read the row and column using scanf. You don't have to tell the user that you're reading the row and column before you allocate memory and run through the loop for reading each value in the matrix.

Isnt that what I'm doing when I am writing this in my code
row_size1= A[0][0]; //first element is row
col_size1= A[0][1];

No. Read two integers into separate variables. The matrix can't be allocated until you know how big to make it, so you can't use A[0][0] and A[0][1] until you get the size from the user.

I was pointing out a memory leak in the code you had provided - If you want to not do it that way any more, that's ok.

Thank you for the memory leak. I kind of mentioned in the beginning that this was the trick in the program I wasn't able to do. Sorry I dodnt mean to confuse you

I dont get how to read 2 integers from 6 integers that the user has provided without using A[0][1].
Could you demonstrate it please

It looks like you already know how to read two integers from the user:

Code:

scanf("%d %d",&row_size1,&col_size1);

(I took this line from your post from an hour ago)