SingleThreadModel bør jeg bruge det(performance)?

Det må koste lidt i object creation & destruction.

Næppe specielt mærkbart.

Men det ville undre mig meget hvis det var nødvendigt.

Hvis du kun bruger metode lokale variable i doGet/doPost, så virker det også
en en multithreaded kontekst.

Jeg benytter så metoder og variabler fra en anden pakke, men det ser ud til, at jeg godt kan undgå singlethreadmodel - nu har jeg i hvert fald fået renset ud i den sammenblanding, der ellers opstår...

Kan du komme på et par eksempler, hvor man normalt ville bruge singlethreadmodel?

Smid i øvrigt bare et svar :)

Nej det var da vist et held at den viste noget rigtigt før - nu blanden den det sammen igen - tror bare jeg holder mig til singlethreadmodel for en sikkerheds skyld :)

STM er bedre end at fylde din servlet med synchronized

svar

hvis du poster din servlet kode så kan vi jo kigge på den

det er som sagt meget usædvaneligt at have brug for STM

Det lyder godt :)
Den kommer her:

import.....

public class PieChart extends HttpServlet implements SingleThreadModel
{
  String title;
  String xSize;
  String ySize;
  String[] dataName;
  String[] dataValue;

  public void init(ServletConfig config) throws ServletException
  {
    super.init(config);
  }

  public void doGet(HttpServletRequest request, HttpServletResponse response) throws ServletException, IOException
  {
    title = null;
    xSize = null;
    ySize = null;
    dataName = null;
    dataValue = null; 

    response.setContentType("image/jpg");
    title = request.getParameter("title");
    ySize = request.getParameter("ySize");
    xSize = request.getParameter("xSize");
    String[] dataNames = request.getParameterValues("dataName");
    String[] dataValues = request.getParameterValues("dataValue");

//Default values for image size
    int x = 500;
    int y = 300;
   
// Create a simple pie chart
    DefaultPieDataset pieDataset = new DefaultPieDataset();
   
    if(title!=null)
      {}
    else
      title = "No title!";

    if(dataNames!=null && dataValues!=null)
    {
      if(dataNames.length==dataValues.length)
      {
        for(int i=0; i<dataValues.length; i++)
        {
          pieDataset.setValue(dataNames[i], new Double(dataValues[i]));
        }       
      }
      else
        pieDataset.setValue("Not the same amount of DataNames and DataValues",0);
    }
    else
        pieDataset.setValue("No DataNames and DataValues",0);

    JFreeChart chart = ChartFactory.createPieChart
    (title, // Title
    pieDataset, // Dataset
    true, // Show legend
    true, // Use tooltips
    false // Configure chart to generate URLs?
    );

    if(xSize!=null && ySize!=null)
    {
      x = Integer.parseInt(xSize);
      y = Integer.parseInt(ySize);
    }

    BufferedImage bi = chart.createBufferedImage(x, y);

    try {
      ChartUtilities.writeBufferedImageAsJPEG(response.getOutputStream(), bi);
    } catch (FileNotFoundException e) {
      e.printStackTrace();
    } catch (ImageFormatException e) {
      e.printStackTrace();
    } catch (IOException e) {
      e.printStackTrace();
    } 
  }
}

Ned i doGet med dem her:

  String title;
  String xSize;
  String ySize;
  String[] dataName;
  String[] dataValue;

!!!!

okay det hjalp jo en del! :)
hvordan kan det være at det ikke var nok at sige:
    title = null;
    xSize = null;
    ySize = null;
    dataName = null;
    dataValue = null;
?

Uden STM så er der kun et objekt af typen PieChart, flere tråde kalder doGet metoden
på samme objekt

når de er erklæret som instans variable så betyder det at der kun er en kopi
af dem nemlig for det ene objekt og så kører alle trådene på den samme kopi og
det går galt

de lokale variable i metoden bliver der derimod oprettte en kopi af for
hvert kald til metoden og dermed får hver tråd sin egen kopi

Forrygende forklaring Arne - det var jeg slet ikke klar over :)

Nu vi er ved performancen, så er der vel ikke rigtigt mere at optimere, hvis alt skal erklæres i metoden?

Det tror jeg ikke.

Jeg vil tro at det er Tomcat og JFreeChart koden som bruger >98% af CPU og ikke
din kode.

Det er super Arne, så håber jeg at folkene bag JFreeChart har gjort det godt :)

Du får mange tak for hjælpen