Package script:

tar jar and run.sh

package.sh

@@ -0,0 +1,14 @@
+#!/bin/bash
+
+GIT=$(command -v git)
+HEAD=$(command -v head)
+CUT=$(command -v cut)
+
+JAR_ARTIFACT="out/artifacts/Hajj_simulation_jar/Hajj-simulation.jar"
+
+if [ -z $"GIT" ] || [ -z $"HEAD" ] || [ -z $"CUT" ]; then
+	echo "Tools missing"
+	exit 1
+else
+	tar -cvf Hajj-Simulation-$(git log | head -n1 | cut -d' ' -f2).tar.gz $JAR_ARTIFACT run.sh
+fi

run.sh

@@ -0,0 +1,9 @@
+#!/bin/bash
+
+jav=$(command -v java)
+
+if [ -z "$jav" ]; then
+	>&2 echo "Can't find java installation"
+else
+	java -jar out/artifacts/Hajj_simulation_jar/Hajj-simulation.jar
+fi

src/Bus.java

@@ -1,11 +1,11 @@
 
 public class Bus extends CivilVehicle {
 	
-	private int UID;
+	private String UID;
 	private Campaign campaign;
 	private static int numeberOfBuses;
-    private final short TIME_TO_FIX = 20; //in minutes
+    private final int TIME_TO_FIX = 20; //in minutes
-    public static final short MAX_FORWARD = 900; //Meter/Min
+    public static final int MAX_FORWARD = 900; //Meter/Min
 
     public static final double STD_BUS_SIZE = 10;
 
@@ -40,11 +40,11 @@ public class Bus extends CivilVehicle {
 
     private void generateUID() {
         numeberOfBuses++;
-        this.UID = numeberOfBuses;
+        this.UID = String.format("BUS%04d", numeberOfBuses);
     }
 
     public String getUID(){
-        return String.format("BUS%04d", UID);
+        return this.UID;
     }
 
     public Campaign getCampaign() {

src/GUI_History.java

@@ -467,6 +467,7 @@ public class GUI_History {
 	}
 
 	public String avgTimeOfTrip() {
+		//TODO: does output diff value even after all have arrived.
 		Calendar now = new GregorianCalendar();
 		now.setTime(currenttimeManager);
 		Calendar from = (GregorianCalendar)now.clone();

src/GUI_ViewStreet.java

@@ -251,7 +251,7 @@ public class GUI_ViewStreet {
 		if (vehicles.isEmpty()) return;
 		vehicleData = new Object[vehicles.size()][6];
 		for (int i = 0; i < vehicles.size(); i++) {
-			vehicleData[i][0] = vehicles.get(i).getUID();
+			vehicleData[i][0] = vehicles.get(i).getUID();// TODO: There is an Exception error here;
 			if (vehicles.get(i) instanceof Bus)
 				vehicleData[i][1] = ((Bus)vehicles.get(i)).getCampaign().getHotelDistrict().name();
 			else vehicleData[i][1] = "Local Vehicle";

src/MakkahCity.java

@@ -56,6 +56,7 @@ public class MakkahCity {
 
 	public static void main(String[] args) {
 
+		//TODO: fix state errors
 		t.start();
 		//Gen Camp
 		campPerDistrict[District.ALMANSOOR.ordinal()] = new ArrayList<>();
@@ -438,7 +439,7 @@ public class MakkahCity {
 			clearDoneCivilVehicles();
 			addCivilVehicleNoise();
 			for (Vehicle vehicle : listOfVehicles) {
-				if (vehicle.getRoute() == null || vehicle.isArrivedToDest())
+				if (vehicle.getRoute() == null)
 					continue;
 				Route route = vehicle.getRoute();
 				double currentLocation = vehicle.getCurrentLocation();
@@ -457,7 +458,10 @@ public class MakkahCity {
 					if (!vehicle.isArrivedToDest()) {
 						double factor = 1-(vehicle.getCurrentStreet().capcityPoint(vehicle.getCurrentLocation(),
 										vehicle.getCurrentLocation()+1000,vehicle)) ;
-						vehicle.move(vehicle.getMaxSpeed()*factor);
+						if (vehicle instanceof Bus) vehicle.move(Bus.MAX_FORWARD * factor );
+						else if (vehicle instanceof Sedan) vehicle.move(Sedan.MAX_FORWARD * factor );
+						else if (vehicle instanceof SUV) vehicle.move(SUV.MAX_FORWARD * factor );
+						else if (vehicle instanceof Truck) vehicle.move(Bus.MAX_FORWARD * factor );
 					}
 				}
 			}
@@ -499,7 +503,7 @@ public class MakkahCity {
 			clearDoneCivilVehicles();
 			addCivilVehicleNoise();
 			for (Vehicle vehicle : listOfVehicles) {
-				if (vehicle.getRoute() == null || vehicle.isArrivedToDest())
+				if (vehicle.getRoute() == null)
 					continue;
 				Route route = vehicle.getRoute();
 				double currentLocation = vehicle.getCurrentLocation();
@@ -517,7 +521,10 @@ public class MakkahCity {
 					if (!vehicle.isArrivedToDest()) {
 						double factor = 1-(vehicle.getCurrentStreet().capcityPoint(vehicle.getCurrentLocation(),
 								vehicle.getCurrentLocation()+1000,vehicle)) ;
-						vehicle.move(vehicle.getMaxSpeed()* factor);
+						if (vehicle instanceof Bus) vehicle.move(Bus.MAX_FORWARD * factor );
+						else if (vehicle instanceof Sedan) vehicle.move(Sedan.MAX_FORWARD * factor );
+						else if (vehicle instanceof SUV) vehicle.move(SUV.MAX_FORWARD * factor );
+						else if (vehicle instanceof Truck) vehicle.move(Bus.MAX_FORWARD * factor );
 					}
 				}
 			}
@@ -585,7 +592,7 @@ public class MakkahCity {
 						stdStreet[i].getPercentRemainingCapacity());
 			}
 			String input = in.next();
-			int index = Integer.parseInt(input);
+			int index = Integer.parseInt(input);//TODO: unhandled ex
 			showStreet(stdStreet[index]);
 		}
 		if (choice.equals("4")){
@@ -904,7 +911,7 @@ public class MakkahCity {
 			if (street.getName() == StreetNames.IBRAHIM_ALKHALIL2) numOfSedan = (int) (numOfSedan * 1.2);
 			for (int x = 0; x < numOfSedan; x++) {
 				Sedan car = new Sedan(getRandom(4, 5));
-				double pointOfEntry = getRandom(0, street.getLength());
+				double pointOfEntry = getRandom(0, street.getLength());//TODO: consider getLength - x
 				if (street.capcityPoint(pointOfEntry, pointOfEntry+1500) < 1){
 					listOfVehicles.add(car);
 					car.setCurrentLocation(pointOfEntry);
@@ -917,7 +924,7 @@ public class MakkahCity {
 			if (street.getName() == StreetNames.FOURTH_HIGHWAY1) numOfTruck = (int) (numOfTruck * 0.5);
 			if (street.getName() == StreetNames.FOURTH_HIGHWAY2) numOfTruck = (int) (numOfTruck * 0.5);
 			if (street.getName() == StreetNames.STREET3) numOfTruck = (int) (numOfTruck * 1.5);
-			if (street.getName() == StreetNames.IBRAHIM_ALKHALIL2) numOfTruck = (int) (numOfTruck * 1.2);
+			if (street.getName() == StreetNames.IBRAHIM_ALKHALIL2) numOfSedan = (int) (numOfSedan * 1.2);
 			for (int x = 0; x < numOfTruck; x++) {
 				Truck car = new Truck(getRandom(4, 5));
 				double pointOfEntry = getRandom(0, street.getLength());
@@ -932,7 +939,7 @@ public class MakkahCity {
 			if (street.getName() == StreetNames.FOURTH_HIGHWAY1) numOfSUV = (int) (numOfSUV * 0.5);
 			if (street.getName() == StreetNames.FOURTH_HIGHWAY2) numOfSUV = (int) (numOfSUV * 0.5);
 			if (street.getName() == StreetNames.STREET3) numOfSUV = (int) (numOfSUV * 1.5);
-			if (street.getName() == StreetNames.IBRAHIM_ALKHALIL2) numOfSUV = (int) (numOfSUV * 1.2);
+			if (street.getName() == StreetNames.IBRAHIM_ALKHALIL2) numOfSUV = (int) (numOfSedan * 1.2);
 			for (int x = 0; x < numOfSUV; x++) {
 				SUV car = new SUV(getRandom(4, 5));
 				double pointOfEntry = getRandom(0, street.getLength());
@@ -943,6 +950,7 @@ public class MakkahCity {
 					car.setCurrentStreet(street);
 				}
 			}
+
 		}
 	}
 
@@ -950,6 +958,12 @@ public class MakkahCity {
 		return currenttimeManager;
 	}
 
+	/**
+	 * Find shortest path without respect to traffic
+	 * @param campaign
+	 * @return
+	 */
+	
 	private static Route getBestRoute(Campaign campaign , Mashier mashier) {
 		//ArrayList<Route> routes = (ArrayList<Route>) Arrays.asList(getRoutesToDistrict(campaign.getHotelDistrict()));
 		Route[] routes = getRoutesToDistrict(campaign.getHotelDistrict());
@@ -985,11 +999,6 @@ public class MakkahCity {
 		return sortingRoute;
 	}
 	
-	/**
-	 * Find shortest path without respect to traffic
-	 * @param campaign
-	 * @return
-	 */
 	public static Route getShortestRoute(Campaign campaign, Mashier mashier) {
 		Route[] routes = getRoutesToDistrict(campaign.getHotelDistrict());
 		Route route = null;

src/SUV.java

@@ -1,9 +1,9 @@
 
 public class SUV extends CivilVehicle {
-	private short UID;
+	private String UID;
-	private static short numeberOfSUV;
+	private static int numeberOfSUV;
-    private final short TIME_TO_FIX = 15; //in minutes
+    private final int TIME_TO_FIX = 15; //in minutes
-    public static final short MAX_FORWARD = 1300;
+    public static final int MAX_FORWARD = 1300;
 
     public SUV(double vehicleSize){
         super(vehicleSize);
@@ -21,11 +21,12 @@ public class SUV extends CivilVehicle {
     
     private void generateUID() {
     	numeberOfSUV++;
-    	this.UID = numeberOfSUV;
+    	this.UID = String.format("SUV%04d", numeberOfSUV);
+    	
     }
     
     public String getUID(){
-        return String.format("SUV%04d", UID);
+        return this.UID;
     }
     
 

src/Sedan.java

@@ -1,10 +1,10 @@
 
 public class Sedan extends CivilVehicle {
 
-	private short UID;
+	private String UID;
-	private static short numeberOfSedan;
+	private static int numeberOfSedan;
-    private final short TIME_TO_FIX = 15; //in minutes
+    private final int TIME_TO_FIX = 15; //in minutes
-    public static final short MAX_FORWARD = 1500; // Meter/Min
+    public static final int MAX_FORWARD = 1500; // Meter/Min
 
     public Sedan(double vehicleSize){
         super(vehicleSize);
@@ -16,11 +16,12 @@ public class Sedan extends CivilVehicle {
     
     private void generateUID() {
     	numeberOfSedan++;
-    	this.UID = numeberOfSedan;
+    	this.UID = String.format("Sedan%04d", numeberOfSedan);
+    	
     }
     
     public String getUID(){
-        return String.format("Sedan%04d", UID);
+        return this.UID;
     }
 
     @Override

src/Truck.java

@@ -1,8 +1,8 @@
 
 public class Truck extends CivilVehicle {
-	private short UID;
+	private String UID;
-	private static short numeberOfTruck;
+	private static int numeberOfTruck;
-    private final short TIME_TO_FIX = 20; //in minutes
+    private final int TIME_TO_FIX = 20; //in minutes
 
     @Override
     public int getMaxSpeed() {
@@ -19,13 +19,12 @@ public class Truck extends CivilVehicle {
     }
     private void generateUID() {
     	numeberOfTruck++;
-    	this.UID = numeberOfTruck;
+    	this.UID = String.format("Truck%04d", numeberOfTruck);
     }
     		  
     	    public String getUID(){
-        return String.format("Truck%04d", UID);
+    	        return this.UID;
+    	    }
     	}
 
-}
-