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 , , reacties: 91, views: 13.664 •

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+166+210+30
Wij zetten de braces op de volgende regel en in sommige gevallen, waar de complexiteit laag is, wordt het nog wel eens gebruikt om 4 regels te 'besparen' wat de leesbaarheid kan vergroten:

Zie dit voorbeeld: http://pastebin.com/qfkvGZbe

Met geneste if's e.d. worden braces wel altijd gebruikt.

[Reactie gewijzigd door JoostBaksteen op 8 januari 2013 14:09]

ACM in 'plan: Tweakers komt met nieuwe implementatie cookienotificatie'
We hebben de melding sowieso alleen voor Nederlandse ip's, dat scheelt al een hoop gedoe. Daarnaast sluiten we ons bekende ipranges van crawlers ook uit van de melding, hoewel er vast nog wel af en toe een robot doorheen valt die 'm wel gaat krijgen...
Jij hebt waarschijnlijk alleen de 'balk' onderin, niet de melding die je zal moeten accepteren voordat je überhaupt op de site kan komen.
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.
Gedeeltelijk wel. Als jij geen toestemming geeft dan moeten wij een site zonder cookies aanbieden en dat is voor ons technisch vrij ingewikkeld, dus bieden wij geen cookieloze site aan.
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.
Krijgen we ooit weer de oude wel/niet kruisjes/vinkjes terug in de pricewatch?

Mateloos irritant dat ik kan aanklikken wat ik WEL wil maar niet wat ik NIET wil. Waarom functionalitiet eruit slopen?

Nu kom ik bij cases uit op Micro-ATX : WEL, alleen krijg ik dan ook allemaal ATX kasten. Als ik daar nou net zo als vroeger een kruisje bij kon zetten was het probleem opgelost en zou ik alleen maar kleinere kasten krijgen.
Ja, die zullen wel een keer terugkeren. Helaas zijn er nog heel veel dingen die we ook willen doen en worden we gehinderd door eindige hoeveelheid tijd. Dus ik kan je niet vertellen wanneer dat wordt :)

Op dit item kan niet meer gereageerd worden.



Populair:Apple iPhone 6Samsung Galaxy Note 4Apple iPad Air 2FIFA 15Motorola Nexus 6Call of Duty: Advanced WarfareApple WatchWorld of Warcraft: Warlords of Draenor, PC (Windows)Microsoft Xbox One 500GBTablets

© 1998 - 2014 Tweakers.net B.V. Tweakers is onderdeel van De Persgroep en partner van Computable, Autotrack en Carsom.nl Hosting door True

Beste nieuwssite en prijsvergelijker van het jaar 2013