java - Pinpointing defect in a method to find the maximum double slice sum of an array -


the task in question taken https://codility.com/programmers/lessons/9-maximum_slice_problem/max_double_slice_sum/.

i paraphrase task description here:

let slice of array (possibly empty) contiguous part of array. is, a[i],..,a[j-1] (where 0 <= <= j < length(a)) slice of a.

let sum of slice sum of elements contained within slice.

the task find maximal combined sum of pair of slices separated precisely 1 element in first , last element of array not present in either slice nor used separator. is, max{a[i+1]+..+a[j-1]+a[j+1]+..+a[k-1] | 0 <= < j < k < length(a)}.

my question not how solve task (i have found working solution online) rather why following solution fails work in cases. every test case have tried on succeeds, codility's large_random test produced wrong answer (my code produced: 2004640 correct answer: 2403731). unfortunately, not input caused error.

what causes below code return incorrect answer in exceptional cases?

public class maxdoubleslicesum {     public int solution(int[] a) {         final int n = a.length;         int minimum = integer.max_value;         int currentslice = 0;         int maxslice = 0;         integer dropvalue = null;         int uptodrop = 0;         (int = 1; < n - 1; i++){             if (a[i] < 0){                 if (dropvalue == null){                     dropvalue = a[i];                     uptodrop = currentslice;                         }                 else{                     /* find maximum of following:                      * 1. include a[i] , keep dropping dropvalue.                      * 2. make a[i] new dropped element , include dropvalue.                      * 3. make a[i] new dropped element , not include uptodrop , dropvalue.                      */                     int includingai = currentslice + a[i]; //1                     int includingolddropvalue = currentslice + dropvalue; //2                     int discardingolddropvalue = currentslice - uptodrop; //3                     if (includingai > includingolddropvalue && includingai > discardingolddropvalue){                         currentslice = includingai;                     }                     else if (includingolddropvalue > discardingolddropvalue){                         dropvalue = a[i];                         currentslice = includingolddropvalue;                         uptodrop = currentslice;                     }                     else{                         dropvalue = a[i];                         currentslice = discardingolddropvalue;                         uptodrop = currentslice;                     }                 }             }             else{                 currentslice += a[i];             }             maxslice = math.max(maxslice, currentslice);             minimum = math.min(minimum, a[i]);         }         if (dropvalue == null){             maxslice -= minimum;         }         return maxslice;     } }


Comments

Post a Comment

Popular posts from this blog

python Tkinter Capturing keyboard events save as one single string -

is possible store keyboard events 1 string? code below stores 1 char , prints it. card reader or bar code reader, contains collection of character/ string not 1 character @ time. goal save char pressed text variable. from tkinter import * root = tk() def key(event): text= event.char text+= event.char print ("pressed", text) def callback(event): frame.focus_set() print ("clicked at", event.x, event.y) frame = frame(root, width=100, height=100) frame.bind("<key>", key) frame.bind("<button-1>", callback) frame.pack() root.mainloop() currently, creating text variable , everytime key() function gets called, , text stores last character typed. you can define text module level variable , , use module level text inside key function - text = '' def key(event): global text text+= event.char print("pressed", text)
Read more

android - InAppBilling registering BroadcastReceiver in AndroidManifest -

i'm implementing inapp billing in android app. works well, however, i'm trying decouple broadcast receiver activity manifest. suggestion in android's trivialdrive sample: // important: dynamically register broadcast messages updated purchases. // register receiver here instead of <receiver> in manifest // because call getpurchases() @ startup, therefore can ignore // broadcasts sent while app isn't running. // note: registering listener in activity bad idea, done here // because sample. regardless, receiver must registered after // iabhelper setup, before first call getpurchases(). currently there's class extends broadcastreceiver : public class iabbroadcastreceiver extends broadcastreceiver { /** * intent action receiver should filter for. */ public static final string action = "com.android.vending.billing.purchases_updated"; private final iabbroadcastlistener mlistener; public iabbroadcastreceiver(iabbroadcastlistener listener) { m
Read more

javascript - Z-index in d3.js -

i have map , when hover effect on region of map below yellow line , circle, last ones disappear. how can solve problem? http://jsfiddle.net/ot21n9qx/ var arc = g.append("path") .style("fill", "none") .style("stroke", "yellow") .style("stroke-width", 2) .attr("d", "m" + pathorigin[0] + "," + pathorigin[1] + " q" + svgpoint[0] + "," + pathorigin[1] + " " + svgpoint[0] + "," + svgpoint[1]); var circlesize = d3.scale.linear() .domain([0, 0.5, 1]) .range([4, 10, 4]); svg not have z order draws objects in order created. need create path , circle after create highlight regions. here forked fiddle: http://jsfiddle.net/o00fzgaf/1/
Read more