Re: problems with using print data - C / C++ / MFC Discussion Boards

dusk855-Oct-10 16:03

5-Oct-10 16:03

hi,i'm currently doing a project using a software,Waspmote. I have some problems with my codings,it gave me 2 errors when i tried correcting it,

In function 'void loop()':
error: expected initializer before 'uint8_t'

In function 'void loop()':
error: a function-definition is not allowed here before '{' token

could anyone kindly help me take a look at the codings and suggest ways for me to improve on it.The code is as shown below:

int n=0;

void setup(){
// setup for Serial port over USB
USB.begin();
USB.println("USB port started...");
USB.close();

// Powering BT
BT.ON();

// Init
if(!BT.init()) USB.println("Init OK");
}
void loop()
{

// Searching for devices
if(!BT.scanNetwork()) USB.println("Scan OK");
else USB.println("Scan failed");

// Printing the devices that have been found
for(int l=0;l<BT.devices_found;l++)
{
n=0;
USB.print("---- DEVICE ");
USB.print(l,DEC);
USB.print(" ---- ");
USB.print("MAC: ");
for(int m=0;m<12;m++)
{
USB.print(BT.discovered_devices[l].mac_address[m],BYTE);
}
USB.print(" -- NAME: ");
while(BT.discovered_devices[l].name[n]!='\0')
{
USB.print(BT.discovered_devices[l].name[n],BYTE);
n++;
}
n=0;
USB.print(" -- CoD: ");
while(BT.discovered_devices[l].CoD[n]!='\0')
{
USB.print(BT.discovered_devices[l].CoD[n],BYTE);
n++;
}
USB.println("");

delay(2000);
}

// Creating a connection to a device
if(!BT.createConnection("0017E5F1CF74","02"))
{
USB.println("");
USB.print("--- CONNECTION -- MTU: ");
USB.print(BT.connection_mtu[0],BYTE);
USB.print(BT.connection_mtu[1],BYTE);
USB.println(BT.connection_mtu[2],BYTE);
}
else USB.println("Connection failed");

delay(1000);

void printData(char* data )
{
uint8_t uart=0;

printString(data,uart);
printByte('\r',uart);
printByte('\n',uart);

}
delay(2000);

return ;
}

Re: problems with using print data

Peter_in_27805-Oct-10 16:25

Peter_in_2780

5-Oct-10 16:25

Check where you put your braces { }. It looks to me like your "printData()" function is INSIDE "loop()". Indenting your code neatly NEVER hurts and often (in cases like this) helps you see what you've written.

Software rusts. Simon Stephenson, ca 1994.

Re: problems with using print data

dusk855-Oct-10 17:08

dusk85

5-Oct-10 17:08

thanks for the reply.I tried removing the brackets but it still doesn't work,if possible may i just trouble you to help me correct the codings because i'm rather new in programming.

Re: problems with using print data

Peter_in_27805-Oct-10 17:23

Peter_in_2780

5-Oct-10 17:23

I didn't say remove the braces. You need to get them matching correctly, like this:

func1() {    // open 1
  ...
  ...
}            // close 1

func2() {    // open 2    
  ...
  ...
}            // close 2

What you had is more like

func1() {
  ...
  ...
  func2() {
    ...
    ...
  }

}

See how the indentation makes it easy to spot the problem?

Software rusts. Simon Stephenson, ca 1994.

Re: problems with using print data

dusk855-Oct-10 17:41

dusk85

5-Oct-10 17:41

After looking through the codes and compiling, I still get an error:

In function 'void loop()':
error: a function-definition is not allowed here before '{' token

C++

int n=0;

void setup()
  {                              
  // setup for Serial port over USB
  USB.begin();
  USB.println("USB port started...");
  USB.close();
  
  // Powering BT
  BT.ON();
  
  // Init
  if(!BT.init()) USB.println("Init OK");
  }                              

void loop()

{
   // Searching for devices
  if(!BT.scanNetwork()) USB.println("Scan OK");
  else USB.println("Scan failed");
  
  // Printing the devices that have been found
  for(int l=0;l<BT.devices_found;l++)
  
  {
    n=0;
    USB.print("---- DEVICE ");
    USB.print(l,DEC);
    USB.print(" ----  ");
    USB.print("MAC: ");
    for(int m=0;m<12;m++)
    
    {
      USB.print(BT.discovered_devices[l].mac_address[m],BYTE);
    }
    
    USB.print(" -- NAME: ");
    while(BT.discovered_devices[l].name[n]!='\0')
    {
      USB.print(BT.discovered_devices[l].name[n],BYTE);
      n++;
    }
    
    n=0;
    USB.print(" -- CoD: ");
    while(BT.discovered_devices[l].CoD[n]!='\0')
    {
      USB.print(BT.discovered_devices[l].CoD[n],BYTE);
      n++;
    }
    USB.println("");
    
    delay(2000);
  }
  

 
  // Creating a connection to a device
  if(!BT.createConnection("0017E5F1CF74","02"))
  {
    USB.println("");
    USB.print("--- CONNECTION -- MTU: ");
    USB.print(BT.connection_mtu[0],BYTE);
    USB.print(BT.connection_mtu[1],BYTE);
    USB.println(BT.connection_mtu[2],BYTE);    
  }
  else USB.println("Connection failed");
  
  
  delay(1000);
  
  void printData(char* data )
  {
  uint8_t uart=0;
  
  printString(data,uart);
  printByte('\r',uart);
  printByte('\n',uart);

  }
 delay(2000);
   
 return  ;
}

Re: problems with using print data

Peter_in_27805-Oct-10 18:00

Peter_in_2780

5-Oct-10 18:00

I just pasted that code into my syntax-highlighting editor, and it tells me that the very last close brace matches the one immediately after void loop(). So void printData() is still "inside" void loop(). Again, the indenting should give you a clue - void printData() isn't in column 1 like the other functions are.

Software rusts. Simon Stephenson, ca 1994.

Re: problems with using print data

dusk855-Oct-10 20:06

dusk85

5-Oct-10 20:06

I taken

C++

void printData

out from the

C++

loop

.There are no more errors,but when i upload the codings to the waspmote IDE, it doesn't send out the text to external devices through the bluetooth device.Do you have any recommendations for me to modify my existing source code?

Re: problems with using print data

Peter_in_27805-Oct-10 20:17

Peter_in_2780

5-Oct-10 20:17

dusk85 wrote:
Do you have any recommendations for me to modify my existing source code?
Quote Selected Text

The short answer is no. I am happy to help with getting your code to compile, but I'm not going into your bluetooth issues. I suggest you try a forum for bluetooth or waspmote (whatever that is).

Cheers,
Peter

Software rusts. Simon Stephenson, ca 1994.

Re: problems with using print data

Luc Pattyn5-Oct-10 16:40

Luc Pattyn

5-Oct-10 16:40

You really should never present unformatted code. Using PRE tags would preserve indentation and make things readable, and maybe the mistake obvious.

Smile | :)

Luc Pattyn [Forum Guidelines] [Why QA sucks] [My Articles] Nil Volentibus Arduum

Please use <PRE> tags for code snippets, they preserve indentation, and improve readability.

VS2010 & SDK 7.1

Fareed Rizkalla5-Oct-10 12:43

Fareed Rizkalla

5-Oct-10 12:43

Does anyone know if I can make the SDK 7.1 directories the default instead of using the SDK 7.0! Sniff | :^)

I know that you can configure them on each solution, but it is very boring to set them on each solution.

Or should I copy and replace files of 7.0A with 7.1. Roll eyes | :rolleyes:

(Might even free up some disk space in the process)

MySQL 5.0 and Visual C++ 6....do they work together?

ns5-Oct-10 7:48

5-Oct-10 7:48

Does anyone know if the new mysql libraries can be called from the old old vc6 (on windows xp)?
thanks!

Re: MySQL 5.0 and Visual C++ 6....do they work together?

_Flaviu17-Mar-13 21:09

_Flaviu

17-Mar-13 21:09

Yes, they perfectly work togheter ... swend me an email, and I'll send to you a test application ...

String

Fareed Rizkalla5-Oct-10 6:16

Fareed Rizkalla

5-Oct-10 6:16

If I assign a string to a variable.

char* t = "Hello World!";

Doesn't this string get a fixed array with a null terminator at the end?

Re: String

Richard MacCutchan5-Oct-10 6:49

Richard MacCutchan

5-Oct-10 6:49

Yes. Take a look at it in the debugger if you are not sure.

Just say 'NO' to evaluated arguments for diadic functions! Ash

Re: String

Fareed Rizkalla5-Oct-10 8:20

Fareed Rizkalla

5-Oct-10 8:20

wcscat_s keeps throwing an error that the string isn't null terminated! :S

Re: String

Moak5-Oct-10 8:36

Moak

5-Oct-10 8:36

Are you sure it doesn't throw an exception, because the memory provided in strDestination argument is read-only?

Chat in Europe Java | [Coffee]

Now with 24% more Twitter

Re: String

Richard MacCutchan5-Oct-10 9:05

Richard MacCutchan

5-Oct-10 9:05

Fareed Rizkalla wrote:
wcscat_s keeps throwing an error that the string isn't null terminated!

Yes, I'm not surprised since you are sending it a char* (i.e. ASCII string) but wcscat_s() expects Unicode.

Just say 'NO' to evaluated arguments for diadic functions! Ash

Re: String

Moak5-Oct-10 6:59

Moak

5-Oct-10 6:59

No. It assigns a pointer to a fixed null-terinated string, there is a slight difference between arrays and pointers. A writable block of memory is not allocated and you should get a compiler warning because of the unsafe conversion... change the code to:

const char* t = "Hello World!";

Hope this helps Smile | :)

Chat in Europe Java | [Coffee]

Now with 24% more Twitter

Re: String

Richard MacCutchan5-Oct-10 21:37

Richard MacCutchan

5-Oct-10 21:37

Moak wrote:
a pointer to a fixed null-terinated string,

Which is simply an array of characters.

Moak wrote:
there is a slight difference between arrays and pointers.

I would say there is a huge difference.

Just say 'NO' to evaluated arguments for diadic functions! Ash

Re: String

Luc Pattyn5-Oct-10 8:37

Luc Pattyn

5-Oct-10 8:37

every literal string gets a NULL appended automatically by the compiler.

However there are 8-bit and 16-bit characters and strings, and you can't just mix them.
If you use wcscat_s, you want wide char strings, the example in MSDN is:

wchar_t wszStr[] = L"1a1g";

Luc Pattyn [Forum Guidelines] [Why QA sucks] [My Articles] Nil Volentibus Arduum

Please use <PRE> tags for code snippets, they preserve indentation, and improve readability.

Cannot display icon in tree. Help!

Brad Reichenbach5-Oct-10 4:44

Brad Reichenbach

5-Oct-10 4:44

Hello. I created a new icon in the resource view called "IDI_OBJECT_GLOBAL_VAR". In my "View" cpp, I added the icon to my HICON ( along with a bunch of other icons that work) using the following:

hIcon[ICON_INDEX_OBJ_GLOBAL_VAR] = AfxGetApp()->LoadIcon(IDI_OBJECT_GLOBAL_VAR);

This is then added to a tree list :

for( GUShort index = 0;
index < ICON_INDEX_MAXIMUM;
index++ )
{
m_TreeImageList.Add( hIcon[ index ] );
}

GetTreeCtrl().SetImageList( &m_TreeImageList, TVSIL_NORMAL );

When I go to insert an item with the icon, I use the following:

// Add the new item into the tree beneath the parent item
hNewItem = GetTreeCtrl().InsertItem( label,
ICON_INDEX_OBJ_GLOBAL_VAR,
ICON_INDEX_OBJ_GLOBAL_VAR,
parentTreeItem,
hInsertAfter );

The item itself is placed in the tree, but the icon NEVER displays. It's just blank. Any help on this would be appreciated. Thanks!

Re: Cannot display icon in tree. Help!

Niklas L5-Oct-10 8:22

Niklas L

5-Oct-10 8:22

Have you checked the return values of LoadIcon and Add? Since you want to use your indexes as above, I suggest you put in an index check.

for( GUShort index = 0;
     index < ICON_INDEX_MAXIMUM;
     index++ )
{
    VERIFY(m_TreeImageList.Add( hIcon[ index ] ) == index);
}

Who knows? You might be surprised!

(The point is, if any of your icons fail to be added, the indexes are screwed up)

home

Extract numbers from string

Anu_Bala5-Oct-10 1:12

Anu_Bala

5-Oct-10 1:12

Hi I have strings like
GROUP23
TREND12
GRAPH120
GRAPH01
From this i have to extract the numbers only like 23,12,120,01...

I tried atoi(),but it returns 0.
I tried Right(),

int iPanNo;
CString sNo;
CString sName = "GROUP23";
sNo = sName.Right(2);
iPanNo = atoi(sNo);

The above code works fine and it returns '23'.
But it fails for GRAPH120.For this it returns 20 not 120.

How can i do that?

Anu

Re: Extract numbers from string

Richard MacCutchan5-Oct-10 1:29

Richard MacCutchan

5-Oct-10 1:29

You need to parse your string into alpha and numeric portions and then apply the conversion on the numeric portion. Take a look at the methods of CString() to see if there is a builtin that will do it for you, or use something like isdigit()[^]. The rest should be easy.

Just say 'NO' to evaluated arguments for diadic functions! Ash

Re: Extract numbers from string

Anu_Bala5-Oct-10 2:04

Anu_Bala

5-Oct-10 2:04

Thank you.I coded like

for (int i = 0; i < sNo.GetLength(); i++) 
 { 
   Number = sNo.GetAt(i);            
   if(isdigit(Number))  
      res += Number; 
 }

And it works.
Thanks.

Anu

Use Ctrl+Left/Right to switch messages, Ctrl+Up/Down to switch threads, Ctrl+Shift+Left/Right to switch pages.