[project-s] インポート周りの修正とリファクタリング (VOICEVOX#1614)

src/components/Sing/ScoreSequencer.vue

@@ -172,7 +172,7 @@ export default defineComponent({
     const dragMoveCurrentY = ref();
     const dragDurationCurrentX = ref();
     // 分解能（Ticks Per Quarter Note）
-    const tpqn = computed(() => state.score?.resolution ?? 480);
+    const tpqn = computed(() => state.score?.tpqn ?? 480);
     // ノート
     const notes = computed(() => state.score?.notes ?? []);
     // 拍子
@@ -263,7 +263,7 @@ export default defineComponent({
         note: {
           id,
           position,
-          midi: noteNumber,
+          noteNumber,
           duration,
           lyric,
         },
@@ -337,10 +337,10 @@ export default defineComponent({
           }
           isNotesChanged = true;
           const position = note.position + amountPositionX;
-          const noteNumber = note.midi + amountPositionY;
+          const noteNumber = note.noteNumber + amountPositionY;
           return {
             ...note,
-            midi: noteNumber,
+            noteNumber,
             position,
           };
         } else {
@@ -506,16 +506,16 @@ export default defineComponent({
       }
       const newNotes = state.score.notes.map((note) => {
         if (selectedNoteIds.value.includes(note.id)) {
-          const noteNumber = Math.min(note.midi + 1, 127);
+          const noteNumber = Math.min(note.noteNumber + 1, 127);
           return {
             ...note,
-            midi: noteNumber,
+            noteNumber,
           };
         } else {
           return note;
         }
       });
-      if (newNotes.some((note) => note.midi > 127)) {
+      if (newNotes.some((note) => note.noteNumber > 127)) {
         return;
       }
       store.dispatch("REPLACE_ALL_NOTES", { notes: newNotes });
@@ -527,16 +527,16 @@ export default defineComponent({
       }
       const newNotes = state.score.notes.map((note) => {
         if (selectedNoteIds.value.includes(note.id)) {
-          const noteNumber = Math.max(note.midi - 1, 0);
+          const noteNumber = Math.max(note.noteNumber - 1, 0);
           return {
             ...note,
-            midi: noteNumber,
+            noteNumber,
           };
         } else {
           return note;
         }
       });
-      if (newNotes.some((note) => note.midi < 0)) {
+      if (newNotes.some((note) => note.noteNumber < 0)) {
         return;
       }
       store.dispatch("REPLACE_ALL_NOTES", { notes: newNotes });
@@ -622,7 +622,6 @@ export default defineComponent({
     });
 
     return {
-      measureNum,
       beatsPerMeasure,
       beatWidth,
       gridCellWidth,

src/components/Sing/SequencerNote.vue

@@ -84,15 +84,15 @@ export default defineComponent({
   setup(props, { emit }) {
     const store = useStore();
     const state = store.state;
-    const tpqn = computed(() => state.score?.resolution ?? 480);
+    const tpqn = computed(() => state.score?.tpqn ?? 480);
     const zoomX = computed(() => state.sequencerZoomX);
     const zoomY = computed(() => state.sequencerZoomY);
     const positionX = computed(() => {
       const noteStartTicks = props.note.position;
       return tickToBaseX(noteStartTicks, tpqn.value) * zoomX.value;
     });
     const positionY = computed(() => {
-      const noteNumber = props.note.midi;
+      const noteNumber = props.note.noteNumber;
       return noteNumberToBaseY(noteNumber + 0.5) * zoomY.value;
     });
     const barHeight = computed(() => getKeyBaseHeight() * zoomY.value);

src/components/Sing/ToolBar.vue

@@ -249,7 +249,7 @@ export default defineComponent({
     });
 
     const snapTypeSelectOptions = computed(() => {
-      const tpqn = store.state.score?.resolution ?? 480;
+      const tpqn = store.state.score?.tpqn ?? 480;
       return getSnapTypes(tpqn).map((snapType) => {
         if (isTriplet(snapType)) {
           return { snapType, label: `1/${(snapType / 3) * 2}（三連符）` };

src/infrastructures/AudioRenderer.ts

@@ -411,17 +411,17 @@ export interface Instrument {
    * ノートオンをスケジュールします。
    * すでに指定されたノート番号でノートオンがスケジュールされている場合は何も行いません。
    * @param contextTime ノートオンを行う時刻（コンテキスト時間）
-   * @param midi MIDIノート番号
+   * @param noteNumber MIDIノート番号
    */
-  noteOn(contextTime: number, midi: number): void;
+  noteOn(contextTime: number, noteNumber: number): void;
 
   /**
    * ノートオフをスケジュールします。
    * すでに指定されたノート番号でノートオフがスケジュールされている場合は何も行いません。
    * @param contextTime ノートオフを行う時刻（コンテキスト時間）
-   * @param midi MIDIノート番号
+   * @param noteNumber MIDIノート番号
    */
-  noteOff(contextTime: number, midi: number): void;
+  noteOff(contextTime: number, noteNumber: number): void;
 
   /**
    * 発音中のすべての音に対して、ノートオフのスケジュールを行います。
@@ -436,7 +436,7 @@ export interface Instrument {
 export type NoteEvent = {
   readonly noteOnTime: number;
   readonly noteOffTime: number;
-  readonly midi: number;
+  readonly noteNumber: number;
 };
 
 /**
@@ -502,8 +502,8 @@ class NoteEventScheduler implements EventScheduler {
         this.startContextTime + (event.noteOffTime - this.startTime);
 
       if (event.noteOnTime < untilTime) {
-        this.instrument.noteOn(noteOnContextTime, event.midi);
-        this.instrument.noteOff(noteOffContextTime, event.midi);
+        this.instrument.noteOn(noteOnContextTime, event.noteNumber);
+        this.instrument.noteOff(noteOffContextTime, event.noteNumber);
         this.index++;
       } else break;
     }
@@ -647,7 +647,7 @@ export type Envelope = {
 };
 
 type SynthVoiceParams = {
-  readonly midi: number;
+  readonly noteNumber: number;
   readonly oscillatorType: OscillatorType;
   readonly envelope: Envelope;
 };
@@ -656,7 +656,7 @@ type SynthVoiceParams = {
  * シンセサイザーのボイスです。音を合成します。
  */
 class SynthVoice {
-  readonly midi: number;
+  readonly noteNumber: number;
   private readonly oscillatorNode: OscillatorNode;
   private readonly gainNode: GainNode;
   private readonly envelope: Envelope;
@@ -678,7 +678,7 @@ class SynthVoice {
   }
 
   constructor(audioContext: BaseAudioContext, params: SynthVoiceParams) {
-    this.midi = params.midi;
+    this.noteNumber = params.noteNumber;
     this.envelope = params.envelope;
 
     this.oscillatorNode = new OscillatorNode(audioContext);
@@ -692,11 +692,11 @@ class SynthVoice {
 
   /**
    * MIDIノート番号を周波数に変換します。
-   * @param midi MIDIノート番号
+   * @param noteNumber MIDIノート番号
    * @returns 周波数（Hz）
    */
-  private midiToFrequency(midi: number) {
-    return 440 * 2 ** ((midi - 69) / 12);
+  private midiToFrequency(noteNumber: number) {
+    return 440 * 2 ** ((noteNumber - 69) / 12);
   }
 
   /**
@@ -715,7 +715,7 @@ class SynthVoice {
     this.gainNode.gain.linearRampToValueAtTime(1, t0 + atk);
     this.gainNode.gain.setTargetAtTime(sus, t0 + atk, dcy);
 
-    const freq = this.midiToFrequency(this.midi);
+    const freq = this.midiToFrequency(this.noteNumber);
     this.oscillatorNode.frequency.value = freq;
 
     this.oscillatorNode.start(contextTime);
@@ -797,16 +797,16 @@ export class PolySynth implements Instrument {
    * ノートオンをスケジュールします。
    * すでに指定されたノート番号でノートオンがスケジュールされている場合は何も行いません。
    * @param contextTime ノートオンを行う時刻（コンテキスト時間）
-   * @param midi MIDIノート番号
+   * @param noteNumber MIDIノート番号
    */
-  noteOn(contextTime: number, midi: number) {
+  noteOn(contextTime: number, noteNumber: number) {
     const exists = this.voices.some((value) => {
-      return value.isActive && value.midi === midi;
+      return value.isActive && value.noteNumber === noteNumber;
     });
     if (exists) return;
 
     const voice = new SynthVoice(this.audioContext, {
-      midi,
+      noteNumber,
       oscillatorType: this.oscillatorType,
       envelope: this.envelope,
     });
@@ -822,11 +822,11 @@ export class PolySynth implements Instrument {
    * ノートオフをスケジュールします。
    * すでに指定されたノート番号でノートオフがスケジュールされている場合は何も行いません。
    * @param contextTime ノートオフを行う時刻（コンテキスト時間）
-   * @param midi MIDIノート番号
+   * @param noteNumber MIDIノート番号
    */
-  noteOff(contextTime: number, midi: number) {
+  noteOff(contextTime: number, noteNumber: number) {
     const voice = this.voices.find((value) => {
-      return value.isActive && value.midi === midi;
+      return value.isActive && value.noteNumber === noteNumber;
     });
     if (!voice) return;