How to write code based on the typedef? thread.

Yup got it

SID = n1 * 2^16 + n2 * 2^8 + n3;

Also Vulpes can you tell me a possible solution for the last post which we were discussing that UINT32 was holding a string. I have multiple data types which will be sent from another function of which one of them is hex value whose max value can be 4 bytes and the other type is string. so what can i do for this type of scenario? Possible i should end up with something by tomorrow. Hope you will come up with some possible solution.

if there is any solution then state some example how that can be implemented.

Sam Hobbs i am not kidding seriously it's getting compiled. If you want i will upload you the whole code.  I just copy pasted some part of the code not the whole. there are lot of files in the project. if needed i can send you all of them. And if it comes to declaration of concat_string and static string i have declared in some other place which i haven't posted in the last post. all the code which i wrote is giving the intended results apart from small issues which had with the pointers and the one which you and Vulpes pointed out.

When you write code the way you are, it is extremely easy to overlook problems. That is the main point that I want to make clear.

As best as I can see, about the line:


I think that Vulpes just said "I'm wondering how this line is even compiling".

I tried to explain what would happen if that code was to compile. I still doubt that it compiles. If it compiles for you then there is something you are not telling us. Are you posting from another universe?

Note that I was wrong; it is taking a while for me to get back into the C++ mode. In that call to strlen the bit_format_ref would be de-referenced by the * so that the contents of bit_format_ref would be used as an address and the data at that address would be searched.

Your latest code does not compile. The line shown above does not compile. Note that bit_format_ref is not even initialized in that code. Also concat_string is not declared. The declaration of static_string will not work; you will get compiler errors from it too.

Yup i just noticed it after both(Sam and Vulpes) of you pointed it out. I know it's a blunder, i couldn't even believe that i did it. probably i over looked i guess. But i don't know whether it's a bug from visual studio or whatever but that line is giving me exact length of the string. I wonder how is that happening if UINT32 cannot store any char values. Probably both of you needs to have a serious look into that. Below is the code which got compiled and giving the exact length of the string and final output which i was intended to get is also fine.
I kept a break point and checked the value of the size as well, it's giving the absolute value. After working too hard yesterday night i made it working finally. But any possible issues being a bad programmer in C especially, you should be able to clear them out,

Please look at the following code:

        UINT32 * bit_format_ref;
        UINT8 Size;
        UINT32 *temp;
        UINT8 static_string[32] = {0};
        UINT8 array_index = HMI_STRLEN(concat_string);
        

        Size = strlen(*bit_format_ref);
        temp = *(bit_format_ref);
        sprintf(static_string, "%s", temp);
        len = 0;
        while(static_string[len] != '\0')
        {
            if(array_index <= 31)
            {
                concat_string[array_index++] = static_string[len++];
            }
        }

Karthik, I have no doubt whatsoever that I could solve your prolems, but you must be willing to satisfy your requirements using more professional code. Note that I am not saying that you must use good code, even if it means not satisfying requirements.

I am saying that I will help you in a manner that your are more likely to get when you ask for assistance in a site where there are many developers experienced with C++. Most experienced programmers would choke on the use of a UINT32* in a strlen. I truly doubt that you must do that.

A UINT32* is a pointer to a 4-byte integer. I am surprised that strlen(*bit_format_ref) compiles if bit_format_ref is a UINT32*, but I will assume that that is the declaration of bit_format_ref and the code compiles. Note that for Intel architecture processors that most Windows implementations use, integers are stored in reverse order. If bit_format_ref is a UINT32* then *bit_format_ref would be the UINT32; the 32-bit double-word. If the UINT32 has any bytes that are zero (NULL) then strlen ends there. Otherwise strlen will go through memory until it either gets a memory access violation or gets a byte with zero.

I've stared at this on several occasions today in the hope of some inspiration and I've alighted on this line:

Size = strlen(*bit_format_ref);

Given that the strlen function takes a const char* argument, I'm wondering how this line is even compiling given that it's been passed a UINT32 which is an unsigned long?

If you're trying to get the length of the bit_format_ref array, then you won't be able to do it like this.

Either you'll need to pass the length of the array as a separate parameter when you pass the array or you'll need to add a further 'null' element at the end of the array. You can then iterate through it until you get to the 'null' to calculate its length.

Before you do anything else, I'd try doing a:

printf("%d\n", Size);

to see what value is being assigned to Size. If it's not the value you're expecting, then I'd try assigning that value explicitly to Size and seeing whether it works properly then.

i tried so many ways but i couldn't find the answer for accessing the string from the array using the following code. Can you please check out that!

UINT32 * bit_format_ref;
UINT8 len;
UINT8 array_index;
UINT8 concat_string[];

if(bit_format == 0x01)
  {
    /* Static string that needs to displayed */
    Size = strlen(*bit_format_ref);
    for(len = 0; len < Size; len++)
    {
      if(array_index <= 31)
      {
          concat_string[array_index] = (UINT8) (bit_format_ref[len + 1]);
        array_index++;

      }
    }
  }

Note: Please respond at the earliest as i already took lot of time for this.

Actually, I wish we did have something like scanf in .NET as Console::ReadLine is crap!

However, printf and Console::WriteLine are not too dissimilar in functionality :)

i couldn't solve the earlier issue but i solved the scanf and printf stuff that's what i meant in the last post.
i haven't worked on the console application previously so i was thinking that there might some syntax like console.readline() or writeline() for c language in visual studio. but i made a try with scanf and printf which worked and hence i said that i solved it. i don't want to waste your time by looking into that as i solved it.

I tried debugging that many times but couldn't succeed in finding the solution regarding the issue where i tried reading the string. i am not sure what the problem could be?
i checked type casting the data type to UINT8 as well which didn't give the desired result.

To read stuff from the console and write it to the console, you can use the scanf and printf functions, respectively. For example:

char s[20];
printf("\nInput a string : ");
scanf("%s", &s);
printf("You entered %s", s);

int i;
printf("\nInput an integer : ");
scanf("%d", &i);
printf("You entered %d", i);

Not sure what you meant by your edit - do you mean the previous problem has now been solved? 

ok vulpes i will try o debug that.
//EDITED please don't try the below post this is solved it
In the mean time can you tell me how to read and write to console window.
i want to read all kinds of values actually from console window like string, char, numerical an hex values.
i want to write concat_string array finally as output.
Basically i am trying to pass arguments to the following function from console window

static void hmi_gfx_mgr_new_interface(UINT16 text_id, void * dynamic_data_vp)
{
  //write code here
}

how can i do that?

Note: i should accept the arguments when i press enter and finally i should show the concatenated string.
and then it should loop again and should ask to enter arguments again (infinite loop).

I still can't see anything wrong with the code :(

Perhaps if you could give an example, saying what the results are and what they should be, that would help to resolve it.

yup i want to carry on looping even after incrementing index.
i am just trying to


I will make things simple now
 
I have a void pointer array as follows:
 
void *l_language_Text_data_VPA[] =  {
                                                                 l_TripResetLine3_Text_Data_VPA,
                                                                 l_Dummy_Text_Data_VPA,
                                                                 l_TripMenuTitle_Text_Data_VPA
                                                                        };
 
which in turn are defined as follows:
void *l_TripResetLine3_Text_Data_VPA[4] =
                                                                       {
                                                                             l_TripResetLine3_Text_Format_U8A,
                                                                             T_411_A,
                                                                             T_411_B,
                                                                             0x01
                                                                       };

void *l_Dummy_Text_Data_VPA[6] =
                                                                       {
                                                                             l_Dummy_Text_Format_U8A,
                                                                             T_438_B,
                                                                             T_414_C,
                                                                             "abcdefghijklmnop",
                                                                             T_414_A,
                                                                             0x32
                                                                       };

void *l_TripMenuTitle_Text_Data_VPA[4] =
                                                                       {
                                                                             l_TripMenuTitle_Text_Format_U8A,
                                                                             T_413_B,
                                                                             T_613_B,
                                                                             0x11
                                                                       };
which in turn as follows:
 
UINT8 l_TripResetLine3_Text_Format_U8A[2] = {0x03, 0x30};
UINT8 l_Dummy_Text_Format_U8A[3] = {0x05, 0x10, 0x02};
UINT8 l_TripMenuTitle_Text_Format_U8A[2] = {0x03, 0x20};
 
Now I have to point to the array l_Dummy_Text_Data_VPA elements with below line
cat_string[index] = bit_format_ref[len];
in which
else if(bit_format == 0x01) // this statement is true when it's a string like "abcdefghijklmnop"  as stated in l_Dummy_Text_Data_VPA

  {

    /* Static string that needs to displayed */

    Size = HMI_STRLEN(*bit_format_ref, 0);

    for(len = 0; len < Size; len++)

    {

      if(index <= 31)

      {

        cat_string[index] = bit_format_ref[len]; // here i am trying copy "aabcdefghijklmnop" to concat_string array

        index++;

      }

    }

  }

I'm looking at this bit of code:

it's working fine Vulpes but in the later code which i wrote is showing some issues.
which i'm marking in pink color in previous post whose link is
(http://www.c-sharpcorner.com/Forums/Thread/146097/how-to-write-code-based-on-the-typedef.aspx).

possibly the pointer is causing problem.
when i try to access the string marked in red color in the above referred post with the pointer bit_format_ref.//EDITED

1. the type of HmStrLngTable1 is
 static unsigned char const HmStrLngTable1[]

2. what ever may be the number of elements which start with 0x1F. if it starts with 0x1F then we should point to the location which is pointed by the combination of the bytes following 0x1F. //EDITED

3. and about third point they both should be combined back to back and then
HmStrLngTable1[combined value] gives the exact value in the array.

For the benefit of others reading this, it's a continuation of this thread (http://www.c-sharpcorner.com/Forums/Thread/146097/how-to-write-code-based-on-the-typedef.aspx).

A few queries before I try to write some code:

1. What is the exact type of the HmStrLngTable1 array? It appears to be int or unsigned int but I thought it might be one of your typedefs.

2. What happens if there is more than one element with the value 0x1F?  Should those after the first still be concatenated with the other elements?

3. What exactly do you mean by concatenation? It looks like the hex representations should all be laid end to end (presumably in ascending order of array index) but should they then be converted back to a number or left as a string?