expected `}' at end of input












0















int scvVal = 6;
int drvVal = 9;
int sensorVal = A1;
void setup() {
Serial.begin(9600);
pinMode(6, OUTPUT);
pinMode(9, OUTPUT);
}
void loop(){
sensorVal = analogRead(A1);


Serial.print("Sensor Value: ");
Serial.print(sensorVal);

float voltage = (sensorVal*5.0)/1024.0;
Serial.print("Volts: ");
Serial.print(voltage);

float pressure_pascal = (400.0*((float)voltage-0.5))*1000000.0;
float pressure_bar = pressure_pascal/10e5;
Serial.print("Pressure = ");
Serial.print(pressure_bar);
Serial.println(" bars");
Serial.print("Pressure = ");
{
for(int a = 255; a>=0;a--)
{
analogWrite(6,a);
if (pressure_bar = 1600){
analogWrite (6, a);
if (pressure_bar > 1600){
analogWrite (6, a);
}
for(int a = 0; a<256; a++)
{
analogWrite(6, a);
{
if(pressure_bar < 1600){
analogWrite (6, a);
}
for(int b = 255; b>=0;b--)
{
analogWrite(9, b);
{
if (pressure_bar == 1600){
analogWrite (9, b);
if(pressure_bar < 1600){
analogWrite (9, b);
}
for(int b = 0; b<256; b++)
{
analogWrite(9, b);
{
if (pressure_bar > 1600){
analogWrite (9, b);
}
}









share|improve this question









New contributor




Đức Hòang is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
Check out our Code of Conduct.





















  • Welcome to Arduino:SE. Your life would be so much easier if you indented your code correctly. It would then be fairly obvious where the missing } should be.

    – sempaiscuba
    13 hours ago











  • do you help me?

    – Đức Hòang
    13 hours ago











  • @Duc Hoang: Are you formatting your code correctly, that it is readable? Select in in the question editor and click on the {} button

    – chrisl
    13 hours ago











  • Never before have I see such random use of brackets... ;)

    – Majenko
    12 hours ago






  • 1





    do you help me? .... you have to start by helping yourself ..... read and understand the answers below ..... do not copy and paste code without thinking

    – jsotola
    12 hours ago
















0















int scvVal = 6;
int drvVal = 9;
int sensorVal = A1;
void setup() {
Serial.begin(9600);
pinMode(6, OUTPUT);
pinMode(9, OUTPUT);
}
void loop(){
sensorVal = analogRead(A1);


Serial.print("Sensor Value: ");
Serial.print(sensorVal);

float voltage = (sensorVal*5.0)/1024.0;
Serial.print("Volts: ");
Serial.print(voltage);

float pressure_pascal = (400.0*((float)voltage-0.5))*1000000.0;
float pressure_bar = pressure_pascal/10e5;
Serial.print("Pressure = ");
Serial.print(pressure_bar);
Serial.println(" bars");
Serial.print("Pressure = ");
{
for(int a = 255; a>=0;a--)
{
analogWrite(6,a);
if (pressure_bar = 1600){
analogWrite (6, a);
if (pressure_bar > 1600){
analogWrite (6, a);
}
for(int a = 0; a<256; a++)
{
analogWrite(6, a);
{
if(pressure_bar < 1600){
analogWrite (6, a);
}
for(int b = 255; b>=0;b--)
{
analogWrite(9, b);
{
if (pressure_bar == 1600){
analogWrite (9, b);
if(pressure_bar < 1600){
analogWrite (9, b);
}
for(int b = 0; b<256; b++)
{
analogWrite(9, b);
{
if (pressure_bar > 1600){
analogWrite (9, b);
}
}









share|improve this question









New contributor




Đức Hòang is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
Check out our Code of Conduct.





















  • Welcome to Arduino:SE. Your life would be so much easier if you indented your code correctly. It would then be fairly obvious where the missing } should be.

    – sempaiscuba
    13 hours ago











  • do you help me?

    – Đức Hòang
    13 hours ago











  • @Duc Hoang: Are you formatting your code correctly, that it is readable? Select in in the question editor and click on the {} button

    – chrisl
    13 hours ago











  • Never before have I see such random use of brackets... ;)

    – Majenko
    12 hours ago






  • 1





    do you help me? .... you have to start by helping yourself ..... read and understand the answers below ..... do not copy and paste code without thinking

    – jsotola
    12 hours ago














0












0








0








int scvVal = 6;
int drvVal = 9;
int sensorVal = A1;
void setup() {
Serial.begin(9600);
pinMode(6, OUTPUT);
pinMode(9, OUTPUT);
}
void loop(){
sensorVal = analogRead(A1);


Serial.print("Sensor Value: ");
Serial.print(sensorVal);

float voltage = (sensorVal*5.0)/1024.0;
Serial.print("Volts: ");
Serial.print(voltage);

float pressure_pascal = (400.0*((float)voltage-0.5))*1000000.0;
float pressure_bar = pressure_pascal/10e5;
Serial.print("Pressure = ");
Serial.print(pressure_bar);
Serial.println(" bars");
Serial.print("Pressure = ");
{
for(int a = 255; a>=0;a--)
{
analogWrite(6,a);
if (pressure_bar = 1600){
analogWrite (6, a);
if (pressure_bar > 1600){
analogWrite (6, a);
}
for(int a = 0; a<256; a++)
{
analogWrite(6, a);
{
if(pressure_bar < 1600){
analogWrite (6, a);
}
for(int b = 255; b>=0;b--)
{
analogWrite(9, b);
{
if (pressure_bar == 1600){
analogWrite (9, b);
if(pressure_bar < 1600){
analogWrite (9, b);
}
for(int b = 0; b<256; b++)
{
analogWrite(9, b);
{
if (pressure_bar > 1600){
analogWrite (9, b);
}
}









share|improve this question









New contributor




Đức Hòang is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
Check out our Code of Conduct.












int scvVal = 6;
int drvVal = 9;
int sensorVal = A1;
void setup() {
Serial.begin(9600);
pinMode(6, OUTPUT);
pinMode(9, OUTPUT);
}
void loop(){
sensorVal = analogRead(A1);


Serial.print("Sensor Value: ");
Serial.print(sensorVal);

float voltage = (sensorVal*5.0)/1024.0;
Serial.print("Volts: ");
Serial.print(voltage);

float pressure_pascal = (400.0*((float)voltage-0.5))*1000000.0;
float pressure_bar = pressure_pascal/10e5;
Serial.print("Pressure = ");
Serial.print(pressure_bar);
Serial.println(" bars");
Serial.print("Pressure = ");
{
for(int a = 255; a>=0;a--)
{
analogWrite(6,a);
if (pressure_bar = 1600){
analogWrite (6, a);
if (pressure_bar > 1600){
analogWrite (6, a);
}
for(int a = 0; a<256; a++)
{
analogWrite(6, a);
{
if(pressure_bar < 1600){
analogWrite (6, a);
}
for(int b = 255; b>=0;b--)
{
analogWrite(9, b);
{
if (pressure_bar == 1600){
analogWrite (9, b);
if(pressure_bar < 1600){
analogWrite (9, b);
}
for(int b = 0; b<256; b++)
{
analogWrite(9, b);
{
if (pressure_bar > 1600){
analogWrite (9, b);
}
}






arduino-uno programming






share|improve this question









New contributor




Đức Hòang is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
Check out our Code of Conduct.











share|improve this question









New contributor




Đức Hòang is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
Check out our Code of Conduct.









share|improve this question




share|improve this question








edited 12 hours ago









Michel Keijzers

6,97251939




6,97251939






New contributor




Đức Hòang is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
Check out our Code of Conduct.









asked 13 hours ago









Đức HòangĐức Hòang

41




41




New contributor




Đức Hòang is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
Check out our Code of Conduct.





New contributor





Đức Hòang is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
Check out our Code of Conduct.






Đức Hòang is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
Check out our Code of Conduct.













  • Welcome to Arduino:SE. Your life would be so much easier if you indented your code correctly. It would then be fairly obvious where the missing } should be.

    – sempaiscuba
    13 hours ago











  • do you help me?

    – Đức Hòang
    13 hours ago











  • @Duc Hoang: Are you formatting your code correctly, that it is readable? Select in in the question editor and click on the {} button

    – chrisl
    13 hours ago











  • Never before have I see such random use of brackets... ;)

    – Majenko
    12 hours ago






  • 1





    do you help me? .... you have to start by helping yourself ..... read and understand the answers below ..... do not copy and paste code without thinking

    – jsotola
    12 hours ago



















  • Welcome to Arduino:SE. Your life would be so much easier if you indented your code correctly. It would then be fairly obvious where the missing } should be.

    – sempaiscuba
    13 hours ago











  • do you help me?

    – Đức Hòang
    13 hours ago











  • @Duc Hoang: Are you formatting your code correctly, that it is readable? Select in in the question editor and click on the {} button

    – chrisl
    13 hours ago











  • Never before have I see such random use of brackets... ;)

    – Majenko
    12 hours ago






  • 1





    do you help me? .... you have to start by helping yourself ..... read and understand the answers below ..... do not copy and paste code without thinking

    – jsotola
    12 hours ago

















Welcome to Arduino:SE. Your life would be so much easier if you indented your code correctly. It would then be fairly obvious where the missing } should be.

– sempaiscuba
13 hours ago





Welcome to Arduino:SE. Your life would be so much easier if you indented your code correctly. It would then be fairly obvious where the missing } should be.

– sempaiscuba
13 hours ago













do you help me?

– Đức Hòang
13 hours ago





do you help me?

– Đức Hòang
13 hours ago













@Duc Hoang: Are you formatting your code correctly, that it is readable? Select in in the question editor and click on the {} button

– chrisl
13 hours ago





@Duc Hoang: Are you formatting your code correctly, that it is readable? Select in in the question editor and click on the {} button

– chrisl
13 hours ago













Never before have I see such random use of brackets... ;)

– Majenko
12 hours ago





Never before have I see such random use of brackets... ;)

– Majenko
12 hours ago




1




1





do you help me? .... you have to start by helping yourself ..... read and understand the answers below ..... do not copy and paste code without thinking

– jsotola
12 hours ago





do you help me? .... you have to start by helping yourself ..... read and understand the answers below ..... do not copy and paste code without thinking

– jsotola
12 hours ago










2 Answers
2






active

oldest

votes


















3














Some guidelines for programming:



Start each statement on a new line



int scvVal = 6; 
int drvVal = 9;
int sensorVal = A1;
void setup() {
Serial.begin(9600);
pinMode(6, OUTPUT);
pinMode(9, OUTPUT);
} void loop(){
sensorVal = analogRead(A1);

Serial.print("Sensor Value: "); Serial.print(sensorVal);

float voltage = (sensorVal*5.0)/1024.0; Serial.print("Volts: "); Serial.print(voltage);

float pressure_pascal = (400.0*((float)voltage-0.5))*1000000.0;
float pressure_bar = pressure_pascal/10e5;
Serial.print("Pressure = ");
Serial.print(pressure_bar);
Serial.println(" bars");
Serial.print("Pressure = "); {
for(int a = 255; a>=0;a--) {
analogWrite(6,a);
if (pressure_bar = 1600){
analogWrite (6, a);
if (pressure_bar > 1600){
analogWrite (6, a); }
for(int a = 0; a<256; a++) {
analogWrite(6, a); {
if(pressure_bar < 1600){
analogWrite (6, a); }
for(int b = 255; b>=0;b--) {
analogWrite(9, b); {
if (pressure_bar == 1600){
analogWrite (9, b);
if(pressure_bar < 1600){
analogWrite (9, b); }
for(int b = 0; b<256; b++) {
analogWrite(9, b); {
if (pressure_bar > 1600){
analogWrite (9, b);
} }


Use brackets and align them



.. so each pair of { and } is on the same column and align code with brackets to be indented 2, 3 or 4 spaces



int scvVal = 6; 
int drvVal = 9;
int sensorVal = A1;
void setup()
{
Serial.begin(9600);
pinMode(6, OUTPUT);
pinMode(9, OUTPUT);
}
void loop()
{
sensorVal = analogRead(A1);

Serial.print("Sensor Value: ");
Serial.print(sensorVal);

float voltage = (sensorVal*5.0)/1024.0; Serial.print("Volts: ");
Serial.print(voltage);

float pressure_pascal = (400.0*((float)voltage-0.5))*1000000.0;
float pressure_bar = pressure_pascal/10e5;
Serial.print("Pressure = ");
Serial.print(pressure_bar);
Serial.println(" bars");
Serial.print("Pressure = ");
{
for(int a = 255; a>=0;a--)
{
analogWrite(6,a);
if (pressure_bar = 1600)
{
analogWrite (6, a);
if (pressure_bar > 1600)
{
analogWrite (6, a);
}
for(int a = 0; a<256; a++)
{
analogWrite(6, a);
{
if(pressure_bar < 1600)
{
analogWrite (6, a);
}
for(int b = 255; b>=0;b--)
{
analogWrite(9, b);
{
if (pressure_bar == 1600)
{
analogWrite (9, b);
if(pressure_bar < 1600)
{
analogWrite (9, b);
}
for(int b = 0; b<256; b++)
{
analogWrite(9, b);
{
if (pressure_bar > 1600)
{
analogWrite (9, b);
}
}


Remove superfluous brackets



As you can see, you are having some superfluous/missing brackets. What you want is probably something like:



int scvVal = 6; 
int drvVal = 9;
int sensorVal = A1;
void setup()
{
Serial.begin(9600);
pinMode(6, OUTPUT);
pinMode(9, OUTPUT);
}
void loop()
{
sensorVal = analogRead(A1);

Serial.print("Sensor Value: ");
Serial.print(sensorVal);

float voltage = (sensorVal*5.0)/1024.0; Serial.print("Volts: ");
Serial.print(voltage);

float pressure_pascal = (400.0*((float)voltage-0.5))*1000000.0;
float pressure_bar = pressure_pascal/10e5;
Serial.print("Pressure = ");
Serial.print(pressure_bar);
Serial.println(" bars");
Serial.print("Pressure = ");

for(int a = 255; a>=0;a--)
{
analogWrite(6,a);
if (pressure_bar = 1600)
{
analogWrite (6, a);
}
if (pressure_bar > 1600)
{
analogWrite (6, a);
}
}
for(int a = 0; a<256; a++)
{
analogWrite(6, a);
if(pressure_bar < 1600)
{
analogWrite (6, a);
}
}
for(int b = 255; b>=0;b--)
{
analogWrite(9, b);
if (pressure_bar == 1600)
{
analogWrite (9, b);
}
if(pressure_bar < 1600)
{
analogWrite (9, b);
}
}
for(int b = 0; b<256; b++)
{
analogWrite(9, b);
if (pressure_bar > 1600)
{
analogWrite (9, b);
}
}
}


Remove duplicate code



In the if statement, you perform twice the same code (both you send an analogWrite with the same value, so the code (from the for loops can be reduced to:



  for(int a = 255; a>=0;a--) 
{
analogWrite(6,a);
if (pressure_bar >= 1600)
{
analogWrite (6, a);
}
}
for(int a = 0; a<256; a++)
{
analogWrite(6, a);
if(pressure_bar <= 1600)
{
analogWrite (6, a);
}
}
for(int b = 255; b>=0;b--)
{
analogWrite(9, b);
if (pressure_bar <= 1600)
{
analogWrite (9, b);
}
}
for(int b = 0; b<256; b++)
{
analogWrite(9, b);
if (pressure_bar >= 1600)
{
analogWrite (9, b);
}
}
}


Unclear



What is unclear why you first write an analog value, and directly afterward (in each if) you write the same value. I'm not sure what you really want here. Either you want to write to a different pin, or you want a delay (to have the value having some effect), I'm just not sure.






share|improve this answer

































    1














    Look at your open brackets { vs closing brackets, }. You need the same number of each. If have this in the Arduino IDE and use CTRL-T, they are somewhat easy to find. I just looked, and I can spot them pretty easily.



    int scvVal = 6; 
    int drvVal = 9;
    int sensorVal = A1;
    void setup()
    {
    Serial.begin(9600);
    pinMode(6, OUTPUT);
    pinMode(9, OUTPUT);
    }
    void loop()
    {
    sensorVal = analogRead(A1);
    Serial.print("Sensor Value: ");
    Serial.print(sensorVal);
    float voltage = (sensorVal*5.0)/1024.0;
    Serial.print("Volts: ");
    Serial.print(voltage);
    float pressure_pascal = (400.0*((float)voltage-0.5))*1000000.0;
    float pressure_bar = pressure_pascal/10e5;
    Serial.print("Pressure = ");
    Serial.print(pressure_bar);
    Serial.println(" bars");
    Serial.print("Pressure = ");
    { << bracket for what?
    for(int a = 255; a>=0;a--) { << missing closing bracket
    analogWrite(6,a);
    if (pressure_bar = 1600){ << missing closing bracket
    analogWrite (6, a);
    if (pressure_bar > 1600){
    analogWrite (6, a);
    }
    for(int a = 0; a<256; a++) {
    analogWrite(6, a);
    { << wrong direction
    if(pressure_bar < 1600){
    analogWrite (6, a);
    }
    for(int b = 255; b>=0;b--) {
    analogWrite(9, b);
    { << wrong direction
    if (pressure_bar == 1600){ << missing closing bracket
    analogWrite (9, b);
    if(pressure_bar < 1600){ << missing closing bracket
    analogWrite (9, b);
    }
    for(int b = 0; b<256; b++) {
    analogWrite(9, b);
    { << wrong direction
    if (pressure_bar > 1600){
    analogWrite (9, b);
    }
    } << closing bracket for loop





    share|improve this answer
























      Your Answer






      StackExchange.ifUsing("editor", function () {
      return StackExchange.using("schematics", function () {
      StackExchange.schematics.init();
      });
      }, "cicuitlab");

      StackExchange.ready(function() {
      var channelOptions = {
      tags: "".split(" "),
      id: "540"
      };
      initTagRenderer("".split(" "), "".split(" "), channelOptions);

      StackExchange.using("externalEditor", function() {
      // Have to fire editor after snippets, if snippets enabled
      if (StackExchange.settings.snippets.snippetsEnabled) {
      StackExchange.using("snippets", function() {
      createEditor();
      });
      }
      else {
      createEditor();
      }
      });

      function createEditor() {
      StackExchange.prepareEditor({
      heartbeatType: 'answer',
      autoActivateHeartbeat: false,
      convertImagesToLinks: false,
      noModals: true,
      showLowRepImageUploadWarning: true,
      reputationToPostImages: null,
      bindNavPrevention: true,
      postfix: "",
      imageUploader: {
      brandingHtml: "Powered by u003ca class="icon-imgur-white" href="https://imgur.com/"u003eu003c/au003e",
      contentPolicyHtml: "User contributions licensed under u003ca href="https://creativecommons.org/licenses/by-sa/3.0/"u003ecc by-sa 3.0 with attribution requiredu003c/au003e u003ca href="https://stackoverflow.com/legal/content-policy"u003e(content policy)u003c/au003e",
      allowUrls: true
      },
      onDemand: true,
      discardSelector: ".discard-answer"
      ,immediatelyShowMarkdownHelp:true
      });


      }
      });






      Đức Hòang is a new contributor. Be nice, and check out our Code of Conduct.










      draft saved

      draft discarded


















      StackExchange.ready(
      function () {
      StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2farduino.stackexchange.com%2fquestions%2f63354%2fexpected-at-end-of-input%23new-answer', 'question_page');
      }
      );

      Post as a guest















      Required, but never shown

























      2 Answers
      2






      active

      oldest

      votes








      2 Answers
      2






      active

      oldest

      votes









      active

      oldest

      votes






      active

      oldest

      votes









      3














      Some guidelines for programming:



      Start each statement on a new line



      int scvVal = 6; 
      int drvVal = 9;
      int sensorVal = A1;
      void setup() {
      Serial.begin(9600);
      pinMode(6, OUTPUT);
      pinMode(9, OUTPUT);
      } void loop(){
      sensorVal = analogRead(A1);

      Serial.print("Sensor Value: "); Serial.print(sensorVal);

      float voltage = (sensorVal*5.0)/1024.0; Serial.print("Volts: "); Serial.print(voltage);

      float pressure_pascal = (400.0*((float)voltage-0.5))*1000000.0;
      float pressure_bar = pressure_pascal/10e5;
      Serial.print("Pressure = ");
      Serial.print(pressure_bar);
      Serial.println(" bars");
      Serial.print("Pressure = "); {
      for(int a = 255; a>=0;a--) {
      analogWrite(6,a);
      if (pressure_bar = 1600){
      analogWrite (6, a);
      if (pressure_bar > 1600){
      analogWrite (6, a); }
      for(int a = 0; a<256; a++) {
      analogWrite(6, a); {
      if(pressure_bar < 1600){
      analogWrite (6, a); }
      for(int b = 255; b>=0;b--) {
      analogWrite(9, b); {
      if (pressure_bar == 1600){
      analogWrite (9, b);
      if(pressure_bar < 1600){
      analogWrite (9, b); }
      for(int b = 0; b<256; b++) {
      analogWrite(9, b); {
      if (pressure_bar > 1600){
      analogWrite (9, b);
      } }


      Use brackets and align them



      .. so each pair of { and } is on the same column and align code with brackets to be indented 2, 3 or 4 spaces



      int scvVal = 6; 
      int drvVal = 9;
      int sensorVal = A1;
      void setup()
      {
      Serial.begin(9600);
      pinMode(6, OUTPUT);
      pinMode(9, OUTPUT);
      }
      void loop()
      {
      sensorVal = analogRead(A1);

      Serial.print("Sensor Value: ");
      Serial.print(sensorVal);

      float voltage = (sensorVal*5.0)/1024.0; Serial.print("Volts: ");
      Serial.print(voltage);

      float pressure_pascal = (400.0*((float)voltage-0.5))*1000000.0;
      float pressure_bar = pressure_pascal/10e5;
      Serial.print("Pressure = ");
      Serial.print(pressure_bar);
      Serial.println(" bars");
      Serial.print("Pressure = ");
      {
      for(int a = 255; a>=0;a--)
      {
      analogWrite(6,a);
      if (pressure_bar = 1600)
      {
      analogWrite (6, a);
      if (pressure_bar > 1600)
      {
      analogWrite (6, a);
      }
      for(int a = 0; a<256; a++)
      {
      analogWrite(6, a);
      {
      if(pressure_bar < 1600)
      {
      analogWrite (6, a);
      }
      for(int b = 255; b>=0;b--)
      {
      analogWrite(9, b);
      {
      if (pressure_bar == 1600)
      {
      analogWrite (9, b);
      if(pressure_bar < 1600)
      {
      analogWrite (9, b);
      }
      for(int b = 0; b<256; b++)
      {
      analogWrite(9, b);
      {
      if (pressure_bar > 1600)
      {
      analogWrite (9, b);
      }
      }


      Remove superfluous brackets



      As you can see, you are having some superfluous/missing brackets. What you want is probably something like:



      int scvVal = 6; 
      int drvVal = 9;
      int sensorVal = A1;
      void setup()
      {
      Serial.begin(9600);
      pinMode(6, OUTPUT);
      pinMode(9, OUTPUT);
      }
      void loop()
      {
      sensorVal = analogRead(A1);

      Serial.print("Sensor Value: ");
      Serial.print(sensorVal);

      float voltage = (sensorVal*5.0)/1024.0; Serial.print("Volts: ");
      Serial.print(voltage);

      float pressure_pascal = (400.0*((float)voltage-0.5))*1000000.0;
      float pressure_bar = pressure_pascal/10e5;
      Serial.print("Pressure = ");
      Serial.print(pressure_bar);
      Serial.println(" bars");
      Serial.print("Pressure = ");

      for(int a = 255; a>=0;a--)
      {
      analogWrite(6,a);
      if (pressure_bar = 1600)
      {
      analogWrite (6, a);
      }
      if (pressure_bar > 1600)
      {
      analogWrite (6, a);
      }
      }
      for(int a = 0; a<256; a++)
      {
      analogWrite(6, a);
      if(pressure_bar < 1600)
      {
      analogWrite (6, a);
      }
      }
      for(int b = 255; b>=0;b--)
      {
      analogWrite(9, b);
      if (pressure_bar == 1600)
      {
      analogWrite (9, b);
      }
      if(pressure_bar < 1600)
      {
      analogWrite (9, b);
      }
      }
      for(int b = 0; b<256; b++)
      {
      analogWrite(9, b);
      if (pressure_bar > 1600)
      {
      analogWrite (9, b);
      }
      }
      }


      Remove duplicate code



      In the if statement, you perform twice the same code (both you send an analogWrite with the same value, so the code (from the for loops can be reduced to:



        for(int a = 255; a>=0;a--) 
      {
      analogWrite(6,a);
      if (pressure_bar >= 1600)
      {
      analogWrite (6, a);
      }
      }
      for(int a = 0; a<256; a++)
      {
      analogWrite(6, a);
      if(pressure_bar <= 1600)
      {
      analogWrite (6, a);
      }
      }
      for(int b = 255; b>=0;b--)
      {
      analogWrite(9, b);
      if (pressure_bar <= 1600)
      {
      analogWrite (9, b);
      }
      }
      for(int b = 0; b<256; b++)
      {
      analogWrite(9, b);
      if (pressure_bar >= 1600)
      {
      analogWrite (9, b);
      }
      }
      }


      Unclear



      What is unclear why you first write an analog value, and directly afterward (in each if) you write the same value. I'm not sure what you really want here. Either you want to write to a different pin, or you want a delay (to have the value having some effect), I'm just not sure.






      share|improve this answer






























        3














        Some guidelines for programming:



        Start each statement on a new line



        int scvVal = 6; 
        int drvVal = 9;
        int sensorVal = A1;
        void setup() {
        Serial.begin(9600);
        pinMode(6, OUTPUT);
        pinMode(9, OUTPUT);
        } void loop(){
        sensorVal = analogRead(A1);

        Serial.print("Sensor Value: "); Serial.print(sensorVal);

        float voltage = (sensorVal*5.0)/1024.0; Serial.print("Volts: "); Serial.print(voltage);

        float pressure_pascal = (400.0*((float)voltage-0.5))*1000000.0;
        float pressure_bar = pressure_pascal/10e5;
        Serial.print("Pressure = ");
        Serial.print(pressure_bar);
        Serial.println(" bars");
        Serial.print("Pressure = "); {
        for(int a = 255; a>=0;a--) {
        analogWrite(6,a);
        if (pressure_bar = 1600){
        analogWrite (6, a);
        if (pressure_bar > 1600){
        analogWrite (6, a); }
        for(int a = 0; a<256; a++) {
        analogWrite(6, a); {
        if(pressure_bar < 1600){
        analogWrite (6, a); }
        for(int b = 255; b>=0;b--) {
        analogWrite(9, b); {
        if (pressure_bar == 1600){
        analogWrite (9, b);
        if(pressure_bar < 1600){
        analogWrite (9, b); }
        for(int b = 0; b<256; b++) {
        analogWrite(9, b); {
        if (pressure_bar > 1600){
        analogWrite (9, b);
        } }


        Use brackets and align them



        .. so each pair of { and } is on the same column and align code with brackets to be indented 2, 3 or 4 spaces



        int scvVal = 6; 
        int drvVal = 9;
        int sensorVal = A1;
        void setup()
        {
        Serial.begin(9600);
        pinMode(6, OUTPUT);
        pinMode(9, OUTPUT);
        }
        void loop()
        {
        sensorVal = analogRead(A1);

        Serial.print("Sensor Value: ");
        Serial.print(sensorVal);

        float voltage = (sensorVal*5.0)/1024.0; Serial.print("Volts: ");
        Serial.print(voltage);

        float pressure_pascal = (400.0*((float)voltage-0.5))*1000000.0;
        float pressure_bar = pressure_pascal/10e5;
        Serial.print("Pressure = ");
        Serial.print(pressure_bar);
        Serial.println(" bars");
        Serial.print("Pressure = ");
        {
        for(int a = 255; a>=0;a--)
        {
        analogWrite(6,a);
        if (pressure_bar = 1600)
        {
        analogWrite (6, a);
        if (pressure_bar > 1600)
        {
        analogWrite (6, a);
        }
        for(int a = 0; a<256; a++)
        {
        analogWrite(6, a);
        {
        if(pressure_bar < 1600)
        {
        analogWrite (6, a);
        }
        for(int b = 255; b>=0;b--)
        {
        analogWrite(9, b);
        {
        if (pressure_bar == 1600)
        {
        analogWrite (9, b);
        if(pressure_bar < 1600)
        {
        analogWrite (9, b);
        }
        for(int b = 0; b<256; b++)
        {
        analogWrite(9, b);
        {
        if (pressure_bar > 1600)
        {
        analogWrite (9, b);
        }
        }


        Remove superfluous brackets



        As you can see, you are having some superfluous/missing brackets. What you want is probably something like:



        int scvVal = 6; 
        int drvVal = 9;
        int sensorVal = A1;
        void setup()
        {
        Serial.begin(9600);
        pinMode(6, OUTPUT);
        pinMode(9, OUTPUT);
        }
        void loop()
        {
        sensorVal = analogRead(A1);

        Serial.print("Sensor Value: ");
        Serial.print(sensorVal);

        float voltage = (sensorVal*5.0)/1024.0; Serial.print("Volts: ");
        Serial.print(voltage);

        float pressure_pascal = (400.0*((float)voltage-0.5))*1000000.0;
        float pressure_bar = pressure_pascal/10e5;
        Serial.print("Pressure = ");
        Serial.print(pressure_bar);
        Serial.println(" bars");
        Serial.print("Pressure = ");

        for(int a = 255; a>=0;a--)
        {
        analogWrite(6,a);
        if (pressure_bar = 1600)
        {
        analogWrite (6, a);
        }
        if (pressure_bar > 1600)
        {
        analogWrite (6, a);
        }
        }
        for(int a = 0; a<256; a++)
        {
        analogWrite(6, a);
        if(pressure_bar < 1600)
        {
        analogWrite (6, a);
        }
        }
        for(int b = 255; b>=0;b--)
        {
        analogWrite(9, b);
        if (pressure_bar == 1600)
        {
        analogWrite (9, b);
        }
        if(pressure_bar < 1600)
        {
        analogWrite (9, b);
        }
        }
        for(int b = 0; b<256; b++)
        {
        analogWrite(9, b);
        if (pressure_bar > 1600)
        {
        analogWrite (9, b);
        }
        }
        }


        Remove duplicate code



        In the if statement, you perform twice the same code (both you send an analogWrite with the same value, so the code (from the for loops can be reduced to:



          for(int a = 255; a>=0;a--) 
        {
        analogWrite(6,a);
        if (pressure_bar >= 1600)
        {
        analogWrite (6, a);
        }
        }
        for(int a = 0; a<256; a++)
        {
        analogWrite(6, a);
        if(pressure_bar <= 1600)
        {
        analogWrite (6, a);
        }
        }
        for(int b = 255; b>=0;b--)
        {
        analogWrite(9, b);
        if (pressure_bar <= 1600)
        {
        analogWrite (9, b);
        }
        }
        for(int b = 0; b<256; b++)
        {
        analogWrite(9, b);
        if (pressure_bar >= 1600)
        {
        analogWrite (9, b);
        }
        }
        }


        Unclear



        What is unclear why you first write an analog value, and directly afterward (in each if) you write the same value. I'm not sure what you really want here. Either you want to write to a different pin, or you want a delay (to have the value having some effect), I'm just not sure.






        share|improve this answer




























          3












          3








          3







          Some guidelines for programming:



          Start each statement on a new line



          int scvVal = 6; 
          int drvVal = 9;
          int sensorVal = A1;
          void setup() {
          Serial.begin(9600);
          pinMode(6, OUTPUT);
          pinMode(9, OUTPUT);
          } void loop(){
          sensorVal = analogRead(A1);

          Serial.print("Sensor Value: "); Serial.print(sensorVal);

          float voltage = (sensorVal*5.0)/1024.0; Serial.print("Volts: "); Serial.print(voltage);

          float pressure_pascal = (400.0*((float)voltage-0.5))*1000000.0;
          float pressure_bar = pressure_pascal/10e5;
          Serial.print("Pressure = ");
          Serial.print(pressure_bar);
          Serial.println(" bars");
          Serial.print("Pressure = "); {
          for(int a = 255; a>=0;a--) {
          analogWrite(6,a);
          if (pressure_bar = 1600){
          analogWrite (6, a);
          if (pressure_bar > 1600){
          analogWrite (6, a); }
          for(int a = 0; a<256; a++) {
          analogWrite(6, a); {
          if(pressure_bar < 1600){
          analogWrite (6, a); }
          for(int b = 255; b>=0;b--) {
          analogWrite(9, b); {
          if (pressure_bar == 1600){
          analogWrite (9, b);
          if(pressure_bar < 1600){
          analogWrite (9, b); }
          for(int b = 0; b<256; b++) {
          analogWrite(9, b); {
          if (pressure_bar > 1600){
          analogWrite (9, b);
          } }


          Use brackets and align them



          .. so each pair of { and } is on the same column and align code with brackets to be indented 2, 3 or 4 spaces



          int scvVal = 6; 
          int drvVal = 9;
          int sensorVal = A1;
          void setup()
          {
          Serial.begin(9600);
          pinMode(6, OUTPUT);
          pinMode(9, OUTPUT);
          }
          void loop()
          {
          sensorVal = analogRead(A1);

          Serial.print("Sensor Value: ");
          Serial.print(sensorVal);

          float voltage = (sensorVal*5.0)/1024.0; Serial.print("Volts: ");
          Serial.print(voltage);

          float pressure_pascal = (400.0*((float)voltage-0.5))*1000000.0;
          float pressure_bar = pressure_pascal/10e5;
          Serial.print("Pressure = ");
          Serial.print(pressure_bar);
          Serial.println(" bars");
          Serial.print("Pressure = ");
          {
          for(int a = 255; a>=0;a--)
          {
          analogWrite(6,a);
          if (pressure_bar = 1600)
          {
          analogWrite (6, a);
          if (pressure_bar > 1600)
          {
          analogWrite (6, a);
          }
          for(int a = 0; a<256; a++)
          {
          analogWrite(6, a);
          {
          if(pressure_bar < 1600)
          {
          analogWrite (6, a);
          }
          for(int b = 255; b>=0;b--)
          {
          analogWrite(9, b);
          {
          if (pressure_bar == 1600)
          {
          analogWrite (9, b);
          if(pressure_bar < 1600)
          {
          analogWrite (9, b);
          }
          for(int b = 0; b<256; b++)
          {
          analogWrite(9, b);
          {
          if (pressure_bar > 1600)
          {
          analogWrite (9, b);
          }
          }


          Remove superfluous brackets



          As you can see, you are having some superfluous/missing brackets. What you want is probably something like:



          int scvVal = 6; 
          int drvVal = 9;
          int sensorVal = A1;
          void setup()
          {
          Serial.begin(9600);
          pinMode(6, OUTPUT);
          pinMode(9, OUTPUT);
          }
          void loop()
          {
          sensorVal = analogRead(A1);

          Serial.print("Sensor Value: ");
          Serial.print(sensorVal);

          float voltage = (sensorVal*5.0)/1024.0; Serial.print("Volts: ");
          Serial.print(voltage);

          float pressure_pascal = (400.0*((float)voltage-0.5))*1000000.0;
          float pressure_bar = pressure_pascal/10e5;
          Serial.print("Pressure = ");
          Serial.print(pressure_bar);
          Serial.println(" bars");
          Serial.print("Pressure = ");

          for(int a = 255; a>=0;a--)
          {
          analogWrite(6,a);
          if (pressure_bar = 1600)
          {
          analogWrite (6, a);
          }
          if (pressure_bar > 1600)
          {
          analogWrite (6, a);
          }
          }
          for(int a = 0; a<256; a++)
          {
          analogWrite(6, a);
          if(pressure_bar < 1600)
          {
          analogWrite (6, a);
          }
          }
          for(int b = 255; b>=0;b--)
          {
          analogWrite(9, b);
          if (pressure_bar == 1600)
          {
          analogWrite (9, b);
          }
          if(pressure_bar < 1600)
          {
          analogWrite (9, b);
          }
          }
          for(int b = 0; b<256; b++)
          {
          analogWrite(9, b);
          if (pressure_bar > 1600)
          {
          analogWrite (9, b);
          }
          }
          }


          Remove duplicate code



          In the if statement, you perform twice the same code (both you send an analogWrite with the same value, so the code (from the for loops can be reduced to:



            for(int a = 255; a>=0;a--) 
          {
          analogWrite(6,a);
          if (pressure_bar >= 1600)
          {
          analogWrite (6, a);
          }
          }
          for(int a = 0; a<256; a++)
          {
          analogWrite(6, a);
          if(pressure_bar <= 1600)
          {
          analogWrite (6, a);
          }
          }
          for(int b = 255; b>=0;b--)
          {
          analogWrite(9, b);
          if (pressure_bar <= 1600)
          {
          analogWrite (9, b);
          }
          }
          for(int b = 0; b<256; b++)
          {
          analogWrite(9, b);
          if (pressure_bar >= 1600)
          {
          analogWrite (9, b);
          }
          }
          }


          Unclear



          What is unclear why you first write an analog value, and directly afterward (in each if) you write the same value. I'm not sure what you really want here. Either you want to write to a different pin, or you want a delay (to have the value having some effect), I'm just not sure.






          share|improve this answer















          Some guidelines for programming:



          Start each statement on a new line



          int scvVal = 6; 
          int drvVal = 9;
          int sensorVal = A1;
          void setup() {
          Serial.begin(9600);
          pinMode(6, OUTPUT);
          pinMode(9, OUTPUT);
          } void loop(){
          sensorVal = analogRead(A1);

          Serial.print("Sensor Value: "); Serial.print(sensorVal);

          float voltage = (sensorVal*5.0)/1024.0; Serial.print("Volts: "); Serial.print(voltage);

          float pressure_pascal = (400.0*((float)voltage-0.5))*1000000.0;
          float pressure_bar = pressure_pascal/10e5;
          Serial.print("Pressure = ");
          Serial.print(pressure_bar);
          Serial.println(" bars");
          Serial.print("Pressure = "); {
          for(int a = 255; a>=0;a--) {
          analogWrite(6,a);
          if (pressure_bar = 1600){
          analogWrite (6, a);
          if (pressure_bar > 1600){
          analogWrite (6, a); }
          for(int a = 0; a<256; a++) {
          analogWrite(6, a); {
          if(pressure_bar < 1600){
          analogWrite (6, a); }
          for(int b = 255; b>=0;b--) {
          analogWrite(9, b); {
          if (pressure_bar == 1600){
          analogWrite (9, b);
          if(pressure_bar < 1600){
          analogWrite (9, b); }
          for(int b = 0; b<256; b++) {
          analogWrite(9, b); {
          if (pressure_bar > 1600){
          analogWrite (9, b);
          } }


          Use brackets and align them



          .. so each pair of { and } is on the same column and align code with brackets to be indented 2, 3 or 4 spaces



          int scvVal = 6; 
          int drvVal = 9;
          int sensorVal = A1;
          void setup()
          {
          Serial.begin(9600);
          pinMode(6, OUTPUT);
          pinMode(9, OUTPUT);
          }
          void loop()
          {
          sensorVal = analogRead(A1);

          Serial.print("Sensor Value: ");
          Serial.print(sensorVal);

          float voltage = (sensorVal*5.0)/1024.0; Serial.print("Volts: ");
          Serial.print(voltage);

          float pressure_pascal = (400.0*((float)voltage-0.5))*1000000.0;
          float pressure_bar = pressure_pascal/10e5;
          Serial.print("Pressure = ");
          Serial.print(pressure_bar);
          Serial.println(" bars");
          Serial.print("Pressure = ");
          {
          for(int a = 255; a>=0;a--)
          {
          analogWrite(6,a);
          if (pressure_bar = 1600)
          {
          analogWrite (6, a);
          if (pressure_bar > 1600)
          {
          analogWrite (6, a);
          }
          for(int a = 0; a<256; a++)
          {
          analogWrite(6, a);
          {
          if(pressure_bar < 1600)
          {
          analogWrite (6, a);
          }
          for(int b = 255; b>=0;b--)
          {
          analogWrite(9, b);
          {
          if (pressure_bar == 1600)
          {
          analogWrite (9, b);
          if(pressure_bar < 1600)
          {
          analogWrite (9, b);
          }
          for(int b = 0; b<256; b++)
          {
          analogWrite(9, b);
          {
          if (pressure_bar > 1600)
          {
          analogWrite (9, b);
          }
          }


          Remove superfluous brackets



          As you can see, you are having some superfluous/missing brackets. What you want is probably something like:



          int scvVal = 6; 
          int drvVal = 9;
          int sensorVal = A1;
          void setup()
          {
          Serial.begin(9600);
          pinMode(6, OUTPUT);
          pinMode(9, OUTPUT);
          }
          void loop()
          {
          sensorVal = analogRead(A1);

          Serial.print("Sensor Value: ");
          Serial.print(sensorVal);

          float voltage = (sensorVal*5.0)/1024.0; Serial.print("Volts: ");
          Serial.print(voltage);

          float pressure_pascal = (400.0*((float)voltage-0.5))*1000000.0;
          float pressure_bar = pressure_pascal/10e5;
          Serial.print("Pressure = ");
          Serial.print(pressure_bar);
          Serial.println(" bars");
          Serial.print("Pressure = ");

          for(int a = 255; a>=0;a--)
          {
          analogWrite(6,a);
          if (pressure_bar = 1600)
          {
          analogWrite (6, a);
          }
          if (pressure_bar > 1600)
          {
          analogWrite (6, a);
          }
          }
          for(int a = 0; a<256; a++)
          {
          analogWrite(6, a);
          if(pressure_bar < 1600)
          {
          analogWrite (6, a);
          }
          }
          for(int b = 255; b>=0;b--)
          {
          analogWrite(9, b);
          if (pressure_bar == 1600)
          {
          analogWrite (9, b);
          }
          if(pressure_bar < 1600)
          {
          analogWrite (9, b);
          }
          }
          for(int b = 0; b<256; b++)
          {
          analogWrite(9, b);
          if (pressure_bar > 1600)
          {
          analogWrite (9, b);
          }
          }
          }


          Remove duplicate code



          In the if statement, you perform twice the same code (both you send an analogWrite with the same value, so the code (from the for loops can be reduced to:



            for(int a = 255; a>=0;a--) 
          {
          analogWrite(6,a);
          if (pressure_bar >= 1600)
          {
          analogWrite (6, a);
          }
          }
          for(int a = 0; a<256; a++)
          {
          analogWrite(6, a);
          if(pressure_bar <= 1600)
          {
          analogWrite (6, a);
          }
          }
          for(int b = 255; b>=0;b--)
          {
          analogWrite(9, b);
          if (pressure_bar <= 1600)
          {
          analogWrite (9, b);
          }
          }
          for(int b = 0; b<256; b++)
          {
          analogWrite(9, b);
          if (pressure_bar >= 1600)
          {
          analogWrite (9, b);
          }
          }
          }


          Unclear



          What is unclear why you first write an analog value, and directly afterward (in each if) you write the same value. I'm not sure what you really want here. Either you want to write to a different pin, or you want a delay (to have the value having some effect), I'm just not sure.







          share|improve this answer














          share|improve this answer



          share|improve this answer








          edited 12 hours ago

























          answered 13 hours ago









          Michel KeijzersMichel Keijzers

          6,97251939




          6,97251939























              1














              Look at your open brackets { vs closing brackets, }. You need the same number of each. If have this in the Arduino IDE and use CTRL-T, they are somewhat easy to find. I just looked, and I can spot them pretty easily.



              int scvVal = 6; 
              int drvVal = 9;
              int sensorVal = A1;
              void setup()
              {
              Serial.begin(9600);
              pinMode(6, OUTPUT);
              pinMode(9, OUTPUT);
              }
              void loop()
              {
              sensorVal = analogRead(A1);
              Serial.print("Sensor Value: ");
              Serial.print(sensorVal);
              float voltage = (sensorVal*5.0)/1024.0;
              Serial.print("Volts: ");
              Serial.print(voltage);
              float pressure_pascal = (400.0*((float)voltage-0.5))*1000000.0;
              float pressure_bar = pressure_pascal/10e5;
              Serial.print("Pressure = ");
              Serial.print(pressure_bar);
              Serial.println(" bars");
              Serial.print("Pressure = ");
              { << bracket for what?
              for(int a = 255; a>=0;a--) { << missing closing bracket
              analogWrite(6,a);
              if (pressure_bar = 1600){ << missing closing bracket
              analogWrite (6, a);
              if (pressure_bar > 1600){
              analogWrite (6, a);
              }
              for(int a = 0; a<256; a++) {
              analogWrite(6, a);
              { << wrong direction
              if(pressure_bar < 1600){
              analogWrite (6, a);
              }
              for(int b = 255; b>=0;b--) {
              analogWrite(9, b);
              { << wrong direction
              if (pressure_bar == 1600){ << missing closing bracket
              analogWrite (9, b);
              if(pressure_bar < 1600){ << missing closing bracket
              analogWrite (9, b);
              }
              for(int b = 0; b<256; b++) {
              analogWrite(9, b);
              { << wrong direction
              if (pressure_bar > 1600){
              analogWrite (9, b);
              }
              } << closing bracket for loop





              share|improve this answer




























                1














                Look at your open brackets { vs closing brackets, }. You need the same number of each. If have this in the Arduino IDE and use CTRL-T, they are somewhat easy to find. I just looked, and I can spot them pretty easily.



                int scvVal = 6; 
                int drvVal = 9;
                int sensorVal = A1;
                void setup()
                {
                Serial.begin(9600);
                pinMode(6, OUTPUT);
                pinMode(9, OUTPUT);
                }
                void loop()
                {
                sensorVal = analogRead(A1);
                Serial.print("Sensor Value: ");
                Serial.print(sensorVal);
                float voltage = (sensorVal*5.0)/1024.0;
                Serial.print("Volts: ");
                Serial.print(voltage);
                float pressure_pascal = (400.0*((float)voltage-0.5))*1000000.0;
                float pressure_bar = pressure_pascal/10e5;
                Serial.print("Pressure = ");
                Serial.print(pressure_bar);
                Serial.println(" bars");
                Serial.print("Pressure = ");
                { << bracket for what?
                for(int a = 255; a>=0;a--) { << missing closing bracket
                analogWrite(6,a);
                if (pressure_bar = 1600){ << missing closing bracket
                analogWrite (6, a);
                if (pressure_bar > 1600){
                analogWrite (6, a);
                }
                for(int a = 0; a<256; a++) {
                analogWrite(6, a);
                { << wrong direction
                if(pressure_bar < 1600){
                analogWrite (6, a);
                }
                for(int b = 255; b>=0;b--) {
                analogWrite(9, b);
                { << wrong direction
                if (pressure_bar == 1600){ << missing closing bracket
                analogWrite (9, b);
                if(pressure_bar < 1600){ << missing closing bracket
                analogWrite (9, b);
                }
                for(int b = 0; b<256; b++) {
                analogWrite(9, b);
                { << wrong direction
                if (pressure_bar > 1600){
                analogWrite (9, b);
                }
                } << closing bracket for loop





                share|improve this answer


























                  1












                  1








                  1







                  Look at your open brackets { vs closing brackets, }. You need the same number of each. If have this in the Arduino IDE and use CTRL-T, they are somewhat easy to find. I just looked, and I can spot them pretty easily.



                  int scvVal = 6; 
                  int drvVal = 9;
                  int sensorVal = A1;
                  void setup()
                  {
                  Serial.begin(9600);
                  pinMode(6, OUTPUT);
                  pinMode(9, OUTPUT);
                  }
                  void loop()
                  {
                  sensorVal = analogRead(A1);
                  Serial.print("Sensor Value: ");
                  Serial.print(sensorVal);
                  float voltage = (sensorVal*5.0)/1024.0;
                  Serial.print("Volts: ");
                  Serial.print(voltage);
                  float pressure_pascal = (400.0*((float)voltage-0.5))*1000000.0;
                  float pressure_bar = pressure_pascal/10e5;
                  Serial.print("Pressure = ");
                  Serial.print(pressure_bar);
                  Serial.println(" bars");
                  Serial.print("Pressure = ");
                  { << bracket for what?
                  for(int a = 255; a>=0;a--) { << missing closing bracket
                  analogWrite(6,a);
                  if (pressure_bar = 1600){ << missing closing bracket
                  analogWrite (6, a);
                  if (pressure_bar > 1600){
                  analogWrite (6, a);
                  }
                  for(int a = 0; a<256; a++) {
                  analogWrite(6, a);
                  { << wrong direction
                  if(pressure_bar < 1600){
                  analogWrite (6, a);
                  }
                  for(int b = 255; b>=0;b--) {
                  analogWrite(9, b);
                  { << wrong direction
                  if (pressure_bar == 1600){ << missing closing bracket
                  analogWrite (9, b);
                  if(pressure_bar < 1600){ << missing closing bracket
                  analogWrite (9, b);
                  }
                  for(int b = 0; b<256; b++) {
                  analogWrite(9, b);
                  { << wrong direction
                  if (pressure_bar > 1600){
                  analogWrite (9, b);
                  }
                  } << closing bracket for loop





                  share|improve this answer













                  Look at your open brackets { vs closing brackets, }. You need the same number of each. If have this in the Arduino IDE and use CTRL-T, they are somewhat easy to find. I just looked, and I can spot them pretty easily.



                  int scvVal = 6; 
                  int drvVal = 9;
                  int sensorVal = A1;
                  void setup()
                  {
                  Serial.begin(9600);
                  pinMode(6, OUTPUT);
                  pinMode(9, OUTPUT);
                  }
                  void loop()
                  {
                  sensorVal = analogRead(A1);
                  Serial.print("Sensor Value: ");
                  Serial.print(sensorVal);
                  float voltage = (sensorVal*5.0)/1024.0;
                  Serial.print("Volts: ");
                  Serial.print(voltage);
                  float pressure_pascal = (400.0*((float)voltage-0.5))*1000000.0;
                  float pressure_bar = pressure_pascal/10e5;
                  Serial.print("Pressure = ");
                  Serial.print(pressure_bar);
                  Serial.println(" bars");
                  Serial.print("Pressure = ");
                  { << bracket for what?
                  for(int a = 255; a>=0;a--) { << missing closing bracket
                  analogWrite(6,a);
                  if (pressure_bar = 1600){ << missing closing bracket
                  analogWrite (6, a);
                  if (pressure_bar > 1600){
                  analogWrite (6, a);
                  }
                  for(int a = 0; a<256; a++) {
                  analogWrite(6, a);
                  { << wrong direction
                  if(pressure_bar < 1600){
                  analogWrite (6, a);
                  }
                  for(int b = 255; b>=0;b--) {
                  analogWrite(9, b);
                  { << wrong direction
                  if (pressure_bar == 1600){ << missing closing bracket
                  analogWrite (9, b);
                  if(pressure_bar < 1600){ << missing closing bracket
                  analogWrite (9, b);
                  }
                  for(int b = 0; b<256; b++) {
                  analogWrite(9, b);
                  { << wrong direction
                  if (pressure_bar > 1600){
                  analogWrite (9, b);
                  }
                  } << closing bracket for loop






                  share|improve this answer












                  share|improve this answer



                  share|improve this answer










                  answered 13 hours ago









                  CrossRoadsCrossRoads

                  1,58117




                  1,58117






















                      Đức Hòang is a new contributor. Be nice, and check out our Code of Conduct.










                      draft saved

                      draft discarded


















                      Đức Hòang is a new contributor. Be nice, and check out our Code of Conduct.













                      Đức Hòang is a new contributor. Be nice, and check out our Code of Conduct.












                      Đức Hòang is a new contributor. Be nice, and check out our Code of Conduct.
















                      Thanks for contributing an answer to Arduino Stack Exchange!


                      • Please be sure to answer the question. Provide details and share your research!

                      But avoid



                      • Asking for help, clarification, or responding to other answers.

                      • Making statements based on opinion; back them up with references or personal experience.


                      To learn more, see our tips on writing great answers.




                      draft saved


                      draft discarded














                      StackExchange.ready(
                      function () {
                      StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2farduino.stackexchange.com%2fquestions%2f63354%2fexpected-at-end-of-input%23new-answer', 'question_page');
                      }
                      );

                      Post as a guest















                      Required, but never shown





















































                      Required, but never shown














                      Required, but never shown












                      Required, but never shown







                      Required, but never shown

































                      Required, but never shown














                      Required, but never shown












                      Required, but never shown







                      Required, but never shown







                      Popular posts from this blog

                      Færeyskur hestur Heimild | Tengill | Tilvísanir | LeiðsagnarvalRossið - síða um færeyska hrossið á færeyskuGott ár hjá færeyska hestinum

                      He _____ here since 1970 . Answer needed [closed]What does “since he was so high” mean?Meaning of “catch birds for”?How do I ensure “since” takes the meaning I want?“Who cares here” meaningWhat does “right round toward” mean?the time tense (had now been detected)What does the phrase “ring around the roses” mean here?Correct usage of “visited upon”Meaning of “foiled rail sabotage bid”It was the third time I had gone to Rome or It is the third time I had been to Rome

                      Slayer Innehåll Historia | Stil, komposition och lyrik | Bandets betydelse och framgångar | Sidoprojekt och samarbeten | Kontroverser | Medlemmar | Utmärkelser och nomineringar | Turnéer och festivaler | Diskografi | Referenser | Externa länkar | Navigeringsmenywww.slayer.net”Metal Massacre vol. 1””Metal Massacre vol. 3””Metal Massacre Volume III””Show No Mercy””Haunting the Chapel””Live Undead””Hell Awaits””Reign in Blood””Reign in Blood””Gold & Platinum – Reign in Blood””Golden Gods Awards Winners”originalet”Kerrang! Hall Of Fame””Slayer Looks Back On 37-Year Career In New Video Series: Part Two””South of Heaven””Gold & Platinum – South of Heaven””Seasons in the Abyss””Gold & Platinum - Seasons in the Abyss””Divine Intervention””Divine Intervention - Release group by Slayer””Gold & Platinum - Divine Intervention””Live Intrusion””Undisputed Attitude””Abolish Government/Superficial Love””Release “Slatanic Slaughter: A Tribute to Slayer” by Various Artists””Diabolus in Musica””Soundtrack to the Apocalypse””God Hates Us All””Systematic - Relationships””War at the Warfield””Gold & Platinum - War at the Warfield””Soundtrack to the Apocalypse””Gold & Platinum - Still Reigning””Metallica, Slayer, Iron Mauden Among Winners At Metal Hammer Awards””Eternal Pyre””Eternal Pyre - Slayer release group””Eternal Pyre””Metal Storm Awards 2006””Kerrang! Hall Of Fame””Slayer Wins 'Best Metal' Grammy Award””Slayer Guitarist Jeff Hanneman Dies””Bullet-For My Valentine booed at Metal Hammer Golden Gods Awards””Unholy Aliance””The End Of Slayer?””Slayer: We Could Thrash Out Two More Albums If We're Fast Enough...””'The Unholy Alliance: Chapter III' UK Dates Added”originalet”Megadeth And Slayer To Co-Headline 'Canadian Carnage' Trek”originalet”World Painted Blood””Release “World Painted Blood” by Slayer””Metallica Heading To Cinemas””Slayer, Megadeth To Join Forces For 'European Carnage' Tour - Dec. 18, 2010”originalet”Slayer's Hanneman Contracts Acute Infection; Band To Bring In Guest Guitarist””Cannibal Corpse's Pat O'Brien Will Step In As Slayer's Guest Guitarist”originalet”Slayer’s Jeff Hanneman Dead at 49””Dave Lombardo Says He Made Only $67,000 In 2011 While Touring With Slayer””Slayer: We Do Not Agree With Dave Lombardo's Substance Or Timeline Of Events””Slayer Welcomes Drummer Paul Bostaph Back To The Fold””Slayer Hope to Unveil Never-Before-Heard Jeff Hanneman Material on Next Album””Slayer Debut New Song 'Implode' During Surprise Golden Gods Appearance””Release group Repentless by Slayer””Repentless - Slayer - Credits””Slayer””Metal Storm Awards 2015””Slayer - to release comic book "Repentless #1"””Slayer To Release 'Repentless' 6.66" Vinyl Box Set””BREAKING NEWS: Slayer Announce Farewell Tour””Slayer Recruit Lamb of God, Anthrax, Behemoth + Testament for Final Tour””Slayer lägger ner efter 37 år””Slayer Announces Second North American Leg Of 'Final' Tour””Final World Tour””Slayer Announces Final European Tour With Lamb of God, Anthrax And Obituary””Slayer To Tour Europe With Lamb of God, Anthrax And Obituary””Slayer To Play 'Last French Show Ever' At Next Year's Hellfst””Slayer's Final World Tour Will Extend Into 2019””Death Angel's Rob Cavestany On Slayer's 'Farewell' Tour: 'Some Of Us Could See This Coming'””Testament Has No Plans To Retire Anytime Soon, Says Chuck Billy””Anthrax's Scott Ian On Slayer's 'Farewell' Tour Plans: 'I Was Surprised And I Wasn't Surprised'””Slayer””Slayer's Morbid Schlock””Review/Rock; For Slayer, the Mania Is the Message””Slayer - Biography””Slayer - Reign In Blood”originalet”Dave Lombardo””An exclusive oral history of Slayer”originalet”Exclusive! Interview With Slayer Guitarist Jeff Hanneman”originalet”Thinking Out Loud: Slayer's Kerry King on hair metal, Satan and being polite””Slayer Lyrics””Slayer - Biography””Most influential artists for extreme metal music””Slayer - Reign in Blood””Slayer guitarist Jeff Hanneman dies aged 49””Slatanic Slaughter: A Tribute to Slayer””Gateway to Hell: A Tribute to Slayer””Covered In Blood””Slayer: The Origins of Thrash in San Francisco, CA.””Why They Rule - #6 Slayer”originalet”Guitar World's 100 Greatest Heavy Metal Guitarists Of All Time”originalet”The fans have spoken: Slayer comes out on top in readers' polls”originalet”Tribute to Jeff Hanneman (1964-2013)””Lamb Of God Frontman: We Sound Like A Slayer Rip-Off””BEHEMOTH Frontman Pays Tribute To SLAYER's JEFF HANNEMAN””Slayer, Hatebreed Doing Double Duty On This Year's Ozzfest””System of a Down””Lacuna Coil’s Andrea Ferro Talks Influences, Skateboarding, Band Origins + More””Slayer - Reign in Blood””Into The Lungs of Hell””Slayer rules - en utställning om fans””Slayer and Their Fans Slashed Through a No-Holds-Barred Night at Gas Monkey””Home””Slayer””Gold & Platinum - The Big 4 Live from Sofia, Bulgaria””Exclusive! Interview With Slayer Guitarist Kerry King””2008-02-23: Wiltern, Los Angeles, CA, USA””Slayer's Kerry King To Perform With Megadeth Tonight! - Oct. 21, 2010”originalet”Dave Lombardo - Biography”Slayer Case DismissedArkiveradUltimate Classic Rock: Slayer guitarist Jeff Hanneman dead at 49.”Slayer: "We could never do any thing like Some Kind Of Monster..."””Cannibal Corpse'S Pat O'Brien Will Step In As Slayer'S Guest Guitarist | The Official Slayer Site”originalet”Slayer Wins 'Best Metal' Grammy Award””Slayer Guitarist Jeff Hanneman Dies””Kerrang! Awards 2006 Blog: Kerrang! Hall Of Fame””Kerrang! Awards 2013: Kerrang! Legend”originalet”Metallica, Slayer, Iron Maien Among Winners At Metal Hammer Awards””Metal Hammer Golden Gods Awards””Bullet For My Valentine Booed At Metal Hammer Golden Gods Awards””Metal Storm Awards 2006””Metal Storm Awards 2015””Slayer's Concert History””Slayer - Relationships””Slayer - Releases”Slayers officiella webbplatsSlayer på MusicBrainzOfficiell webbplatsSlayerSlayerr1373445760000 0001 1540 47353068615-5086262726cb13906545x(data)6033143kn20030215029