How do I find and remove objects in an array?

[VirtualVirgin]

I can't seem to get any matches with this syntax:

[d.healey]

@VirtualVirgin The problem is each time you write {} you are creating a new object. So in your indexOf and remove functions you are creating new objects.

Instead you need to reuse existing instances of those objects.

const obj1 = {a: 1, b: 2};
const obj2 = {c: 3, d: 4};

const arr = [obj1, obj2];

Console.print(arr.indexOf(obj2)); // 1

[VirtualVirgin]

@d-healey
Yikes! I have a mess then. I have to name every object inside the array by declaring a variable for it?

[d.healey]

@VirtualVirgin said in How do I find and remove objects in an array?:

I have to name every object inside the array by declaring a variable for it?

Nope. That's just the way of explaining why you were having the issue you described. There are more practical ways to do this kind of thing but the best approach will depend on exactly what you need to do and the context in which you're doing it. I need more info.

[VirtualVirgin]

@d-healey I'm working on a primitive piano roll for MIDI note editing and having trouble with note selection.
To try this out, just double-click on an empty cell to create a note, and double-click again to destroy it.
I think I fixed the adding and removing of array objects.

What I am working on now is trying to get the selection methods to work properly for each condition. If I select a note by clicking, that is working, but my click-drag selection is not (even though I can see it is populating my "selectedNotes" array properly).

The issue seems to be with my "isSelected" inline function, which is working for the click selection, but not the drag selection.

After that, I need to fix the selection drag to move the notes, as it was working but I broke it somehow.

HiseSnippet 2814.3ocwZrsaabbcorYfW1Fzjh9VeYrApAYjLMorbrsrkijokufXYKH4KMPPHXztCIGqk6Pr6RKs1guTf7GzW6+R9TxmPetujdNyrWlYuPIKjhRXXwYly4Lm6myLC2MP3vBCEAVMZ853oLqF+wl6G6GMdvXJ2254O1pwsatC8Xt+HxtbpufrmvyibCxDp+LpG4DQvwjgh.RHyi4DwE9DOwHti0ihmRCCYtVMZbomhjpg8ksje92e2indTeGV9TVVuUvcXufOgGkO6ta98bOumPcYulOQC50174NB+ABOwLfsuTydVSoNGSGwdIEAaolVOiFN1pw2zzwYs6wt6vUuSuUuG61tqczp86QWs28t2cXza2y4n6wnqN7HmdVM9hsc4Qhf8inQrPqFW9QB238GKNwWsAukGxOxigC5asOrypoehvyEEQbVqAi4dt6lpOCs.prat18RJs6eo4NbWd174Z4uRt.IGCcEXikLYuKYvd80YudZrWErTCMV5xJV5qatuS.eZT9JH+7GZ9b+HVvPJXmzYEErVK8e9qMu4MSbIBPWhVsFH.L7i5NgdL6IAvfLJzteud8Vgbmd85b+Vs.iWXD4Cz.xTpOyirAIEyQrnAhISE9vf1WaWb09WCQA1pWJhXjPPI.F5VHx9vDCXddg.AN3v6S.X1JHfFShDR3XjOAb1IqPbDdj4DwQuGbPCkNqHpgRh3.DXe9GY.M9DBX35j0VAQC9R+UuKYtjtOMf6RBAvj33BrvHLfXCxPpWHSBxiSmLDsQYvowfx4TgILWTZzX7zo0DJoH3RnglhQFke0vggrHIee55DP6FC+eB+lrFJpoLq1tCAofiT.hq+LOOMF.CeCf+R8G4gJa.HotWZI.qE2OB3kYQbeVKosqKrM6hSumZ11Cm4qHS6QcHs9TKB7A24S3tQigMLZLODMyuCG2FLso.LlwGMNRChmImPGDzV8tD5nn2MyrecQamInOKkhIjVCXz9lCr+rI6AS.PBVbiogbL3zqkO4QdPplumEmfvA8WgbqUHe6Jj6tBoeOvHa2xV4MbBHILxHvyokMZHZinCaLfUu6K+xCJvP3jKuLn1rU5sbzPSuDM7KOnfPiSJQCQAP0lj7Aw7T.ODouIW6ceSHhy0P2fzF4qkI86jffRGJkpTbPoiAw0S.ysTDAxC7oXnb.heaOI0FEvhktfRcF4XXDJlcxIEeHostBEjG+HvaJTxF+MvbzA0FofqIZ3mQnympJPa0eB6J2Ybi6H8pewhXjbpMWaSXP.8m2ddxXdDSseuC+pbqDfxHn31TTMl5jfZZxQ.uMBBt7cyAZT2gPEv8fHx1GbJDguRtUbEM6ygIAI1yao1mVsz2BU1NblrrKcAdbapyXs3UjbJmHhx8yvEBVraI+n5cijvWiuTl0Ob+jrdI682Uo50kFbLjHrCY8JWFWJmlelJr4XsEcU0jYdQ7aTQxvVo9qloPKovpzAgGMdKuoioYybj2L1JjdcuUmp4cyMoKxEF.VdOPJZ.Bj2+j5nEjpJUAH+eCU.lwh3AA2gD5PHNmfbEVXC8T0bRqiSTQeko6XQ.+iX7sWB0wvDY.BBUVpNNlF99vedPZdYXPZ5sBdlF9b7ZbyTphWfEmfhjmnbEhk+quJp8THI9oqBy.+MF9KxdudL24XfKCqSM8AVPD2wPXf3iYS7KHOuWIOuWIOCjIreeMxCFo89pCwzjBvuF9GHKJQ+yRLlmzL0SgtCjYc.KIXhmHlExZw8QTHYoDvtw.X1JB2x3rx4dBTrSKnqZCnXEcELY0zy.xrntBpyYY8bfOyB65vKqbtCMZb2gdBQP6SS.WtiF7gp1rFnsy7s.g+lZacZVCLe.tAODpMSt90qp5LNMR3TPpnteGMuf.Vzr.eUGeqm0125pd+zCYS.T1+Vq4Ra5fwLn5FvSTkokcJOD52k6mm1ujIFAbaIbo4++jA4ypWDJln0b2AHvG11ojWbBdNxb+arQRgCb.pdjUPxlEFjmEdtxuzQJC4mmLoi3xLdwhG.eaqLig6IsiFMZqXe3D.jMdHdPfRrGvAonOP5wrXzKJGH5xhC6ojTjHPeN1IpinfYL.Br0h7IUGcHUvUcXKiCIio9tXF27Vr2Ame.0yCy+pUyl8A3fRF8YKs7anE5JAoKD.q9Rrtu6UU1bECAyict5xNZ1n7tRlRCfCUCkATBnhHiEP1OUep11vY2BEdrtSCfiAzNJ.OtmjtnRovpWSRyTc65jqA8GTzRhnAcJYmksULCNZ6Mb7fLYqS1x0kf0MXS.dPqylblyUB+.D7Rtmx.1Bt7I.YBnQyR34jxabh6AZC89lzHPYhTJn3p5AE+zOkDTb0RAEYMM1wbr9.oIoj9y.b6hG3zzw9+mRy77cR0z8YXF5NcV3XkLpQjruIO2X.SdH01ZPj5cmzYbhWk5.vYmjOKgSktTReOHUDDYe0eGbxrKEVXlrCiKf.CUnjg8pSZza5GsqhPkjo3hY2R.xFx1D1iMgM4Hnnu738341P3LPrNU4BrXfLY5Y43wnAsK37V59Gj2cQVJp3zuC8lg8otN4fhoujM9z6vzhgmIGWvr+lotT3DZ51cTkueJeULWB.Wk12TBTaxC8aDJ2kRuYOWiyUcCMqU4vFI7wDsXMC3gIpvuHM2kocYBc5EIbOoiDmjC04Fm1Xhj6WFkl4kC0OGtiZVDvWDWsV2sZO9Ukp+DBW2Y4L56teZieS39EOvTlu2ocJaThWDlw5EcKsmqlgI8zOu8bQXFWpPe0wdxyAhWfEdpgX4IG.eJ4.7Kw8O77liMO3ZaeWiHqgbepGd2pg0DdIa24MSOi7p10TlSdWW1WjBb11oTVVh7ythlBu4xFbvVURtynho.0qWYj83b5DqeuoUeLKcHOqCaoC647HWEQ47cvKsXvmvAWBGYdH3XHltAkB.qvA8fdGVSLWU.2+vZBy.xubkXr5g0DdEWGF2BuhWn4T6jFTMitzs7FK217ZDOi1IsMSIaWLmrzdHKezs9610.3B2NW2y9ZdKj+u7BsKOE9A3qk07udHp7u90qE1GfVnZWONgVINdODsKK.3G.FuxK1ofRYtYGxKNOo9ajL.aoYgETxyTV5MhNibnxpklex7ID9nW0q7aKOkKbZeRwkFNrx0vmUCeaNvAqpkwWUMXQH11eF1o3JfODd8ho.Z03xlOGYy5eNR8WK0Q8JeZ.J7etOO5USY908FpVIOMH9zkIbE.Zj7gK+yIObIy8sb1ISEAQVbW7kKSG12Rx35uyr0ad9ioQzTxATF1so3c4gBRiGy9.2godWT6lOlEdbjXpD1jWkzpwUN2a9o4a7OrYrNWHy4lMwlebSUrY1L+5OuIOhMI8Its+h+0wO62vE9.P7cEgbzN8C5OodUr3Wl9vtXAAI2cklp2TUi090e9WxYMx+3WLYMq3hrl0+bSpmm3jzai.zZ+olaAmpOehoz.fMz3GckhU4Gf9qZtivclGMx78vwe0.IK.RqwiPiOrrOnEhMTAKsn2vu1GIewug+4kc+5l6xibFWM+tTE7K3A9+Z9M4mbvW1b6gCgTU4L6ka9j+9B+8E76CqndS3QPqpAbvgq4KmMYeve1gAbhO3FhYNZrDFiqF2CGiZl8Y9txA+F7IYw933FIK1OcQqITm.wO5nxLf+.GthbFfm7k+XPratCNlTQpfIbW9O53XRpRHt5EEwacQQbsKJh29hh32dQQ7NWTDu6YiH9ygYqYQhIpvIKqc1caYp6FM11mBdiROWq+KPJw4zT

[d.healey]

@VirtualVirgin said in How do I find and remove objects in an array?:

I'm working on a primitive piano roll

Have you seen the built in piano roll with the MIDI player? I think it might just be a viewer, I haven't played around with it. Lots of docs though.

[VirtualVirgin]

@d-healey I did take a look at it, but I don't see any way to edit the notes from the viewer. I just think it outputs the graphics from a MIDI file, so I though it was just as well if I try to build my own.

[d.healey]

@VirtualVirgin Yeah I think you're right about the viewer. But you can use the player as the backend to manage the tracks, notes, etc. and build your own UI to display and edit.

[d.healey]

For the original problem you might be able to make use of some of the fancier array functions like map, filter, find, some, etc. Or just use a good ol' fashioned loop to manually search the array for the object you're interested in.

[VirtualVirgin]

@d-healey said in How do I find and remove objects in an array?:

For the original problem you might be able to make use of some of the fancier array functions like map, filter, find, some, etc. Or just use a good ol' fashioned loop to manually search the array for the object you're interested in.

I am using filter and map in this script, but I am new to them so I may not be using them correctly.

My trouble at the moment is detecting which notes are selected

"selectedNotes" is an array that is getting populated properly to tell me which noteCells are selected,
but then I run this inline function to check and this only works somethimes:

// check for selected notes
inline function isSelected(cell)
{
	local sRow = selectedNotes.some(obj => obj.row === cell.row);
	local sCol = selectedNotes.some(obj => obj.col === cell.col);
	if (sRow && sCol)
		return true;
	else
		return false;
};

I am trying to use that like "array.contains()", but in this case for specific objects ("cell").
The "cell" is an object with the coordinates i.e. {"row" : 60, "col" : 4}
This works when I use click or shift click to select my notes, but not when a drag selection has been made. Not sure why as they both populate the selectedNotes array.

[d.healey]

@VirtualVirgin For those functions (some, filter, etc.) the first parameter should be a function.

Edit: I just realised you're doing the javascript arrow function thing, we don't have that in HISE.

[VirtualVirgin]

@d-healey said in How do I find and remove objects in an array?:

@VirtualVirgin For those functions (some, filter, etc.) the first parameter should be a function.

Edit: I just realised you're doing the javascript arrow function thing, we don't have that in HISE.

No, the arrow function definitely works:

[d.healey]

@VirtualVirgin

Oh that's good to know, thanks!

Now I can do this kind of thing:

Engine.showYesNoWindow("A title", "Some text", response => {
	Console.print(response);
});

You could replace this

	if (sRow && sCol)
		return true;
	else
		return false;

With return sRow && sCol

But this doesn't solve your issue. Can you make a simple snippet that demonstrates the issue?

[VirtualVirgin]

@d-healey
I found where the problem is taking place:

            selectedNotes.push
            (
	            selectedNotes.filter(function (c) 
	            {
	                var cx = c.col * cellWidth;
	                var cy = height - (c.row + 1) * cellHeight;
	
	                return 
	                (
	                    cx + cellWidth > x1 &&
	                    cx < x2 &&
	                    cy + cellHeight > y1 &&
	                    cy < y2
	                );
	            })
            );

This selectedNotes.push is pushing an object inside an array to the selectedNotes array, whereas I only need it to push an object to the array.
So the current result is looking like [[{"row":42, "col":4}]]
but I need it to just look like [{"row":42, "col":4}]
I am not sure how I modify that to just use a the object and not have it wrapped in that extra array.

[d.healey]

@VirtualVirgin Add [0] after the parenthesis

[VirtualVirgin]

@d-healey Almost...
that turns it into an object, but it only returns one selection to the list instead of all of the noteCells under the selection rectangle area.
Without the [0] it returns all of them, but inside an array.

[d.healey]

@VirtualVirgin In that case after it returns them, just grab the [0] element from that array.

[VirtualVirgin]

@d-healey Whoops... is there a panic button of some sort for feedback loops? I got stuck in one.

[d.healey]

@VirtualVirgin What do you mean?

[VirtualVirgin]

@d-healey I added a for loop after that function and it seems to be stuck in the loop