expect procedure name in procedure call(newbie)

by R » Thu, 30 Dec 2004 19:37:03 GMT

Hello.

I've got 'expect procedure name in procedure call' warning but
I think my code is good
Inside testclass.adb I have Create function and when I'm trying to call
it from
main.adb unit I receive that error.

Below are full codes of my Ada units.

And by the way - how can I dynamically allocate memory for e.g. 10
elements(array of Floats)?
How can I reallocate them to 20 elements or 4?
How can I free the memory?

thanks in advance
best regards R

Codes:

main.adb:
--------
with testclass;
procedure Main is
object : testclass.rec1_Access;
begin
testclass.Create(object, 10);
end Main;

testclass.ads:
--------------
package testclass is
type rec1 is tagged private;
type rec1_Access is access rec1;
function Create(this: rec1_Access; s: Integer) return Integer;
private
type rec1 is tagged record
field: Integer;
end record;
end testclass;

testclass.adb:
--------------
package body testclass is
function Create(this: rec1_Access; s: Integer) return Integer is
begin
this.field :=s;
		return this.field;
	end Create;
end testclass;

by **Florian Weimer** » Thu, 30 Dec 2004 21:07:30 GMT

> testclass.Create(object, 10);

You are ignoring the return value of Create, which is not allowed.

by **Martin Dowie** » Thu, 30 Dec 2004 21:32:09 GMT



In Ada you don't need to play around with memory 
allocation/reallocation/deallocation as much in other lower level 
languages. If you know in advance the maximum number you are going to 
have then you could use a discriminated array, e.g.

    type Number_Of_Floats is range 0 .. 20;
    --  Never going to be more than 20.

    type Array_Of_Floats is array (Number_Of_Floats range <>) of Float;

    type Group_Of_Floats (Length : Number_Of_Floats := 0) is record
       F : Array_Of_Floats (1 .. Length);
    end record;

    My_Floats : Group_Of_Floats;
begin
    ...

If you really don't know the number of items you are going to have then 
use one of the many container libraries available - you wouldn't 
re-invent the wheel and re-code the STL if you were using C++ would you?

Here is a link to a page of such libraries:

 http://www.**--****.com/ +Libraries

Look under "Data Types".

Cheers

-- Martin

by **Martin Dowie** » Thu, 30 Dec 2004 21:34:35 GMT



Typo - that should be a "discriminated record".

by R » Thu, 30 Dec 2004 22:03:24 GMT

OK so i added new variable ret(Integer) and added:
ret := testclass.Create(object, 10);

but now Create raises an expception: CONSTRAINT_ERROR : testclass.adb:5
access check failed

where the Create looks like:
function Create(this: rec1_Access; s: Integer) return Integer is
begin
this.field := s; -- this is the 5th line - access check error
return this.field;
end Create;

basicly  the codes are the same from my first post - I only folow Your
instuction not to ignore the return value.

Is my access type wrong? it point to rec1 tagged record.
thanks in advance for Your help
best regards R

by **Mark Lorenzen** » Thu, 30 Dec 2004 22:24:45 GMT

"R" < XXXX@XXXXX.COM > writes:


The parameter 'this' is of an access type. As the actual parameter you
pass a non-initialised variable. In Ada, all variables of access type
have the value 'null' by default. So you pass 'Create' a null value
and the dereference it.

Your example seems to be a translation of a C++ function. I would do
the following instead:

1) Make 'Create' a procedure instead of a function with the following
   signature:

   Create (This : out Rec1; S : in Integer);

2) procedure Main is
     Object : Testclass.Rec1;
   begin
     Testclass.Create(Object, 10);
   end Main;

Do not fiddle around with pointers as much as you do. It seems like
you try to program in Ada the C++ way.

- Mark Lorenzen

by **Martin Krischik** » Fri, 31 Dec 2004 03:46:49 GMT




If you learn for Hobby then I  suggest a container lib with support for
indefinite object:

 http://www.**--****.com/ :Ada:Libraries:MultiPurpose:AdaCL#Components.

If not you have to do it the hard way:

 http://www.**--****.com/ :Ada:Types:access


You call the parameter this - but Create is a "static" method. Read:

 http://www.**--****.com/ :Ada:OO#Primitive_operations


your are aware that this is null?


-- 
mailto:// XXXX@XXXXX.COM 
 http://www.**--****.com/

expect procedure name in procedure call(newbie)

expect procedure name in procedure call(newbie)

Jump to:

Next

expect procedure name in procedure call(newbie)

Re: expect procedure name in procedure call(newbie)

Re: expect procedure name in procedure call(newbie)

Re: expect procedure name in procedure call(newbie)

Re: expect procedure name in procedure call(newbie)

Re: expect procedure name in procedure call(newbie)

Re: expect procedure name in procedure call(newbie)

Similar Threads:

Who is online