Sikkerhetsinstillinger i IE ødelegger

[Manfred]

Jeg får i tillegg helt mark av norske navn på klasser og filer, men det er en annen sak. Det blir så ufattelig uoversiktlig med noen engelske funksjonsnavn (de som er i rammeverket) og noen norske om hverandre. Utrolig inkonsekvent.

 

Hvorfor ha en egen klasse for å koble til databasen? Jeg kobler til databasen min med en kodelinje uten å lage noe jalla-API selv, liksom.

[ze5400]

EDIT: Nervermind, jeg så ikke return

 

 

...

4 Nesting av IF setninger...

If fosterId = 0 Then
  tall = 2
Else
' MASSE LINJER MED KODE
End If
Return tall

 

Bør heller skrives som:

If fosterId = 0 Then
  Return 2
End If

' MASSE LINJER MED KODE

 

 

 

Nå sikter du til dette eksemplet eller generelt?

 

Jeg er ganske sikker på at når man legger kode i else skal den kjøres hvis en hendelse intreffer/ikke intreffer, og vitsen er litt vekk hvis den kjøres uansett:P

Jeg ville ikke lagt "MASSELINJERMEDKODE" i else, men jeg ville heller kalt en Sub eller funksjon i else. Det er meget mulig jeg er helt på bærtur da jeg kun programmerer som hobby + at det er litt tidlig ennå.

Endret 19. oktober 2006 av ze5400

[Manfred]

...

4 Nesting av IF setninger...

If fosterId = 0 Then
  tall = 2
Else
' MASSE LINJER MED KODE
End If
Return tall

 

Bør heller skrives som:

If fosterId = 0 Then
  Return 2
End If

' MASSE LINJER MED KODE

 

 

 

Nå sikter du til dette eksemplet eller generelt?

 

Jeg er ganske sikker på at når man legger kode i else skal den kjøres hvis en hendelse intreffer/ikke intreffer, og vitsen er litt vekk hvis den kjøres uansett:P

Jeg ville ikke lagt "MASSELINJERMEDKODE" i else, men jeg ville heller kalt en Sub eller funksjon i else. Det er meget mulig jeg er helt på bærtur da jeg kun programmerer som hobby + at det er litt tidlig ennå.

7102149[/snapback]

 

Lille venn... Den kjøres ikke uansett når man har en Return i If'en, vet du...

[wolf5]

En grei måte å fjerne sql injection er å gå over til Stored Procedures og adde hver parameter i parameter arrayen istedet for å bygge en streng.

 

Da slipper du å tenke på å replace ' her og der, samt alle de andre variantene av ' tegnet (urlencodet osv).

 

Men ville plassert det høyt på listen ja. Det finnes mange hobby hackere der ute og om de får nyss om et sted hvor de kan prøve ut sin "leetness" så setter de i gang.

 

Men uansett. Mye av svarene her er helt off topic. Greit at man poengtererer ut bedre kodepraksis men det er ikke det som er problemet. Alle begynner en plass og som oftest er kodepraksisen suspekt i begynnerfasen, men dette er noe som modnes med tiden.

Endret 19. oktober 2006 av wolf5

[Manfred]

Det er ikke hensiktsmessig å bruke SP til alt. Det blir rett og slett alt for mye på alle småspørringer. Men har ikke .NET egne sånne "querybuilder"-saker hvor man legger til med "AddParam" eller slike ting? Synes jeg så noe slikt en gang.

[ze5400]

...

4 Nesting av IF setninger...

If fosterId = 0 Then
  tall = 2
Else
' MASSE LINJER MED KODE
End If
Return tall

 

Bør heller skrives som:

If fosterId = 0 Then
  Return 2
End If

' MASSE LINJER MED KODE

 

 

 

Nå sikter du til dette eksemplet eller generelt?

 

Jeg er ganske sikker på at når man legger kode i else skal den kjøres hvis en hendelse intreffer/ikke intreffer, og vitsen er litt vekk hvis den kjøres uansett:P

Jeg ville ikke lagt "MASSELINJERMEDKODE" i else, men jeg ville heller kalt en Sub eller funksjon i else. Det er meget mulig jeg er helt på bærtur da jeg kun programmerer som hobby + at det er litt tidlig ennå.

7102149[/snapback]

 

Lille venn... Den kjøres ikke uansett når man har en Return i If'en, vet du...

7102368[/snapback]

 

 

 

Oops

 

Hadde ikke helt våkna, la ikke merke til at det var en return der

 

Flaut til tusen detta

[wolf5]

Om det finnes en querybuilder så ville jeg brukt det ja. Ville for all del ungått "manuell" strengbygging for å unngå svakheten med injection det kan gi.

 

Fordelene med SP'er at de fungerer som funksjoner. Endres det i databasen så trenger ikke utvikler endre noe i koden for SP'ens grensesnitt forblir gjerne den samme. Fungerer som et komponent lag mellom database og kode.

 

Rene SQL strenger har den svakheten at man må endre overalt i koden sin straks man gjør mindre endringer i databasen. Men alt kommer selvsagt an på størrelsen på prosjektet. Store prosjekter bør bruke SP'er for å unngå mye vedlikehold ved endringer i databasem.

[j000rn]

Men uansett. Mye av svarene her er helt off topic. Greit at man poengtererer ut bedre kodepraksis men det er ikke det som er problemet. Alle begynner en plass og som oftest er kodepraksisen suspekt i begynnerfasen, men dette er noe som modnes med tiden.

7102373[/snapback]

 

Jeg syntes det er helt greit å få tips ang. kodepraksis bra begynnelsen av. Hvis koden hadde fulgt de tipsene jeg gav hadde det vært mye lettere å finne feilen(e). Ing spurte heller ikke om det var noen sikkerhetshull. Var det off-topic (les: feil) å tipse om dette?

Man er jo her for å få hjelp. At man får mer hjelp enn man spør om bør jo bare være en bonus

[wolf5]

Joda helt greit det, men når jeg leste det var det som om alt dreide seg om hvordan koden var skrevet istedet for selve problemet.

 

Er vel greiest å skrive et svar på spørsmålet og deretter komme med tillegget :-)

 

Blir lett å spinne off topic + at det var så sabla mye tekst angående måten å kode på. + resten av folka hoppet på samme hest og glemte nesten topicen.