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.627 •

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)

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.
Een if statement zonder braces wordt door Sonar altijd als major issue aangeduid, terwijl wij dat in sommige gevallen wel gebruiken.
Waarom?
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).
Kunnen jullie uitleggen hoe jullie het doen met de cookie wall mbt de zoekmachines, en andere bots? Wil ook graag een cookiewall implementeren. Nu kan ik google bot etc wel excluden, maar wat als google zelf langskomt met een nieuw ip of andere googlebot wordt hij dus geblokkeerd door een cookiewall.
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
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]

Omdat het in sommige situaties leesbaarder uitpakt.
Het 'geen toegang geven bij niet accepteren van de cookie-ding' is toch geen onderdeel van de wet?
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]

Bij ons is het geen enkele geval dat een if statement op 1 regel, zonder braces iets leesbaarder maakt...

Je wordt er dan ook zwaar op afgerekend door je team hier }>

[Reactie gewijzigd door Phoenix_the_II op 8 januari 2013 14:10]

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.
IMHO is dat bad practice.
Heb te vaak meegemaakt dat 'iets' niet werkte omdat men vergeten was om de braces toe te voegen op het moment dat men de if statement aan het uitbreiden was.

Daarom zeg ik altijd braces gebruiken bij een if statement
In dit voorbeeld kan je ook gewoon "return bar" doen.
Uhm, is dat niet: return (bar) ? Of ben ik nu een voorbeeld uit het verband aan het trekken?
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.
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.
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 :+
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
Totale chaos op T.net HQ op dit moment. Hier had niemand aan gedacht :P

zal wel niet
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.

Op dit item kan niet meer gereageerd worden.



Populair: Apple iPhone 6 iPhone Apple iPhone 6 Plus Smartphones Laptops Microsoft Games Apple Activision Smartwatches

© 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