[webtigerteam]

Kändes inte helt OK att köra ett perl-script med root-rättigheter som uppdaterar sig själv via nätet ...

Jag har gjort ett script i NodeJS (JavaScript) för er som kör Nginx:

https://github.com/Z3TA/letsencrypt-nodejs-nginx

Fritt att kopiera och modifiera.

[Nerix]

Ursprungligen postat av webtigerteam

Kändes inte helt OK att köra ett perl-script med root-rättigheter som uppdaterar sig själv via nätet ...

Jag har gjort ett script i NodeJS (JavaScript) för er som kör Nginx:

https://github.com/Z3TA/letsencrypt-nodejs-nginx

Fritt att kopiera och modifiera.

Tog en snabb titt på koden. Du har en del try-catch block på asynkrona metod-anrop som inte kommer fungera. Ett tips är att använda en linter om du är osäker på vad som fungerar och ej.

Edit: EsLint med Googles regler ger 499 fel och Flow Type runt 100.

[webtigerteam]

Ursprungligen postat av Nerix

Tog en snabb titt på koden. Du har en del try-catch block på asynkrona metod-anrop som inte kommer fungera. Ett tips är att använda en linter om du är osäker på vad som fungerar och ej.

Edit: EsLint med Googles regler ger 499 fel och Flow Type runt 100.

Tack för din feedback! readFileSync är dock en synkron funktion !!

Angående "linters" så är min erfarenhet att de nästan aldrig upptäcker buggar. Av de 499 "felen" du hittade beror troligtvis de flesta på att jag använder tab i stället för mellanslag som indentering ...

[Nerix]

Ursprungligen postat av webtigerteam

Tack för din feedback! readFileSync är dock en synkron funktion !!

Helt rätt, mitt fel.

Ursprungligen postat av webtigerteam

Angående "linters" så är min erfarenhet att de nästan aldrig upptäcker buggar. Av de 499 "felen" du hittade beror troligtvis de flesta på att jag använder tab i stället för mellanslag som indentering ...

Stängde av indenteringsreglerna för filen. En majoritet försvann, men 196 återstår. Några exempel är '==' istället för '===', dubbeldeklarering av variabler, ohanterade callback-fel, importering utanför headern, djup nästade callback, långa rader, användning av variabler ej ännu deklarerade.

Vad är din uppfattning av en linter, dess användning och vilken funktion anser du att den fyller?

[webtigerteam]

Ursprungligen postat av Nerix

Vad är din uppfattning av en linter, dess användning och vilken funktion anser du att den fyller?

En linter kan vara bra om man arbetar i ett team, och kommit överens om att koden ska se ut på ett speciellt sätt. Jag tycker dock det läggs allt för mycket tid på meta-programmering, till exempel omstrukturering av koden för att hålla raderna under 80 tecken, vilket ibland kan kräva kluriga lösningar som gör koden svårare att förstå.

Ursprungligen postat av Nerix

Stängde av indenteringsreglerna för filen. En majoritet försvann, men 196 återstår. Några exempel är '==' istället för '===', dubbeldeklarering av variabler, ohanterade callback-fel, importering utanför headern, djup nästade callback, långa rader, användning av variabler ej ännu deklarerade.

importering i headern uppmanar till att använda globala variabler! Jag försöker undvika globala variabler så långt som möjligt. En unik grej med modul-systemet common-js är att man kan importera moduler lokalt!! Koden blir så mycket lättare att förstå då.


'==' istället för '===' ... Om jag skulle skriva if(foo===undefined) skulle jag även behöva lägga till if(foo===null). Så det är lite av en bekvämlighet, jag fångar två flugor i en smäll. Man bör dock undvika att jämföra olika typer. Ex: if("42"==42) . Men vad hjälper det om det blir false i stället för true !? Eventuella buggar kvarstår ändå. Man bör i stället konvertera alla nummer till den typ man vill ha. Ex: var age=parseInt(request.form.age) för att vara på den säkra sidan.

[tartareandesire]

Ursprungligen postat av webtigerteam

Angående "linters" så är min erfarenhet att de nästan aldrig upptäcker buggar. Av de 499 "felen" du hittade beror troligtvis de flesta på att jag använder tab i stället för mellanslag som indentering ...

Den är inte heller till för att upptäcka buggar, det har du andra verktyg till. Den är till för att få en bättre och mer överskådlig kodstruktur för alla som tittar på koden och minska risken för att man skapar buggar från första början.

[webtigerteam]

Ursprungligen postat av tartareandesire

Den är inte heller till för att upptäcka buggar, det har du andra verktyg till.

Kan du rekommendera några verktyg som hittar buggar i JavaScript?

[jayzee]

Ursprungligen postat av webtigerteam

Kan du rekommendera några verktyg som hittar buggar i JavaScript?

Lycka till att hitta sånt till ett otypat och direkt tolkat programmeringsspråk.
Lint/hint är bara ämnade för kodkvalité.

[Nerix]

Ursprungligen postat av webtigerteam

En linter kan vara bra om man arbetar i ett team, och kommit överens om att koden ska se ut på ett speciellt sätt.

Nej, de primära syftet är att hitta kod som rent semantiskt är rätt, alltså där kompilatorn säger ok, men där chansen är stor att du och interpretatorn har delade meningar. Te.x semikolon och den icke-determinism som avsaknaden kan innebära.

Ursprungligen postat av webtigerteam

Jag tycker dock det läggs allt för mycket tid på meta-programmering, till exempel omstrukturering av koden för att hålla raderna under 80 tecken, vilket ibland kan kräva kluriga lösningar som gör koden svårare att förstå.

Meta-programmering har inget med struktur att göra.

Förstår du de statistiska sambandet mellan buggar och kodkomplexitet?

Ursprungligen postat av webtigerteam

importering i headern uppmanar till att använda globala variabler! Jag försöker undvika globala variabler så långt som möjligt.

Nej. Importering av extern kod i de flesta Javascript-JIT är blockerande. Gör du en import mitt i koden så blockar detta hela eventloopen och då även gränssnittet.

För att inte nämna de optimeringsproblem din JIT utsätts för när kod med sidoeffekter laddas in under körning. Har du te.x koll på hur icke-deterministisk kod påverkar din prestanda?

Ursprungligen postat av webtigerteam

En unik grej med modul-systemet common-js är att man kan importera moduler lokalt!! Koden blir så mycket lättare att förstå då.

Så vitt jag vet så tillåter alla importeringssystem detta.

Ursprungligen postat av webtigerteam

'==' istället för '===' ... Om jag skulle skriva if(foo===undefined) skulle jag även behöva lägga till if(foo===null). Så det är lite av en bekvämlighet, jag fångar två flugor i en smäll.

I de fallet är `if(foo)` att rekommendera. Att du försöker jämföra med undefined och null tyder på att du har andra, mer fundamentala problem, i din implementering. Fråga dig själv varför dessa världen introducerades från första början- Odefinderade världen representerar ett odefinerat tillstånd, alltså en bugg.

Ursprungligen postat av webtigerteam

Man bör dock undvika att jämföra olika typer. Ex: if("42"==42) . Men vad hjälper det om det blir false i stället för true !? Eventuella buggar kvarstår ändå. Man bör i stället konvertera alla nummer till den typ man vill ha. Ex: var age=parseInt(request.form.age) för att vara på den säkra sidan.

Tvetydlig kod är vad som skapar buggar (och i de här fallet även prestandaproblem). På samma vis som `==` operatorn inte ska användas så ska `++` inte användas.

Om vi bortsätt från koden du postade så är de största misstaget du gör att tro att du är smartare än miljön du befinner dig i. För att kunna avgöra huruvida felen lintern påpekade är värt att fixa eller ej så behöver du grundläggande kunskap inom bl.a beräkningsmodeller, kompilatorer, typsystem, matematisk statistik och miljön koden ska köras i, te.x Googles V8-motor.

Varför inte bygga upp nödvändig kunskap parallellt och förlita dig på de verktyg som finns?

Ursprungligen postat av jayzee

Lycka till att hitta sånt till ett otypat och direkt tolkat programmeringsspråk.
Lint/hint är bara ämnade för kodkvalité.

FlowType (https://flowtype.org/) hjälper till med de här. Den kan, utan annoteringar, härleda typer och fel i din kod. Te.x `function(x) {x.length}(null) // => Error: null is not a string;