Send SMS using Dallas temperature sensor DS18S20

Using Arduino Programming Questions
1

Hi,
i have connect Dallas Temperature sensor DS18S20 with my gsm shield and i am trying to send an sms with the temperature of my room.i tried a lot and now i wrote code without errors but when i make the upload and open the serial port nothing happens!
I upload the example from ide (send sms) and everything is ok (GSM initialized)
i write again the code with DS18S20 but again nothing! Even one serial.print...
Here is my code

// Include the GSM library
#include <GSM.h>
//Libraries for temperature
#include <OneWire.h>
#include <DallasTemperature.h>

// PIN Number
#define PINNUMBER ""
// Data wire is plugged into pin 2 on the Arduino
#define ONE_WIRE_BUS 2

GSM gsmAccess;
GSMVoiceCall vcs;
GSM_SMS sms;
 
// Setup a oneWire instance to communicate with any OneWire devices
// (not just Maxim/Dallas temperature ICs)
OneWire oneWire(ONE_WIRE_BUS);
 
// Pass our oneWire reference to Dallas Temperature.
DallasTemperature sensors(&oneWire);
// initialize the library instance

char numtel[20];
const int sensorPin = A2;
// Array to hold the number for the incoming call
float tmp = gettemp();
//float gettemp();


void setup()
{  // initialize serial communications and wait for port to open:
  Serial.begin(9600);

  Serial.println("kaneis klhsh pairneis mnm");
 
  // connection state
  boolean notConnected = true;
  sensors.begin();

  // Start GSM shield
  // If your SIM has PIN, pass it as a parameter of begin() in quotes
  while (notConnected)
  {
    if (gsmAccess.begin(PINNUMBER) == GSM_READY)
      notConnected = false;
    else
    {
      Serial.println("Not connected");
     
      delay(1000);
    }
  }

  // This makes sure the modem correctly reports incoming events
  vcs.hangCall();
  Serial.println("GSM ok");
  Serial.println("pere thl");
}

void loop()
{
  // Check the status of the voice call
  switch (vcs.getvoiceCallStatus())
  {
    case IDLE_CALL: // Nothing is happening

      break;

    case RECEIVINGCALL: // Yes! Someone is calling us

      Serial.println("driiin driiin");

      // Retrieve the calling number
      vcs.retrieveCallingNumber(numtel, 20);

      // Print the calling number
      Serial.print("arithmos:");
      Serial.println(numtel);

      // Answer the call, establish the call
      vcs.answerCall();

//----------------
       // sms text
  Serial.print("h thermokrasia einai: ");
  //b = gettemp();
  Serial.print(tmp);
 
  // send the message
  sms.beginSMS(numtel);
  sms.print(tmp);
  //sms.print(txtMsg);
  sms.endSMS();
  Serial.println("\nto minima stalthike!\n");
//---------------
  vcs.hangCall();
      break;
  }
  delay(1000);
}


float gettemp()
{
  // call sensors.requestTemperatures() to issue a global temperature
  // request to all devices on the bus
  //Serial.print(" Requesting temperatures...");
  sensors.requestTemperatures(); // Send the command to get temperatures
 // Serial.println("DONE");

  //Serial.print("Temperature for Device 1 is: ");
 
  //Serial.print(sensors.getTempCByIndex(0)); // Why "byIndex"?
   return float (sensors.getTempCByIndex(0));
    //Serial.print(a);
    // You can have more than one IC on the same bus.
    // 0 refers to the first IC on the wire
 delay (10000);
}
2

I don't know about your problem specifically, but I know that this

#define PINNUMBER ""

looks bad, and this

delay(1000);

is not recommended.

3

Send SMS using Dallas temperature sensor DS18S20

That's not unlike trying to send an SMS using an ice cream cone and a box of marbles.

Sending an SMS based on the temperature read from a Dallas temperature sensor is a horse of a different color.

// Array to hold the number for the incoming call
float tmp = gettemp();

What nonsense. That comment has nothing to do with the code.

Calling the gettemp() function here is useless.

If you are not getting any serial print statements to work, you need to ditch 90% of the code. Have nothing more than setup() and loop() with NOTHING attached. In setup(), get the port ready and print something.

Only when that works should you add the hardware with no code that admits that it exists. If the serial prints still work, start adding code to deal with the hardware. Test after every little addition.