ProcessManager.java

[pisco]

Hey Leute
ich hab 2 Klassen geschrieben jedoch krieg ich immer die gleichen Fehler und zwar in der ProzessManager-Klasse. Weis leider nicht was der Compiler meint:

/**
* A Process (a running program)
*/ 
public class Process
{
	
	private int id;
	private int total;
	private int current;
	
        public Process (int id, int total)
        {
		this.id=id;
		this.total=total;//alle Anweisungen...
		this.current = 0;//bereits ausgeführte Anweisungen...
                
        }
	
        public void execute ()
        {
		current++;
		
		
        }
	
              public int restDuration ()
        {
                
		
        	return (total-current);
        }
	
	        public int getId ()
        {
                
        	return this.id;
        }
	
	        public boolean finished ()
        {
		if(current==total)                
			return true;
		else
			return false;
        }
}

und jetz die FEHLERHAFTE Klasse:

import java.util.*;
/**
* Prozess manager, manages a list of processes.
*/
public class ProcessManager
{
	ArrayList<Process> proc;//Anzahl der von ihm zu verwaltenden Prozesse 
        int procLim;//das Limit an ausführbaren Prozessen
	
	
        public ProcessManager (int processLimit)
        {
        	proc = new ArrayList<Process>();
		this.procLim=processLimit;
		
		
        }
	
	public boolean hasFreeSlot ()
        {
		return proc.size()<procLim;
		
        }
	
	public boolean assignProcess (Process newProc)
        {
		if(newProc.hasFreeSlot()){
			
			procLim.add(newProc);
			return true;
		} else{
			return false;
		}
		
        }
	
        public int nextToFinish ()
        {
                int i = Integer.MAX_VALUE;
                int pid = -1;
		Process p;
		
                for(int j=0; j<proc.size();j++)
                {
                	if(p.restDuration()<i)
                	{
                		i=p.restDuration();
                		pid=p.getId();
                	}
                }
                
                return pid;
        }
	
	
	public void execute ()
        {
        	Iterator<Process> i=proc.iterator();
        	
        	while(i.hasNext())
        	{
        		Process p=i.next();
        		p.execute();
        		if(p.finished())
        		{
        			i.remove();
        		}
        	}
        }
        
	public void list()
        {
		Process p;
        	for(int i=0; i<proc.size();i++)
        	{
        		System.out.println(p.getId()+" "+p.restDuration());
        	}
        } 
	
}

Mein Compiler sagt was von Zeile 27 und 29... glaub ich^^

ich freu mich schon auf eure Antworten und bedank mich im Vorraus für eure Hilfe
greetz pisco

 

[eRaaaa]

Mein Compiler sagt was von Zeile 27 und 29... glaub ich^^

Jop und das macht dich dann nicht stutzig so dass du dir die Zeilen zusammen mit der Fehlermeldung anschaust?

        if(newProc.hasFreeSlot()){
            
            procLim.add(newProc);

a) In der Klasse Process gibts keine Methode

hasFreeSlot()

(meinst du hier evtl. nur [c]if (hasFreeSlot())[/c]) ?
b) procLim ist ein int, also 1. kein Objekt 2. was sollte add(Process) bewirken? Meinst du an der Stelle nicht viel eher proc.add(..) ?
---
c) Zeile 41 ist auch komisch Process p; for(int j=0; j<proc.size();j++) gleiches Spiel in Zeile 73 !

 

[pisco]

hey was meinst du mit c) ?

wie ist das komisch?

        public int nextToFinish ()
        {
                int i = Integer.MAX_VALUE;
                int pid = -1;
		
                for(Process p: plist)
                {
                	if(p.restDuration()<i)
                	{
                		i=p.restDuration();
                		pid=p.getId();
                	}
                }
                
                return pid;
        }

        public void list()
        {
        	for(Process p:plist)
        	{
        		System.out.println(p.getId()+" "+p.restDuration());
        	}
        }

ein Kollege hat mir gerade seine Lösung des Problems geschickt... nur der verwendet foreach -Schleifen und irgendwie check ich die ned....

Wie kann man die den in normalefor-schleifen umwandeln?

greetz und danke für deine Antwort

 

[eRaaaa]

Ändere in dem Code von deinem Kollegen einfach plist zu proc um, das sollte es schon gerwesen sein wenn ich mich jetzt nicht verguckt habe.

wie ist das komisch?

Du deklarierst Process p; und versuchst einige Zeilen später einfach eine Methode restDuration() darauf aufzurufen... p ist/wäre aber null(lokale Variablen müssen IMMER initialisiert werden)...um deinen Code also zu korrigieren müsstest du dir halt erst das process-Objekt holen, du könntest also

p = proc.get(i);

aufrufen, dann hast du auch ein konkretes Objekt auf dme du entsprechende Methoden aufrufen kannst !

 

[pisco]

hey danke
und was ist jetz der unterschied zwischen meiner Version und der der Foreach schleife...

                for(Process p: plist)
                {
                    if(p.restDuration()<i)

in dieser version wird p doch irgendiwe über die foreach schleife deklariert oder?

greetz

 

[eRaaaa]

The For-Each Loop

Galileo Computing :: Java ist auch eine Insel (8. Auflage) – 12 Datenstrukturen und Algorithmen

 

[pisco]

hey danke
für die Links trotzdem hab ich noch nicht verstanden wie man das in eine normale For-schleife umschreibt...

greetz

 

[eRaaaa]

Na so wie du es in execute auch schon gemacht hast, mit einem Iterator

	public int nextToFinish() {
		int i = Integer.MAX_VALUE;
		int pid = -1;
		Process p;
		for (Iterator<Process> iterator = proc.iterator(); iterator.hasNext();) {
			p = iterator.next();
			if (p.restDuration() < i) {
				i = p.restDuration();
				pid = p.getId();
			}
		}
		return pid;
	}

oder wie ich schon sagte halt mit proc.get(index); ....