Changeset 15004 for trunk/FACT++/scripts

Timestamp:

03/09/13 16:27:00 (12 years ago)

Author:

tbretz

Message:

Code cleanup.

File:

• trunk/FACT++/scripts/Observation_class.js (modified) (1 diff)

trunk/FACT++/scripts/Observation_class.js

@@ -1 +1 @@
 'use strict';
 
-/* **************
- * this file contains just the implementation of the 
- * Observation class (I know there are no classes in javascript...)
- */
+//
+// this file contains just the implementation of the
+// Observation class (I know there are no classes in javascript...)
+//
 
-// Constructor
-function Observation ( oArg ) {
+function Observation(obj)
+{
+    if (typof(obj)!='object')
+        throw new Error("Observation object can only be constructed using an object.");
+
+    if (!obj.date)
+        throw new Error("Observation object must have a 'date' parameter");
+
+    // FIXME: Check transisiton from summer- and winter-time!!
+    var utc = new Date(obj.date);
+    if (isNaN(utc.valueOf()))
+        throw new Error(obj.date+' is not a valid Date String.'+
+                        ' Try something like "2013-01-08 23:05 UTC" .');
+
+    this.start  = utc;
+    this.task   = obj.task ? oArg.task.toUpperCase() : "DATA";
+    this.source = obj.source;
+    this.ra     = obj.ra;
+    this.dec    = obj.dec;
     
-    if (oArg['date'])
+    switch (this.task)
     {
-        // FIXME: Check transisiton from summer- and winter-time!!
-        var utc = new Date(oArg.date);
-        if (isNaN(utc.valueOf()))
-        {
-            throw new Error(startDateString+' is not a valid Date String.'+
-                        ' Try something like "2013-01-08 23:05 UTC" .');
-        }
-        this.start = utc
+    case 'DATA':
+        if (this.source == undefined)
+            throw new Error("Observation must have either 'source' or 'task' " +
+                            "if 'task' == 'data' it must have also have 'source' ");
+        break;
+
+    case 'STARTUP':
+        if (this.source != undefined)
+            console.out("warning. Observation with task='startup' also has source defined");
+        break;
+
+    case 'SHUTDOWN':
+        if (this.source != undefined)
+            console.out("warning. Observation with task='shutdown' also has source defined");
+        break;
+
+    case 'RATESCAN':
+        if (this.source == undefined && (this.ra == undefined || this.dec == undefined))
+            throw new Error("Observation must have either 'source' or 'ra' & 'dec' " +
+                            "if 'task' == 'ratescan'");
+        break;
+
+    default:
+        throw new Error(" the task of this observation:"+this.task+
+                        "is not implemented. use one of: DATA, STARTUP, SHUTDOWN, RATESCAN.");
     }
-    else
-    {
-        throw new Error("Observation object must have a 'date' parameter");
-    }
-    
-    if (oArg['task'])
-        this.task = oArg.task.toUpperCase()
-    else
-        this.task = 'DATA'
-        
-    if (oArg['source'])
-        this.source = oArg.source
-    else
-        this.source = undefined
-        
-    if (oArg['ra'])
-        this.ra = oArg.ra
-    else
-        this.ra = undefined
-        
-    if (oArg['dec'])
-        this.dec = oArg.dec
-    else
-        this.dec = undefined
-    
-    
-    
-    switch( this.task)
-    {
-        case 'DATA':
-            if (this.source == undefined)
-                throw new Error(
-                    "Observation must have either 'source' or 'task' " +
-                    "if 'task' == 'data' it must have also have 'source' ");
-            break;
-        case 'STARTUP':
-            if ( this.source != undefined)
-            {
-                console.out("warning. Observation with task='startup' also has source defined");
-            }
-            break;
-        case 'SHUTDOWN':
-            if ( this.source != undefined)
-            {
-                console.out("warning. Observation with task='shutdown' also has source defined");
-            }
-            break;
-        case 'RATESCAN':
-            if ( this.source == undefined && (this.ra == undefined || this.dec == undefined) )
-            {
-                throw new Error(
-                    "Observation must have either 'source' or 'ra' & 'dec' " +
-                    "if 'task' == 'ratescan'");
-            }
-            break;
-        default:
-            throw new Error(" the task of this observation:"+this.task+ 
-                            "is not implemented. use one of: DATA, STARTUP, SHUTDOWN, RATESCAN.");
-    }
-    
-    
 }
 
 // method toString()
 Observation.prototype.toString = function() 
-{ 
+{
     if (this.source)
         return this.task + "  " + this.source+" ["+this.start+"]" ;
     else
         return this.task + " ["+this.start+"]" ;
-
 }