file upload

Du skal erklære myData før din if - og så putte data i den sidst i din if

Eller flytte skrivning af filen ind i din if - det giver vel ikke mening ellers.

Ok, hvordan er det jeg erklære myData?

Læs nu kommentar nummer 2.

Men du erklærer og putter data i din variabel myData i linien:

  byte[] myData = new Byte[nFileLen];

Og undskyld, men, altså, hvorfor i alverden bøvler du med "file upload", når du ikke ved hvordan man erklærer en variaable i C#? Ville det ikke være mere formålstjenligt at lære sproget først?

Undskyld, men jo. Jeg kan sproget nogenlunde, og jeg vil hele tiden lære mere selvfølgelig. Så jeg lærer sproget imens jeg laver hjemmesider.

Jeg tænkte også at man erklærede på den måde, men var ikke sikker på at det hed at erklære. Men det var jo også gjordt.

Men det var rigtigt, nu satte jeg resten op i if sætningen. Og så er jeg kommet videre fra fejlen. Men nu virker det bare ikke. Jeg har prøvet at uploade en fil som hedder: test.xls Men den lægger sig ikke på serveren.

ved du hvad problemmet kunne være?

Nu ser min kode sådan ud:

protected void Button2_Click(object sender, EventArgs e)
    {
        string sSavePath;

        sSavePath = "../admin";

        if (FileUpload.PostedFile != null)
        {
            // Check file size (mustn’t be 0)
            HttpPostedFile myFile = FileUpload.PostedFile;
            int nFileLen = myFile.ContentLength;
            if (nFileLen == 0)
            {
                lblOutput_excel.Text = "No file was uploaded.";
                return;
            }
           

            if (System.IO.Path.GetExtension(myFile.FileName).ToLower() != ".xls")
            {
                lblOutput.Text = "The file must have an extension of xls";
                return;

            }
         
            byte[] myData = new Byte[nFileLen];
            myFile.InputStream.Read(myData, 0, nFileLen);

            System.IO.FileStream newFile
                        = new System.IO.FileStream(Server.MapPath(sSavePath),
                                                  System.IO.FileMode.Create);
            newFile.Write(myData, 0, myData.Length);
            newFile.Close();
       
        }

Det gør den nok ikke. Hvilket filnavn ville den skulle have?  Er "admin" et katalog?  Hvad kommer der ud af Server.MapPath(sSavePath) ? Hvor bruger du myFile.FileName?

Hvad mener du med hvilket filnavn den skulle have. Det burde da være ligemeget hvad filnavnet er, bare den ender med .xls . admin er en mappe på roden.

Jeg forstår ikke helt hvad du mener med hvad der kommer ud af Server.MapPath(sSavePath)?

Og her bruger jeg (myFile.FileName):
if (System.IO.Path.GetExtension(myFile.FileName).ToLower() != ".xls")

Det betyder jo bare at hvis ændelsen ikke er .xls så skal den eksekvere dette.

Håber det var svar nok.

mvh. Adam

Så lad mig prøve at lede dig på sporet på anden vis: Bruger du filnavnet på det sted du vil skrive filen på serveren.

Nej jeg bruger ikke filnavnet noget andet sted, det er bare meningen at man skal kunne uploade hvilken som helst xls fil til serveren.

Det jeg skal bruge den til er at excel filens data skal indlæses til sql serveren, men det virker fint.
Men for at det kan virker skal filen ligge på serveren.

Jeg håber jeg har forstået dit spørgsmål rigtigt?

Når du bruger "new System.IO.FileStream(...)" skal du angive det filnavn, som indholdet skal gemmes i. Gør du det?

Nej det gør jeg vel ikke så.
men hvilket indhold er det du mener?
Der er jo ikke noget filnavn som indhold skal gemmes i.
Der er bare en mappe som filen som man uploader skal gemmes i.

Er det denne kodestump du mener at jeg skal ændre?

System.IO.FileStream newFile
                    = new System.IO.FileStream(Server.MapPath(sSavePath),
                                              System.IO.FileMode.Create);
            newFile.Write(myData, 0, myData.Length);
            newFile.Close();

Ja. Du skal jo angive et filnavn, når du bruger FileStream.

Nå ok, nu er jeg ikke så kendt med FileStream.

Hvor skal jeg så angive filnavnet i denne kod:

new System.IO.FileStream(Server.MapPath(sSavePath)

Og vil det så sige at det ikke er muligt at uploade hvilken som helst fil man vil?

Du kan uploade hvad du vil (store filer måske ikke). Men filen skal have et navn på serveren. Første argument til "new FileStream" er filnavnet. Du angiver (formentlig) ikke et filnavn.

Nå ok, så hvis jeg har forstået det rigtigt.
Så det der sker med "new FileStream" er at når man uploader en fil så bliver den automatisk omdøbt til det filnavn man angiver.

Er det rigtigt forstået?

Muligvis korrekt - hvad mener du?

Hvis jeg for eksempel uploader en fil ved navn test.xls
Men der i koden står new FileStream(database.xls)

Kommer filnavnet på serveren så til at hedder database.xls istedet for test.xls?

Forstod du det spørgsmål?

Filen, der oprettes af FileStream-klasse, kommer til at hedde det du skriver som første argument til "new FileStream".

Jeg er ikke sikker, men jeg tror måske vi misfårstår hinanden.
Der skal ikke oprettes nogen fil.

Filen som skal lægges på serveren skal jo bare kopieres fra computeren til serveren.
Så filen eksistere allerede på computeren.

"Der skal ikke oprettes nogen fil." siger du i dit sidste indlæg. Tidligere skriver du "Jeg har prøvet at uploade en fil som hedder: test.xls Men den lægger sig ikke på serveren."

Jeg forstår vist ikke hvad du vil.

ok, jeg vil ligge en fil som jeg har på min computer ned i en mappe på mit webhotel.
Den skal ikke oprette en ny fil, det er jo en fil som allerede eksistere på min computer som jeg bare vil have op på mit webhotel ved hjælp af file upload

Var det bedre forklaret?

Ja og nej ;)  Den kan kun komme til at ligge på webhotellet som en ny fil. Alternativet, som ikke opretter en ny fil, er at "bruge" indholdet af den uploadede fil til et eller andet (tælle tal sammen, eller hvad det kan være), og ikke gemme indholdet.

Nå ok, så har jeg nok misforstået, hvordan man bruger fileupload.
Så skal jeg vel oprette en fil.
Men det er jo ligesom hvis man skal uploade et billede til webhotellet, så opretter man da ikke en ny vel? kopiere den ikke bare billedet fra din computer til webhotellet?

Det er nok en strid om ord. Den opretter en ny fil på webhotellet, med det samme indhold.

Nå ok, så forstår jeg :-D

Nu tilbage til det oprindelige så.

hvad var det så ved det her jeg skulle ændre og til hvad?

new System.IO.FileStream(Server.MapPath(sSavePath)

Hvilket navn skal filen have på serveren?

database.xls

Du kan så fx rette

  sSavePath = "../admin";

til

  sSavePath = "../admin/database.xls";

forudsat dit katalog "admin" ligger der allerede.

når du siger katalog, mener du så mappe?

Ja. Kært barn har mange navne.

Nu har jeg prøvet at gøre det men det virker stadig ikke, nu ser hele koden sådan ud:

{
        string sSavePath;

        sSavePath = "../admin/test.xls";

        if (FileUpload.PostedFile != null)
        {
            // Check file size (mustn’t be 0)
            HttpPostedFile myFile = FileUpload.PostedFile;
            int nFileLen = myFile.ContentLength;
            if (nFileLen == 0)
            {
                lblOutput_excel.Text = "No file was uploaded.";
                return;
            }
           

            if (System.IO.Path.GetExtension(myFile.FileName).ToLower() != ".xls")
            {
                lblOutput.Text = "The file must have an extension of xls";
                return;

            }
         
            byte[] myData = new Byte[nFileLen];
            myFile.InputStream.Read(myData, 0, nFileLen);

            System.IO.FileStream newFile
                        = new System.IO.FileStream(Server.MapPath(sSavePath),
                                                  System.IO.FileMode.Create);
            newFile.Write(myData, 0, myData.Length);
            newFile.Close();
       
        }

Oplysningen "men det virker stadig ikke," er for upræcis til at vi kan bruge det til noget.

Nå ok undskyld. Det jeg mener er at når jeg prøver at uploade en fil, siger den stadig: No file was uploaded.

Og uploader den ikke til serveren.

Ligesom før.

Ok - det var da rart at vide. :)

Der er som sådan ikke noget galt med din kode - jeg kører den (ligner meget) et sted uden problemer. Men der kan så være noget galt i det du ikke viser.

Ja nu har jeg faktisk lige fået den til at virke.
En dum fejl, jeg har lavet. Jeg kaldte FileUpload istedet for File1.

Dum lille fejl.

Men tak for alt din hjælp. Det hjalp meget.

så lukker jeg lige