2013-08-20 7 views
11

Der f # -Code geht buchstäblich 500 mal langsamer als der C# -Code. Was mache ich falsch? Ich habe versucht, den Code für beide Sprachen im Grunde gleich zu machen. Es macht keinen Sinn, dass SetPixel in f # so viel langsamer wäre.Bitmap.SetPixel wirkt in f # langsamer als in C#

F #:

module Imaging 
open System.Drawing; 
#light 
type Image (width : int, height : int) = class 
    member z.Pixels = Array2D.create width height Color.White 

    member z.Width with get() = z.Pixels.GetLength 0 

    member z.Height with get() = z.Pixels.GetLength 1 

    member z.Save (filename:string) =  
    let bitmap = new Bitmap(z.Width, z.Height) 
    let xmax = bitmap.Width-1 
    let ymax = bitmap.Height-1 
    let mutable bob = 0; 
    for x in 0..xmax do 
     for y in 0..ymax do 
     bitmap.SetPixel(x,y,z.Pixels.[x,y]) 
    bitmap.Save(filename) 

    new() = Image(1280, 720) 
end 
let bob = new Image(500,500) 
bob.Save @"C:\Users\White\Desktop\TestImage2.bmp" 

C#:

using System.Drawing; 

namespace TestProject 
{ 
public class Image 
{ 

    public Color[,] Pixels; 
    public int Width 
    { 
     get 
     { 
      return Pixels.GetLength(0); 
     } 
    } 
    public int Height 
    { 
     get 
     { 
      return Pixels.GetLength(1); 
     } 
    } 

    public Image(int width, int height) 
    { 
     Pixels = new Color[width, height]; 
     for (int x = 0; x < Width; x++) 
     { 
      for (int y = 0; y < Height; y++) 
      { 
       Pixels[x, y] = Color.White; 
      } 
     } 
    } 

    public void Save(string filename) 
    { 
     Bitmap bitmap = new Bitmap(Width, Height); 
     for (int x = 0; x < bitmap.Width; x++) 
     { 
      for (int y = 0; y < bitmap.Height; y++) 
      { 
       bitmap.SetPixel(x, y, Pixels[x, y]); 
      } 
     } 
     bitmap.Save(filename); 
    } 
} 
class Program 
{ 
    static void Main(string[] args) 
    { 
     Image i = new Image(500, 500); 
     i.Save(@"C:\Users\White\Desktop\TestImage2.bmp"); 
    } 
} 
} 
+4

Gibt es einen Grund, warum Sie nicht 'LockBits' an erster Stelle verwenden möchten? Das wäre deutlich effizienter, vermute ich. –

+0

Was ist der CLR-Typ von 'z.Pixels'? Vielleicht ein spezieller f # -Typ? – usr

+0

Ich würde ein 1D-Array in f # erstellen und auf einmal in ein Bitmap konvertieren. Wird einfacher und sehr effizient sein. –

Antwort

17

Ihre Definition der Pixels Eigenschaft in F # falsch ist: jedesmal, wenn sein Wert (zB in der inneren Schleife von Save) zugegriffen wird, die Die Definition wird neu bewertet. Sie sollten dieses Formular benutzen, statt:

member val Pixels = Array2D.create width height Color.White 

Dies wird die rechte Seite genau einmal bewerten, wenn der Konstruktor aufgerufen wird, und dann den Wert zwischenzuspeichern.

Verwandte Themen