1. Just change

   void sendSMS()

   to

   void sendSMS(const String& thisIsAString)

   You can then access the parameter inside the function withthisIsAString.

2. No, you do not need a prototype.

• 8
  I would rather advise to pass the String by reference, to avoid additional code to be executed for nothing (copy-constructor, destructor):void sendSMS(String& thisIsAString) or even better, a const reference, if the string argument is not to be modified by the function:void sendSMS(const String& thisIsAString)

  jfpoilpret

  – jfpoilpret

  2014-06-24 17:05:33 +00:00

  CommentedJun 24, 2014 at 17:05

I'd say to never use String again. When code gets bigger and memory usage will be critical you'll hit a dead-end. I know it's more convenient, but give char arrays a shot. Something like:

bool sendSMS(int remoteNumber, char *finalstr){  bool isFinished = 0;  sms.beginSMS(remoteNumber);  for (int i=0;i<sizeof(finalstr);i++){      sms.print(finalstr);  }  sms.endSMS();  lcd.setCursor(0, 0);  lcd.print("Message sent!");  delay(10000);  isFinished = 1;  return isFinished;}

I changed the function to bool. It means that you can use it inside an if statement, where you would want it completed before continuing with your code.

You will need to allocate memory yourself for the array; declare it like this:

char stringArray[33] = {'\0'};

Here, I allocate 32 bytes for data and one additional byte for the character that means "end of string" (it's that \0).

You can change

void sendSMS()

to

void sendSMS(String myinputstring)

you can usemyinputstring in your code.

• That's just a repeat of the other answer.

  sempaiscuba

  – sempaiscuba

  2018-09-21 13:16:10 +00:00

  CommentedSep 21, 2018 at 13:16
• Not a repetition of the other answer. This one isby copy, notby reference. Prefer the other answer to avoid copying of the object.

  Daniel F

  – Daniel F

  2020-08-29 11:28:02 +00:00

  CommentedAug 29, 2020 at 11:28

• programming
• c++