Update moreJs-6.js

heartwhite · web-flow · commit 9e93e786b91a · 2019-05-24T13:26:37.000+02:00
1 - condition's (function) parameter changed as 'age' from 'years'. this way more consistent with lines above.
2- if statement added for the value of 'type' parameter. console logs the vehicle only if it has vehicle type. otherwise logs an warning
diff --git a/Week3/moreJs-6.js b/Week3/moreJs-6.js
@@ -5,15 +5,19 @@ Create a function called vehicle, like before,
  prints 'a blue used car'
 */
 
-function vehicle (color, type, age){
-    let types = [' car' , ' motorbike'];
-    console.log('a ' + color +' ' + condition(age) + types[(type-1)]);
+function vehicle(color, type, age) {
+  let types = [' car', ' motorbike', 'truck', 'bus'];
+  if (type > 0 && type <= types.length) {
+    console.log('a ' + color + ' ' + condition(age) + types[type - 1]);
+  } else {
+    console.log('!NO SUCH VEHICLE HERE');
+  }
 }
-function condition(years){
-    if (years<2){
-        return 'new';
-    } else {
-        return 'used';
-    }
+function condition(age) {
+  if (age < 2) {
+    return 'new ';
+  } else {
+    return 'used ';
+  }
 }
-vehicle('red' , 1, 3);
+vehicle('red', 3, 3);
