Calculations do not work

Follow

Stijn

• March 01, 2016 18:37

Hello!

Can anyone tell me why I get errors in my calculations?

please look http://screencast.com/t/MEUSeApLYJL

or: ((@Maat boor === "16 mm") && (@Aantal verdiepingen === "1 verdieping")) {
15;
} else {
"";
}

and this one

if(@Voornaam && @Tussenvoegsels && @Achternaam) {

@Voornaam + ' ' + @Tussenvoegsels + ' ' + @Achternaam;

} else if (@Voornaam) {

@Voornaam;

} else if (@Tussenvoegsels) {

@Tussenvoegsels;

} else if (@Achternaam) {

@Achternaam;

} else {

"";

}

 

this was my example from the pdf provided by podio

 

if(@Name && @Company) { @Name + ", " + @Company; } else if (@Name) { @Name; } else if (@Company) { @Company; } else { ""; }

 

Hope someone can help me out!

0

• 

  Rainer Grabowski

  • March 02, 2016 12:34

  Hi Stijn,

  I think in the first calculation there's missing an if: 

  if(@Maat boor == "16 mm" && @Aantal verdiepingen == "1 verdieping") {
  15;
  } else {
  "";
  }

  You can write it also this way: 

  @Maat boor == "16 mm" && @Aantal verdiepingen == "1 verdieping" ? 15 : ""
  
  

  I assume, the second code you need to avoid that there "null" is shown if a field is empty.The code looks correct - do you get an error notification for it too? 

  I would do it in shorter way:

  (@Voornaam + ' ' + @Tussenvoegsels + ' ' + @Achternaam).replace(/(null )|(null)/g,"")

  One tip: If you have to use the same variable token mulitple times a code you should declare it as var in the beginning, so you have to select the token only once from the popup list (this makes the code more readable and you can cope parts of it): 
  
  

  var vn = @Voornaam;
  var tv = @Tussenvoegsels;
  var an = @Achternaam;
  if(vn && tn && @an) {
  vn + ' ' + tn + ' ' + an;
  ...
  and so on 
  
  

  Here you find some rules and tips for using variables.

  Rainer

   

  1

  Comment actions Permalink
• 

  Henrik Huhtinen

  • March 03, 2016 18:44

  Well, doesn't the first one miss "if" from the beginning?
  
  Also, remember that calculation field's type is set by its first return. If it returns a number first, it wants to return a number. If it returns text, it wants to return text. To reset this the calculation field needs to be deleted and recreated. 

  1

  Comment actions Permalink
• 

  Jean-Claude Massé

  • March 04, 2016 18:38

  The first one, is because you don't call anything first (like if) try 

  if((@Maat boor == "16 mm" && @Aantal verdiepingen == "1 verdieping")) {
  15;
  } else {
  "";
  }

  In the second one, all your @field are compare to... nothing!

  1

  Comment actions Permalink
• 

  Jean-Claude Massé

  • March 04, 2016 18:39

  You can also use something like

  @Maat boor == "16 mm" && @Aantal verdiepingen == "1 verdieping" ? 15 : ""

  1

  Comment actions Permalink
• 

  Stijn

  • March 05, 2016 22:24

  Thanx everyone for the help and input!

   

  0

  Comment actions Permalink

Please sign in to leave a comment.