Cookies op Tweakers

Tweakers maakt gebruik van cookies, onder andere om de website te analyseren, het gebruiksgemak te vergroten en advertenties te tonen. Door gebruik te maken van deze website, of door op 'Ga verder' te klikken, geef je toestemming voor het gebruik van cookies. Je kunt ook een cookievrije versie van de website bezoeken met minder functionaliteit. Wil je meer informatie over cookies en hoe ze worden gebruikt, bekijk dan ons cookiebeleid.

Meer informatie

Door , , 91 reacties, 13.781 views •

De eerste iteratie van 2013 zit erop en hoewel alle devvers lekker vakantie hebben opgenomen en het erg rustig was op kantoor, zijn er toch nog 83 tickets weggewerkt.

Cookiemuur

Ook wij ontkomen niet aan de cookiewetgeving. De balk die wij eerst toonden, was helaas niet voldoende om aan alle eisen van de wet te voldoen en dus moest deze worden aangepast. Het resultaat hebben jullie allemaal al gezien ;)

Vooral kleine dingetjes

Naast de cookies zijn er vooral kleine dingen opgelost en zijn we begonnen met een aantal zaken die nog steeds in ontwikkeling zijn.

Op het forum is de tijdsnotatie van oude topics aangepast. Voorheen hadden oude topics een tijdsnotatie als '10-'05'; dit wordt nu weergegeven als: '18-10-2005'. Hierdoor wordt niet alleen de dag zichtbaar, maar is ook de overzichtelijkheid van de listings verbeterd.

In de karmaberekening hebben we een aanpassing gedaan om de schaalverandering van de reviews te verwerken. Sinds Tweakers 7 worden reviews niet meer met 1 tot 5 gewaardeerd, maar met een score van 0 tot 3. Nu krijg je weer het juiste karma voor een review die als +3 is beoordeeld.

Verder zijn er nog wat bugjes in V&A opgelost. Zo kon je soms een doorlooptijd opgeven, terwijl de advertentie geen veiling was en werd het startbod in sommige gevallen op 0 gezet als je een veiling bewerkte.

Codekwaliteit

Om de codekwaliteit in de gaten te houden en te verbeteren hebben we de code-quality-tool Sonar geïnstalleerd.

Sonar

We hebben nu een beter overzicht van de kwaliteit van de code en worden gewezen op plekken waar we misschien naar moeten kijken. Zo laat Sonar plekken zien met een hoge cyclomatische complexiteit of waar 'bad practices' gebruikt worden. De tool moet nog wel wat gefinetuned worden. Zo is er een aantal zaken die Sonar nu nog als major beschouwt, maar die bij ons wel zijn toegestaan. Een if statement zonder braces wordt door Sonar altijd als major issue aangeduid, terwijl wij dat in sommige gevallen wel gebruiken.

Reacties (91)

Reactiefilter:-191091+164+23+30
Moderatie-faq Wijzig weergave
Een beetje developer doet wel aan TDD ;)

Sluit me aan bij de rest die unitTests willen zien en anti regressie zijn :D
Een unit test coverage van 0.0%? Is dat omdat er geen unit test coverage wordt gemeten of (en dat hoop ik niet) is er geen unit test coverage?
sonar doet me heel erg denken aan de static analyzer die je bij Xcode krijgt ( volgens mij onderdeel van CLANG/LLVM ).

De static analyzer zorgt vaak ervoor dat je geen memory leaks ( of in ieder geval is de kans veel kleiner… ) meer over houd.
In de karmaberekening hebben we een aanpassing gedaan om de schaalverandering van de reviews te verwerken. Sinds Tweakers 7 worden reviews niet meer met 1 tot 5 gewaardeerd, maar met een score van 0 tot 3. Nu krijg je weer de juiste karma voor een review die als +3 is beoordeeld.
Hmm, dat maakt nog een best verschil volgens mij. Alles bij elkaar heb ik nu 20K erbij sinds begin van dit jaar :P
Ik vermoed dat je voorlopig nog wel onze karma-koning blijft. ;)
Hmm, ik hoop dat jullie de unittest metric nog niet aangesloten hebben in Sonar anders maak ik me diep zorgen :P. Tenslotte code zonder testen bestaat niet.

[Reactie gewijzigd door retakenroots op 8 januari 2013 14:03]

Klopt, de daadwerkelijke coverage is nog altijd niet heel bijzonder, maar wel een stuk beter dan op dit plaatje te zien is :)
Vraagje:
Kan er niet iets met JavaScript of zo worden gemaakt dat in 1 tabblad cookies accepteren dat de andere open tabs automatisch verversen i.p.v. dat ik elke tab opnieuw moet vernieuwen?

Ik gebruik RSS en dan open ik enkel artikelen die inleidingen / titels hebben die me aanspreken in een aparte tab.
Wat voor reader gebruik je?
Een FireFox-addon genaamd Feed Sidebar 6
1.513 accessors? Ga gauw eens Lombok integreren, ik betwijfel of die accessors meer zijn dan

[code]
public String getString() {
return this.string;
}
[/code]

wat natuurlijk keihard boilerplate code is. Of, als jullie avontuurlijk zijn, ga polyglot programmeren en gebruik Scala's case classes voor 'data' classes.

En die duplicatie en gebrek aan test coverage kan natuurlijk echt niet, :p.
Dat zijn inderdaad de standaard triviale accessors. Gelukkig kan IDEA of Eclipse die automatisch voor je genereren, waardoor we niet hoeven te klooien met allerlei andere vage talen.
Sommige private vars hoeven bijvoorbeeld helemaal geen accessor te hebben.

We hebben Sonar sowieso nog niet helemaal naar wens en smaak afgestemd en onder andere nog niet geintegreerd met de unittests. De effectieve coverage is wel wat hoger dan hier staat, hoewel zeker niet zo hoog als zou moeten. Afgezien daarvan vermoed ik dat de duplication niet helemaal te vermijden is, maar we hebben sowieso nog niet niet heel uitgebreid gezocht waar we wat aan (zouden) moeten doen.
Gelukkig kan IDEA of Eclipse die automatisch voor je genereren, waardoor we niet hoeven te klooien met allerlei andere vage talen.
Sommige private vars hoeven bijvoorbeeld helemaal geen accessor te hebben.
Lombok is echter geen vage taal, maar een annotation processor. Daarmee verplaats je het genereren van getters en setters van je IDE naar je compiler, waardoor je source overzichtelijker blijft. Best leuk idee, alhoewel ik het zelf ook niet gebruik. :)

IDE's snappen dit over het algemeen ook gewoon, omdat support voor annotation processors een van de standaardfeatures van Java 6 is.
Ik ben een maand of 2 geleden bij het beginnen aan een nieuwe baan voor het eerst in aanraking gekomen met Lombok. Heerlijk! Het maakt je classes zoveel compacter! Het scheelt niet zozeer in het werk (annotatie typen of toetscombi indrukken in IntelliJ om boiletplate te genereren, is verwaarloosbaar) maar vooral in overzichelijkheid en leesbaarheid. Bovendien bij echt pure data/model classes scheelt het wel echt in werk (kun je nl gewoon de gehele class annoteren).
Mooi dat tweakers steeds verbeteringen aan blijft brengen. Alleen jammer dat die Cookie waarschuwing verplicht is. Ik word nogal moe van al die waarschuwingen op internet.
Maar zodra je eenmaal de consent bij ons gegeven hebt, zul je die niet meer zien. :) Mocht je uiteraard vanaf een ander device komen waar je niet bent ingelogd, dan natuurlijk wel :+
Mag je die consent koppelen aan een user-account en dan over verschillende apparaten toepassen, ja?
moet je nog wel even inloggen...op de apparaten..en als je al ingelogt bent dan accepteer je al cookies...
Tenzij je dus al op die apparaten was ingelogd. Maar die groep wordt natuurlijk nu steeds kleiner.
Hoe zit het eigenlijk met upcoming nieuws? Dit was ook in 1 keer weg, aangezien je ervoor Karma kon inleveren, kan ik me goed voorstellen dat deze vraag voor meerdere mensen geldt. Ik heb er al een tijdje en ticket voor open staan. Ik kan me goed voorstellen dat deze niet op de voorpagina van de to-do list staan, maar een reactie had ik op zich wel verwacht van T.net:
[Feat] Karmastore upcoming nieuws
Upcoming news zal (iig voorlopig) niet terugkomen. Er waren maar een paar gebruikers ervan, wat te weinig was om er veel tijd en moeite in te steken. Ingeleverde karma is gewoon weer teruggegeven. Mbt het ticket: we kunnen helaas niet overal direct op reageren.
Bedankt voor de update, jammer dat het niet meer door gaat.

Ik veracht inderdaad geen reactie binnen 2 weken, vandaar vreemde reactie:
we kunnen helaas niet overal direct op reageren.
Datum van ticket: Woensdag 24 oktober 2012 16:52
Datum van vandaag: 08 januari 2013 ;)
Upcoming news zal (iig voorlopig) niet terugkomen. Er waren maar een paar gebruikers ervan, wat te weinig was om er veel tijd en moeite in te steken.
Wel jammer, vond het altijd wel handig (ook om te zien wat er al gesubmit was).
Ook wij ontkomen niet aan de cookiewetgeving. De balk die wij eerst toonden, was helaas niet voldoende om aan alle eisen van de wet te voldoen en dus moest deze worden aangepast. Het resultaat hebben jullie allemaal al gezien ;)
Het is wel ontzettend jammer dat die cookiepopup de html achter de pagina niet alvast inlaad.. Ik was bezig met een Tweakers windows 8 applicatie voor de AppRace die ik uiteindelijk niet gereleased heb door deze popup. (Ik had reacties in mijn applicatie weergegeven, maar dit werkt niet meer door de popup)
Een if statement zonder braces wordt door Sonar altijd als major issue aangeduid, terwijl wij dat in sommige gevallen wel gebruiken.
Haha, deze discussie heb ik toevallig gister nog met iemand gevoerd. Het is gewoon een style van programmeren, maar braces gebruiken is altijd aan te raden (Vind ik persoonlijk overzichtelijker).
Het is wel ontzettend jammer dat die cookiepopup de html achter de pagina niet alvast inlaad.. Ik was bezig met een Tweakers windows 8 applicatie voor de AppRace die ik uiteindelijk niet gereleased heb door deze popup. (Ik had reacties in mijn applicatie weergegeven, maar dit werkt niet meer door de popup)
We kunnen nu eenmaal niet eenvoudig een site zonder (third-party) cookies serveren. Als tijdens het verschijnen van de cookiemuur in de achtergrond de pagina geladen zou worden, zou je al cookies kunnen krijgen voordat je hiervoor toestemming hebt gegeven.
Helemaal mee eens, was ook mijn feedback toen de .plan over de nieuwe cookiepopup werd gepubliceerd. Ook moet er eens goed worden gekeken naar deze popup om mobiele devices want daar is dit soms echt dramatisch.

Op dit item kan niet meer gereageerd worden.



HTC One (M9) Samsung Galaxy S6 Grand Theft Auto V Microsoft Windows 10 Apple iPad Air 2 FIFA 15 Motorola Nexus 6 Apple iPhone 6

© 1998 - 2015 de Persgroep Online Services B.V. Tweakers vormt samen met o.a. Autotrack en Carsom.nl de Persgroep Online Services B.V. Hosting door True