file i/o

[mike hengst]

Ok I have worked this to being pretty colse to what I need. The only problem left(Ihope) is that not everything I want to be outputted to the orderout file is showing up. The data must be being passed to the super class because the output I am getting hs been calculated, which is done in the super class(Order). I am only getting the extended price with the handling charge added on. What am I not doing that is causing this???HELP. Here is the code for the two classes in question. I think it may have to do with the toString() itself and will keep working on it, bu tI would appreciate a second(or more) set of eyes to look at it...Thanks

 package java163.project6;
import java.text.*;
import java163.util.*;
import java.io.*;
import java.util.*;
/**
 *  class extends order.
 *
 */
public class OrderWithCharge extends Order {
    private static  double[]  charge          = new double[4];
    private static  int[]     qty             = new int[4];
                    double    newprice        = 0;
                    double    total           = 0;
    private         double    handlingCharge;
    private         int       amt;

    /**
     *  Constructor for the OrderWithCharge object
     *
     *@param  name   customer name
     *@param  id     customer id
     *@param  qt     quantity ordered
     *@param  price  price of item
     *@param  item   item name/description
     */
    OrderWithCharge(String name, int id, int qt, double price, String item) {
        super(name, id, qt, price, item);
    }

    /**
     *  Method calls parent class method and overrides and adds fee.
     *
     */
    public void computePrice() {
        try {
            super.computePrice();
            super.getQtyOrdered();
            String           strLine  = null;
            BufferedReader   in       = new BufferedReader(new FileReader("data/charges.dat"));
            strLine = in.readLine();
            StringTokenizer  tok      = new StringTokenizer(strLine);
            int              num      = tok.countTokens();
            int              i        = 0;
            boolean          eof      = false;
            String           str      = null;
            while(!eof) {
                str = in.readLine();
                if(str == null) {
                    eof = true;
                }
                else {
                    while(tok.hasMoreTokens()) {
                        qty[i] = Integer.parseInt(tok.nextToken());
                        charge[i] = Double.parseDouble(tok.nextToken());
                        ++i;
                    }
                }
            }
            int              sub      = 3;
            while(sub >= 0 && qtyOrdered < qty[sub]) {
            handlingCharge = charge[sub];
                --sub;
            }
            total = super.getExtendedPrice() + handlingCharge;
            in.close();
        }
        catch(IOException e) {
            System.out.println("The error is " + e);
        }
        finally{
        }
    }
    /**
     *  Formats display of total price
     *
     *@return    String containing formatted total
     */
    public String toString() {
        DecimalFormat  df  = new DecimalFormat("$####.00");
        StringBuffer   sb  = new StringBuffer();
        super.toString();
        sb.append("\n With handling charge : " + df.format(total));
        return sb.toString();
    }
} 

package java163.project6;
import java.text.*;
import java163.util.*;
import java.io.*;
import java.util.*;
//Jones 1111 7 12.99 Widget
//Smith 1212 12 15.35 Thingamajig
//Jackson 2323 5 14.57 Whatchamacallit
//Doe 2222 2 13.05 SomethingOrOther
public class ProcessOrders{
    private static String[] name = new String[4];
    private static int[] id = new int[4];
    private static int[] qt = new int[4];
    private static double[] prc = new double[4];
    private static String[] item = new String[4];
    public void processOrder(){
       
       try{
           BufferedReader in = new BufferedReader(new FileReader("data/orderin.txt"));
          PrintWriter out = new PrintWriter(new FileWriter("data/orderout.txt"));
           boolean eof = false;
     
                    
                
       while(!eof) {    
               String line = in.readLine();
                if(line == null) {
                    eof = true;
                }
                else{

                        StringTokenizer tok = new StringTokenizer(line);
                         String name = tok.nextToken();
                         int id = Integer.parseInt(tok.nextToken());
                         int qt = Integer.parseInt(tok.nextToken());
                         double prc = Double.parseDouble(tok.nextToken());
                         String item = tok.nextToken();
                    OrderWithCharge owc = new OrderWithCharge(name,id,qt,prc,item);
                    owc.setCustName(name);
                    owc.setCustNumber(id);
                    owc.setQtyOrdered(qt);
                    owc.setUnitPrice(prc);
                    owc.setItem(item);
                    owc.computePrice();
                    
                    System.out.println(owc.toString());
                    out.println(owc.toString());                       
     
                }
           
       }
                    in.close();
                    out.close();
       } 
        catch(IOException e){
            System.out.println("There was an error1 " + e);
        } 
//     try{
//          PrintWriter out = new PrintWriter(new FileWriter("data/orderout.txt"));          
          
 //         for(int x=0;x<4;++x){
//              System.out.println(name[x]);
//              System.out.println(id[x]);
//              System.out.println(qt[x]);
//              System.out.println(prc[x]);
//              System.out.println(item[x]);
//              out.println(name[x]);
//              out.println(id[x]);
//              out.println(qt[x]);
//              out.println(prc[x]);
 //             out.println(item[x]);
//          }
//           out.close();    
//      }
//      catch(Exception e){
//          System.out.println("There was an error2 " + e);
//      }
        
    }
}  

[Jim Yingst]

Usually when you're continuing a previous topic, it's best to put your new post as an "Add" to the existing topic, rather than as a separate topic. The old topic will appear "new" once it has a new post in it (i.e. the date/time appearing next to the topic will be the date/time of the most recent post, and the topic will appear at the top of the topic list until something even newer is posted). By adding to the previous post, you make it easier for everyone to have the complete context of your problem. And you prevent anyone from wasting their time responding to the original post if you've already solved some of the problems (as in this case) - they can see the most recent version of your code in the same thread, without having to search elsewhere. Which is usually easier for all of us. Anyway...
I think your primary problem right now is in the OrderWithCharge toString() method. You call super.toString() - but you don't do anything with the String that this method returns. I think you want to append this to your StringBuffer somewhere, e.g.:

    sb.append(super.toString());

Some other observations...
In class ProcessOrders:
The various private static arrays aren't used for anything, are they? If you do need to save the order info for use later, I'd use either a single OrderWithCharge[] array, or a Collection such as ArrayList containing OrderWithCharge elements.
Initializing all your array lengths to 4 seems questionable - what if this number needs to change later? I'd replace each 4 with a variable (probably static and final), such as MAX_RECORD_COUNT. Then it's easy to change later if you need to. OF course, if you use some sort of Collection you probably won't need to specify a maximum size anyway - the Collection will simply grow as you add elements to it.
Inside the processOrder() method, there seems to be a lot of redundant stuff. The OrderWithCharge constructor appears to set name,id,qt,prc,item - there's no need to set these again immediately after using owc.setCustName(name) etc.
In class OrderWithCharge:
Again, the static arrays are pretty suspicious looking - why are they needed? It seems that all the data you need are already declared as instance (non-static) fields of either Order or OrderWithCharge - just use these fields.
It looks like your computeCharges() method is doing a lot more work than it needs to - for each individual order, it's rereading the quantities and recalculating the charges for all the orders. Why? It seems that quantities have already been read from the orderin.txt file - are the quantities in charges.dat different? If they are, then you shoudl probably use different names to distinguish the two types of quantitites. In any event, if you need to read the file charges.dat in addition to orderin.txt, just do it once, in a static method somewhere - and put the data you've read into the appropriate fields in your array/Collection or OrderWithCharge objects.
Hope that helps. Good luck...
[ December 15, 2002: Message edited by: Jim Yingst ]