2012-01-02 38 views
1

我希望有人能帮助我。我正在尝试创建一个上传程序,可以上传一个或多个mp3文件并将其属性添加到数据库(歌曲标题,专辑等)。我不确定我遇到的问题是由于数据库问题还是因为我不太了解CommonsMultipartFile的工作原理。总之,这里是我的代码”grails错误PatchedDefaultFlushEventListener

查看:

<g:form name="fileupload" url="[action:'uploads',controller:'fileResource']" method="POST" enctype="multipart/form-data">   
    <span>Add files...</span> 
    <input type="file" name="files" multiple>   
</g:form> 

fileResource:

def uploads = {     
    Collection result = [] 
    if (request instanceof MultipartHttpServletRequest) { 
     MultipartHttpServletRequest multiRequest = (MultipartHttpServletRequest)request; 
     CommonsMultipartFile file = (CommonsMultipartFile)multiRequest.getFile("files");    
      println"one" 
      moveFile(file) 
      println"three"    
     result << [name: file.originalFilename, size:file.size] 
    }   
    render result as JSON 
} 

private moveFile(CommonsMultipartFile file){ 
    println"two" 
    try{    
     def userId = getUserId() 
     def newFileName = java.util.UUID.randomUUID().toString()      
     def userGuid = SecUser.get(userId.id).uid 
     def webRootDir = servletContext.getRealPath("/")    
     File myFile = new File(webRootDir + "/myUsers/${userGuid}/music",newFileName + ".") 
     file.transferTo(myFile) 
     MP3File mp3file = new MP3File(myFile); 
     String duration = mp3file.getAudioHeader().getTrackLength() 
     String bitRate = mp3file.getAudioHeader().getBitRate() 
     String encodingType = mp3file.getAudioHeader().getEncodingType() 
     String getFormat = mp3file.getAudioHeader().getFormat() 
     String trackLength = mp3file.getAudioHeader().getTrackLength() 
     String fileName = file.originalFilename 
     String  artistName = "" 
     String  albumName = "" 
     String  songName = "" 

     def getArtistId="" 
     try{ 
      if (mp3file.hasID3v2Tag()){ 
       //println("has id3v2 tag") 
       ID3v24Tag id3v24tag = mp3file.getID3v2TagAsv24(); 
       artistName = (id3v24tag.getFirst(ID3v24Frames.FRAME_ID_ARTIST).trim()) 
       albumName = (id3v24tag.getFirst(ID3v24Frames.FRAME_ID_ALBUM).trim()) 
       songName = (id3v24tag.getFirst(ID3v24Frames.FRAME_ID_TITLE).trim()) 
      }else if (mp3file.hasID3v1Tag()) { 
       //println("has id3v1 tag") 
       ID3v1Tag tag = mp3file.getID3v1Tag(); 
       artistName = tag.getFirst(FieldKey.ARTIST).trim() 
       albumName = tag.getFirst(FieldKey.ALBUM).trim() 
       songName = tag.getFirst(FieldKey.TITLE).trim() 
      } 
      else{ 
       println("this format not yet supported:" + getFormat) 
      } 
     }catch (Exception ex) { 
      log.error("ERROR: FILE NOT PROCESSED") 
     } 

     try{ 
      def oArtist = "" 
      def c = Artist.createCriteria() 
      def getMyArtist = c.get { 
       eq('artistName', artistName) 
       secUser { 
        eq('id', userId.id) 
       } 
      } 
      if (getMyArtist == null){ 
       //artist does not exist 
       // 1c. add artist 
       // 1d. add album 
       // 1e. add song 
       // 1f. move song 
       // 1g. DONE 
       oArtist= new com.jason.score.Artist(
        artistName : artistName , 
        secUser : userId.id 
       ) 
       userId.addToArtist(oArtist).save(flush:true) 
       def oAlbum  = new Album 
       (
        albumName:  albumName 
       ) 
       oArtist.addToAlbum(oAlbum).save(flush:true) 

       def oSong = new Song 
       (
        fileLocation: webRootDir + "myUsers/${userGuid}/music/", 
        songBitRate: bitRate, 
        songDuration: duration, 
        songEncodeType: encodingType, 
        songName: songName 
       ) 
       oAlbum.addToSong(oSong).save(flush:true) 
      }else{ 
       //artist exists with that username 
       //need album name and artist id 
       def d = Album.createCriteria() 
       def getMyAlbum = d.get { 
       eq('albumName', albumName) 
       artist { 
        eq('id', getMyArtist.id) 
       } 
         } 
       if(getMyAlbum == null){ 
        //    3. add album 
        //    3a. add song 
        //    3b. move song 
        //    3c. DONE         
        Artist findArtist = Artist.get(getMyArtist.id) 
        def oAlbum  = new Album 
        (
         albumName:  albumName 
        ) 
        findArtist.addToAlbum(oAlbum).save(flush:true) 
        def oSong = new Song 
        (
         fileLocation: webRootDir + "myUsers/${userGuid}/music/", 
         songBitRate: bitRate, 
         songDuration: duration, 
         songEncodeType: encodingType, 
         songName: songName 
        ) 
        oAlbum.addToSong(oSong).save(flush:true) 
        }else{ 
         //album does exist 
         //check if song exists with album id        
         def e = Song.createCriteria() 
         def getMySong = e.get { 
          eq('songName', songName) 
          album { 
           eq('id', getMyAlbum.id) 
          } 
         } 
         if (getMySong==null){ 
          Album findAlbum = Album.get(getMyAlbum.id)         
          def oSong = new Song 
          (
           fileLocation: webRootDir + "myUsers/${userGuid}/music/", 
           songBitRate: bitRate, 
           songDuration: duration, 
           songEncodeType: encodingType, 
           songName: songName 
          ) 
          findAlbum.addToSong(oSong).save(flush:true) 
         }else{ 
          //it already exists, so do nothing 
         }     


        } 
       }       
     }catch (Exception ex){ 
      log.error("ERROR: ADDING TO DB: " + ex) 
     } 
    }catch (Exception ex){ 
     log.error("error" + ex) 
    } 
} 

    private getUserId(){ 
    return SecUser.get(springSecurityService.principal.id) 
} 
private getUser(){ 
    return SecUser.get(springSecurityService.principal) 
} 

我得到的错误是:

2012-01-02 14:25:25,314 [http-8080-1] ERROR events.PatchedDefaultFlushEventListener - Could not synchronize database state with session 
org.hibernate.StaleObjectStateException: Row was updated or deleted by another transaction (or unsaved-value mapping was incorrect): [com.jason.score.SecUser#1] 
at org.hibernate.persister.entity.AbstractEntityPersister.check(AbstractEntityPersister.java:1792) 
at org.hibernate.persister.entity.AbstractEntityPersister.update(AbstractEntityPersister.java:2435) 
at org.hibernate.persister.entity.AbstractEntityPersister.updateOrInsert(AbstractEntityPersister.java:2335) 
at org.hibernate.persister.entity.AbstractEntityPersister.update(AbstractEntityPersister.java:2635) 
at org.hibernate.action.EntityUpdateAction.execute(EntityUpdateAction.java:115) 
at org.hibernate.engine.ActionQueue.execute(ActionQueue.java:279) 
at org.hibernate.engine.ActionQueue.executeActions(ActionQueue.java:263) 
at org.hibernate.engine.ActionQueue.executeActions(ActionQueue.java:168) 
at org.codehaus.groovy.grails.orm.hibernate.events.PatchedDefaultFlushEventListener.performExecutions(PatchedDefaultFlushEventListener.java:46) 
at org.hibernate.event.def.DefaultFlushEventListener.onFlush(DefaultFlushEventListener.java:50) 
at org.hibernate.impl.SessionImpl.flush(SessionImpl.java:1027) 
at org.springframework.orm.hibernate3.HibernateTemplate$28.doInHibernate(HibernateTemplate.java:883) 
at org.springframework.orm.hibernate3.HibernateTemplate.doExecute(HibernateTemplate.java:406) 
at org.springframework.orm.hibernate3.HibernateTemplate.executeWithNativeSession(HibernateTemplate.java:374) 
at org.springframework.orm.hibernate3.HibernateTemplate.flush(HibernateTemplate.java:881) 
at org.codehaus.groovy.grails.orm.hibernate.metaclass.SavePersistentMethod$1.doInHibernate(SavePersistentMethod.java:58) 
at org.springframework.orm.hibernate3.HibernateTemplate.doExecute(HibernateTemplate.java:406) 
at org.springframework.orm.hibernate3.HibernateTemplate.execute(HibernateTemplate.java:339) 
at org.codehaus.groovy.grails.orm.hibernate.metaclass.SavePersistentMethod.performSave(SavePersistentMethod.java:53) 
at org.codehaus.groovy.grails.orm.hibernate.metaclass.AbstractSavePersistentMethod.doInvokeInternal(AbstractSavePersistentMethod.java:179) 
at org.codehaus.groovy.grails.orm.hibernate.metaclass.AbstractDynamicPersistentMethod.invoke(AbstractDynamicPersistentMethod.java:59) 
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method) 
at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:39) 
at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:25) 
at java.lang.reflect.Method.invoke(Method.java:597) 
at org.codehaus.groovy.runtime.callsite.PojoMetaMethodSite$PojoCachedMethodSite.invoke(PojoMetaMethodSite.java:188) 
.... 

2012-01-02 14: 25:25,379 [http-8080-1] ERROR errors.GrailsExceptionResolver - 处理请求时发生异常:[POST] /com.jason.score/fileResource/uploads Stacktrace如下: org.springframework.orm.hibernate3.HibernateOptimisticLockingFailureException:具有标识符[1]的类[com.jason.score.SecUser]的对象:乐观锁定失败;嵌套的异常是org.hibernate.StaleObjectStateException:行由另一个事务更新或删除(或未保存值映射不正确):[com.jason.score.SecUser#1] at java.lang.Thread.run(Thread。 java:662) 引起:org.hibernate.StaleObjectStateException:行被另一个事务更新或删除(或未保存值映射不正确):[com.jason.score.SecUser#1] ... 1更多

+0

“getUserId()”方法/闭包的外观如何?堆栈跟踪的底部是什么 - 有没有关于错误源于您的自定义代码的信息? – david 2012-01-02 20:26:47

+0

我添加了getUserId()方法。错误日志很大,以适应这里的所有。我只是在错误日志的底部添加了原始文章 – jason 2012-01-02 20:31:36

+0

,当我一次只上传一个文件时,一切正常。只有当我处理多个文件时才会发生这种情况。 – jason 2012-01-02 20:40:05

回答

1

该错误归因于Hibernate。您的标准查询可能存在错误。这看起来像一个可能的候选人

oAlbum.addToSong(oSong).save(flush:true) 

那些应该是单独的操作。标准查询应该由服务层中的方法封装。对于这样简单的操作,使用GORM动态查找器的标准可能看不出多少好处。

你也试图在一个地方做得太多。重构您的控制器以使用服务来执行这些操作。将moveFile()分成多个解析MP3文件的方法,然后将其传递给另一种方法,将其添加到用户的歌曲中。

如果直接在域中创建关系,则不需要使用用户的ID。

为每个原子片写一些测试,它应该使错误更容易找到。

+0

感谢您的回复。我对grails仍然陌生,所以我有一些跟进问题。首先,你在封装服务层的方法是什么意思?另外,我该如何折射我的控制器以使用服务?再次感谢您的帮助 – jason 2012-01-02 20:48:40

+0

如果您还不熟悉Grails,那么我建议您查看Manning的Ledbrook&Smith的Grails in Action。如果您想要免费的教程,请查看Scott Davis的Mastering Grails系列。这个特定的人处理服务。 http://www.ibm.com/developerworks/java/library/j-grails05208/index.html另外,请查看用户指南http://www.grails.org/doc/latest/guide/services。HTML – 2012-01-04 02:09:34